From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.0 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id B6686C4360F for ; Tue, 26 Mar 2019 17:24:24 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 89F7C2087C for ; Tue, 26 Mar 2019 17:24:24 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="cfsCMV+G" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 89F7C2087C Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=redhat.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender:Content-Type: Content-Transfer-Encoding:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:Message-ID:From: References:To:Subject:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=gBz8W3LnC2q7mb284ibGIdRsZNq7VHPiNKxa7sFo8oI=; b=cfsCMV+GhFnlz3/53u7emTRjA yqnLBHMjGHfcR0mZ9kMLmjndfAT6ReasdEseLaIF1QBvaInUWneNCB6tYpqWiN2DQkFrgbY/mwMdA VYwcGM/795OhXaGsUWsVN4ignPHcOix0gqwWbhMUOEDkRNsxlZ/7C6L4ZH2HRbqH0Bl3wyvt32WuF SuA3wBguBOb31s7eFT7GOEH+tXj5iG5m+A+TXDulMZz6pTrP4E1jeLKHh4PPDXE3PBd9uPr6OO1aA dajivNRJKd2tdLiJoYGZvCyckNOL4tQQan/tuIJkTQV05oDV6gCp1z9idNlFgfJ1rorZiuB+0+mgM 7uKudm6Kw==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1h8po8-00018x-Ga; Tue, 26 Mar 2019 17:24:16 +0000 Received: from mx1.redhat.com ([209.132.183.28]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1h8po4-000180-1V for linux-arm-kernel@lists.infradead.org; Tue, 26 Mar 2019 17:24:14 +0000 Received: from smtp.corp.redhat.com (int-mx04.intmail.prod.int.phx2.redhat.com [10.5.11.14]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id D695B30917EE; Tue, 26 Mar 2019 17:24:10 +0000 (UTC) Received: from [10.3.116.169] (ovpn-116-169.phx2.redhat.com [10.3.116.169]) by smtp.corp.redhat.com (Postfix) with ESMTP id 08FDB5DABA; Tue, 26 Mar 2019 17:24:06 +0000 (UTC) Subject: Re: [PATCH net v2 0/2] thunderx: fix receive buffer page recycling To: Jesper Dangaard Brouer References: <155361559092.14454.4615730926574762068.email-sent-by-dnelson@cyan> <08849eac-c185-ee0d-3907-85beb392899b@redhat.com> <20190326171844.014b1918@carbon> From: Dean Nelson Message-ID: <2bf5a8d9-3fec-28b5-ecb0-bba1fb88b2df@redhat.com> Date: Tue, 26 Mar 2019 12:24:06 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.6.0 MIME-Version: 1.0 In-Reply-To: <20190326171844.014b1918@carbon> Content-Language: en-US X-Scanned-By: MIMEDefang 2.79 on 10.5.11.14 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.41]); Tue, 26 Mar 2019 17:24:11 +0000 (UTC) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190326_102412_118266_D86885ED X-CRM114-Status: GOOD ( 22.67 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Sunil Goutham , Robert Richter , netdev@vger.kernel.org, Vadim Lomovtsev , David Miller , linux-arm-kernel@lists.infradead.org Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 3/26/19 11:18 AM, Jesper Dangaard Brouer wrote: > On Tue, 26 Mar 2019 11:09:10 -0500 > Dean Nelson wrote: > >> On 3/26/19 10:53 AM, Dean Nelson wrote: >>> In attempting to optimize receive buffer page recycling for XDP, commit >>> 773225388dae15e72790 ("net: thunderx: Optimize page recycling for XDP") >>> inadvertently introduced two problems for the non-XDP case, that will be >>> addressed by this patch series. >>> >>> Dean Nelson (2): >>> thunderx: enable page recycling for non-XDP case >>> thunderx: eliminate extra calls to put_page() for pages held for recycling >>> >>> drivers/net/ethernet/cavium/thunder/nicvf_queues.c | 30 ++++++++++++++---------------- >>> 1 file changed, 14 insertions(+), 16 deletions(-) >>> >> >> >> I'm really sorry, I completely forgot to add a description >> of what the changes were made by v2 (which was just to add >> the 'Fixes:' lines to the changelogs). > > That should be okay, for such a small change. Okay. >> And I forgot to add you to the CC-list. I'm assuming I >> should repost? Right? (And do you want to be added?) >> >> It's just one of those days. > > I can live with not getting Cc'ed (but I would prefer it), but some > other adjustments. By your phrase "but some other adjustments", are you indicating that I should post a v3 with the below items addressed (and while I'm at it fix the above two)? Or is the following just for my education for the future? > > There should not be a newline between "Fixes:" and "Signed-off-by:". Good to know. I'd actually searched netdev mailing list for examples. And the one patch I happened to look at, had a blank line separating it and the "Signed-off-by:" line that followed. And just looking again, I saw a couple that way. But they're definitely in the minority. > And nitpicking, I believe that we usually only show the first 12 chars > of the commit id. I have setup git to use: 'git show --pretty=fixes' I had looked at Documentation/process/submitting-patches.rst and had read the following... If you want to refer to a specific commit, don't just refer to the SHA-1 ID of the commit. Please also include the oneline summary of the commit, to make it easier for reviewers to know what it is about. Example:: Commit e21d2170f36602ae2708 ("video: remove unnecessary platform_set_drvdata()") removed the unnecessary platform_set_drvdata(), but left the variable "dev" unused, delete it. You should also be sure to use at least the first twelve characters of the SHA-1 ID. The kernel repository holds a *lot* of objects, making collisions with shorter IDs a real possibility. Bear in mind that, even if there is no collision with your six-character ID now, that condition may change five years from now. Which has the first 20 chars in its example, and speaks of using at least 12 chars. So now I just went back and re-read that section and the paragraph that followed it (which previously had been outside of the view of my terminal window)... If your patch fixes a bug in a specific commit, e.g. you found an issue using ``git bisect``, please use the 'Fixes:' tag with the first 12 characters of the SHA-1 ID, and the one line summary. Do not split the tag across multiple lines, tags are exempt from the "wrap at 75 columns" rule in order to simplify parsing scripts. For example:: Fixes: 54a4f0239f2e ("KVM: MMU: make kvm_mmu_zap_page() return the number of pages it actually freed") Which does say that you are correct. Just the first 12 chars should be used. Like I said earlier... just one of those days. :-) > > From my ~/.gitconfig: > --------------------- > [core] > abbrev = 12 > > [pretty] > fixes = Fixes: %h (\"%s\") > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel