From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33401) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1agdxc-0007Fu-SS for qemu-devel@nongnu.org; Thu, 17 Mar 2016 15:51:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1agdxX-0008Mi-RH for qemu-devel@nongnu.org; Thu, 17 Mar 2016 15:51:56 -0400 Received: from mx2.parallels.com ([199.115.105.18]:55533) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1agdxX-0008MX-LB for qemu-devel@nongnu.org; Thu, 17 Mar 2016 15:51:51 -0400 References: <1458200278-11940-1-git-send-email-sameeh@daynix.com> From: "Denis V. Lunev" Message-ID: <56EB0AC8.8080701@virtuozzo.com> Date: Thu, 17 Mar 2016 22:51:36 +0300 MIME-Version: 1.0 In-Reply-To: <1458200278-11940-1-git-send-email-sameeh@daynix.com> Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/2] e1000: Introducing an upper bound of interrupts List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Sameeh Jubran , qemu-devel@nongnu.org, Jason Wang Cc: Dmitry Fleytman , Yan Vugenfirer On 03/17/2016 10:37 AM, Sameeh Jubran wrote: > This patch series introduces an upper bound for the number of interrupts > per second. This feature is supported by the real hardware, however up > until now it wasn't implemented in e1000. This feature is very > significant, it can prevent an interrupt storm by giving the driver > a bounded inter-interrupt interval to handle interrupts. > > This patch was made after observing an interrupt storm in Windows 10 > when disabling e1000. > > How reproducible: > > Steps to reproduce: > 1. Start Win 10 guest with e1000 device. > 2. Go to device manager and try to disable and enable the device. > 3. After several enable/disable to the device the guest hangs when > the device is being disabled. > > Actual results: > Guest hang after click OK button. > > Expected results: > Device is disabled. > > After applying the patch the guest no longer hangs, and an Iperf test > ran successfully. > > Sameeh Jubran (2): > e1000: Fixing interrupts pace. > Revert "e1000: fix hang of win2k12 shutdown with flood ping" > > hw/net/e1000.c | 13 ++++++++----- > 1 file changed, 8 insertions(+), 5 deletions(-) > In general I support the idea to have the minimal limit and drop original patch. Though I'd better keep the same delay as it was. The limit for physical card is good for physical card. They have limitations which are not available in virtualization world. AFAIR I have used 250 as the number from the article of the original author of ITR support as the best value. Den