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 DF46DA034F; Wed, 31 Mar 2021 13:00:29 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 75F0B140E6C; Wed, 31 Mar 2021 13:00:29 +0200 (CEST) Received: from szxga04-in.huawei.com (szxga04-in.huawei.com [45.249.212.190]) by mails.dpdk.org (Postfix) with ESMTP id 6CF96406A3 for ; Wed, 31 Mar 2021 13:00:27 +0200 (CEST) Received: from DGGEMS411-HUB.china.huawei.com (unknown [172.30.72.59]) by szxga04-in.huawei.com (SkyGuard) with ESMTP id 4F9Nb02PDRz19KC7; Wed, 31 Mar 2021 18:58:20 +0800 (CST) Received: from [10.67.103.128] (10.67.103.128) by DGGEMS411-HUB.china.huawei.com (10.3.19.211) with Microsoft SMTP Server id 14.3.498.0; Wed, 31 Mar 2021 19:00:23 +0800 To: Ferruh Yigit , Thomas Monjalon CC: , Andrew Rybchenko References: <1616748961-11239-1-git-send-email-humin29@huawei.com> <751cf194-7de2-4f9b-3f1a-fd2ad9c941f2@huawei.com> <26287890.AfWHT1PW0D@thomas> From: "Min Hu (Connor)" Message-ID: <4e244417-a55e-a472-7aff-a3129cdf0c49@huawei.com> Date: Wed, 31 Mar 2021 19:00:24 +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: Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 8bit X-Originating-IP: [10.67.103.128] X-CFilter-Loop: Reflected Subject: Re: [dpdk-dev] [PATCH 2/2] net/hns3: support IEEE 1588 PTP 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/3/31 17:26, Ferruh Yigit 写道: > On 3/31/2021 8:28 AM, Thomas Monjalon wrote: >> 31/03/2021 04:35, Min Hu (Connor): >>> 在 2021/3/30 21:59, Ferruh Yigit 写道: >>>> On 3/26/2021 8:56 AM, Min Hu (Connor) wrote: >>>>> Add hns3 support for new ethdev APIs to enable and read IEEE1588/ >>>>> 802.1AS PTP timestamps. >>>>> >>>>> Signed-off-by: Min Hu (Connor) >>>>> --- a/drivers/net/hns3/hns3_cmd.h >>>>> +++ b/drivers/net/hns3/hns3_cmd.h >>>>> @@ -123,6 +123,12 @@ enum hns3_opcode_type { >>>>>        HNS3_OPC_CLEAR_MAC_TNL_INT      = 0x0312, >>>>>        HNS3_OPC_CONFIG_FEC_MODE        = 0x031A, >>>>> +#ifdef RTE_LIBRTE_IEEE1588 >>>>> +    /* PTP command */ >>>>> +    HNS3_OPC_PTP_INT_EN             = 0x0501, >>>>> +    HNS3_OPC_CFG_PTP_MODE           = 0x0507, >>>>> +#endif >>>>> + >>>> >>>> Hi Connor, >>>> >>>> Does it needs to be a compile time configuration? What happens if it is >>>> always enabled, or controlled by device argument? >>>> . >>> Hi Ferruh, >>>     Firstly the "RTE_LIBRTE_IEEE1588" origins from the config file in >>> DPDK. >>> Almost every nic driver use this macro in compile time. >>>     For me, I think using this macro give one option for users to >>> decide if his APPs contains this module. For example, in loT field, >>> some microprocessor has small memory or small disk, So the APPs should >>> be as small as possible. So, if user does not need "PTP", the APPs no >>> need to contain it. >>>     Well, another top, if is always enabled, for HNS3 PMD, it will >>> work well for our nic. If user want to use "PTP", just call API. If user >>> does not use it, it also doesn't matter. But we advise that if user >>> don't need this function, just turn it off. >>>     Thanks. >> >> Disabling at compile-time does not reduce the footprint significantly. >> RTE_LIBRTE_IEEE1588 should disappear, so I advise not using it >> in new code. Instead, you could enable/disable at runtime if needed. >> > > I am aware that 'RTE_LIBRTE_IEEE1588' already exists and used by some > drivers, but as you said they are from times we had a config option for > it, for the new support I believe it is better to have runtime > configuration. > > And I agree with Thomas that it shouldn't increase the footprint much to > always compile the support in, if you have numbers please share. > Well, thanks Thomas, Ferruh, If there is no help to reduce the footprint significantly, I agree to delete "RTE_LIBRTE_IEEE1588". As for runtime config, there is no need to enable or disable PTP. Because We support the function, if user want to use it, just call related APIs, if not, don't use it. So, I will delete the "RTE_LIBRTE_IEEE1588" in v3, what's your opinion? .