netdev - Re: [net 1/3] i40e: fix un-necessary Tx hangs

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: <20150106090440.00006a7d@unknown>
Date:	Tue, 6 Jan 2015 09:04:40 -0800
From:	Jesse Brandeburg <jesse.brandeburg@...el.com>
To:	Jeff Kirsher <jeffrey.t.kirsher@...el.com>, <davem@...emloft.net>
Cc:	<netdev@...r.kernel.org>, <nhorman@...hat.com>,
	<sassmann@...hat.com>, <jogreene@...hat.com>
Subject: Re: [net 1/3] i40e: fix un-necessary Tx hangs

On Tue, 6 Jan 2015 00:44:22 -0800
Jeff Kirsher <jeffrey.t.kirsher@...el.com> wrote:

> From: Jesse Brandeburg <jesse.brandeburg@...el.com>
> 
> When the driver was polling with interrupts disabled the hardware
> will occasionally not write back descriptors.  This patch causes
> the driver to detect this situation and force an interrupt to
> fire which will flush the stuck descriptor.  Does not conflict
> with NAPI because if we are already polling the napi_schedule is
> ignored.  Additionally the extra interrupts are rate limited, so
> don't cause a burden to the CPU.

Dave, sorry to do this but we discovered a bug in this patch (where
this patch impacts performance) and didn't interrupt Jeff before he sent
it (my mistake) I will prep a v2 for Jeff.

Please do not apply.
--
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