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 8D6CF423AC; Wed, 11 Jan 2023 11:46:19 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 78D6B40691; Wed, 11 Jan 2023 11:46:19 +0100 (CET) Received: from NAM11-DM6-obe.outbound.protection.outlook.com (mail-dm6nam11on2067.outbound.protection.outlook.com [40.107.223.67]) by mails.dpdk.org (Postfix) with ESMTP id 6BF2A4014F for ; Wed, 11 Jan 2023 11:46:18 +0100 (CET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=SlnXssPBqGrOnFns6V8mpp/js9T3hL5dErc6IvBURMDa6VFAJJ8G/K8W/u50Hzo/N7PDLOamzx3FPROSTxq85c2APSwRA7N05fbRmPKXXywv7Yi2MS0ajVppDEGwtT68d3SQ+BK56ILWBUk2XLidP7dB0apbLUVFY+6krFXWsDCc0DQWwyUIBJSjd4nEhSFWHK6mkcBiBGz1/zVYKfomf67rdJpWYIHkpdQ5gA2WHmetDivN+Qcj3eFWLsUOrn+6n8krKsUuyu+yZWywvDcOteNrqIU1xLnxcaxp+X8ZZSPt01SssLKWKR7PxByAyIGlds1duexymWtpz5207hYbtw== 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=4Jd5BFlXbd7MsE6h16MiUqa1iN1GhGZppdaGDKHynLo=; b=P58Ml1EgXSRVLM9HmnklZ6knxk/j6xvHFLCwH2PekAV34J+AkznaVDGxzPd4HJI4o+cTq3KlPR2RB+kLg9XRPB1w5b8XMMT39VYmewzUQE5nGMI9zxBIXwSWqhaRohCJfCxy9/dvZMi9wiCy4e5m/ufltlydkVrYlw86idHcwnOIDErfuFlYmgNzGctj4hFwfU/iZbw+ZppitatNF8xL4sgnNHRCSBKD392dK/5asI1nNMtBkZBDFznaB0CZbkgR/ctvkMJY+4oXcr1aoYorTtqWSl9qw3ivdDJEv79FeIFY9octJuaUpcUj/nG5GRCmUluwDtWm7Dc1uLRoXTQheQ== 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=4Jd5BFlXbd7MsE6h16MiUqa1iN1GhGZppdaGDKHynLo=; b=FUqOKKhIYow6YX6flJ/Hv32NUjkTvdSPji+ICylvl/JfS/YkKV8M8D1dgkqRYL5t95PTN/SwZexxu2ZCLisEC5Ue2p6jnUsA1dRt54VyBaamcr4ze3fDWxjkR5RgwFy7R0C9F8CpVWTN8WRhx7/hysr3GYDDXLiXg+DV8sXkOrM= 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 SN7PR12MB6886.namprd12.prod.outlook.com (2603:10b6:806:262::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5986.18; Wed, 11 Jan 2023 10:46:16 +0000 Received: from CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::4807:1f44:5e04:e05a]) by CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::4807:1f44:5e04:e05a%9]) with mapi id 15.20.5986.018; Wed, 11 Jan 2023 10:46:16 +0000 Message-ID: Date: Wed, 11 Jan 2023 10:46:10 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.6.1 Subject: Re: [PATCH V4 0/5] app/testpmd: support mulitple process attach and detach port Content-Language: en-US From: Ferruh Yigit To: "lihuisong (C)" , dev@dpdk.org, andrew.rybchenko@oktetlabs.ru Cc: thomas@monjalon.net, liudongdong3@huawei.com, huangdaode@huawei.com, fengchengwen@huawei.com References: <20220825024425.10534-1-lihuisong@huawei.com> <20221206092649.8287-1-lihuisong@huawei.com> <6c448463-b3f0-d128-b144-d4e93c5c11d7@amd.com> <0a87026d-770a-dd4f-4c48-fdd3c16992dd@huawei.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-ClientProxiedBy: LO4P302CA0040.GBRP302.PROD.OUTLOOK.COM (2603:10a6:600:317::15) To CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR12MB4294:EE_|SN7PR12MB6886:EE_ X-MS-Office365-Filtering-Correlation-Id: 5abc0c0f-37e3-483a-9490-08daf3c11087 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: dN//E2F4JOZOCD3UVGj83+uHE7KYshsZuZ6cqLtHh75a23GtgE82SheCKuO7AhzURGdzV2YC/lKDrSei1jf+6B5F5N1AbdOZeFX6M3LEV0HYVVvLoITDuOXhBj7o/6kEDsDRKbrouzXcvP1YXI89InanIElm4j4uYoy14WtoxYEDlQuixnnJA86AhV/TEN7lRzvGysuEwNrmBgn7EoS8NICOguUU7TCkJ8TINfzdg5tUw2Ek7ljGOzcAhYYT3MsmVOz2fE8ktgKcWo7sqv4IWgUyjaQ97jTuDwqSxUTehyD7vna4Ksv51oRAphdSE0xryat0s0kyjlblNlHcWMVUWX03zRL0STtKgStw1v34tZQqp3cx3QUVLye1+vem018Ht40Buqa2s9iUFe7JvBh3KTIB38ZzRuhz01uWleCKjCaBx+cdsQbHRu2QW8ZloOEHImEhbRsiCXlf0Rxr3x3N/7nhEpPwA0GJQqzGSmTXAAwxgC+9Fy36bMCvWegq22dIuZqDEgP2rwARUHxYnzt/POFwGgoIvXAAjMmBBNwX9FCotd4kKU4PFYDx5ljSW2EAd1yHE/dET/39pGZLxVG9JTPUM5Uzuv+yQBwBbuVinpBYRh3bfscrUrJoniVhFjT7jlG6NXjdl2UR7WrKs0qrtXr+hUM3lIC6pgU2mlArkSKiewLmXcX8ae5dhpPi8IIUQiPCozXNOiuvSOvp28DUzhTxPhH8SYBqSRFOehuBmHI= 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:(13230022)(4636009)(136003)(376002)(396003)(346002)(366004)(39860400002)(451199015)(8676002)(66946007)(66556008)(66476007)(316002)(4326008)(2906002)(5660300002)(8936002)(44832011)(41300700001)(36756003)(53546011)(83380400001)(31696002)(6666004)(6486002)(478600001)(6506007)(38100700002)(2616005)(26005)(6512007)(86362001)(186003)(31686004)(43740500002)(45980500001); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?TGpIeU5FMm9NRlNPSUZvOXNENVJEQzBtUFVlMlRPdy9KbFh0bnU3WGxBWHFU?= =?utf-8?B?dVRMSEM4WDNwOEw2QWlIUHIzWkYveCtKM2FldG1qY0ZVU3JhMWxoaGd1RkFO?= =?utf-8?B?V3dhTXVxVmlYQkRlV2gzQ2ZsNGFFRllPeXJnUVBHUEw5bm1BMHNrWWp6NGIr?= =?utf-8?B?L1M0eWpaUGdYM2kyY3lBaWZhbzQyR1JDMGpEcGxWR2pRU0x1WENTWTlybHI3?= =?utf-8?B?dStiTHBoUVc2c0ZrS2tEYzdBblMzTEg0UVEyaVd0dEFWY0MwSnhleDN1cTMy?= =?utf-8?B?RnhKMkxaNlFPZGxHS2lUeFBvcEJhdzV5ZzFSVFo4eXE3di9vWkdyUWlZVStO?= =?utf-8?B?NExad1ZDR1FRSzhlOE5hb2lEcGxXbU1zTmtzUTl5VXFyZytQNlR5N2RTMWw4?= =?utf-8?B?WG1OamFUM3Y1ZVBGVG5xbkRhazJkTnJQNG5lTG9RNDZFekhTc0dKN3I0cWRh?= =?utf-8?B?bTJEY3F2Vk5XZXNkUy95SGJJUDFwbG9qVXI1R211V0xkeVBCWXRoSEpsS0R2?= =?utf-8?B?Yi8xd3lGY011dEhWTlByVU9jc0pGSk50ZDdRK0VSTlVOSy9LYVpOUlpHcXY4?= =?utf-8?B?eks1b2Evb0pKZ1RNZEltNEZMOUE0MEFQclFpSTZlUkx2dHYvK2dpbTQ2L3dq?= =?utf-8?B?TWNFTUQrc0dRdnBMNkFHQXF2VkpHbWg1YjFCMEJQcmptSlVmYm1lZ2lGYnlE?= =?utf-8?B?YS9SRFZHZHdlS2hNQUdUL08vV050Vi8xVDJWSHViY1pDaUp0OGhVbG9LVE04?= =?utf-8?B?OC9lZXNCa1g2TWRRc3Fnb1F5aW9jSDhnaDVLZ2ZCNXlSN3FGQmJEb24rNTFD?= =?utf-8?B?L0JDV0dVb0lPQ1lyb21aRHpna28xN2tHZTBjTGhPeHdIelcvdEVjSm1TUUV0?= =?utf-8?B?dlVzNG44U3g1U3A4dFA2ci94R0Y2L29YTk1GL0Z6NzNYbUpWcUVnT0o2dHBL?= =?utf-8?B?OUxzZXROakhRWXcwYkhoTW1yRmRVSlJObVVYeXZJWDAwWGVMS2FzY2hONnBl?= =?utf-8?B?bHdncTlkTHcvWDEvT1Z5MEdCRjlhUEZ0RmpJbmNSdGFPNVUrT0RpSGF0UGJw?= =?utf-8?B?cGVwQ3F6K3k0U2FnUEtsaElXODdScFljZ1BtNXEvOUZmbTdvRC9QVmxDcVlh?= =?utf-8?B?d0VyRS81cWlxTXVUN2xucWxJM3RzR2xHVnJXREwzYm1BeEtSSDA5YStDMExP?= =?utf-8?B?c2lmVVVCK0N5OWpPVXdTRXRLcEVoR1YxdDgxVGJpSkZKejNUdW1vZ2RyOUtv?= =?utf-8?B?ZE50azd1RlU4a2J1Nk5QNUVyMG13czNjNjdUMTJTU0Fyc2p6aWhEN1RnakFu?= =?utf-8?B?TGxiZmhtTy9qdE9KL1lqVHRMdDNBakxwUzJhQ0xjeklrWFZqakpHWml0NUF6?= =?utf-8?B?RTMvTWNCM3A4a1crTXl2Y2g0YWUyRndyZFduL0Y3SW9PM1NrN0dwSWhaUG9k?= =?utf-8?B?TnJ2MGYwOGgwSmZIdFU2YjNXUjJHamQzZmJ4SkpEaHB0bFF4czlKa2h1MXpz?= =?utf-8?B?NEJlVWJxdFJXOThnMG1aZTJFTkJ5TWpVYk1kNjAvaVFjK1RCRVVYQnhvSzVY?= =?utf-8?B?TXZwUkpBMmhIY04yeTJaQ1RwelBjMDkzMWhPVW5Kci9WNTlmZlJQOUE2ekhp?= =?utf-8?B?VEVJekF0ZGRUa0lETlY1VU41TjVNZ0hVRUh1T05wMkxFcDZrK3NhSUpxTEs4?= =?utf-8?B?MkF5eUlTMWtVT2lBWENXTCtOSlRIVTNTcFRORlltMFczUkdkU2lPeVJPekFn?= =?utf-8?B?KzlSaElSbXh1aEJOVHpBaWhxTTEreGQxdEhrcTR3cC8wbFNDTVpvRzFJUUc0?= =?utf-8?B?cnNScytjWk1XZzVxNENFWjc1eG05b1I0UXdKWjVUZ2x0WHV4aEZtZXE4Wjhw?= =?utf-8?B?aVBzWGRYYWVyaXZjTTZxNkxwUTlRK0U5WkNOa1pWckFIWk5TMTJuVVphWU5o?= =?utf-8?B?a2hOYklJTGFoWTFKNEhLdDk5RTVBbmN0eW5UbGU5dEpqV1Z0SjM2b1Jna3Ri?= =?utf-8?B?ckRDcGNKYWViazlQUU5kSVc3enVsSUtkVVlxR0p3Z3RCcFlCalNLZnR0RnAv?= =?utf-8?B?M0pVWHdKY0tDVHcwZERodFRyT1RkMTZZV0oyVENNeG9wU2QwZG5PclVsRFVX?= =?utf-8?Q?PmNeLdydSAu6nZMHfVnMsG9q0?= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 5abc0c0f-37e3-483a-9490-08daf3c11087 X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB4294.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Jan 2023 10:46:15.9603 (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: dkARi93J6sksxe5op1JDnO/ocSlG+DfsO3kQ0x+nHbEydIJ1Y0pHAH1/M0hhXkgl X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR12MB6886 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/11/2023 10:27 AM, Ferruh Yigit wrote: > On 1/11/2023 12:53 AM, lihuisong (C) wrote: >> >> 在 2023/1/11 0:51, Ferruh Yigit 写道: >>> On 12/6/2022 9:26 AM, Huisong Li wrote: >>>> This patchset fix some bugs and support attaching and detaching port >>>> in primary and secondary. >>>> >>>> --- >>>>   -v4: fix a misspelling. >>>>   -v3: >>>>     1) merge patch 1/6 and patch 2/6 into patch 1/5, and add >>>> modification >>>>        for other bus type. >>>>     2) add a RTE_ETH_DEV_ALLOCATED state in rte_eth_dev_state to resolve >>>>        the probelm in patch 2/5. >>>>   -v2: resend due to CI unexplained failure. >>>> >>>> Huisong Li (5): >>>>    drivers/bus: restore driver assignment at front of probing >>>>    ethdev: fix skip valid port in probing callback >>>>    app/testpmd: check the validity of the port >>>>    app/testpmd: add attach and detach port for multiple process >>>>    app/testpmd: stop forwarding in new or destroy event >>>> >>> Hi Huisong, >>> >>> I haven't checked the patch in detail yet, but I can see it gives some >>> ABI compatibility warnings, is this expected: >> This is to be expected. Because we insert a device state, >> RTE_ETH_DEV_ALLOCATED, >> before RTE_ETH_DEV_ATTACHED for resolving the issue patch 2/5 mentioned. >> We may have to announce it. What do you think? > > If there is an actual ABI break, it can't go in this release, need to > wait LTS release and yes needs deprecation notice in advance. > > But not all enum value change warnings are real break, need to > investigate all warnings one by one. > Need to investigate if old application & new dpdk library may cause any > unexpected behavior for application. > OR, appending new enum item, `RTE_ETH_DEV_ALLOCATED`, to the end of the enum solves the issue, although logically it won't look nice. Perhaps order can be fixed in next LTS, to have more logical order, but not quite sure if order worth the disturbance may cause in application. >>> >>> 1 function with some indirect sub-type change: >>> >>>    [C] 'function int dpaa_eth_eventq_attach(const rte_eth_dev*, int, u16, >>> const rte_event_eth_rx_adapter_queue_conf*)' at dpaa_ethdev.c:1149:1 has >>> some indirect sub-type changes: >>>      parameter 1 of type 'const rte_eth_dev*' has sub-type changes: >>>        in pointed to type 'const rte_eth_dev': >>>          in unqualified underlying type 'struct rte_eth_dev' at >>> ethdev_driver.h:50:1: >>>            type size hasn't changed >>>            1 data member change: >>>              type of 'rte_eth_dev_state state' changed: >>>                type size hasn't changed >>>                1 enumerator insertion: >>>                  'rte_eth_dev_state::RTE_ETH_DEV_ALLOCATED' value '1' >>>                2 enumerator changes: >>>                  'rte_eth_dev_state::RTE_ETH_DEV_ATTACHED' from value '1' >>> to '2' at rte_ethdev.h:2000:1 >>>                  'rte_eth_dev_state::RTE_ETH_DEV_REMOVED' from value '2' >>> to '3' at rte_ethdev.h:2000:1 >>> >>> 1 function with some indirect sub-type change: >>> >>>    [C] 'function int rte_pmd_i40e_set_switch_dev(uint16_t, rte_eth_dev*)' >>> at rte_pmd_i40e.c:3266:1 has some indirect sub-type changes: >>>      parameter 2 of type 'rte_eth_dev*' has sub-type changes: >>>        in pointed to type 'struct rte_eth_dev' at ethdev_driver.h:50:1: >>>          type size hasn't changed >>>          1 data member change: >>>            type of 'rte_eth_dev_state state' changed: >>>              type size hasn't changed >>>              1 enumerator insertion: >>>                'rte_eth_dev_state::RTE_ETH_DEV_ALLOCATED' value '1' >>>              2 enumerator changes: >>>                'rte_eth_dev_state::RTE_ETH_DEV_ATTACHED' from value '1' >>> to '2' at rte_ethdev.h:2000:1 >>>                'rte_eth_dev_state::RTE_ETH_DEV_REMOVED' from value '2' to >>> '3' at rte_ethdev.h:2000:1 >>> >>> 1 function with some indirect sub-type change: >>> >>>    [C] 'function rte_eth_dev* rte_eth_dev_allocate(const char*)' at >>> ethdev_driver.c:72:1 has some indirect sub-type changes: >>>      return type changed: >>>        in pointed to type 'struct rte_eth_dev' at ethdev_driver.h:50:1: >>>          type size hasn't changed >>>          1 data member change: >>>            type of 'rte_eth_dev_state state' changed: >>>              type size hasn't changed >>>              1 enumerator insertion: >>>                'rte_eth_dev_state::RTE_ETH_DEV_ALLOCATED' value '1' >>>              2 enumerator changes: >>>                'rte_eth_dev_state::RTE_ETH_DEV_ATTACHED' from value '1' >>> to '2' at rte_ethdev.h:2000:1 >>>                'rte_eth_dev_state::RTE_ETH_DEV_REMOVED' from value '2' to >>> '3' at rte_ethdev.h:2000:1 >>> >>> ... there are more warnings for same issue ... >>> >>> . >