On 08:42 Fri 21 Dec     , [EMAIL PROTECTED] wrote:
> On Fri, Dec 21, 2007 at 04:30:37PM +0000, Sasha Khapyorsky wrote:
> > 
> > Initialize sm portid in ib_resolve_smlid() - likely it fixes crash when
> > in debug mode uninitialized direct path array printed.
> > 
> > Signed-off-by: Sasha Khapyorsky <[EMAIL PROTECTED]>
> 
> Tested by: Arthur Kepner <[EMAIL PROTECTED]>

Thanks!

> Sasha - seems to fix things up when using the debug flags, 
> furthermore, it seems to make the original problem (the 
> failure of "smpquery -G ...") go away:

After looking more in the code I see that it is very possible - when
smpquery asks SA for PathRecord it uses PKey index value from sm_id
which was uninitialized before the patch. Assuming that PKey index
value was garbaged PathRecord query has been rejected by CA even
before it reached SM.

Seems we fixed both bugs in this patch (by initializing sm_id) - cool.
Thanks for the reporting and the great help with debugging this.

Sasha
_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to