From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 0E8C1A0A02; Sat, 17 Apr 2021 09:42:38 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 8BAAC4068F; Sat, 17 Apr 2021 09:42:37 +0200 (CEST) Received: from szxga05-in.huawei.com (szxga05-in.huawei.com [45.249.212.191]) by mails.dpdk.org (Postfix) with ESMTP id 9497740143 for ; Sat, 17 Apr 2021 09:42:35 +0200 (CEST) Received: from DGGEMS406-HUB.china.huawei.com (unknown [172.30.72.59]) by szxga05-in.huawei.com (SkyGuard) with ESMTP id 4FMlMs2Lr2zPrTT; Sat, 17 Apr 2021 15:39:37 +0800 (CST) Received: from [10.67.103.128] (10.67.103.128) by DGGEMS406-HUB.china.huawei.com (10.3.19.206) with Microsoft SMTP Server id 14.3.498.0; Sat, 17 Apr 2021 15:42:29 +0800 To: Stephen Hemminger , Kevin Traynor CC: , , , References: <1618046334-39857-1-git-send-email-humin29@huawei.com> <1618555931-25858-1-git-send-email-humin29@huawei.com> <20210416092853.39a1517e@hermes.local> From: "Min Hu (Connor)" Message-ID: <61364ea0-0f43-8114-da4e-29d671868c0e@huawei.com> Date: Sat, 17 Apr 2021 15:42:29 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.3.1 MIME-Version: 1.0 In-Reply-To: <20210416092853.39a1517e@hermes.local> Content-Type: text/plain; charset="gbk"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.67.103.128] X-CFilter-Loop: Reflected Subject: Re: [dpdk-dev] [PATCH v6] ethdev: add sanity checks in control APIs X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" ÔÚ 2021/4/17 0:28, Stephen Hemminger дµÀ: > On Fri, 16 Apr 2021 11:22:02 +0100 > Kevin Traynor wrote: > >>> + if (dev_conf == NULL) { >>> + RTE_ETHDEV_LOG(ERR, >>> + "Cannot configure ethdev port %u to NULL dev_conf\n", >> >> The others use a natural sounding names instead of argument name. If you >> wanted to match that it could be "..to NULL conf" > > I would prefer that error messages don't try to be English sentences. > The wording ends up awkward. and overly wordy. > If function name is automatically included by RTE_ETHDEV_LOG() then > Just: > RTE_ETHDEV_LOG(ERR, "NULL ethdev") > should be enough for programmer to find/fix the problem > . Hi, Stephen, thanks your for comment. But I am on side with Andrew Rybchenko. "- log messages should be human readable (i.e. I'd avoid usage of function name)" >