Subject: Re: May 14 scsi changes: driver support needed?
To: Manuel Bouyer <bouyer@antioche.lip6.fr>
From: Ignatios Souvatzis <is@netbsd.org>
List: current-users
Date: 05/21/2001 18:39:25
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