From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id D06A06CC1 for ; Tue, 8 May 2018 11:02:27 +0200 (CEST) X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga105.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 08 May 2018 02:02:26 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.49,377,1520924400"; d="scan'208";a="54095659" Received: from bricha3-mobl.ger.corp.intel.com ([10.237.221.55]) by orsmga001.jf.intel.com with SMTP; 08 May 2018 02:02:24 -0700 Received: by (sSMTP sendmail emulation); Tue, 08 May 2018 10:02:23 +0100 Date: Tue, 8 May 2018 10:02:22 +0100 From: Bruce Richardson To: Andy Green Cc: Andrew Rybchenko , dev@dpdk.org Message-ID: <20180508090222.GC20636@bricha3-MOBL.ger.corp.intel.com> References: <152575364588.56689.3300796065057551728.stgit@localhost.localdomain> <152575382842.56689.4589071928538784307.stgit@localhost.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Organization: Intel Research and Development Ireland Ltd. User-Agent: Mutt/1.9.4 (2018-02-28) Subject: Re: [dpdk-dev] [PATCH 12/18] drivers: net: sfc: fix another strncpy size and NUL 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, 08 May 2018 09:02:28 -0000 On Tue, May 08, 2018 at 04:18:41PM +0800, Andy Green wrote: > > > On 05/08/2018 03:36 PM, Andrew Rybchenko wrote: > > Many thanks. I guess the most of below notes are applicable to many other > > patches in the series. > > > > Signed-off-by: ,  Fixes: and Cc: stable@dpdk.org tags are missing. See [1]. > > Everybody's project has different prejudices. > > > Changeset summary should start from "net/sfc: " > > I.e. something like: > > net/sfc: fix strncpy size and NUL > > Yeah if that's what you like. > > > (it looks like "another" is useless in the original subject) > > It captures my feeling at having to wade through making 18 fixes before I > could compile the project on current Fedora. > > > In general all patches should pass ./devtools/check-git-log.sh and > > ./devtools/checkpatches.sh > > (which requires path to Linux kernel checkpatches.pl). > > Can you help me understand why adding CRLFs at 80 cols on the gcc errors I > pasted helps anything at all? The patches actually fix problems in the > code. > I don't think there is any need to wrap the error messages since they are pasted verbatim. > If you don't care about Coverity, let me know and I will register this > project there and send you fixes when I have time. > FYI: The DPDK project is already registered in coverity and scanned regularly. We do try and fix as many issues it flags as possible, but not everything gets dealt with as quickly as we'd like, sadly. https://scan.coverity.com/projects/dpdk-data-plane-development-kit Regards, /Bruce