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 7AA8B4234A; Tue, 10 Oct 2023 11:55:37 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0B91740297; Tue, 10 Oct 2023 11:55:37 +0200 (CEST) Received: from NAM02-SN1-obe.outbound.protection.outlook.com (mail-sn1nam02on2062.outbound.protection.outlook.com [40.107.96.62]) by mails.dpdk.org (Postfix) with ESMTP id 79B0E40278 for ; Tue, 10 Oct 2023 11:55:35 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=D5bSZ/IzQ+WlXb5mncvCg9mVJCkl7T04vw5O43ajdOTc5LMpmeEL/qgLX77eHQvtJMmCa8td0AiiESdgeP20WnAYHKtfAPNwjqUFXUMELg1FDIZwxhenebbeT8q59GTI08ivBoQYY/6bMkbfLkV7oPUBvex7NiRPO3GCb3bqSOiS6dtZLLfNO9n7Zdh5ewewRS7UxzY0SGv/KVZKye8VT+nlbVQ2Kr0OZ+wQe/NTkbRhqlYMh9kuDOQcG8ek4YPBeOmF3JSM2Q6dQxV6Y/TRSb/I9gD7MU+XAQyzWFQz91jDEWF5uoO6TzgQ8ViT9ozm/uztn8Mo9spgAHFImpYzyw== 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=0lx80QD66kdzeuoRGzNbq4wW2j20wGIVBvxfUnjKaNg=; b=UpADqAEflI9PCeGRPIPPhT6GN6DO+uz8TvIucXX4MtANf7+eHuNuCcPWv+dA2W5hZ4phDD8EPPf+8pySV8Nvqqcyewjc53wIA9Avn1956N3z0eI3LLyyeKzXX4hhOVmjeWX2IMivQm5p3NxZa9m8PlvOLCOJ3Hz4YsfpyMcEbArryY3WgSxrYC5QoHxmBPWDSc0rGVqvVAuDCcsNUDvjjF+x34H1bfy3xA72Iiseye8b4cjErq1rqPCrDNYAxdcQIg1HmVi+KreSm5bFaFttFLWdBpw2b6lvX/frzwQr8b+Ioms+jDsMKPwQpRG8xJLLdiHbRIY+Ml+hNjeafzIWBQ== 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=0lx80QD66kdzeuoRGzNbq4wW2j20wGIVBvxfUnjKaNg=; b=Lua6875+RCyaPZuB0y8jyyH1Wmy8rB8SrmfsU5LZzT64Vt74KdLCNGgvgRUzMZcOmzct3lrfCSLJztONeAz+vIlfGj2Div1bnO+r5YmEvok/HBmaCECyVFOWF1UNfvJxczJw4jaZ8PKDUHRmZfR9f6QHR91uowo0SSGjsU9OapU= 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 BN9PR12MB5226.namprd12.prod.outlook.com (2603:10b6:408:11f::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6863.38; Tue, 10 Oct 2023 09:55:33 +0000 Received: from CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::102f:c69b:d3e5:7fe8]) by CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::102f:c69b:d3e5:7fe8%4]) with mapi id 15.20.6863.032; Tue, 10 Oct 2023 09:55:32 +0000 Message-ID: <8eb416b3-e571-4692-a8fd-7b89cf9b00bb@amd.com> Date: Tue, 10 Oct 2023 10:55:26 +0100 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH] ethdev: remove init_color from METER_MARK action To: Gregory Etelson , dev@dpdk.org, Ori Kam , Andrew Rybchenko Cc: mkashani@nvidia.com, rasland@nvidia.com, Aman Singh , Yuying Zhang , Matan Azrad , Viacheslav Ovsiienko , Suanming Mou , Thomas Monjalon References: <20230808100036.215477-1-getelson@nvidia.com> Content-Language: en-US From: Ferruh Yigit Autocrypt: addr=ferruh.yigit@amd.com; keydata= xsFNBGJDD3EBEAC/M7Tk/DfQSmP1K96vyzdhfSBzlCaGtcxNXorq4fALruqVsD3oi0yfyEz9 4YN8x7py0o9EL8ZdpOX0skc0AMCDAaw033uWhCn0GLMeGRKUbfOAPvL6ecSDvGD7CJIO9j0J eZUvasBgPdM/435PEr9DmC6Ggzdzt8IuG4PoLi5jpFSfcqxZFCCxLUDEo/w0nuguk2FTuYJg B2zEZ4JTBZrw7hIHiFh8D8hr6YA6a5uTofq1tr+l048lbtdFUl8TR0aIExVzE4Z8qKZlcE+9 RQaewjK5Al1jLE4sHdmd3GN+IvgDF3D/fLsi25SKJDeGSdeHkOmaX0qGeM4WKIfU6iARRCiQ N3AmBIxZ/A7UXBKLaOyZ+/i3sE6Wb53nrO4i8+0K2Qwyh6LjTeiJAIjYKN43ppxz3DaI+QwQ vI+uyHr4Gg0Da9EPPz/YyKauSeOZCfCB5gIfICO0j6x0SCl8uQ2nLpjxcZkf0gjcwUzP3h+S 3x6NfDji9YEij0zczW/dcSpGgZ6vsFpPrtnP9ZXy6J53yp0kJtOJoOlkEFFdU2yCZnCDseum CoudmGLZVvS0/DzHDJejq+3kK3FDGktZBOxZIIpal+nFqS7lVgOZc4+huVv3jyhzoAUOEyXA XK5j6o7g8STUY+z33QNnHpdLvecMwuzmvqy0jR54yAbZ64mB9QARAQABzSNGZXJydWggWWln aXQgPGZlcnJ1aC55aWdpdEBhbWQuY29tPsLBlwQTAQgAQQIbAwULCQgHAgYVCgkICwIEFgID AQIeAQIXgAIZARYhBEm7aYjps5XGsPHCElRTPtCKKm/6BQJjb9DJBQkC+3/YAAoJEFRTPtCK Km/6d94P/irGq3mPa7LamXMIioQ8i6ppMSjpr8g+SxH9RnzbmoUjUY6hVzCpXYxEejiJHubg 7lwD+bOocYpiU8Pe0UncVBhIPNk/dIWQAyH0IWK1nd+hOnjxHv3AQpP80Be2o1mUn3oq/b+B QYiyvvre2gIugYq0hzLcG7z4zREeT+Nl0DMDfnLx+Tj2FAOHrOfudUqjdr/VjF5PoTK0bxnb Brqulp0I0ft7hNsufhzf+TlqaB5l0eQO4gDo5xGTP97TT1cGYsXVMsyDhHqW6P3cuj8kkLkO Ch4oq8OLL8GElgRy5y5svU6CJu3f8hT9aiqIoaRpfNv6N8iEk6g4/HNN+uydb9YCA+1pbwBx skmDhtxQrmXpI+Fmfq0aX2NmqHcy6JR8Ekm/nfwLAW/aDpwoMImVN5pPXtOBSjYi4pj7hiJk UA0ZFi8HCW+fic85p8MXn1GPDmbO82Mm3JVJ1uJNAzGsBbW5SP1ol6+XTeaSxcmzFlrDs8vL XQJMRTpcG6Pti7GbbsOtldvmxRQpq9PmCgQg9IrPXpMoaKE3WwtiSiCALs0kYj4F+hhgAfjI RsOuaAJTnKYW31pC/QdroMNAEqIpLveql03jj+xD9ntJIVAsXiCitsLIXzKP0L5tcHkoVLC3 NAjVYq6XkAEssPF4mMRLoEwpvxOytHe1BCJoRl0YA0aQzsFNBGJDD3EBEAC8fBFQHej8qgIG CBzoIEd1cZgPIARlIhRudODXoNDbwA+zJMKtOVwol3Hh1qJ2/yZP11nZsqrP4fyUvMxrwhDe WBWFVDbWHLnqXMnKuUU1vQMujbzgq/4Rb9wSMW5vBL6YxhZng+h71JgS/9nVtzyaTtsOTrJi 6nzFSDx6Wbza2jYvL9rlK0yxJcMEiKwZQ/if4KcOesD0rtxomU/iSEv6DATcJbGXP6T93nPl 90XksijRKAmOwvdu3A8IIlxiSSVRP0lxiHOeR35y6PjHY2usfEDZZOVOfDfhlCVAIBZUZALv VmFOVSTYXeKgYa6Ooaf72+cHM3SgJIbYnevJfFv8YQW0MEAJ/IXE7B1Lk+pHNxwU3VBCrKnA fd/PTvviesuYRkrRD6qqZnINeu3b2DouVGGt2fVcGA38BujCd3p8i7azoGc7A6cgF7z9ETnr ANrbg1/dJyDmkDxOxVrVquTBbxJbDy2HaIe9wyJTEK2Sznpy62DaHVY+gfDQzexBXM10geHC IIUhEnOUYVaq65X3ZDjyAQnNDBQ4uMqSHZk8DpJ22X+T+IMzWzWl+VyU4UZXjkLKPvlqPjJk 1RbKScek5L2GhxHQbPaD76Hx4Jiel0vm2G+4wei8Ay1+0YRFkhySxogU/uQVXHTv63KzQMak oIfnN/V2R0ucarsvMBW+gwARAQABwsF8BBgBCAAmAhsMFiEESbtpiOmzlcaw8cISVFM+0Ioq b/oFAmNv0PAFCQL7f/8ACgkQVFM+0Ioqb/oU9hAAisAJJ09j/kiEeA1HGKUg3DyFnYW22zRL z1IHYmcHQanMx4+FAwsb1bonVldQMIYNJ9z69UNIU16zIqLZt3D5QK7Je5F5q4NZgfP2jtjX rc3jyu0PuDerFZQyxNcYhhmlF9JO9NV976WYbBAM3AAr0TDohUk+YicYTKab+ZHYOABXbqHX qi+bzDCH1vKBDfpJviprthhK1tIABUK5lsp9aFwFU2jfu2J3XX3pwQhPMjtn2C4v37XUnOqM SJPr2HLU27IW5I3BBpJn7dZE/BkmDmnceHF8E4tRlXCQ/cf81+eErL832sfhVJo1MpubKh3B j89ZLmw0cDXZP2hqlC9vdXnoWWHI8PDPkiw9z11yLrT9Wd4cTJTjFV49Z9G96rgnOtWDcZ1J xHBAnXYQ7V2k9abY+ZqY1PtAPr/smqzQfPHXyTAhRVZp2f3yQmj7UqB07FJuLQjJ4CGk97Qx ///qeSg7D8x/sGEA8Yp2jgIj6u7tspuz+RgBYrWpfI0VIHhWED0jNXzqztrVX18CzA6r0ReY SG+CSyycKkco6UI9ZeUrXFwnoUJqIJY9wTJo5gD9EgGIbItv3qWTq29dihLYpzd6dqB708k7 4RBz9051oLaFVGkguFw/tXmA17nx+c2uR5jx8wb4j68umG2X++0dSta5eHAVhmtXvrqPW6Ku 3FQ= In-Reply-To: <20230808100036.215477-1-getelson@nvidia.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-ClientProxiedBy: PR3P189CA0061.EURP189.PROD.OUTLOOK.COM (2603:10a6:102:b4::6) To CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR12MB4294:EE_|BN9PR12MB5226:EE_ X-MS-Office365-Filtering-Correlation-Id: 9becc0f3-26a4-4287-2557-08dbc9770adb X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: ME2UE6S+RNdiyTK03oOQ/iPGtz8JH+8zUbcOtuGzbRaiXYon7WinynxLjGlflwJrGbE+/1I03Nq8gPbRLPOsXy7JzK5ZVaKNi8rkrvhcJoxTvsxNuf5OtHX5XyjHi9Znj4+avTEpDiE8zmGyO/sm04UY7z0HbA5p6JjaP3J3xPVs560stopHH4GGllt7pYK3YdNWPGuOuMdcKr3PFVFFGSYl5RroqBb9QQ3qwmcLenz4+LOdLBy8R349/+kB3V4Qhdf/Yz0QT/LCUB6g6LUL8D9XgUUkTAkEdS/g6O4JCRvhzrPa+yrdfYnGXGzjGDHMD2JWKM45KD8N+ByxZrxfGlUg14rNTmeNh6twN7up7G/6uwsaBQ/u0ILnZEBhZL+uOifcFUa4EvTvg4qlL+NTjIdqhMIgpAIKyQKY9zBZzK5CjGzPoJUdw9FxHXWLaNeicYCylQ6UrMdd3Zzsac7lufx1SE4yb2v+RRP+KRmTxufRPFU84TjC+jLArBLPpM8FsMKHSwoQGohbJyQzUzTeaw/9CKaJ5bqWkDOM+6ry/zNIbuext8X95rwAFVMjqwdGhoQf19hnGm9MJnNunDF+EbgmHkPgKeVA/c9p93C2Z8QCqC03gh0wRLQmOxlkcnry8z6/y2NGAX+xCK5VexcOCA== 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:(13230031)(376002)(39860400002)(136003)(396003)(346002)(366004)(230922051799003)(186009)(1800799009)(64100799003)(451199024)(31686004)(2616005)(26005)(53546011)(54906003)(31696002)(38100700002)(36756003)(86362001)(83380400001)(2906002)(8676002)(4326008)(6666004)(7416002)(41300700001)(6512007)(8936002)(478600001)(6506007)(66476007)(6486002)(316002)(5660300002)(44832011)(110136005)(66946007)(66556008)(45980500001)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Q2dxNVFjYkRyQ2ZVSm1NU3lZbElUWFJXZEw3Ykk1NHRidXljVHMva3pCRXZG?= =?utf-8?B?VTJzcWlhWlRBQS94aGpqOEhXd282SjVUK2V1Y2NJMlVtTkgrU3BYdm4wLzdt?= =?utf-8?B?K0NVcnQvb09vSUVhdzhtNWdiVUlFZytVeC9QbTVwRy9xU3JFMDRHZHgyMGVJ?= =?utf-8?B?UmxGRUFDVUh1cWRQZkFrUmZUcDg4clR3d0RVZlNXeG9PTm9ReUZQZlFNeFF6?= =?utf-8?B?VmJTNmw3OEQ4YjlXM2lpTzVVQmZ5VnZBQ0pwd0JtZ2VoSUdKNmpKNGY4dDRx?= =?utf-8?B?djJPQUJZWDZaWG1FM3d5ZHNQNEU1QVRXTUJwRE1oNzBJS3JHSTh3bTBHbWpp?= =?utf-8?B?T2NjRHhxTnJzUHdLRks0cXdDdzU0RGVsV1N1NU8vUUQxL28yWFlrdXkvTFhr?= =?utf-8?B?Zmp4WnNvMC9lZ2hldzJnbkFBbkRYdUVRT3BXWGd0OVVEV2tKdnIyRE1WM1hP?= =?utf-8?B?TmNSVTZhdFVrOFJLZWpZMGd3Vkd2cVRoZzBaNkNkR1BGWGU5dkxYUytnemFV?= =?utf-8?B?aStaR1BOanRHc29YNzg0dGZKYjdpYTM4bGRRUEZnRm1XcFhadzNVa2hGK0lX?= =?utf-8?B?YWUrSFlXcjZnaGxRdWk0eXFPWmRBOVI0YlpaSjRiL2JwZ0pLZk5SbVREZkhx?= =?utf-8?B?bXQ4QjBZQVBlSHJuT0x4RUhlZ2JDZURrMjkybThHbmlRMmh2TGhtaFNnUnp4?= =?utf-8?B?QTF5SVVjeEZ1cW54a0Q3cWlxTVdPM1crVFkvSHpRamlKOFZpbUdZR1Nwbkg5?= =?utf-8?B?V1BBdWF4TUIyVlVKeGZxbnpVWXpjVngrS1Z3Y3ZRVWpYcXVMV1MxYzhyUmM1?= =?utf-8?B?ZXQxclJ2SDdZeVU5S2JHcGw3RWlNOGFlQW5XbHZZOWRvZ2FNdGR5Y3p2Wjg0?= =?utf-8?B?QXFkZnprZ1BkeGhmcU80OHM2SWRUYmp6REhWcHJQT0hZdXNBVDRiODBMc05H?= =?utf-8?B?TVd5WGcwVm1HYlkrWDFCWUFXejFlOXIremVoS3lPZ2xtVDNRTnFvZlNHQlJG?= =?utf-8?B?ZVRkNExFdC93ZkM1dEEzL2RqMHAvMUN4SEY5M2FINlZkcjBSVXh3Vis2VDZ4?= =?utf-8?B?V0V2NXF5Q3M1WWpJVHJOYXFZM2FXTGxuTHdOY0FVNHlPUU51a25BWVpTbkx6?= =?utf-8?B?dVg4VHNRNVJJY3BNMjJWY094NUFibWt4eTR4Q3g2L2xRVjNGblg4WXpjL0xU?= =?utf-8?B?QTR1dEJNU1BsMXNJVEtYSk1LOFZqMnZTSFdWalRWMHlNV1cyc3I5TklBQVJF?= =?utf-8?B?S1J0NWNhR2svVVZGRG1KaC9XNXRQNE8wUjNTc2U4VWcxMTlDazlWNllpcHlW?= =?utf-8?B?VzJjc2pvWDZwdFdnY3BlTWxkSWU0eTRxYnN5NWV4RU11NHEyblo1MDRvWGMz?= =?utf-8?B?YkVmK0w5ZkhVTlNTTVFvREx1dmt5NWF6VmoxM1hWY1lFNWZVeVk0ZzB5a0xC?= =?utf-8?B?UjYyb0RZVXJ2TVJaNFhEK0p6MjdNa1lCdmVKYkwxOVZaaDQ1Q2RCRytKMFdt?= =?utf-8?B?YU5QK3BaNlkwZVhONVg2U3IxazB2TzBOMng1Q1JUZXB6NWI5TDQxc043bm9Z?= =?utf-8?B?c2ZRQWhxcmVKdEsyNnZ4VS8vOVJlM2xYZW45eFY0ZUU5UFJhN1pqTzRKOEdF?= =?utf-8?B?Y0pPd1hhWExYZ2lYSTBwY3N0cXkremVwQU5GaVVZRm5yWFozdnJ5RlRTMVA4?= =?utf-8?B?NDlTa2pwZjRRV3VPSldoTTBUMnFsa1BxM0p0UTdKcjRNa0RaQ0VZbW5DSnBR?= =?utf-8?B?QTI0YnVpL044TjJ6cEk2TzRySWR2NmlMZ0tSalQvemFQcUhIbDNRUXBxN25w?= =?utf-8?B?UnRNVGJTOUE0R2E1OTJQcVUwRytUOWNkL2tRbDZsVVBvanhuMTJrQmJ0MGV4?= =?utf-8?B?bWh5Z3VKV0pLMDFoOFRJRDZyb0FnaHYwOFpPbWpzRWVvUGJxK2xFMnZjNno1?= =?utf-8?B?a2J0OGVPbXBqRnYzVWlyYjRIejlPZ3prakF1ZDFkQkhpMG9hZnM1VjlkbC9X?= =?utf-8?B?ek5UL1VUUmFKekZoUzhPM21DSjR3SWl5c2NiSmRJbTAyMFBNL01jdjJpWkgw?= =?utf-8?B?alhZa1NvUm9SUE9aT2p6TWdUQmU2RzErc09Sa1YxNlQ3SnM0a1ZhWnI2SGw3?= =?utf-8?Q?cijzvLFnuFN/KiWGwXeWCfvxc?= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 9becc0f3-26a4-4287-2557-08dbc9770adb X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB4294.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Oct 2023 09:55:32.5799 (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: 5FadidWS49vHbIAjjFsQMO+OkCnxEs0b5u8rjpL06xp1+f3RGDl2EKCh445DGcO7 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN9PR12MB5226 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 8/8/2023 11:00 AM, Gregory Etelson wrote: > Indirect list API defines 2 types of action update: > • Action mutable context is always shared between all flows > that referenced indirect actions list handle. > Action mutable context can be changed by explicit invocation > of indirect handle update function. > • Flow mutable context is private to a flow. > Flow mutable context can be updated by indirect list handle > flow rule configuration. > > `METER_MARK::init_color` is flow resource. > Current flows implementation placed `init_color` in the > `rte_flow_action_meter_mark` making it action level resource. > > The patch removes `init_color` from the `rte_flow_action_meter_mark` > structure. > > API change: > The patch removed: > • struct rte_flow_action_meter_mark::init_color > > • struct rte_flow_update_meter_mark::init_color_valid > > Signed-off-by: Gregory Etelson > Ori, et al., mechanics of the patches looks good, and mlx is the only user, if there is no objection it will be merged in a few days.