[openib-general] [PATCH] IB/mthca: fix MAD extended header format

Michael S. Tsirkin mst at mellanox.co.il
Mon Oct 30 08:03:46 PST 2006


Quoting r. Roland Dreier <rdreier at cisco.com>:
> Subject: Re: [PATCH] IB/mthca: fix MAD extended header format
> 
>  > Roland, the offsets look wrong. Am I missing something?
>  > Please review vs the spec, and if correct consider for 2.6.19.
> 
> I just applied your original patch without checking it carefully :)
> Yes, the offsets in your patch look like they match the PRM now.
> However:
> 
>  > +#define MAD_IFC_RLID_OFFSET   0x10e
> 
> I don't see anything about RLID in either the latest Tavor or Arbel
> specs -- they say the RLID goes in the input modifier (as the current
> mthca code also does).  So is putting the RLID into the mailbox
> needed?  Or should we follow the docs?

Old PRM versions (1.01) used to have rlid there - that's why it was in original
coed and I just kept it.  It kind of looked safest to keep it around, just in
case there's some old firmware that wants it there.

Most likely we can drop this - worst case user will need to upgrade
the firware. You decide.

-- 
MST




More information about the general mailing list