On 2020-05-11 00:34, Arun Easi wrote: > How did you pick the list of structures for this one? IOCB > structures make sense, but why "ct_sns_req", for e.g.? Adding > support for a different request may alter the structure size. > > Wondering what made you add all these checks? Anything tripped > while making some changes? Hi Arun, For this patch I selected all data structures that have one or more little endian or big endian members. Such data structures either are used to interface with the firmware or represent FC frame information. I agree that struct ct_sns_req is special. I will leave the size check for that data structure out when I repost this patch series. Bart.
