Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit bf15fe99 authored by Stephen Hemminger's avatar Stephen Hemminger Committed by David S. Miller
Browse files

sky2: receive counter update 



Since it is likely that there are multiple packets received per
interrupt, only update the receive counters once after all
packets are processed.

Signed-off-by: default avatarStephen Hemminger <shemminger@vyatta.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 6c83504f

drivers/net/sky2.c

+19 −10
Original line number Original line Diff line number Diff line
@@ -2357,11 +2357,25 @@ static inline void sky2_tx_done(struct net_device *dev, u16 last) 
	}

	}

}

}





static inline void sky2_rx_done(struct sky2_hw *hw, unsigned port,

				unsigned packets, unsigned bytes)

{

	if (packets) {

		struct net_device *dev = hw->dev[port];



		dev->stats.rx_packets += packets;

		dev->stats.rx_bytes += bytes;

		dev->last_rx = jiffies;

		sky2_rx_update(netdev_priv(dev), rxqaddr[port]);

	}

}



/* Process status response ring */

/* Process status response ring */

static int sky2_status_intr(struct sky2_hw *hw, int to_do, u16 idx)

static int sky2_status_intr(struct sky2_hw *hw, int to_do, u16 idx)

{

{

	int work_done = 0;

	int work_done = 0;

	unsigned rx[2] = { 0, 0 };

	unsigned int total_bytes[2] = { 0 };

	unsigned int total_packets[2] = { 0 };





	rmb();

	rmb();

	do {

	do {
@@ -2388,7 +2402,8 @@ static int sky2_status_intr(struct sky2_hw *hw, int to_do, u16 idx) 
		le->opcode = 0;

		le->opcode = 0;

		switch (opcode & ~HW_OWNER) {

		switch (opcode & ~HW_OWNER) {

		case OP_RXSTAT:

		case OP_RXSTAT:

			++rx[port];

			total_packets[port]++;

			total_bytes[port] += length;

			skb = sky2_receive(dev, length, status);

			skb = sky2_receive(dev, length, status);

			if (unlikely(!skb)) {

			if (unlikely(!skb)) {

				dev->stats.rx_dropped++;

				dev->stats.rx_dropped++;
@@ -2406,9 +2421,6 @@ static int sky2_status_intr(struct sky2_hw *hw, int to_do, u16 idx) 
			}

			}





			skb->protocol = eth_type_trans(skb, dev);

			skb->protocol = eth_type_trans(skb, dev);

			dev->stats.rx_packets++;

			dev->stats.rx_bytes += skb->len;

			dev->last_rx = jiffies;





#ifdef SKY2_VLAN_TAG_USED

#ifdef SKY2_VLAN_TAG_USED

			if (sky2->vlgrp && (status & GMR_FS_VLAN)) {

			if (sky2->vlgrp && (status & GMR_FS_VLAN)) {
@@ -2487,11 +2499,8 @@ static int sky2_status_intr(struct sky2_hw *hw, int to_do, u16 idx) 
	sky2_write32(hw, STAT_CTRL, SC_STAT_CLR_IRQ);

	sky2_write32(hw, STAT_CTRL, SC_STAT_CLR_IRQ);





exit_loop:

exit_loop:

	if (rx[0])

	sky2_rx_done(hw, 0, total_packets[0], total_bytes[0]);

		sky2_rx_update(netdev_priv(hw->dev[0]), Q_R1);

	sky2_rx_done(hw, 1, total_packets[1], total_bytes[1]);



	if (rx[1])

		sky2_rx_update(netdev_priv(hw->dev[1]), Q_R2);





	return work_done;

	return work_done;

}

}