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 A040242DCC; Wed, 5 Jul 2023 13:41:14 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 848384021F; Wed, 5 Jul 2023 13:41:14 +0200 (CEST) Received: from NAM04-MW2-obe.outbound.protection.outlook.com (mail-mw2nam04on2071.outbound.protection.outlook.com [40.107.101.71]) by mails.dpdk.org (Postfix) with ESMTP id B46A240150 for ; Wed, 5 Jul 2023 13:41:13 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=WSHI8WUCyhXOZXvvbSdrvY7Ah3/blySufne14mTgGAUzTCXHAgV23bDSh4gcSiD54WAj1psXNalzHO3G/TemjFhXgLy9iizfB06do3AC3ck8MrEgL5587X6QaHkXdBRxaXTL/p9t7hB58xniQb1lYcrkezcfAzG6roxHPpkv7G32pSwoY6oSLOjH1l5QXmL3ZLioiAMtUkx6uTw5sj/RdfS0lxN2A6VOMPEexRAkHTJjuUYiqdcK+I+om4xZEpbK0cjis7tWwDJJLUfkZA3FORJL8pYLuW9TMbXA6QXJCJK/T+T9vp5+h+Ufxb8kjWtc5ym459ufLc9hROAH8NMFvA== 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=7mY1Bt/mq5niXg+sxQHPNLDLHeDVVpGS7Pk8tEGFLgI=; b=lsWy6Pw8SSgncKH3JZNYZvrlcdtwNko2zsPuqlFXg87UK7ppAMc3MY3gOgRMzmdO28QzENVbWfnScor+PUCpJ5+o/fBDSknHAXtjP97rzYlfS9y7fKvwJxtvz7s/uKr8i5d/I4ZE+wqLnTW5qJsrpxaGi6LYYvioYx/BZebMI4WUBfUVJm+LY6FUvekHbv1FvJcRjVpYL6Y6uONnzuCJBKJBsCPd/b7PvrN7YWTKfldxUVExJ98Hxaa/S4+3ETHiL8E9y38uUZHVGhCfQ24u5n1YfxvIsuAPEZnbxVFFkYngvEyAbPPP3ggkOgMy2HJ10qHeiVYYJrNYJpNb1Q7yXg== 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=7mY1Bt/mq5niXg+sxQHPNLDLHeDVVpGS7Pk8tEGFLgI=; b=SiE0G3/nDUIRwWvWQY9m1YrJu1gztkEOya8cjFjif4MOkt6QvPX+9FfOaCW+4yKgiar3dfEUJwTmJlAuhYDtCP+F0kUsIjwa4VscPA4xBRmUIo5BohTCRqVrfE4PlT+IEkr/tuH9XdA4lycc0FCqqxHXef2cqb0BWIXsm37gHG0= 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 BY5PR12MB4051.namprd12.prod.outlook.com (2603:10b6:a03:20c::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6565.17; Wed, 5 Jul 2023 11:41:11 +0000 Received: from CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::369d:5f05:aaba:ebd8]) by CH2PR12MB4294.namprd12.prod.outlook.com ([fe80::369d:5f05:aaba:ebd8%3]) with mapi id 15.20.6565.016; Wed, 5 Jul 2023 11:41:10 +0000 Message-ID: <51296191-547c-652e-3945-d86a89e2c3d9@amd.com> Date: Wed, 5 Jul 2023 12:41:04 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101 Thunderbird/102.12.0 Subject: Re: [PATCH v2] app/testpmd: fix invalid queue ID when start port Content-Language: en-US To: "lihuisong (C)" , Jie Hai Cc: dev@dpdk.org, liudongdong3@huawei.com, alialnu@nvidia.com, "Mcnamara, John" , Qi Z Zhang , Yuying Zhang , Thomas Monjalon , Shiyang He , Aman Singh , "Jiale, SongX" , David Marchand References: <20230703110232.28494-1-haijie1@huawei.com> <20230704084507.12788-1-haijie1@huawei.com> <29d89a6b-49e0-8789-411f-6a89db177b91@amd.com> <74fb4fc2-6497-0ba9-e95d-71da03afd76e@amd.com> <27022343-946b-ec7e-9beb-4ad9e74e38f8@huawei.com> From: Ferruh Yigit In-Reply-To: <27022343-946b-ec7e-9beb-4ad9e74e38f8@huawei.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-ClientProxiedBy: LO4P265CA0105.GBRP265.PROD.OUTLOOK.COM (2603:10a6:600:2c3::7) To CH2PR12MB4294.namprd12.prod.outlook.com (2603:10b6:610:a9::11) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: CH2PR12MB4294:EE_|BY5PR12MB4051:EE_ X-MS-Office365-Filtering-Correlation-Id: 7599c61e-3b96-4257-97b7-08db7d4cba96 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Rt3BWzId4j9K6Qb94i9AlkDYkNcHbAIfl7P3+RsvBBMBiZhgQgdh5Hx5QZ+v2E8ChBBR7OpEHntFnzsjETIsvZJEXL3Moh1yJYDmwK7voGWzKAK6d1k/NPM50/ExCJPGWIjPeaNcJqZVGUvhFk64+bSLUdBHcmm9qhHa3VulpmMCNGghREv2lumjaNIEZOt83kfZ2qnNwXvK3B5gV8rDr8TzePhWCsT9OWq8Z/G4ZttVKoYLirgkrBswKd4H9h4jmOO8VCsEc/y/CyWzUwM0gSGo41Y8zI3mZv85p4K21FSiWU7nqpBLaWSMZyWX/YvCLI8aAcJLCS5N7aSrTwm8z8FJlm/Zzn4IY7q1WaOXVIJgU2osBLnCSnpvq7TSQynZAyLhGasRqcnFCMDHnEqLt0FKIytwntYnRTguADm9dcAOq4lv5qGJmNjpsUjciaW0omOx3Jp5Q3V79C0+osrQ2cmfGAP4V11UZzt3Kd3JheVntc8XnXnYDvMSUjyt373ynMbrFQAJAqenXWhgqsC1ZO/yXjQj0BvTxpeLgDvi3en5CDzT3ig5f9WlIABYuhXzXBRvYkiLxaGHIGAC/OHJkl3ItBIvB9XrN2HbknRthKSOHkXkiy9ByTDqad5/tvFzuFgCP1cvzBybNL///XrJQQ== 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:(13230028)(4636009)(39860400002)(136003)(346002)(376002)(396003)(366004)(451199021)(41300700001)(110136005)(54906003)(6486002)(36756003)(5660300002)(44832011)(316002)(2906002)(8936002)(8676002)(7416002)(31686004)(66556008)(4326008)(66476007)(6512007)(6666004)(478600001)(38100700002)(186003)(31696002)(86362001)(66946007)(2616005)(26005)(83380400001)(6506007)(53546011)(45980500001)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?TFhpZ0ptS1dQNlBRYktvMnZ1aHlReDV0U2pQRzc3RTh1cjJ3aHZ6YXI0VjdS?= =?utf-8?B?S0Iwd0lhTEVXK1Zyc1MybjczNmhvbVVLd2pHaXZEc3hjRnRqU05Kb0hTN1hM?= =?utf-8?B?N01xZWs4UnlMa00wUmdmQ2EraXliTDcvM0Jpa2g5OVFDOXNPVG1uVnBJNDBr?= =?utf-8?B?dTFQU2xWMzFObUZ3c1dlYVFydjBvVDQ4REE2eTJYUDNSSE5oQWJZWG5LblFY?= =?utf-8?B?VERobzh1WnNvTHZZVUJxclF4K0JYT2hydUdmVXJXOVFvemdlUHkzbVZQczVu?= =?utf-8?B?emVYd0lRVEo2aGovd1ZHeHNzMlF3V1oxWUtHT0hrSU1RbVVDVmdDbGtZRk54?= =?utf-8?B?Z1cwU2IzY2xMRGFVNjNreVNCNmFKOGdlTzdqaFptelFOSEVFeG1USUhacjM3?= =?utf-8?B?MHRDRmw2YWtIcCs1dzk0MzZNVXhuOHRzREF5NDVpYURhQjdaanRTNWYzMkNx?= =?utf-8?B?TFJ1ZFRsTk00bGZQY0FRUCtxcWNaU0JuWnNvM0lvclFxMitlaExUaUZjYVpP?= =?utf-8?B?Q3dkWkhEa3BBNWZCcFVSWFhDRXZ2UEtxSHdmV0xuT1QzL1RnNCtOQ2N5ODhU?= =?utf-8?B?NkwyTnFPZ2NhempCMVZSc0FPbGkyekt1ai9TSXdYSVdTV05ueituWCs5MWxm?= =?utf-8?B?RVU1NmFOaXZmL3BmSVR4N1FSL1k5NHM0dTFIUnVVWWpXcXdhaGNwZnh6L1BX?= =?utf-8?B?WEZhaUxQQU9rQmRkUmZwdHREeThTMHhUTDZGdG1ZUnh4dlp1NHBLeXBUY2hv?= =?utf-8?B?N3NhNHdDUy95bkprWUVoQVdKckg1cUJEbTZkWkJlSzB5bVE0V2VoRnFISTJY?= =?utf-8?B?RU9Hajlwa1ovWFUyZ2QvV2oyaVlxQ0pYOVNqTlJnNHV1ektsdmtuU0Era0xo?= =?utf-8?B?bi9HQW4xd3NsdHJXZ1pFN3hUOTVoNUlrWmFBSUJ6YlZZMjF4eUFSY0dVM083?= =?utf-8?B?cXVnNEJ2ZTY2akVsUTUxd2NtQ0kydVpKUkovbG1saFRPRitjMjBrRTBmbis5?= =?utf-8?B?d1MrMHlJR0hKRVYzc3ZOcEJac3FyQ0hTOFJOT1lJb1phTFVJSXhmbitvMGN2?= =?utf-8?B?WXR2aUt3Mng1YzJQaVAwVWo1VU1vY1R1ZGdvUjc2QTBrN1MrUlFpQzFpczNV?= =?utf-8?B?Y0lPU0t0MGxqaENYbWJzSWVoeXJtOFgxQmJXQjRCTHFWQnlJN0V3UzkrMnhj?= =?utf-8?B?NlAxRHkxWjliTFJHNDlMWEwwUVlNSzk5U0JTdjlnTVpyS2tGOE9kZ2xPWWVz?= =?utf-8?B?R2kwbGJnRmVxcThnYkVSQVNVY2JsbjJ1cCtnNjFGRkJ0ZmxwWG02b3FSMGtS?= =?utf-8?B?VEU3TkZmUnljUERZUTVKR0FXRStRMEs2Z1YxalhJTzh1NUplQXlPQ0NQOTBu?= =?utf-8?B?NlF4MkdzTWhPSHgrTk93cmVwZmp4SmtCc1FLV3M3T3hDZHBzU2dxS09mTllR?= =?utf-8?B?WWRnenZ0UlNsdld4OGt6UEdYT3dzVGlMRUJrRXhLRWlUSjZ2OEx5aitEeUQz?= =?utf-8?B?QUFrTHA4K1p5R3c3M2RRWHFwalR6bGk5aSt5WjI0Y1M5U0IvclU1WTVLUGxS?= =?utf-8?B?UlVyMXhnZi9aMit1Vk8wbnkwZkRYN0l3SWdVb3h0YTZNdXlxY3RSVHo5Y0FQ?= =?utf-8?B?Q2s1SkoxNHNhTEVPOVA4WDRUSE4wSStOY0psVmE0Y2FiTG5LMEVWdGF4ekIv?= =?utf-8?B?QzdsWVVKMk9FczBZK1I2WGRkTnBzSUVGSjZyTXZtL1U4TzZYblllaS9zZHpv?= =?utf-8?B?OFN6QW5jZDVURW5YSlg4RFkrOVY5TEVLdGZ5ak5MakV2VU9sd3ZJaXRFRGYv?= =?utf-8?B?VVp2MTFMRlJQanA5MmVrYzE1NXBMMk9VTDhFM09vM3JIZCtwL0hQY1VRaG5u?= =?utf-8?B?T2k1dkN3a29YTHZ6NlV6b1ZzNlphWVJCVE5mM2xUZmJaRzQyY3VzeTdZZzZF?= =?utf-8?B?c0VnN25DY1NjRytJWFNzQUVpVll1Z0JSQ1Vrd3BHaWZRM2VtNGt2VUdiS2RU?= =?utf-8?B?OStuQVJVUFk0QlRKdGU2aFFZeks4aDRnWHRvd0hGRFM3Z1dHZzdXY3padEMz?= =?utf-8?B?bVhZZnNVLzNrZkFPQVFJcXJlRlBqendNR2psd2RuREVsVXdQV3NtcnhoNmlR?= =?utf-8?Q?iZITfjsh6sTKv8dH/+Rlpmv/R?= X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-Network-Message-Id: 7599c61e-3b96-4257-97b7-08db7d4cba96 X-MS-Exchange-CrossTenant-AuthSource: CH2PR12MB4294.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Jul 2023 11:41:10.5861 (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: poNJiazpgqumzGpLRsZpV3ngnRaKizfX1zcB3Qp3PzzkI9ZK4ieLrTDOnQWd2D9I X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR12MB4051 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 7/5/2023 10:40 AM, lihuisong (C) wrote: > > 在 2023/7/5 16:02, Ferruh Yigit 写道: >> On 7/5/2023 4:16 AM, lihuisong (C) wrote: >>> 在 2023/7/4 18:59, Ferruh Yigit 写道: >>>> On 7/4/2023 9:45 AM, Jie Hai wrote: >>>>> Function update_queue_state updates queue state of all queues >>>>> of all ports, using the queue num nb_rxq|nb_txq stored locally >>>>> by testpmd. An error on the invalid queue ID occurs if we run >>>>> testpmd with two ports and detach-attach one of them and start >>>>> the other port first. This is because the attached port has not >>>>> been configured and has no queues, which differs from nb_rxq|nb_txq. >>>>> The similar error happens in multi-process senoris if secondary >>>>> process attaches a port and starts it. >>>>> >>>>> This patch updates queue state of the specified port, which has >>>>> been configured by primary process. As the secondary process >>>>> cannot configure the ports, make sure that the secondary process >>>>> starts the port only after the primary process has done so. >>> Now look good to me. >>> Acked-by: Huisong Li >>>>> Fixes: 141a520b35f7 ("app/testpmd: fix primary process not polling >>>>> all queues") >>>>> Fixes: 5028f207a4fa ("app/testpmd: fix secondary process packet >>>>> forwarding") >>>>> Cc: stable@dpdk.org >>>>> >>>>> Signed-off-by: Jie Hai >>>>> >>>> The problem description and solution looks reasonable to me, but Intel >>>> testing still reporting the issue. >>>> >>>> There is a chance that the issue Intel side observing is different, >>>> waiting for more information from Intel test team. >>>> >>>> . >> Hi Song, >> >> As far as I understand this patch works with an update from ixgbevf >> driver, can you please confirm? >> And can we have the ixgbevf fix soon, to not block the -rc3? >> > Hi Ferruh, > > Yes, ixgbe is not setting queue status correctly. > Whether tesptmd polls the queue depends on the queue status > (dev->data->rx_queue_state[queue_id]). > But some drivers (not just ixgbe) do not set the status correctly. > Jie is doing this. will be sent ASAP. > Hi Huisong, Jie As you said some drivers don't do it right, and my concern is: a) if we have a side effect from last minute changes from drivers, b) if there are more drivers impacted but not recognized yet That is why my proposal is to revert Jie's fix for this release, record known issue, and merge it back early next release to give more time to drivers adjust. What do you think?