From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id DE0C06A5B for ; Wed, 6 Jul 2016 11:25:28 +0200 (CEST) Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga104.fm.intel.com with ESMTP; 06 Jul 2016 02:25:29 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.28,318,1464678000"; d="scan'208";a="134554245" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.220.95]) by fmsmga004.fm.intel.com with SMTP; 06 Jul 2016 02:25:26 -0700 Received: by (sSMTP sendmail emulation); Wed, 06 Jul 2016 10:25:25 +0025 Date: Wed, 6 Jul 2016 10:25:25 +0100 From: Bruce Richardson To: Thomas Monjalon Cc: dev@dpdk.org, nelio.laranjeiro@6wind.com Message-ID: <20160706092524.GC17404@bricha3-MOBL3> References: <1467729650-925-1-git-send-email-bruce.richardson@intel.com> <6142444.k3HLHFvhXo@xps13> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6142444.k3HLHFvhXo@xps13> Organization: Intel Research and =?iso-8859-1?Q?De=ACvel?= =?iso-8859-1?Q?opment?= Ireland Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Subject: Re: [dpdk-dev] [PATCH] scripts: add check for net driver names without net prefix X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 06 Jul 2016 09:25:29 -0000 On Wed, Jul 06, 2016 at 11:10:48AM +0200, Thomas Monjalon wrote: > 2016-07-05 15:40, Bruce Richardson: > > +# check headline label for net/ prefix if needed > > +bad=$(echo "$headlines" | grep -E --color=always \ > > + -e "^($(ls drivers/net | grep -v Makefile | tr '\n' '|')):" \ > > + | sed 's,^,\t,') > > +[ -z "$bad" ] || printf "Headline missing 'net/' prefix:\n$bad\n" > > This check is definitely a good idea. > We need the same thing for crypto. > Few nits: > - -E is not needed for this regex > - "/base:" cases are not handled because of the colon at the end > of the regex. Good point. Easy enough to fix, I suspect. > - patches touching several drivers are not handled. Examples: > "mlx:" -> "net/mlx:" > "drivers/net:" -> "net:" > Yes, however, those are in the minority of cases, and we can always add special case extra regex's for those - hardcoded like the other regex's in the file. I did the pulling directory list so we don't have to update the checks every time a new driver was added. > By the way, I am looking at an idea, that Nelio sent me, based on > modified files. It can complete this check. Great. I'll look forward to seeing that. If it works out well, by all means drop this patch. If it's not coming any time soon though, I can do a V2 of this patch, but I won't bother right now in the expectation that a better solution is coming. /Bruce