On Thursday 27 July 2006 19:48, Stephen Hemminger wrote:
> --- sky2.orig/drivers/net/forcedeth.c 2006-07-13 12:53:48.000000000 -0700
> +++ sky2/drivers/net/forcedeth.c 2006-07-27 10:45:49.000000000 -0700
> @@ -381,21 +381,21 @@
>
> /* Big endian: should work, but is untested */
> struct ring_desc {
> - u32 PacketBuffer;
> - u32 FlagLen;
> + u32 buf;
> + u32 flaglen;
> };
>
> struct ring_desc_ex {
> - u32 PacketBufferHigh;
> - u32 PacketBufferLow;
> - u32 TxVlan;
> - u32 FlagLen;
> + u32 bufhigh;
> + u32 buflow;
> + u32 txvlan;
> + u32 flaglen;
> };
Shouldn't these two structs be __attribute__((packed)) ?
--
Greetings Michael.
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
