Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754366AbbFVHA7 (ORCPT ); Mon, 22 Jun 2015 03:00:59 -0400 Received: from mail-wi0-f173.google.com ([209.85.212.173]:35496 "EHLO mail-wi0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751787AbbFVHAw (ORCPT ); Mon, 22 Jun 2015 03:00:52 -0400 MIME-Version: 1.0 In-Reply-To: <1504484104.41691434955982414.JavaMail.weblogic@ep2mlwas07a> References: <1504484104.41691434955982414.JavaMail.weblogic@ep2mlwas07a> Date: Mon, 22 Jun 2015 09:00:50 +0200 Message-ID: Subject: Re: [RFC PATCH v2] packet: remove handling of tx_ring From: Frans Klaver To: Maninder Singh Cc: "davem@davemloft.net" , Willem de Bruijn , "dborkman@redhat.com" , Alexander Drozdov , Al Viro , Eyal Birger , "mst@redhat.com" , netdev , "linux-kernel@vger.kernel.org" , PANKAJ MISHRA Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2457 Lines: 74 Hi, On Mon, Jun 22, 2015 at 8:53 AM, Maninder Singh wrote: > Hi Frans, > >>> v1 = replace if()/BUG with BUG_ON() for tx_ring. >>> >>> v2 = >> >>I would keep this below the ---. There's little historical use for >>this version information when it gets merged. >> >>> remove handling of tx_ring in prb_setup_retire_blk_timer >>> for TPACKET_V3 because init_prb_bdqc is called only for NULL tx_ring >>> and thus prb_setup_retire_blk_timer for NULL tx_ring only. >> >>I'd say tx_ring is false, rather than NULL. It's not a pointer (here). >> >> >>> And also in funciton init_prb_bdqc there is no usage of tx_ring. >> >>s,funciton,function, > > Thanks for feedback , please check below changelog if it looks ok, > Then i will share updated patch:- > > v1 = replace if()/BUG with BUG_ON() for tx_ring. > > Signed-off-by: Maninder Singh > Signed-off-by: Frans Klaver No, the Suggested-by: was better for me. You can't go about and add Signed-off-by lines for someone else without permission ;-). > --- > Changes in v2: > > Remove handling of tx_ring in prb_setup_retire_blk_timer > for TPACKET_V3 because init_prb_bdqc is called only for zero tx_ring > and thus prb_setup_retire_blk_timer for zero tx_ring only. > > And also in functon init_prb_bdqc there is no usage of tx_ring. > Thus removing tx_ring from init_prb_bdqc. > > net/packet/af_packet.c | 14 +++++--------- > 1 file changed, 5 insertions(+), 9 deletions(-) > > Thanks > Maninder No, I didn't make myself clear enough, I'm afraid. The info about the different incarnations of your patch should go below the dashes. The whole "Remove handling ...." text should be your commit message, because that is what you want to see in the commit log. Here's an example: Subject: [RFC PATCH v2] packet: remove handling of tx_ring Remove handling of tx_ring in prb_setup_retire_blk_timer for TPACKET_V3 because init_prb_bdqc is called only for zero tx_ring and thus prb_setup_retire_blk_timer for zero tx_ring only. And also in functon init_prb_bdqc there is no usage of tx_ring. Thus removing tx_ring from init_prb_bdqc. Signed-off-by: Maninder Singh Suggested-by: Frans Klaver --- v1..v2: remove BUG() by removing tx_path diffstat & patch -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in Please read the FAQ at http://www.tux.org/lkml/