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 9A446427F6 for ; Tue, 21 Mar 2023 09:41:09 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 87FE8410EF; Tue, 21 Mar 2023 09:41:09 +0100 (CET) Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by mails.dpdk.org (Postfix) with ESMTP id 5B62140A7A; Tue, 21 Mar 2023 09:41:06 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1679388066; x=1710924066; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=jNMaHlhQ5Kq528XFF8BrDsrNaM2nO9NpYdCbJYbDanM=; b=nOlVSUYaYOTU6NM6uoNUpVyzjEYexE4oIdnMRLDzXcu3XZeTAdJoQzhO EF9GpI7IowKaD4VO52a1gHkEEKLqIvJ51YKRnbDHJBo70B61iGE83fmwf spBKH3Lpx5Z92RPuvnm/1kdo3PEW76VWKdB1Ay8FCgIOQ9OlTnQm7JHJZ IM2PA0DHy2fREa/b77bSE+ntCt8lMI+UnNvZcwmO4SqAXZS8xSbofrN9R PCxSA0/NtWLvPAnOl2VGnXvd0AgnBvHlJsAvI+uUMRiiLd/3EYneCrMBf Jo2XJsgQGAfwsYa4GTb53CwaAFH4sjG6LrQ6XuJWETs3BpK/FyTMkNou9 g==; X-IronPort-AV: E=McAfee;i="6600,9927,10655"; a="340427846" X-IronPort-AV: E=Sophos;i="5.98,278,1673942400"; d="scan'208";a="340427846" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by orsmga102.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 21 Mar 2023 01:41:05 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10655"; a="745769566" X-IronPort-AV: E=Sophos;i="5.98,278,1673942400"; d="scan'208";a="745769566" Received: from orsmsx603.amr.corp.intel.com ([10.22.229.16]) by fmsmga008.fm.intel.com with ESMTP; 21 Mar 2023 01:41:05 -0700 Received: from orsmsx611.amr.corp.intel.com (10.22.229.24) by ORSMSX603.amr.corp.intel.com (10.22.229.16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.21; Tue, 21 Mar 2023 01:41:04 -0700 Received: from orsmsx610.amr.corp.intel.com (10.22.229.23) by ORSMSX611.amr.corp.intel.com (10.22.229.24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.21; Tue, 21 Mar 2023 01:41:04 -0700 Received: from ORSEDG602.ED.cps.intel.com (10.7.248.7) by orsmsx610.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.21 via Frontend Transport; Tue, 21 Mar 2023 01:41:04 -0700 Received: from NAM11-DM6-obe.outbound.protection.outlook.com (104.47.57.170) by edgegateway.intel.com (134.134.137.103) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.21; Tue, 21 Mar 2023 01:41:04 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EDKsnPGY8avdl0hc9OVH+q3uJWezzEAnLpBMcTwuhGwVz91yVClB5xoBJQoIrXrSERImse3sO9FXHf7hYY4navHSOgm/ewe9hn/wWEv8+rEcaW87eHEI0U87KfufwhhifnAe4N1tWnKzgfr1mVFi19fY/RW8ILY3IsdqAb3AtK3XrfaenBEvTokVHTXEaZrDzmXCXSTd7xoY7XA959JXU8GHvWv+ENQ+/SujdsU5qoeqhk8KV2HrqtyERiktx1bXKj0/7udmmCkN/0/ntPBWyXBnN9Pghisl3ff9N16Fjm6OloB87Ihmdap0VkNh1ilSeLwoWc3YZpTiGRZVCpaYKw== 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=jNMaHlhQ5Kq528XFF8BrDsrNaM2nO9NpYdCbJYbDanM=; b=Mo9+RTD5GIRRot8xdoCEoBF8jBhCbFDsupAYIVRB1PFO8rlCFJlMDJIXXj8k/PFn82+AvPWS8WVOfWyqQlzn13rFZWOaGfx7EZBNfOMMRK3Sx97hPT5GBT1UXGnNNGvgQVFtWL5QtP4LpiH5OB57pDxsa/ihxjLP1eBVMEWENUnDelf14O/uJlNYyglGoQyaGXtcSP7fuQ7o98NBiagiES0qhYGgFl4J3mQpi8H9jV9ze/HRLI0KN11kFmlZyTBSOtYAB5VZxm+jadv14fgRJvV2UDmIxYozfE8wynVFOhdJaNZq81PzxqcuApGLhAn3uk0CCoh741bmPljttdw3Zg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none Received: from DM6PR11MB3530.namprd11.prod.outlook.com (2603:10b6:5:72::19) by DM4PR11MB6334.namprd11.prod.outlook.com (2603:10b6:8:b5::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6178.37; Tue, 21 Mar 2023 08:41:02 +0000 Received: from DM6PR11MB3530.namprd11.prod.outlook.com ([fe80::2a1b:9bf9:445f:567f]) by DM6PR11MB3530.namprd11.prod.outlook.com ([fe80::2a1b:9bf9:445f:567f%3]) with mapi id 15.20.6178.037; Tue, 21 Mar 2023 08:41:02 +0000 From: "Huang, Wei" To: Thomas Monjalon CC: "dev@dpdk.org" , "david.marchand@redhat.com" , "stable@dpdk.org" , "Xu, Rosen" , "Zhang, Tianfei" , "Zhang, Qi Z" Subject: RE: [PATCH v1] raw/ifpga: remove virtual device unplug operation Thread-Topic: [PATCH v1] raw/ifpga: remove virtual device unplug operation Thread-Index: AQHZWHHvun1srISqFUW9KK/CSBP1OK8D2uEAgACG2SCAAIedgIAAAFNw Date: Tue, 21 Mar 2023 08:41:02 +0000 Message-ID: References: <20230316204445.360330-1-wei.huang@intel.com> <6411222.K2JlShyGXD@thomas> <18059490.sWSEgdgrri@thomas> In-Reply-To: <18059490.sWSEgdgrri@thomas> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DM6PR11MB3530:EE_|DM4PR11MB6334:EE_ x-ms-office365-filtering-correlation-id: 2a6e9f89-81d8-4423-223a-08db29e800cd x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: YvTOpz2rYlHoai1rftCkaO6NtzhnvFndK/66Knz3u/oaKeHeirIovmmxeQLagsW1OoIZ9pEkhk7hQ0SFR+g2HDXTsuSl+Fi5UfYEc+V/FgITvZURUsyne11OyM0uiHpa8SfsqSrcUFVjxCCtqNKqcYML2zoccxlQtipv7lLzVqz6TgBXA0QUjh4WZNepWYDGsiI5MZTw2odXlRsyXQG6GcaoBtib0gd2YUxRv4ErtP1GjUZyvemvsv2BgWN5vAFtpdfg+hx9iplB9ZqEwyPx/0TnYjddmTZxODS9jmYmQm5ylOqiDPrX/c1liQn4c0E1V/cq/akM6ruZWolcpSo+aQXHvF0zum8g3bxgrbpfK/HWLt+bib4E/EfrXOgusb8u12QejbKB5ACtkosS3MngUnpkYdJnPqKs7KE/feamIFv4LIk+DbRkmYwIOmHXdeyeYzSBKwgwFQWfkZRtYGwYV+YX0VnWyBoyUuyDjkMzxyyLlOwc1wfv+uodVLlONmpSd2kpm0VGJOQ5HHIzlL09p2JGJ4uewTUB+3acEeGAJXAWvkx0/ZWO7F01CdQ3iq9sktimGhPeaF/wb4HDREVCCkbF9t1C8lzPtX2nQRAzidk3X4IzIl6eB89EiJmHh/5yuQmhGGMwFJVjNosMdIeHSY3QJckrU+sxrBEBebwJtJLF6NPQl9VO6VuaGkSFPlB7UuyPozW7D7FOQ5Tsu0QNLA== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DM6PR11MB3530.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230025)(376002)(396003)(136003)(346002)(39860400002)(366004)(451199018)(52536014)(66476007)(8676002)(66446008)(5660300002)(4326008)(6916009)(76116006)(64756008)(66556008)(66946007)(8936002)(41300700001)(26005)(186003)(54906003)(316002)(83380400001)(478600001)(2906002)(71200400001)(7696005)(9686003)(53546011)(6506007)(55016003)(33656002)(107886003)(38100700002)(86362001)(38070700005)(122000001)(82960400001); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?IOE5V15ZqUFH+UJzmvXewbncQfvZADwEUnZ9Txqbc1EgBIF8bTs/JpdElh+7?= =?us-ascii?Q?W5uY/BXqCKQ0G/7PpXuB0gHOsSe+mPcEID14ozNhcW8i4s0aSTV8BlQPXIfC?= =?us-ascii?Q?DHPnA94LVc/tEV21XwWmKI3h3UH7B8Fd+gYYxwiTePMSGyfQvslgXJjt/U77?= =?us-ascii?Q?W2W7Wkn3yCXsm4NOSVuwLAsZSHSUqjXOWr5X2kLGwdVpA+fJxOaqkl3rRrGd?= =?us-ascii?Q?WR/oSuCV8mwNVWQGZ6Ks352QYN+SHXW2IPNUUjjjcJdHM5DIGcf4IwIT45cR?= =?us-ascii?Q?4kC+q/316Tjwau231PEFNqOLvqPGpHfmAITRLvC2miSors23KxTXPkzKHG+Y?= =?us-ascii?Q?iPGMFrGfIIflbnVUU8IDWSJzJOfHEhJvRduoODuClDLQ8YMI7rPSapoowMuU?= =?us-ascii?Q?02+h32CEe9UhEtRDBuqlEpPlSigpdYWCxHJoDbAPnrWBuf+mdQXa7vPl8O/s?= =?us-ascii?Q?qft5Ed3l6MGGbhXYSSyS77YdXN7sgt3b1PxAh+2OtaPHhQclE8vOKuqEXF0b?= =?us-ascii?Q?+VzkyDNizgao9QxQ0HzWtGedQMzCrm3dv3HChTL6wHi4BT7OUD9v4zOYEGe1?= =?us-ascii?Q?bEKGCS1J+HocH8Nt+Og+OgcIHwyhOlwBejzWPShh8UI7HnforKy+hv0MYk06?= =?us-ascii?Q?LT44xkRK4xh2DvELYn9yMAieG4iXmMWc8ezZDiKX3/ce2guTd1VbYXjt/5Yv?= =?us-ascii?Q?Dw+xANHfrptxMu6bF5xE1S+2mEodhJ7tl9dLAjOId9ocLHovGZ0m0qigyEGV?= =?us-ascii?Q?dFsyLidB61xfel2o2DmmUXeU+N4NQiTR2eFnnj5l77LfRnN3usaHrK/YKx38?= =?us-ascii?Q?CktugrBbMrQK+ERVA8RpQ8r84WJ3MKTk2X7/oBz12FqDC+2lbFFPxsAUNbIX?= =?us-ascii?Q?uck5lXNLpjdir8z+7JmtmZO58ZFmsAzlbG/ynQdGzsU0zx0zaQZgxnk5+LdB?= =?us-ascii?Q?zJrBv9B28XwD/dHVjap8h7cKDZLlzAbokZh9TSO56TELIJ9jVe0+6q6/6GOu?= =?us-ascii?Q?fXp5eLwMGBknheuKb9edB44TX7nborsRcGtmsXYE5bkvrMa9pgSny5cl9x7d?= =?us-ascii?Q?HINo4K+JtTvnyBp+3Piis4tqYgalIyBTPBowF7kSfv69wqyWLNSSVrRSO3ol?= =?us-ascii?Q?kWdsRFgztkpKmF8XgFkxTRerWfiqaTvvZPQFYmi46Zb3MhK42+EhZhP/Yo5D?= =?us-ascii?Q?/ryfIVR6XYgNlWjBczGc3vsUFvzIARzIIToQ/QtV0ntVBHHO6YkU/01l0Qek?= =?us-ascii?Q?lFojOdvghgZG1mMptUOO4PQy4/1RjImBZWn/jMotbRSmT21h3Syg3Z7ONo8l?= =?us-ascii?Q?zNFR23Lp818untmhTXBuVWgY2m/deC9VzlUO+CUV2f2a9+wOhAli+LZEIToY?= =?us-ascii?Q?shveqFJX0pSrkSFln0Vc6i4QjyqGMS84HSoRXUDik08IYi+KG0lUYYRwbw4C?= =?us-ascii?Q?U2PEj5nuS2JeIan4iCgwQLIeahJZFulcM7MYOcIqJVI/igNNIJeSmsm+lXT0?= =?us-ascii?Q?0OCsW+7AYw1ilWM34zHCKQjUKThF0gQ0yhVu3DiAHWppos3ujgiI57epj+zb?= =?us-ascii?Q?WVccHPKBABHD2h1yP04lA83ZtHCXBiWQ+j67EZ1I?= Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM6PR11MB3530.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 2a6e9f89-81d8-4423-223a-08db29e800cd X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Mar 2023 08:41:02.5496 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: lObL5pJ1DYgedeug4XSTm2+O7qbW+jK2fpAMHSL1UsHe5l7AJU9wBRCNwZBWs42b0Ncmp0Nbl2creci5Cr0hGQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR11MB6334 X-OriginatorOrg: intel.com X-BeenThere: stable@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: patches for DPDK stable branches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: stable-bounces@dpdk.org Hi Thomas, > -----Original Message----- > From: Thomas Monjalon > Sent: Tuesday, March 21, 2023 16:14 > To: Huang, Wei > Cc: dev@dpdk.org; david.marchand@redhat.com; stable@dpdk.org; Xu, > Rosen ; Zhang, Tianfei ; > Zhang, Qi Z > Subject: Re: [PATCH v1] raw/ifpga: remove virtual device unplug operation >=20 > 21/03/2023 01:11, Huang, Wei: > > From: Thomas Monjalon > > > 16/03/2023 21:44, Wei Huang: > > > > VDEV bus has implemented cleanup() function to perform cleanup for > > > > devices on the bus during eal_cleanup(), so there is no need for > > > > ifpga driver to record virtual devices and unplug them. > > > > > > Why no need? > > > If the application wants to explicitly remove a device, what happens? > > > > > > > > EAL will output an error information "Cannot find plugged device (%s)". >=20 > It does not look what we expect. >=20 Let me clear it. With this patch, no error information will be outputted. Without this patch, error information will be outputted. Because bus cleanu= p action will unplug virtual device, then ifpga PMD unplug the virtual devi= ce which is already be cleanup, bus->find_device() returns NULL, EAL output= "Cannot find plugged device (%s)\n" at line 302 in eal_common_dev.c