From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756215AbbCCLSc (ORCPT ); Tue, 3 Mar 2015 06:18:32 -0500 Received: from ns.iliad.fr ([212.27.33.1]:33430 "EHLO ns.iliad.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755012AbbCCLSb (ORCPT ); Tue, 3 Mar 2015 06:18:31 -0500 Message-ID: <54F59885.1000203@freebox.fr> Date: Tue, 03 Mar 2015 12:18:29 +0100 From: Nicolas Schichan User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 To: David Miller CC: tklauser@distanz.ch, balbi@ti.com, w-lkml@lebenslange-mailadresse.de, ebiederm@xmission.com, alexander.h.duyck@redhat.com, netdev@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] bcm63xx_enet: fix poll callback. References: <1425317290-1443-1-git-send-email-nschichan@freebox.fr> <20150302.221547.345365592649851304.davem@davemloft.net> In-Reply-To: <20150302.221547.345365592649851304.davem@davemloft.net> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 03/03/2015 04:15 AM, David Miller wrote: > First, no signoff, that is required for your patch. Hello David, I realized that I had forgotten the signoff-by line too late unfortunately. > Secondly, we strongly recommend that TX buffer reclaim be not > accounted at all in the poll budget. > > Just do all of the TX work, unconditionally, every time the poll > routine is invoked. Do not add it into the work variable that > gets compared against the budget. Pretend it took '0' units of > work. I took inspiration from the mv643xx_eth driver which seems to account the tx buffer reclaim in the work done in its poll callback. I'll send an updated patch. Thanks, -- Nicolas Schichan Freebox SAS