***SPAM*** Re: [ofa-general] Re: [PATCH] Update mad formatting functions.

[Hal Rosenstock]

On Wed, Apr 15, 2009 at 5:03 PM, Ira Weiny <weiny2 at llnl.gov> wrote:

<snip...>

> As an aside, not all LID's are decimal.  Should we change this?
>
> from fields.c
> ...
>        {BE_OFFS(256, 16), "DrSmpDLID", mad_dump_hex},
>        {BE_OFFS(272, 16), "DrSmpSLID", mad_dump_hex},
> ...
>        {BITSOFFS(224, 16), "RedirectLID", mad_dump_hex},
>        {BITSOFFS(480, 16), "TrapLID", mad_dump_hex},
> ...
>        {BITSOFFS(320, 16), "PathRecDLid", mad_dump_hex},
>        {BITSOFFS(336, 16), "PathRecSLid", mad_dump_hex},
> ...
>        {BITSOFFS(288, 16), "McastMemMLid", mad_dump_hex},

The agreement was decimal LIDs for unicast and hex LIDs for multicast.
Permissive LID is unicast so is decimal. This was missed in libibmad
though.

-- Hal

<snip...>