netxen: small xmit optimizations

Signed-off-by: Dhananjay Phadke <dhananjay@netxen.com>
Signed-off-by: David S. Miller <davem@davemloft.net>

diff --git a/drivers/net/netxen/netxen_nic.h b/drivers/net/netxen/netxen_nic.h
index 73cb1164457fdcdfc3d8c7177a49033eb9cfa26a..b5c0d66daf7e75d0a479ef39d5313739dfd9eab8 100644 (file)
--- a/drivers/net/netxen/netxen_nic.h
+++ b/drivers/net/netxen/netxen_nic.h
@@ -358,10 +358,7 @@ struct cmd_desc_type0 {
                __le64 addr_buffer1;
        };
 
-       __le16 buffer1_length;
-       __le16 buffer2_length;
-       __le16 buffer3_length;
-       __le16 buffer4_length;
+       __le16 buffer_length[4];
 
        union {
                struct {

diff --git a/drivers/net/netxen/netxen_nic_init.c b/drivers/net/netxen/netxen_nic_init.c
index f8164345e3b3f66051135824f79ef4686bb1493a..72aba634554a94a2a04e91f4cac20e2c8e77326c 100644 (file)
--- a/drivers/net/netxen/netxen_nic_init.c
+++ b/drivers/net/netxen/netxen_nic_init.c
@@ -955,6 +955,7 @@ int netxen_process_cmd_ring(struct netxen_adapter *adapter)
        int done = 0;
 
        last_consumer = adapter->last_cmd_consumer;
+       barrier(); /* cmd_consumer can change underneath */
        consumer = le32_to_cpu(*(adapter->cmd_consumer));
 
        while (last_consumer != consumer) {
@@ -1005,6 +1006,7 @@ int netxen_process_cmd_ring(struct netxen_adapter *adapter)
         * There is still a possible race condition and the host could miss an
         * interrupt. The card has to take care of this.
         */
+       barrier(); /* cmd_consumer can change underneath */
        consumer = le32_to_cpu(*(adapter->cmd_consumer));
        done = (last_consumer == consumer);
 

diff --git a/drivers/net/netxen/netxen_nic_main.c b/drivers/net/netxen/netxen_nic_main.c
index dfd66eaed1aa9f29f00f3e0422e0c4faf8eb0a8e..c9519843f8cb9e4c1185aded96027706468531b9 100644 (file)
--- a/drivers/net/netxen/netxen_nic_main.c
+++ b/drivers/net/netxen/netxen_nic_main.c
@@ -1212,7 +1212,16 @@ netxen_clean_tx_dma_mapping(struct pci_dev *pdev,
        }
 }
 
-static int netxen_nic_xmit_frame(struct sk_buff *skb, struct net_device *netdev)
+static inline void
+netxen_clear_cmddesc(u64 *desc)
+{
+       int i;
+       for (i = 0; i < 8; i++)
+               desc[i] = 0ULL;
+}
+
+static int
+netxen_nic_xmit_frame(struct sk_buff *skb, struct net_device *netdev)
 {
        struct netxen_adapter *adapter = netdev_priv(netdev);
        struct netxen_hardware_context *hw = &adapter->ahw;
@@ -1245,7 +1254,7 @@ static int netxen_nic_xmit_frame(struct sk_buff *skb, struct net_device *netdev)
 
        /* Copy the descriptors into the hardware    */
        hwdesc = &hw->cmd_desc_head[producer];
-       memset(hwdesc, 0, sizeof(struct cmd_desc_type0));
+       netxen_clear_cmddesc((u64 *)hwdesc);
        /* Take skb->data itself */
        pbuf = &adapter->cmd_buf_arr[producer];
 
@@ -1264,7 +1273,7 @@ static int netxen_nic_xmit_frame(struct sk_buff *skb, struct net_device *netdev)
        netxen_set_tx_frags_len(hwdesc, frag_count, skb->len);
        netxen_set_tx_port(hwdesc, adapter->portnum);
 
-       hwdesc->buffer1_length = cpu_to_le16(first_seg_len);
+       hwdesc->buffer_length[0] = cpu_to_le16(first_seg_len);
        hwdesc->addr_buffer1 = cpu_to_le64(buffrag->dma);
 
        for (i = 1, k = 1; i < frag_count; i++, k++) {
@@ -1277,7 +1286,7 @@ static int netxen_nic_xmit_frame(struct sk_buff *skb, struct net_device *netdev)
                        k = 0;
                        producer = get_next_index(producer, num_txd);
                        hwdesc = &hw->cmd_desc_head[producer];
-                       memset(hwdesc, 0, sizeof(struct cmd_desc_type0));
+                       netxen_clear_cmddesc((u64 *)hwdesc);
                        pbuf = &adapter->cmd_buf_arr[producer];
                        pbuf->skb = NULL;
                }
@@ -1297,21 +1306,18 @@ static int netxen_nic_xmit_frame(struct sk_buff *skb, struct net_device *netdev)
                buffrag->dma = temp_dma;
                buffrag->length = temp_len;
 
+               hwdesc->buffer_length[k] = cpu_to_le16(temp_len);
                switch (k) {
                case 0:
-                       hwdesc->buffer1_length = cpu_to_le16(temp_len);
                        hwdesc->addr_buffer1 = cpu_to_le64(temp_dma);
                        break;
                case 1:
-                       hwdesc->buffer2_length = cpu_to_le16(temp_len);
                        hwdesc->addr_buffer2 = cpu_to_le64(temp_dma);
                        break;
                case 2:
-                       hwdesc->buffer3_length = cpu_to_le16(temp_len);
                        hwdesc->addr_buffer3 = cpu_to_le64(temp_dma);
                        break;
                case 3:
-                       hwdesc->buffer4_length = cpu_to_le16(temp_len);
                        hwdesc->addr_buffer4 = cpu_to_le64(temp_dma);
                        break;
                }