[ofw] RE: IPoIB partitioning support potential bug?

[Fab Tillier]

>  Hi, Yes, this byte swap is correct. Checked upon Voltaire IB switch.

Ok, so does this mean that the pkey value was returned by the HCA driver in host-order?  If so, this is a bug in the HCA driver.

-Fab

> Slava
>
> -----Original Message----- From: ofw-bounces at lists.openfabrics.org
> [mailto:ofw-bounces at lists.openfabrics.org] On Behalf Of
> ofw-request at lists.openfabrics.org Sent: Saturday, May 31, 2008 10:00 PM
> To: ofw at lists.openfabrics.org Subject: ofw Digest, Vol 16, Issue 71
>
> Send ofw mailing list submissions to
>         ofw at lists.openfabrics.org
>
> To subscribe or unsubscribe via the World Wide Web, visit
>         http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw or,
>         via email, send a message with subject or body 'help' to
>         ofw-request at lists.openfabrics.org
>
> You can reach the person managing the list at
>         ofw-owner at lists.openfabrics.org
>
> When replying, please edit your Subject line so it is more specific
> than "Re: Contents of ofw digest..."
>
>
> Today's Topics:
>
>    1.  IPoIB partitioning support potential bug? (Fab Tillier)
>
>
> ----------------------------------------------------------------------
>
> Message: 1 Date: Fri, 30 May 2008 12:18:14 -0700 From: Fab Tillier
> <ftillier at windows.microsoft.com> Subject: [ofw] IPoIB partitioning
> support potential bug? To: "ofw at lists.openfabrics.org"
> <ofw at lists.openfabrics.org> Message-ID:
>
> <52F2F0099A281049A606C312906481CE04DA725C4E at NA-EXMSG-
> W601.wingroup.winde ploy.ntdev.microsoft.com>
>
> Content-Type: text/plain; charset="us-ascii"
>
> Hi folks,
>
> Looking at the code changes to support partitioning in IPoIB (revision
> 1160)...
>
> The __port_get_bcast and __port_create_bcast functions are swapping the
> byte order of the pkey when formatting the MGID of the member record.
> IBAL treats pkeys as network order values always, like all other values
> that go on the wire.
>
> I would think this is a bug, but not sure as I haven't gone over the
> partition support in detail yet.
>
> Is the code right?
> -Fab
>
>
> ------------------------------
>
> _______________________________________________ ofw mailing list
> ofw at lists.openfabrics.org
> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw
>
> End of ofw Digest, Vol 16, Issue 71 ***********************************
> _______________________________________________ ofw mailing list
> ofw at lists.openfabrics.org
> http://lists.openfabrics.org/cgi-bin/mailman/listinfo/ofw