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 22AB541B93; Tue, 31 Jan 2023 23:20:37 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id ED43A40684; Tue, 31 Jan 2023 23:20:36 +0100 (CET) Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2080.outbound.protection.outlook.com [40.107.223.80]) by mails.dpdk.org (Postfix) with ESMTP id 021324067B for ; Tue, 31 Jan 2023 23:20:35 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=UlW+zESONQA+V8AGj9lO5tkyooDfTJoBBMQ0y1fyBrtp4dE4FSKO5BACLFGxNLA66Oey6NXprIMRZlJxpoS6LZOYydMaZF9jH9b1nNn0HQ3NsQP8HgQLYWFSQjpBpEZnQqAH5r3OZ7SrZ3HrJNExPROACf//6ruhFTiwFh8t9HPDg2fH2qTrNvd0KKsGJYIcKYOkEV1hpGWkvuIByVP4OMVPlX1wkmdbEA83oO4yXoaYEIZkyZLC1gGb03evYA17DNCz3wAvBvZUfQlYoiThzr0ng6T8wf2lvWb2k2F+rRsUjqEN9t0VuyrhMrNMhndWr17jdbMGv1HUwuITU1RkPw== 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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=k+nM5VlxfzbBIusrc+M8HwESooXz17fkvYLIDWzhCcQ=; b=TO13DYO86+2wNl/WlLQm3b70yDklQUQ7c9bsckUfYQxSW8rUtZvjC6KBXAfY9zsVB1N0sOUpOVGauJskeJ2XXUXp7igPqAoDfjgt01IsZ9JxRUHPOr5zSnqoYIquH9Kk5sMJjMvZ+WDVfuJ3gDIuGlL5bRt9KM8owSCA/BuMaoLnghRlAbUohs32Fbdj6VY5vAMvb0GDYoUjww5amV6MLA10wQuk+gMOBnC20bIdVr4IDfdRN2u0c7MA1Q0tJl/qmZd/XMejOtOalk3mMwjg56M4hUNBZjTCmjNNxfWpP8gwdacVOJLgthjle/J/wrHuJXPIa9YGgiPWCh2Fiu1ihA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=amd.com; dmarc=pass action=none header.from=amd.com; dkim=pass header.d=amd.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=amd.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=k+nM5VlxfzbBIusrc+M8HwESooXz17fkvYLIDWzhCcQ=; b=ZhzYr9KvWo/ttbjVBWpsiTpPMzhd10Bl62tSYYjCx0ZrCovJiRw+MrGF9Mon67QbYSG+3dXrLbvBQU20DEQzMWxJg8DHCLdTNSF31G7qe67+BI5OrFJNZLPlqSFyIKxW3CV/DUDKoZqt7PvkZfT3+He5AXMzlS1Zk6p6BRqx9ZI= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=amd.com; Received: from CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) by CY8PR12MB7492.namprd12.prod.outlook.com (2603:10b6:930:93::9) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6043.38; Tue, 31 Jan 2023 22:20:32 +0000 Received: from CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::4807:1f44:5e04:e05a]) by CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::4807:1f44:5e04:e05a%8]) with mapi id 15.20.6043.038; Tue, 31 Jan 2023 22:20:32 +0000 Message-ID: <654437f5-2ee7-9d9e-adc9-20c6d4563242@amd.com> Date: Tue, 31 Jan 2023 22:20:16 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.6.1 Content-Language: en-US To: Jerin Jacob Cc: Ankur Dwivedi , "dev@dpdk.org" , David Marchand , Jerin Jacob Kollanukkaran , Andrew Rybchenko , "thomas@monjalon.net" , "mdr@ashroe.eu" , "orika@nvidia.com" , "chas3@att.com" , "humin29@huawei.com" , "linville@tuxdriver.com" , "ciara.loftus@intel.com" , "qi.z.zhang@intel.com" , "mw@semihalf.com" , "mk@semihalf.com" , "shaibran@amazon.com" , "evgenys@amazon.com" , "igorch@amazon.com" , "chandu@amd.com" , Igor Russkikh , "shepard.siegel@atomicrules.com" , "ed.czeck@atomicrules.com" , "john.miller@atomicrules.com" , "ajit.khaparde@broadcom.com" , "somnath.kotur@broadcom.com" , "Maciej Czekaj [C]" , Shijith Thotton , Srisivasubramanian Srinivasan , Harman Kalra , "rahul.lakkireddy@chelsio.com" , "johndale@cisco.com" , "hyonkim@cisco.com" , "liudongdong3@huawei.com" , "yisen.zhuang@huawei.com" , "xuanziyang2@huawei.com" , "cloud.wangxiaoyun@huawei.com" , "zhouguoyang@huawei.com" , "simei.su@intel.com" , "wenjun1.wu@intel.com" , "qiming.yang@intel.com" , "Yuying.Zhang@intel.com" , "beilei.xing@intel.com" , "xiao.w.wang@intel.com" , "jingjing.wu@intel.com" , "junfeng.guo@intel.com" , "rosen.xu@intel.com" , Nithin Kumar Dabilpuram , Kiran Kumar Kokkilagadda , Sunil Kumar Kori , Satha Koteswara Rao Kottidi , Liron Himi , "zr@semihalf.com" , Radha Chintakuntla , Veerasenareddy Burru , Sathesh B Edara , "matan@nvidia.com" , "viacheslavo@nvidia.com" , "longli@microsoft.com" , "spinler@cesnet.cz" , "chaoyong.he@corigine.com" , "niklas.soderlund@corigine.com" , "hemant.agrawal@nxp.com" , "sachin.saxena@oss.nxp.com" , "g.singh@nxp.com" , "apeksha.gupta@nxp.com" , "sachin.saxena@nxp.com" , "aboyer@pensando.io" , Rasesh Mody , Shahed Shaikh , Devendra Singh Rawat , "jiawenwu@trustnetic.com" , "jianwang@trustnetic.com" , "jbehrens@vmware.com" , "maxime.coquelin@redhat.com" , "chenbo.xia@intel.com" , "steven.webster@windriver.com" , "matt.peters@windriver.com" , "bruce.richardson@intel.com" , "mtetsuyah@gmail.com" , "grive@u256.net" , "jasvinder.singh@intel.com" , "cristian.dumitrescu@intel.com" , "jgrajcia@cisco.com" , "mb@smartsharesystems.com" References: <20230112112140.807233-1-adwivedi@marvell.com> <20230120084059.2926575-1-adwivedi@marvell.com> <20230120084059.2926575-3-adwivedi@marvell.com> <614e22c6-8485-0e8d-742e-b3d100f96468@amd.com> <91a635b5-3fe5-b47d-d8ba-44f9b1614bf9@amd.com> From: Ferruh Yigit Subject: Re: [EXT] Re: [PATCH v6 2/6] ethdev: add trace points for ethdev (part one) In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-ClientProxiedBy: LO2P123CA0086.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:138::19) To CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR12MB4294:EE_|CY8PR12MB7492:EE_ X-MS-Office365-Filtering-Correlation-Id: ebe4b8ef-004c-4e6f-8573-08db03d95db3 X-LD-Processed: 3dd8961f-e488-4e60-8e11-a82d994e183d,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: FRqWScBOzKrW+mnZzF33OoDBFB2wXWQv9Qmg0gO2r3152S20XmGbvZmABQCyBjEnZIc2q3mERAzN4ETZ6/kTlHo3AGBkct4tDdElOPIC0WncYosDGlZx+OjOrgbI3P6X7CrMc1JgE42w2Wqoq4lTSezF4ZeAOfO26WYCu5wEMeT3q/zPVtHuhz6IgSk4QrfBeQVFxnPnd1ojsS21JgfVPSYR9O2AnZE9ra9T7RG3BwbzKSC4DAFhGfrzFDvMGsX5pGfjeqRFRMhhENmJzaa5hq7ZZyNQI8i00v0yjmz5pF2nOvCYxtWjBvcZT3LPC8aJimRM4UqVi52hRWC2dTUUOIKPUnkYo74QJYMNYYfUevU8Bk25zpB7D8GtAECcXNvyg4TpdjuaSY2QoxjP5Ic/mzMYcmowfVJd+SCxHTRV0im0GeQP/ih+H4O7KnzQm5a079XU/DegHF88dHBy8Y5Ak51Ec2lms8NRAYut6kLEJGVkiVfoeiorU0GFOFDqzAtK+/l6Myj1517Vfw+btNlJr8PjS3AJQpZgbC5SoVUskvr25CsdVENfYsJhllTREi9f7+HCuzNHI58DhD05lBFIs7k+0iz/EdgRQK/Otx4l5cQS4OnVeBtUgOEjVjBMJ5ltUNoKsfFfhHpGASVYf0nCgf2V63qKQjkcgTGMs5gaBFjlSWC61O3caWle6S5vcBDPOrvonardo5sXX3zpeRm8d151JgL80vb2ERLoo1SbcJs= X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CH2PR12MB4294.namprd12.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230025)(4636009)(39860400002)(376002)(346002)(396003)(136003)(366004)(451199018)(86362001)(38100700002)(66556008)(316002)(31696002)(7406005)(36756003)(66946007)(7416002)(8936002)(5660300002)(7336002)(7366002)(41300700001)(54906003)(4326008)(66476007)(6916009)(8676002)(2906002)(44832011)(2616005)(6486002)(478600001)(186003)(26005)(6506007)(6512007)(53546011)(6666004)(31686004)(45980500001)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ejhUNUFxdXJ3V3hJTm1vMzVVN2lNTXRTakw0SEF5cUdDYkgwSUNYY2NsZy9q?= =?utf-8?B?SHozeDNMVmdwZCtkbHIwY0xyYkxqSzdsOGd4N3Nnd0doeFR6ci9QSVBjK3pa?= =?utf-8?B?a0hreVh6RkVLUCtwdi85MEJwTVFjK0NVV2JFWmp1amhpcW9hYkprcWFONTY1?= =?utf-8?B?MFZ1bmpvOGVGZVdxQksrT3NEWExwejlMTGJwMDFRSERoQXFSWDJSSFFDVnQw?= =?utf-8?B?d1E4ajBvL1RDL2d1TkFvUWEreXFxMUVMY1hpK2RBSDlza0NEUTNvUGVEZ0R6?= =?utf-8?B?NlVnSVhRaUc0ZXF4VFZLVk9RU2prUFUyNFZxN0R5bmlhSTlnOFV6NXpPazJZ?= =?utf-8?B?OWFjYzBERkdNRkhERWd5NGdPSTRkYWF6SkdmTllnRUNNdWFEN2xmRldUMTNF?= =?utf-8?B?L1VMc1krMzNReHd2WHpiRHE5OEc5UTJCTVh3UmtrZ2YxNnlweVZWMG1OZzJR?= =?utf-8?B?Y1hyVnV0Yk5ENElVY29USTA3RVZONFZnL2RKbjFMUTRDOWNpdEo4eDBQQlZW?= =?utf-8?B?bElkbkdILzRVank2Z0pjOVhyTVlrRVRhd3RFUDlPNzNIOVJMVnlPMTFqR1FF?= =?utf-8?B?TDUzaVh5TFk3enFhY1lxNDMxbUxrcHlGYUNmeWUxN2xvRjhQc21sRjEzVDVC?= =?utf-8?B?eDZvRWJhNUMzSTBKZDBTUmtWbWpVQksxTm5nazBla0dMRHQ0c0RJaHpSM0dI?= =?utf-8?B?aXU4bldHWExDenlGeHRtWEYyNFFMelRvM1M1R2RTV2tTS3MzSCtmNzVFclpW?= =?utf-8?B?ZjJEQzh0QTZJZ1JIN0Z6MmtOaWE4QWtaR09wSjF3UFRnY3Y0QjRYZTYzZEpH?= =?utf-8?B?VDUwWVo0TW01SFJ3cUQwN0k0c0FieE5Sem5LdFJZVXpoZEk3Ky9qRVlYMEd6?= =?utf-8?B?VmFZTVU0dy9KU1Ywb2ZQemhvbHFYcnpxWTcvSFF2RFhiamJETFJCamQxdkhv?= =?utf-8?B?RU5IMVQzSzJLN3RnbExVTTlxeGdsaHFjQVZVZVc4bEUyOTdEdWZ1MlRwdTlS?= =?utf-8?B?N2lmL2YwSFNpY1g5bE9HVVRaanpWS2FzVkRkRGJ5WUNEbmM0b1RyMzdSSUYz?= =?utf-8?B?dlpOb1pFMW9FVG5xN043NzNCYlFkOXE1bDJrVFZnWWUyMVVUU2hpV0RFOHBP?= =?utf-8?B?NWt0NU5nQ1orcGFSS2F5aHZBOEZWRzZvNktnNnJna210MjM5TlhxdTNSdTlG?= =?utf-8?B?eFRveU1yVy9uTXlsNEttNUFxLzFoY09oTnRac2liaHdvQmpmM05Hd0M4UlFq?= =?utf-8?B?SVZ2Y0R4OVpYRUhQL2pnMW9USGl6TENUV2ZtcUswTHpOZnl2QmQreXJNbG5p?= =?utf-8?B?VjhscW9RNjlZeDc4US9WYmc2VVU1NEo2SWpTTUNEUElXVFovdEU5T0FnNGIw?= =?utf-8?B?NGhBK2ZJM2ZwVUtkdGRPSzhMZ1llZ2x0U2ZiV1lDeVhSRTdST3ZBTml6YnFM?= =?utf-8?B?NThSMFhTUzM0N3dpa0pYVFJwZnRxSktpUXFYL01ocTdEcGdxSzh0K3RITVlZ?= =?utf-8?B?dEhkV3JSSlRMbG5WSzg3MUN5MzF6WGthN3RTN2tiaUNvcG9IQy93RWEvZlBM?= =?utf-8?B?TVBrTUhLNnpsZzlIRmI1U2tpTzhBcC9IWjZVaDJaYnVvMVFrVThYaTB5Rm5W?= =?utf-8?B?UzYvZzVJNFRQbUI1VHFYZjJSa25wSThFcWZocHpyV1lMaXNRNllGbGpjTHZT?= =?utf-8?B?bFBFOUlBQVBieGsyNXVBVXhSdUthTVNPdyt0TjBoSHN5UWphc1NleVRNaktJ?= =?utf-8?B?c0JBVGc3QkFDeVBlU1N3WGt0OFg3bHlQUG1Nd0w0RGpiK2x0TEpmRjl5ZFJH?= =?utf-8?B?VnlRdEwrL3dzY01teXlLN0MrWHpMMXQ3U1dlcDN3WTJwc2pHRWE2cWxjRnl1?= =?utf-8?B?VlVXRkRPdHEwc0NDQlAvQVJOMXJIREtZT3dzdmp0VjgxUWorM0I3Ny9QMXJY?= =?utf-8?B?SVlOMjhSSVRDQ2FjeUQyNWtQZEZjQTl3NThodkpYbHdvQklZSWJRNjlFdjht?= =?utf-8?B?SEo3REE4Z0ZnOVZKamVYdUpqbDJtWE9CTnphKzd2M1NJN2ZqNFpjU3l2Rkth?= =?utf-8?B?Yy9oOWNvWHlkWHJTMGNRVjBiY0Z0ZDB2blF6VDhPdWxoL0d5N1dUZUNubm1H?= =?utf-8?Q?r1bFELX+Nhs8drG9oX0dcI/il?= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: ebe4b8ef-004c-4e6f-8573-08db03d95db3 X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB4294.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Jan 2023 22:20:32.2433 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: en++aDIIQLIhynZElflptzEYy6GBxlgXe+/W1xKCSrBG30rbFEbYaPaorvAelojS X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR12MB7492 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 On 1/31/2023 6:46 PM, Jerin Jacob wrote: > On Wed, Feb 1, 2023 at 12:09 AM Ferruh Yigit wrote: >> >> On 1/30/2023 4:01 PM, Ankur Dwivedi wrote: >> >> <...> >> >>>>> diff --git a/lib/ethdev/meson.build b/lib/ethdev/meson.build index >>>>> 39250b5da1..f5c0865023 100644 >>>>> --- a/lib/ethdev/meson.build >>>>> +++ b/lib/ethdev/meson.build >>>>> @@ -24,6 +24,7 @@ headers = files( >>>>> 'rte_ethdev.h', >>>>> 'rte_ethdev_trace.h', >>>>> 'rte_ethdev_trace_fp.h', >>>>> + 'rte_ethdev_trace_fp_burst.h', >>>> >>>> Why these trace headers are public? >>>> Aren't trace points only used by the APIs, so I expect them to be internal, so >>>> applications shouldn't need them. Why they are expsed to user. >>> 'rte_ethdev_trace.h' can be made as internal. Not sure about 'rte_ethdev_trace_fp.h' and 'rte_ethdev_trace_fp_burst.h' as the tracepoints in fast path may be called from public inline functions. >> >> Trace calls used by inline functions needs to be public, in this case at >> least 'rte_ethdev_trace_fp_burst.h' needs to be public. >> >> Can you please at least move all trace points that are called by inline >> functions to the same file, 'rte_ethdev_trace_fp_burst.h', to reduce >> number of the header files to make public? Feel free to rename header if >> required. >> >> Meanwhile not sure about adding new header as dependency to end user. >> @Jerin, @Andrew, what do you think > > rte_ethdev_trace_fp_burst.h will be installed through ninja install > and application does not need to directly include this. So this scheme > is OK. Right? I dont see any downside. > Right. It is installed automatically with above meson change, and it is included by 'rte_ethdev.h', so user doesn't need to include it explicitly. Overall there is no functional problem here. Only this header file needs to be included (directly or indirectly) by every application that use ethdev. I would much prefer to have an internal header but not able to because of technical reasons (inline functions). After lots of effort we did to hide as much ethdev internals as we can, now we are exposing some new trace stuff to user. As we can't prevent header to be public, I am just questioning below options to reduce exposure of this header, hoping that it may lead a better solution. > >> 1) to move these trace points to 'rte_ethdev_core.h' >> OR >> 2) disable trace calls in inline functions on compile time, possibly >> with existing 'RTE_ETHDEV_DEBUG_*' macro