netdev - Re: [net 5/7] igb: fix stats for i210 rx_fifo

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <5352F17F.3080907@cogentembedded.com>
Date:	Sun, 20 Apr 2014 01:58:23 +0400
From:	Sergei Shtylyov <sergei.shtylyov@...entembedded.com>
To:	davem@...emloft.net, Todd Fujinaka <todd.fujinaka@...el.com>
CC:	Jeff Kirsher <jeffrey.t.kirsher@...el.com>, netdev@...r.kernel.org,
	gospo@...hat.com, sassmann@...hat.com
Subject: Re: [net 5/7] igb: fix stats for i210 rx_fifo_errors

Hello.

On 04/19/2014 05:59 AM, Jeff Kirsher wrote:

> From: Todd Fujinaka <todd.fujinaka@...el.com>

> RQDPC on i210/i211 is R/W not ReadClear. Clear after reading.

> Signed-off-by: Todd Fujinaka <todd.fujinaka@...el.com>
> Tested-by: Aaron Brown <aaron.f.brown@...el.com>
> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@...el.com>
> ---
>   drivers/net/ethernet/intel/igb/igb_main.c | 4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)

> diff --git a/drivers/net/ethernet/intel/igb/igb_main.c b/drivers/net/ethernet/intel/igb/igb_main.c
> index fb98d46..16430a8 100644
> --- a/drivers/net/ethernet/intel/igb/igb_main.c
> +++ b/drivers/net/ethernet/intel/igb/igb_main.c
> @@ -5193,8 +5193,10 @@ void igb_update_stats(struct igb_adapter *adapter,
>
>   	rcu_read_lock();
>   	for (i = 0; i < adapter->num_rx_queues; i++) {
> -		u32 rqdpc = rd32(E1000_RQDPC(i));
>   		struct igb_ring *ring = adapter->rx_ring[i];
> +		u32 rqdpc = rd32(E1000_RQDPC(i));

    Empty line needed after declaration. It was present before this patch.

> +		if (hw->mac.type >= e1000_i210)
> +			wr32(E1000_RQDPC(i), 0);
>
>   		if (rqdpc) {
>   			ring->rx_stats.drops += rqdpc;

WBR, Sergei

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html