From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 470362BB4 for ; Tue, 21 Feb 2017 13:07:05 +0100 (CET) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 21 Feb 2017 04:07:04 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.35,189,1484035200"; d="scan'208";a="1113591671" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.221.61]) by fmsmga001.fm.intel.com with SMTP; 21 Feb 2017 04:07:02 -0800 Received: by (sSMTP sendmail emulation); Tue, 21 Feb 2017 12:07:02 +0000 Date: Tue, 21 Feb 2017 12:07:01 +0000 From: Bruce Richardson To: Vladyslav Buslov Cc: "dev@dpdk.org" Message-ID: <20170221120701.GA203564@bricha3-MOBL3.ger.corp.intel.com> References: <1487524499-9570-1-git-send-email-vladyslav.buslov@harmonicinc.com> <20170221112330.GA194104@bricha3-MOBL3.ger.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Research and =?iso-8859-1?Q?De=ACvel?= =?iso-8859-1?Q?opment?= Ireland Ltd. User-Agent: Mutt/1.7.2 (2016-11-26) Subject: Re: [dpdk-dev] [PATCH] lpm: extend IPv6 next hop field X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 21 Feb 2017 12:07:05 -0000 On Tue, Feb 21, 2017 at 11:37:09AM +0000, Vladyslav Buslov wrote: > Hello Bruce, > > Thanks for reviewing my code. > > I noticed errors generated by checkpatch. > However all of them are due to original formatting of files. (e.g. code copy-pasted by me as-is from same file or modifications to original lines unrelated to formatting) > I decided to preserve original formatting of files instead of introducing inconsistencies. > I'll create V2 according to code guidelines if it is your preferred approach. > > Regards, > Vlad > Hi Vlad, one of the main reasons we don't wholesale do code-style cleanup on the code is so as to preserve the git history of what patches modified what lines. For any lines you are modifying anyway, this doesn't apply, so feel free to clean up those code lines in the process. It just makes the code cleaner for everyone else, while ensuring that "git blame" still provides useful info. Thanks, /Bruce > > -----Original Message----- > > From: Bruce Richardson [mailto:bruce.richardson@intel.com] > > Sent: Tuesday, February 21, 2017 1:24 PM > > To: Vladyslav Buslov > > Cc: dev@dpdk.org > > Subject: Re: [PATCH] lpm: extend IPv6 next hop field > > > > On Sun, Feb 19, 2017 at 07:14:59PM +0200, Vladyslav Buslov wrote: > > > This patch extend next_hop field from 8-bits to 21-bits in LPM library > > > for IPv6. > > > > > > Added versioning symbols to functions and updated library and > > > applications that have a dependency on LPM library. > > > > > > Signed-off-by: Vladyslav Buslov > > > --- > > Looks good to me. There are a number of checkpatch errors flagged. Can you > > perhaps do a V2 with those fixed. (You can use the checkpatches.sh wrapper > > script in the devtools directory to help). > > > > Once checkpatch errors are cleared: > > > > Acked-by: Bruce Richardson > > > > Regards, > > /Bruce