Commit 6b8f90c2 authored by Lennert Buytenhek's avatar Lennert Buytenhek Committed by Lennert Buytenhek

mv643xx_eth: avoid reading ->byte_cnt twice during receive processing

Currently, the receive processing reads ->byte_cnt twice (once to
update interface statistics and once to properly size the data area
of the received skb), but since receive descriptors live in uncached
memory, caching this value in a local variable saves one uncached
access, and increases routing performance a tiny little bit more.
Signed-off-by: default avatarLennert Buytenhek <buytenh@marvell.com>
parent 2b4a624d
...@@ -477,6 +477,7 @@ static int rxq_process(struct rx_queue *rxq, int budget) ...@@ -477,6 +477,7 @@ static int rxq_process(struct rx_queue *rxq, int budget)
struct rx_desc *rx_desc; struct rx_desc *rx_desc;
unsigned int cmd_sts; unsigned int cmd_sts;
struct sk_buff *skb; struct sk_buff *skb;
u16 byte_cnt;
rx_desc = &rxq->rx_desc_area[rxq->rx_curr_desc]; rx_desc = &rxq->rx_desc_area[rxq->rx_curr_desc];
...@@ -499,6 +500,8 @@ static int rxq_process(struct rx_queue *rxq, int budget) ...@@ -499,6 +500,8 @@ static int rxq_process(struct rx_queue *rxq, int budget)
mp->work_rx_refill |= 1 << rxq->index; mp->work_rx_refill |= 1 << rxq->index;
byte_cnt = rx_desc->byte_cnt;
/* /*
* Update statistics. * Update statistics.
* *
...@@ -508,7 +511,7 @@ static int rxq_process(struct rx_queue *rxq, int budget) ...@@ -508,7 +511,7 @@ static int rxq_process(struct rx_queue *rxq, int budget)
* byte CRC at the end of the packet (which we do count). * byte CRC at the end of the packet (which we do count).
*/ */
stats->rx_packets++; stats->rx_packets++;
stats->rx_bytes += rx_desc->byte_cnt - 2; stats->rx_bytes += byte_cnt - 2;
/* /*
* In case we received a packet without first / last bits * In case we received a packet without first / last bits
...@@ -537,7 +540,7 @@ static int rxq_process(struct rx_queue *rxq, int budget) ...@@ -537,7 +540,7 @@ static int rxq_process(struct rx_queue *rxq, int budget)
* The -4 is for the CRC in the trailer of the * The -4 is for the CRC in the trailer of the
* received packet * received packet
*/ */
skb_put(skb, rx_desc->byte_cnt - 2 - 4); skb_put(skb, byte_cnt - 2 - 4);
if (cmd_sts & LAYER_4_CHECKSUM_OK) { if (cmd_sts & LAYER_4_CHECKSUM_OK) {
skb->ip_summed = CHECKSUM_UNNECESSARY; skb->ip_summed = CHECKSUM_UNNECESSARY;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment