> Since not all SA attributes should be multiple of 8 bytes (specifically
> InformInfo).

I thought the issue was that the C-struct definition was required to be a 
multiple of 8-bytes, not the actual SA attribute definition, ..

>  static inline ib_net16_t OSM_API ib_get_attr_offset(IN const uint32_t
> attr_size)
>  {
> -     CL_ASSERT((attr_size & 0x07) == 0);
>       return (cl_hton16((uint16_t) (attr_size >> 3)));
>  }

which causes this to fail, since it lops off the lower bits of the size, rather 
than rounding up.

I would either expect the assert to always work or the code to round up 
properly.

- Sean
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to