From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 63FDCA04BC for ; Thu, 17 Sep 2020 13:50:16 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 251941D634; Thu, 17 Sep 2020 13:50:16 +0200 (CEST) Received: from EUR01-DB5-obe.outbound.protection.outlook.com (mail-eopbgr150070.outbound.protection.outlook.com [40.107.15.70]) by dpdk.org (Postfix) with ESMTP id 443061D62D; Thu, 17 Sep 2020 13:50:13 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=kvD8t/wrJfIuVlNtiobcy6gfB/WOEFwLfVNBlTKUymjGZb++CLnAxAU6YLMsxB7zOBgjnREIcpUKOr40e9ezeqOnBKk2cCGP6seL2JcP/990PSRteCIDdgTzGhE8awDmqtE6okuqChou6WPGHNh67GB9/OUmX7sx1eEjsbyD/Ymo0Z+MP2U5Ci3/yGupeKBMv9JHcIDjfrzYZT9nUFPmwOrB1esJouAS1TEOJ0BYFUYNLlomkXFyS3CVwKAY+o2iGAaKzfB0W7p5/4jSwnHT5MtJHEuxGOZ+S3DyBXYnO75uuwJEbaSLsbeffKP3Pqcm4xzYTzuyFzunazUPUv9eIg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=qh8jnKmOSwUYd+/zERhrcYt30cvssjUZh55xWwkE3uY=; b=WWtMeNFZw1ocmu1xxVNCerHyOMfqYHIp3AAfdVnzLMfLaIeK2A1OLLjdWRCCfgmIeQ1u0pcl2t0ZeOosM5Y6kvjleXu6azHVCl1xgHKNPplez7htXXxb1zncDDVFLhBgvPk/6lG3BW2J7vatGQUZljl+2WWqzgytT/K/osK+fKGRBc2lWOfPWCzbttb6dfXlPur+02Y0I/oWDqPaIuafvfsFxcNJr0eArUwkv7d81taWfB2iLbRwY/ZqHcBXfKN2pDqUIF1bU37PBH7+Tlb5cGD3L0dR3eC1RMbUJK0zJvFlnEv3jdwMYy4tAjuAoZCl7RbMT8HPYJtilZW7CfyLpg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=oss.nxp.com; dmarc=pass action=none header.from=oss.nxp.com; dkim=pass header.d=oss.nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=NXP1.onmicrosoft.com; s=selector2-NXP1-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=qh8jnKmOSwUYd+/zERhrcYt30cvssjUZh55xWwkE3uY=; b=XoLce2yR7kW2rfX+iEnhhp39INsiG/T63J0O3aAgIvZIo5mWI3W7nSYgN3Ese+cHpNsz9m2XpE6nYqSQ75eIGfeXL4E7TCxetGBCwMwXsMtcoIE2O92GJQdSd8YIk2jO1I0/Hfsr5wp/vm6Srakm7qbMEeTwe0GGw5mkHJyxhU0= Authentication-Results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=oss.nxp.com; Received: from AM6PR04MB4456.eurprd04.prod.outlook.com (2603:10a6:20b:22::25) by AM7PR04MB6774.eurprd04.prod.outlook.com (2603:10a6:20b:104::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3391.15; Thu, 17 Sep 2020 11:50:12 +0000 Received: from AM6PR04MB4456.eurprd04.prod.outlook.com ([fe80::e024:2be5:29c3:31b0]) by AM6PR04MB4456.eurprd04.prod.outlook.com ([fe80::e024:2be5:29c3:31b0%7]) with mapi id 15.20.3391.014; Thu, 17 Sep 2020 11:50:12 +0000 To: Ferruh Yigit , "Di, ChenxuX" , "hemant.agrawal@nxp.com" Cc: "sachin.saxena@nxp.com" , "stable@dpdk.org" , "dev@dpdk.org" , "Richardson, Bruce" References: <20200915024055.72103-1-chenxux.di@intel.com> <974e79a6-6a52-c84c-6ae2-cf802e551444@intel.com> <2ada3c33bf2b4dcf862e6fa4e6a97cbc@intel.com> <59ceb607-4ffc-550e-45bf-293b97248ff3@intel.com> From: Hemant Agrawal Message-ID: Date: Thu, 17 Sep 2020 17:20:03 +0530 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Thunderbird/78.2.2 In-Reply-To: <59ceb607-4ffc-550e-45bf-293b97248ff3@intel.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US X-ClientProxiedBy: SG2PR02CA0066.apcprd02.prod.outlook.com (2603:1096:4:54::30) To AM6PR04MB4456.eurprd04.prod.outlook.com (2603:10a6:20b:22::25) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 Received: from [192.168.1.2] (122.176.3.89) by SG2PR02CA0066.apcprd02.prod.outlook.com (2603:1096:4:54::30) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3370.16 via Frontend Transport; Thu, 17 Sep 2020 11:50:10 +0000 X-Originating-IP: [122.176.3.89] X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-HT: Tenant X-MS-Office365-Filtering-Correlation-Id: 3708bcc1-f3f1-417a-f1fc-08d85affd5bf X-MS-TrafficTypeDiagnostic: AM7PR04MB6774: X-MS-Exchange-SharedMailbox-RoutingAgent-Processed: True X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:7691; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 8p7UZK7VB+DJCTMC0bi4Tu51I47gFpfW7vD940+/DpUlq3HKGvJtk6PtVePZwMEmKbyGEYq6GBQQRkwhnOJOO3hTjT45e0Lzwydjr9zW2T8ORdQOS7JGMJlJGSi8hBnXTmfpk5CNWcbgvV7fpJqndAPAd3e4ziEfy0IMoh17Ln3dufQR2VWELUtWRAlXfikW6SfqhWD//2JoVq4QO9gCIn9suAI/Flzdj9Dn1exbOKMZOrrmklKxhU+Q69Vc4FbX4DFs78FlzZox9cpxZGspcXcWqXfaX6s4xNxxyLwwyKF/VX+Z3o+Ew6G41pihMp8EcZr8ra0rl5YzNOJOxhGEfJTp4laA3VkEpk413H+mCFLcSQ0dpHfX5mXiSqwSP5PK X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM6PR04MB4456.eurprd04.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(376002)(346002)(136003)(396003)(39850400004)(66556008)(66476007)(66946007)(86362001)(478600001)(16576012)(6486002)(316002)(44832011)(956004)(8936002)(8676002)(2616005)(2906002)(4326008)(31696002)(83380400001)(31686004)(110136005)(52116002)(6666004)(54906003)(5660300002)(26005)(53546011)(16526019)(55236004)(186003)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData: Sp3FdiX0+QVbYoM5bJUtxgvS4337+5Du/3O75+NrmG7M/j+BWBdiZ03tI/isyQTg9JK7J9FakeUdOCtFX4InKW3jTqHgEClzjPF4CGzPIBMDKnbE8rVX0bDsKgrv7PJ2inQyr+/Fao56kVRpljVMbHZmFuHg1yLdQbRh/CvWZlBFK1as/EismhrxIWrtKOn5HMxSQq0MJ1MAE1r/pKCBgMxW9DcZqllv853LY4Z2z1CDBYW12nXZ4kTisRV3ItCaO2cD08m1EM8dClNwIHgDhcqQYGur06bZR4lgRQ3nAOlNd0oFfKz4wD74UZH/7dxi3MBK7nP6/6g1AmBsIkBtV979L3JfVIjNU5GPm7zndrZg/EkU/Eehg/PBGQFdXmwHiNBP+0xcmFe3bSahOliDhySr7vOCBpTuG/pLHWw/UhgVCU7Sz3aaMKOHaZxmN7jT+m1B7NKkIqW4MFN1huT2x/JYHK1MT9qru3ox3At7yy++zPYTUtpyLBKlVdwv/Z3P9E0aQQRFtGxkHBTLp4SoUBctTmBlPiajsxs1X90IkkTJi5tGFL6TgcHW6PM+BJSTgzeG4OpJJUXWCHSPiHSGOhqKdz7w1f1Qrha2v71Sx9fvfQdqrzyjyRxWY0fmJuu2ipicXa/uR7RdI6zkVzw5tA== X-OriginatorOrg: oss.nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 3708bcc1-f3f1-417a-f1fc-08d85affd5bf X-MS-Exchange-CrossTenant-AuthSource: AM6PR04MB4456.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Sep 2020 11:50:12.4550 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: CN1mDXAx+KEPgzvHTw/Ecjo3Dp608OdQmz4qc+K2Wo89oXLtz2RV01AZE05QB9QYW0QlbD1wNXWKmgL4IWhQRg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM7PR04MB6774 Subject: Re: [dpdk-stable] [dpdk-dev] [PATCH] net/dpaa2: fix build error about timesync functions X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Sender: "stable" On 9/17/2020 5:08 PM, Ferruh Yigit wrote: > On 9/17/2020 3:03 AM, Di, ChenxuX wrote: >> Hi, >> >>> -----Original Message----- >>> From: Ferruh Yigit >>> Sent: Wednesday, September 16, 2020 11:29 PM >>> To: Di, ChenxuX ; hemant.agrawal@nxp.com >>> Cc: sachin.saxena@nxp.com; stable@dpdk.org; dev@dpdk.org; Richardson, >>> Bruce >>> Subject: Re: [dpdk-stable] [PATCH] net/dpaa2: fix build error about >>> timesync >>> functions >>> >>> On 9/15/2020 3:40 AM, Chenxu Di wrote: >>>> When the build option has '-DRTE_LIBRTE_IEEE1588=1', the announce of >>>> timesync functions will be build. >>>> However the dpdk_conf doesn't hav RTE_LIBRTE_IEEE1588 so that the file >>>> dpaa2_ptp.c will not be build. >>>> It cause the build error. >>>> This patch fixes it by adding set for dpdk_conf. >>>> >>>> Fixes: 184c39d16568 ("net/dpaa2: add DPRTC sub-module") >>>> Cc: stable@dpdk.org >>>> >>>> Signed-off-by: Chenxu Di >>>> --- >>>>    drivers/net/dpaa2/meson.build | 4 ++++ >>>>    1 file changed, 4 insertions(+) >>>> >>>> diff --git a/drivers/net/dpaa2/meson.build >>>> b/drivers/net/dpaa2/meson.build index 6dd0eb274..d9aadfdae 100644 >>>> --- a/drivers/net/dpaa2/meson.build >>>> +++ b/drivers/net/dpaa2/meson.build >>>> @@ -17,6 +17,10 @@ sources = files('base/dpaa2_hw_dpni.c', >>>>            'mc/dpdmux.c', >>>>            'mc/dpni.c') >>>> >>>> +if '-DRTE_LIBRTE_IEEE1588=1' in get_option('c_args') >>> >>> The "RTE_LIBRTE_IEEE1588=1" can fail, >>> all places looking for "#ifdef RTE_LIBRTE_IEEE1588", so a "-Dc_args=- >>> DRTE_LIBRTE_IEEE1588" is more likely, but why not "-Dc_args=- >>> DRTE_LIBRTE_IEEE1588=666" >>> >> >> Yes, I will change it >> >>>> + dpdk_conf.set('RTE_LIBRTE_IEEE1588', 1) endif >>>> + >>>>    if dpdk_conf.has('RTE_LIBRTE_IEEE1588') >>>>        sources += files('mc/dprtc.c') >>>>        sources += files('dpaa2_ptp.c') >>>> >>> >>> Can't we just remove the conditional build: >>> >>>    -if dpdk_conf.has('RTE_LIBRTE_IEEE1588') >>>    -       sources += files('mc/dprtc.c') >>>    -       sources += files('dpaa2_ptp.c') >>>    -endif >>>    +sources += files('mc/dprtc.c') >>>    +sources += files('dpaa2_ptp.c') >> >> The announce of timesync functions are in the #define >> DRTE_LIBRTE_IEEE1588 >> While the define of the functions are in the file 'dpaa2_ptp.c'. >> So they should be both build or not build by whether the build option >> -DRTE_LIBRTE_IEEE1588=1 or not. >> So it seems not a good idea that remove the conditional. >> > > timesyncs_* dev_ops functions defined but not used is not big problem, > only can increase the library size. > > I believe more concern is on: > 'RTE_PMD_REGISTER_DPAA2_OBJECT(dprtc, rte_dpaa2_dprtc_obj);' > which looks like register function to run in constructor when > 'dpaa2_ptp.c' is compiled, but not sure affect of it. > It can be possible to wrap that call with 'RTE_LIBRTE_IEEE1588' ifdef, > that should work. > > It is preferred to remove the compile time flag instead of finding > ways to make it work. > > Let's wait for the dpaa2 maintainers' response, perhaps they can come > with a smart way to remove the compile time flag. Hi Ferruh,    enabling  IEEE1588 causes some performance drop in the dpaa2 performance. That is the reason, we have kept this code in compile time flag. However, we will work in future to make it run-time configurable but that will require some code restructuring and it will be a moderate size work. Regards, Hemant