> Quoting Sean Hefty <[EMAIL PROTECTED]>:
> Subject: Re: [PATCH RFC 1 of 5] IB/sa: QoS support
> 
> >  static const struct ib_field path_rec_table[] = {
> > -   { RESERVED,
> > -     .offset_words = 0,
> > -     .offset_bits  = 0,
> > -     .size_bits    = 32 },
> > -   { RESERVED,
> > -     .offset_words = 1,
> > +   { PATH_REC_FIELD(service_id),
> > +     .offset_words = 2,
> 
> I haven't read the proposed annex, but I would have expected this to be 
> offset_words = 0.  Otherwise, it seems that the service_id would fall over 
> the dgid.

Thanks, good catch.

-- 
MST

_______________________________________________
openib-general mailing list
[email protected]
http://openib.org/mailman/listinfo/openib-general

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

Reply via email to