On Mon, May 21, 2001 at 03:40:58PM +0200, Manuel Bouyer wrote:
> 
> Hum, this means we need to add __attribute__((packed)) to all structs defining
> SCSI commands or results ? Looks strange to me, as this worked before without
> this ... Or maybe we need them only for scsipi_mode_sense and
> scsipi_mode_select, as they're the only ones with union in the struct ?

We just were lucky.

Using structs to map hardware/wire data is compiler dependent.

it seems to work when you ONLY have u_int8_t fields, but mixing with structs
or with unions or with unaligned shorts or longs doesn't create consistent
results.

Regards,
	-is