Thread (5 messages) 5 messages, 2 authors, 2009-08-28

Re: [ewg] [PATCH] IB/ehca: Construct MAD redirect replies from request MAD

From: Hal Rosenstock <hidden>
Date: 2009-08-27 13:31:44

On 8/27/09, Joachim Fenkes [off-list ref] wrote:
Hal Rosenstock [off-list ref] wrote on 26.08.2009 17:15:03:
quoted
Thanks for doing this. It looks sane to me. The only issue I recall that
quoted
appears to be remaining is a better setting of
ClassPortInfo:RespTimeValue
quoted
rather than hardcoding. Perhaps using the value from PortInfo is the way
to go
quoted
(ideally it would be that value from the port to which the the requester
is
quoted
being redirected to but that might not be so easy to get from this port.
I don't think that effort will be necessary or even legal. The requestor
will react to the redirection with another Get(ClassPortInfo) to the
redirection target, which will reply with its own RespTimeValue, so our
driver should speak for itself.

I overreached with my comment on how this works.

 Since we don't know when our MAD
processing and sending of the response is going to be scheduled (we're not
running on real-time constraints here), we play it safe and return 18,
which amounts to roughly a second.

Make sense?

I don't think it should be hard coded. IMO it would be better to default to
18 and somehow able to be adjusted (via a (dynamic) module parameter ?).

-- Hal

Regards
Joachim
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help