From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <david.hunt@intel.com>
Received: from mga14.intel.com (mga14.intel.com [192.55.52.115])
 by dpdk.org (Postfix) with ESMTP id 4E3642B96
 for <dev@dpdk.org>; Thu,  6 Jul 2017 12:39:23 +0200 (CEST)
Received: from orsmga004.jf.intel.com ([10.7.209.38])
 by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;
 06 Jul 2017 03:39:22 -0700
X-ExtLoop1: 1
X-IronPort-AV: E=Sophos;i="5.40,316,1496127600"; d="scan'208";a="105076606"
Received: from dhunt5-mobl.ger.corp.intel.com (HELO [10.237.221.55])
 ([10.237.221.55])
 by orsmga004.jf.intel.com with ESMTP; 06 Jul 2017 03:39:20 -0700
To: Jerin Jacob <jerin.jacob@caviumnetworks.com>
References: <1498751388-41571-2-git-send-email-david.hunt@intel.com>
 <1498830673-56759-1-git-send-email-david.hunt@intel.com>
 <1498830673-56759-2-git-send-email-david.hunt@intel.com>
 <20170703035755.GA6275@jerin>
 <25452a77-c5ae-97e8-b41a-5dfcb9fb19a6@intel.com>
 <20170705052853.GA8031@jerin>
 <dc8aa64b-a5ad-f52a-1173-0be9a5af8ccb@intel.com>
 <20170706033120.GA10973@jerin>
 <7987361f-935e-a689-103d-276fe3a4c36e@intel.com>
Cc: dev@dpdk.org, harry.van.haaren@intel.com, Gage Eads
 <gage.eads@intel.com>, Bruce Richardson <bruce.richardson@intel.com>
From: "Hunt, David" <david.hunt@intel.com>
Message-ID: <5693accf-67e0-cf97-dea5-dae7d21ff971@intel.com>
Date: Thu, 6 Jul 2017 11:39:19 +0100
User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:45.0) Gecko/20100101
 Thunderbird/45.8.0
MIME-Version: 1.0
In-Reply-To: <7987361f-935e-a689-103d-276fe3a4c36e@intel.com>
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [dpdk-dev] [PATCH v5 1/3] examples/eventdev_pipeline: added
 sample app
X-BeenThere: dev@dpdk.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
X-List-Received-Date: Thu, 06 Jul 2017 10:39:23 -0000



On 6/7/2017 11:04 AM, Hunt, David wrote:
>
>
> On 6/7/2017 4:31 AM, Jerin Jacob wrote:
>
>> Note:
>> Change application to new name.
>>
>> checkpatch.sh
>> -----------------
>>
>> WARNING:EMBEDDED_FUNCTION_NAME: Prefer using '"%s...", __func__' to
>> using 'consumer', this function's name, in a string
>> #294: FILE: examples/eventdev_pipeline_sw_pmd/main.c:178:
>> +        printf("# consumer RX=%"PRIu64", time %"PRIu64 "ms, "
>>
>> WARNING:EMBEDDED_FUNCTION_NAME: Prefer using '"%s...", __func__' to
>> using 'worker', this function's name, in a string
>> #453: FILE: examples/eventdev_pipeline_sw_pmd/main.c:337:
>> +        printf("  worker %u thread done. RX=%zu TX=%zu\n",
>>
>> total: 0 errors, 2 warnings, 1078 lines checked
>
> These are false positives. The text in the messages are not meant to 
> be the function name.
> If anything, I would prefer to change the function names to have " 
> _thread"?
>


Or perhaps, better still, change the function names to verbs, i.e. 
produce() consume(), do_work().

Regards,
Dave.