From patchwork Wed Dec 20 22:58:17 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Jim Harris X-Patchwork-Id: 181871 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7300:2483:b0:fb:cd0c:d3e with SMTP id q3csp64850dyi; Wed, 20 Dec 2023 15:11:32 -0800 (PST) X-Google-Smtp-Source: AGHT+IF8lMifOHBv+UVAvwh7tu7bkEef5Q3WN21VPBjoRSYnPhfFVGbhqg33BJgfE5Z/RkNMqP2+ X-Received: by 2002:a05:6a20:5089:b0:190:38cf:15b5 with SMTP id l9-20020a056a20508900b0019038cf15b5mr452998pza.20.1703113892470; Wed, 20 Dec 2023 15:11:32 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703113892; cv=none; d=google.com; s=arc-20160816; b=P9Oo4Tzurq9Zao3b0sH5XGKyBaArbV3x9sROVsfcAZ8K0YcWu5Gord9Ate8XTO+w/9 NRP9WHqjDq3X50CbusWjwPoLof8r1QjFMiruHADy269xGXjilx+2bhgKHffzv1mOk0Gx xs7tw9n0HX6N27KOHyTQBsk0GRZ6fx4W5+muYxWQvrvwNILkcmKWOekBbnEzLNrUBb0W U/SW5XEU7PbhkloZPAyNdolicN9u/FV3PGXQOmtVrLvvczBcExBcFH9DN5RnPGRzLnu8 mlGydI0V5IWpL7y7DqpZjSPCbxw1Gw6RYy6xGEuosNTXsoScAN3xqtzlrBa895y6KrWi F0cQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=references:cms-type:mime-version:list-unsubscribe:list-subscribe :list-id:precedence:content-transfer-encoding:content-id :content-language:accept-language:in-reply-to:message-id:date :thread-index:thread-topic:subject:to:from:dkim-signature :dkim-filter; bh=pWPHRez3lE3xKoymhilbgrompZKvRUVieFbZgZqSp6w=; fh=OtPxTFaFpOC/i6a2eMxR19Qv834xIhrDak+9qqew0ho=; b=qvryTrPAm0bf6IAuYp7of7sEIr54yLBCjoAeA9qL8hWWAa7VHV3theab6smJZCToDv ZxF+zNttloPhpBniv6VhH8q3hUvcYCIQXGjYGaSc67GDh86PbcKqtkKvUxzRua0L2dOF nrvovdwBAyIUA3ZF17hHgkbgIzCBIqkJtfWsFy8JHrSqjHXlOFW8eXXPdOhEM/kirPD+ YkzOG8ExCQlRyWF3L+pI8a3OeMHGB5j2pGVuOh2o8Zf8RoLRZ2w1xhe03/NmVVUhrzpM YScjfAlBLbUN9YF63lCFWA+ovkgsj6HRQ/KoQtCjBSfP4qX7F2cK86wVw4mS542AT16D VgoA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@samsung.com header.s=mail20170921 header.b=cT6EvW6x; spf=pass (google.com: domain of linux-kernel+bounces-7613-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-7613-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=samsung.com Received: from sv.mirrors.kernel.org (sv.mirrors.kernel.org. [2604:1380:45e3:2400::1]) by mx.google.com with ESMTPS id bq6-20020a056a02044600b005c69605f795si526570pgb.75.2023.12.20.15.11.32 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 20 Dec 2023 15:11:32 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-7613-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1; Authentication-Results: mx.google.com; dkim=pass header.i=@samsung.com header.s=mail20170921 header.b=cT6EvW6x; spf=pass (google.com: domain of linux-kernel+bounces-7613-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-7613-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=samsung.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sv.mirrors.kernel.org (Postfix) with ESMTPS id 3D2322831AC for ; Wed, 20 Dec 2023 23:11:32 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 5375B4E1A8; Wed, 20 Dec 2023 22:58:31 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=samsung.com header.i=@samsung.com header.b="cT6EvW6x" X-Original-To: linux-kernel@vger.kernel.org Received: from mailout2.w2.samsung.com (mailout2.w2.samsung.com [211.189.100.12]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 99DB14B5D0; Wed, 20 Dec 2023 22:58:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=samsung.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=samsung.com Received: from uscas1p2.samsung.com (unknown [182.198.245.207]) by mailout2.w2.samsung.com (KnoxPortal) with ESMTP id 20231220225819usoutp02a2b7636fa1a1a83d085f37470b58fa99~irFcombhy3179731797usoutp02i; Wed, 20 Dec 2023 22:58:19 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout2.w2.samsung.com 20231220225819usoutp02a2b7636fa1a1a83d085f37470b58fa99~irFcombhy3179731797usoutp02i DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1703113099; bh=pWPHRez3lE3xKoymhilbgrompZKvRUVieFbZgZqSp6w=; h=From:To:Subject:Date:In-Reply-To:References:From; b=cT6EvW6x3e7bJAP/IZGn/qKfONUzDEXiOSIWIpj7AMOtUHCAJ5O0z42nG3iAosbwE IcF1QFLncvqdY4gWYMkkSyR7NKvJsLjc1US9k+ZTDSZbjBvlJSx9H43Du097VeWvUP T0EHKnhw+sIZr3xRZAa9b8bodqOsVarz9I6NXpjA= Received: from ussmges1new.samsung.com (u109.gpu85.samsung.co.kr [203.254.195.109]) by uscas1p2.samsung.com (KnoxPortal) with ESMTP id 20231220225818uscas1p24cb8016b3d74556b586b975fc854aa4a~irFcKqyMG1793317933uscas1p2D; Wed, 20 Dec 2023 22:58:18 +0000 (GMT) Received: from uscas1p1.samsung.com ( [182.198.245.206]) by ussmges1new.samsung.com (USCPEMTA) with SMTP id CC.12.09678.A8173856; Wed, 20 Dec 2023 17:58:18 -0500 (EST) Received: from ussmgxs2new.samsung.com (u91.gpu85.samsung.co.kr [203.254.195.91]) by uscas1p1.samsung.com (KnoxPortal) with ESMTP id 20231220225818uscas1p1e01e19bc82953b21b473ae1daf17e839~irFblIfRf2098420984uscas1p1A; Wed, 20 Dec 2023 22:58:18 +0000 (GMT) X-AuditID: cbfec36d-85fff700000025ce-fb-6583718a35d0 Received: from SSI-EX2.ssi.samsung.com ( [105.128.3.66]) by ussmgxs2new.samsung.com (USCPEXMTA) with SMTP id EA.4B.09813.A8173856; Wed, 20 Dec 2023 17:58:18 -0500 (EST) Received: from SSI-EX2.ssi.samsung.com (105.128.2.227) by SSI-EX2.ssi.samsung.com (105.128.2.227) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384) id 15.1.2375.24; Wed, 20 Dec 2023 14:58:17 -0800 Received: from SSI-EX2.ssi.samsung.com ([105.128.2.227]) by SSI-EX2.ssi.samsung.com ([105.128.2.227]) with mapi id 15.01.2375.024; Wed, 20 Dec 2023 14:58:17 -0800 From: Jim Harris To: Bjorn Helgaas , "linux-pci@vger.kernel.org" , "linux-kernel@vger.kernel.org" , Leon Romanovsky , "Jason Gunthorpe" , Alex Williamson Subject: [PATCH 1/2] Revert "PCI/IOV: Serialize sysfs sriov_numvfs reads vs writes" Thread-Topic: [PATCH 1/2] Revert "PCI/IOV: Serialize sysfs sriov_numvfs reads vs writes" Thread-Index: AQHaM5gFg92cIiE8Xka1UVYgjQ/z/A== Date: Wed, 20 Dec 2023 22:58:17 +0000 Message-ID: <170311309712.2826.11254260046040224092.stgit@bgt-140510-bm01.eng.stellus.in> In-Reply-To: <170311143880.2826.17853753430536108145.stgit@bgt-140510-bm01.eng.stellus.in> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: Content-ID: <6B10883F28563046AE2025CE533E93D4@ssi.samsung.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-CFilter-Loop: Reflected X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprDKsWRmVeSWpSXmKPExsWy7djXc7pdhc2pBtdnGVp8+9/DZrGkKcPi yr89jBabNjxhsbi8aw6bxdl5x9kc2DwWbCr16G1+x+bxft9VNo/Pm+QCWKK4bFJSczLLUov0 7RK4Mj5+/sVYMIu7YseUT4wNjF+4uhg5OSQETCTedB9j7WLk4hASWMkoMe/+IzYIp5VJ4tXS 18wwVd8ePASzhQTWMEq0XnKDKPrEKDH98UQmCGcZo8S3CQ2sIFVsApoSv66sYQKxRQTmMEnM uOAJYgsLhErsO3+PGSIeJbG/8SAbhK0ncWzHO6B6Dg4WAVWJZQ3xIGFegUiJvq2N7CA2J1D5 nyOfwcoZBcQkvp+CGM8sIC5x68l8JohDBSUWzd4DdbSYxL9dD9kgbEWJ+99fsoOMZwY6bf0u fYhWO4kzyw8wQ9iKElO6H7JDrBWUODnzCQtEq6TEwRU3WEBelBDYwyGx7NZrqF0uEv9W9rFC 2NISf+8ug4pnS6xc3wH2ioRAgUTDkSCIsLXEwj/roU7mk/j76xHjBEblWUg+mIVw3Swk181C ct0sJNctYGRdxSheWlycm55abJiXWq5XnJhbXJqXrpecn7uJEZiATv87nLuDccetj3qHGJk4 GA8xSnAwK4nw7u1sShXiTUmsrEotyo8vKs1JLT7EKM3BoiTOa2h7MllIID2xJDU7NbUgtQgm y8TBKdXApHUoa8a/Y6UHUkweb1AJOSj+d1tJZHTYZZfJe/cZhzqmNW/kcQk4+KboI++b/ZNq 4/9/zv2zor//8cEp8s9nOVnHyVe63zk7bc901/Xzn7/eODf0knPHhQMF+5S0mG0uzlq52ov5 VfFt2Qx35tyeyINtM8si37ILVVQyJho3Zj62bTW9+GrTs3qpaoXs2UtP7+CeP2t5ghC/6vx5 NYdsp3NIddkHai384ef/zn6f3KKMHP2pR/ICGI2aV/90+OGc58OTMydYuuzCGnebO46t+lfe 1a7yuPaf/W3PRL9smxjDeyIBzdcucuyoSLnomfxMMKZUYcm2JSf9ONMkT3JpqTV2sG75fVNA b8HLyUnHlFiKMxINtZiLihMBqaKDJa8DAAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrHIsWRmVeSWpSXmKPExsWS2cDspNtV2JxqsPomr8W3/z1sFkuaMiyu /NvDaLFpwxMWi8u75rBZnJ13nM2BzWPBplKP3uZ3bB7v911l8/i8SS6AJYrLJiU1J7MstUjf LoEr4+PnX4wFs7grdkz5xNjA+IWri5GTQ0LAROLbg4fMXYxcHEICqxgljm99xQThfGKUuDFv GQtIlZDAMkaJ+005IDabgKbErytrmEBsEYE5TBIzLniC2MICoRJrdjYzQsSjJO4eWM4GYetJ HNvxDqieg4NFQFViWUM8SJhXIFKib2sjO8T4SIn2u59ZQWxOoNY/Rz6DtTIKiEl8PwWxillA XOLWk/lMEEcLSCzZc54ZwhaVePn4HyuErShx//tLdpBVzEBnrt+lD9FqJ3Fm+QFmCFtRYkr3 Q3aIEwQlTs58wgLRKilxcMUNlgmM4rOQbJuFMGkWkkmzkEyahWTSAkbWVYzipcXFuekVxUZ5 qeV6xYm5xaV56XrJ+bmbGIFRevrf4egdjLdvfdQ7xMjEwXiIUYKDWUmEd29nU6oQb0piZVVq UX58UWlOavEhRmkOFiVx3rsPNFKFBNITS1KzU1MLUotgskwcnFINTJ236pmytrnvmLP6T/6R mROLQ6fWTI9N2PTLO3VDI//bndOVc5lyp/scMJv2p7ZPzPjsg2vL9qfdUT52+opZm/qB1P6Z 8lErI+d2ljnFd6xcZVMgKN37u+aA4Z0/gXZXtr7Z6RK7Wthy/ux5EQIKx9OSUz9G9B5e5f9/ 3tldLNNzTpx2/as1Z2nbsex8RdU9QqoVz7+5bQq4ueuo5x4erq1yrj/P3PNK3nLFfLfzbt2j hyRmssTXnno+oZV1M8/3VDOXPb1TN7btmnnuVdDevn/xZ6Nftf1gKjxgKfDfdI094/eTJ2Y7 7fzt7njrYYyeT3PPw6c6b1+ZvmM5NftKSeLHh8fvCb34wnX3hNnrAxvXK7EUZyQaajEXFScC ABZb+LNBAwAA X-CMS-MailID: 20231220225818uscas1p1e01e19bc82953b21b473ae1daf17e839 CMS-TYPE: 301P X-CMS-RootMailID: 20231220225818uscas1p1e01e19bc82953b21b473ae1daf17e839 References: <170311143880.2826.17853753430536108145.stgit@bgt-140510-bm01.eng.stellus.in> X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1785844353076479468 X-GMAIL-MSGID: 1785844353076479468 The proper way to detect a change to the num_VFs value is to listen for a sysfs event, not to add a device_lock() on the attribute _show() in the kernel. This reverts commit 35ff867b76576e32f34c698ccd11343f7d616204. Revert had a small conflict, the sprintf() is now changed to sysfs_emit(). Suggested-by: Leon Romanovsky Signed-off-by: Jim Harris Reviewed-by: Leon Romanovsky --- drivers/pci/iov.c | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/drivers/pci/iov.c b/drivers/pci/iov.c index 25dbe85c4217..d4646bdcd887 100644 --- a/drivers/pci/iov.c +++ b/drivers/pci/iov.c @@ -395,14 +395,8 @@ static ssize_t sriov_numvfs_show(struct device *dev, char *buf) { struct pci_dev *pdev = to_pci_dev(dev); - u16 num_vfs; - - /* Serialize vs sriov_numvfs_store() so readers see valid num_VFs */ - device_lock(&pdev->dev); - num_vfs = pdev->sriov->num_VFs; - device_unlock(&pdev->dev); - return sysfs_emit(buf, "%u\n", num_vfs); + return sysfs_emit(buf, "%u\n", pdev->sriov->num_VFs); } /*