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 D6EB242C50; Mon, 12 Jun 2023 18:32:46 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id BCE5B4114A; Mon, 12 Jun 2023 18:32:43 +0200 (CEST) Received: from linux.microsoft.com (linux.microsoft.com [13.77.154.182]) by mails.dpdk.org (Postfix) with ESMTP id CB04D410FC for ; Mon, 12 Jun 2023 18:32:42 +0200 (CEST) Received: by linux.microsoft.com (Postfix, from userid 1086) id 2753620EAB5F; Mon, 12 Jun 2023 09:32:42 -0700 (PDT) DKIM-Filter: OpenDKIM Filter v2.11.0 linux.microsoft.com 2753620EAB5F DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux.microsoft.com; s=default; t=1686587562; bh=SEXuybwwN9UYnfUCkee2ZZ6qV1XNjPJ1tnT9wSqJZmY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=doq/hN0fInH/IO1yLNrRLGr4B/orJafM6UjNLKnLSc/i5EqCWaF0N4HX3UzlCSDa8 YHDnZqxzoUjCz58qH1DF9zl0Q6CfLd4+OB/Lcv1xOOxzyYvObDMQxbYKyDf59KF6tY +3BGlL8YpZfX6WmTBOS0+eFipd+/2bKfXNKtujYw= Date: Mon, 12 Jun 2023 09:32:42 -0700 From: Tyler Retzlaff To: David Marchand Cc: Qiming Yang , Wenjun Wu , dev@dpdk.org, Morten =?iso-8859-1?Q?Br=F8rup?= Subject: Re: [Bug 1249] [dpdk-23.07] ixgbe port stop failed after launch testpmd Message-ID: <20230612163242.GB18991@linuxonhyperv3.guj3yctzbm1etfxqx2vob5hsef.xx.internal.cloudapp.net> References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) 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 Hi David, I'm out of office until Thursday so I am not able to properly look right now. If there has been a broad regression here could I ask for your help in either reverting it until I can look at it properly or if you see the obvious problem fixing it. There were multiple instances of flags being converted to bool in the series it may be that they all need to be backed out. Very disappointing if I missed something here. Sorry for the trouble. On Mon, Jun 12, 2023 at 09:43:23AM +0200, David Marchand wrote: > Hello Tyler, > > (I could not assign this bz to you, please register to bugs.dpdk.org) > > On Mon, Jun 12, 2023 at 9:29 AM wrote: > > > > Bug ID 1249 > > Summary [dpdk-23.07] ixgbe port stop failed after launch testpmd > > Product DPDK > > Version 23.07 > > Hardware All > > OS All > > Status UNCONFIRMED > > Severity normal > > Priority Normal > > Component testpmd > > Assignee dev@dpdk.org > > Reporter linglix.chen@intel.com > > Target Milestone --- > > > > Environment > > OS: Ubuntu 22.04.2 LTS/5.15.0-71-generic > > DPDK version: 8a2de735318b38ba93a041cb2f209889bde02689 > > Compiler: gcc version 11.3.0 > > NIC hardware: 82599ES 10-Gigabit SFI/SFP+ Network Connection 10fb > > NIC driver: ixgbe-5.18.11 > > NIC firmware: 0x000161bf > > > > Test Setup > > > > 1. bind 1 port to vfio-pci > > ./usertools/dpdk-devbind.py -b vfio-pci 0000:b1:00.0 > > > > 2. launch testpmd > > x86_64-native-linuxapp-gcc/app/dpdk-testpmd -l 1-2 -n 4 -- -i > > > > 3.testpmd>port stop 0 > > testpmd> show port info all > > > > Show the output from the previous commands > > testpmd>port stop 0 > > Stopping ports... > > Checking link statuses... > > ixgbe_dev_link_update_share(): Other link thread is running now! > > Done > > testpmd> show port info all > > ixgbe_dev_wait_setup_link_complete(): IXGBE link thread not complete too long > > time! > > need kill to quit testpmd. > > > > Expected Result > > port stop normal. > > > > Regression > > Is this issue a regression: (Y/N)Y > > > > Version the regression was introduced: Specify git id if known. > > commit e90baf6b82f66c880c7e7f8f77fc534980165808 > > Author: Tyler Retzlaff > > Date: Tue Jun 6 14:45:45 2023 -0700 > > > > net/ixgbe: replace legacy atomics with GCC builtin atomics > > > > Replace the use of rte_atomic.h types and functions, instead use GCC > > supplied C++11 memory model builtins. > > > > Signed-off-by: Tyler Retzlaff > > Acked-by: Morten Brørup > > I suspect this change broke handling of multiple ports by converting a > counter atomic to a simple boolean. > Can you have a look? > > > Thanks. > > -- > David Marchand