Wendy, Michael, Ueimor,

Thanks for the patch and feedback. We'll integrate these into our driver and 
post an update asap.

-Amit

On Friday 25 August 2006 03:10, Francois Romieu wrote:

wen xiong [off-list ref] :
[...]

quoted

diff -Nuar old/drivers/net/netxen/netxen_nic_hw.c
new/drivers/net/netxen/netxen_nic_hw.c ---
old/drivers/net/netxen/netxen_nic_hw.c	2006-08-23 12:58:43.000000000
-0500 +++ new/drivers/net/netxen/netxen_nic_hw.c	2006-08-23
13:15:19.000000000 -0500 @@ -313,7 +313,8 @@
 		}
 	}
 	CMD_DESC_TCP_HDR_OFFSET_WRT(desc, skb->h.raw - skb->data);
-	desc->ip_hdr_offset = skb->nh.raw - skb->data;
+	desc->length_tcp_hdr=cpu_to_le32(desc->length_tcp_hdr);

s/=/ = /

(several occurences)

[...]

quoted

diff -Nuar old/drivers/net/netxen/netxen_nic_init.c
new/drivers/net/netxen/netxen_nic_init.c ---
old/drivers/net/netxen/netxen_nic_init.c	2006-08-23 12:58:43.000000000
-0500 +++ new/drivers/net/netxen/netxen_nic_init.c	2006-08-23
13:15:19.000000000 -0500 @@ -494,7 +494,7 @@
 		desc_head = recv_ctx->rcv_status_desc_head;
 		desc = &desc_head[consumer];

-		if ((desc->owner & STATUS_OWNER_HOST))
+		if (((le16_to_cpu(desc->owner)) & STATUS_OWNER_HOST))

Would it make a difference to swab the constant part, i.e.:

	if (desc->owner & cpu_to_le16(STATUS_OWNER_HOST))