Received: by 10.213.65.68 with SMTP id h4csp106150imn; Thu, 15 Mar 2018 10:56:57 -0700 (PDT) X-Google-Smtp-Source: AG47ELuPnq38Dw8ksZiYXIBNhSzRdScjzj8FGK2qj0ZkxrqLMwq8Trmlodqj/buULedurqP+PsKK X-Received: by 2002:a17:902:4481:: with SMTP id l1-v6mr9116329pld.43.1521136617504; Thu, 15 Mar 2018 10:56:57 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1521136617; cv=none; d=google.com; s=arc-20160816; b=uChLcIT3WUSVSTqZSyBJB9CxFhSfl3PWsKfl0VPV5YV/AG2pY+1YjSrnAFSqZMfr+G ClR+Vlc4RPvYSe8IhwFYd+DHSN49QStuhSUB7z4Wy23COjPQ4VvHU2uRirm0CdMcs4HM Q7SVLhSi9OiTIgLK2Jky0JCaMatJrKqGp9+WBlMoS9OaW4jeMfRxMmtaYz6UjM7NH3tJ mxPL/P0PiF5iogGLkW4KQJMyNNdZz9HNZwazj7IZHWphcewNvjGnPzjdwyZTezumesht 3OT6tjVxDPLOSPUmwRyxjOkAWKGDaWvBDyrSOt6xJrPqv71r1MDok6/W9o+HBrLJT/kj s/VA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:content-transfer-encoding :spamdiagnosticmetadata:spamdiagnosticoutput:msip_labels :content-language:accept-language:in-reply-to:references:message-id :date:thread-index:thread-topic:subject:cc:to:from:dkim-signature :arc-authentication-results; bh=MQ1AaCa6wJrMO2iVE+LGfa8T6qw/iM3B4nINKbaEnUs=; b=z++kOo8kAcfZ7XD1XUX/Sb+9AyaeKNVZcQIHFslogFKPhAylcUy1ZxD8YdoAWv5KF5 v4Fxhg+Ui617nRNOx0+xTh2yVUxudj76Xq4WgozXp7LSQM3VQVX5gBnX8/lFuOqyZ3Ay TlmBsdM+bJhkIL1zGc7ES0fBRfr7KOeRjMfqPCknCfaCcXi+BL1XOxk4gWljCjASGEQ9 xz5MQn+hL6H9ByFCkeMCyVJbQPgizzzUUN0FmRmrDjcQ8SAEloSZkiiaRySzp2mxGm5e J49leFnyI1nzu7v3RcHpHBxvTU/L6lwM7ng6fP/Au4GcamXM9AOXNJ93gnqRWIM9aCjy 0XIA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@microsoft.com header.s=selector1 header.b=iCuQhi/S; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=microsoft.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id u1si3697949pgp.498.2018.03.15.10.56.42; Thu, 15 Mar 2018 10:56:57 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@microsoft.com header.s=selector1 header.b=iCuQhi/S; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=microsoft.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752566AbeCORzV (ORCPT + 99 others); Thu, 15 Mar 2018 13:55:21 -0400 Received: from mail-sg2apc01on0130.outbound.protection.outlook.com ([104.47.125.130]:42568 "EHLO APC01-SG2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752337AbeCORzT (ORCPT ); Thu, 15 Mar 2018 13:55:19 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=MQ1AaCa6wJrMO2iVE+LGfa8T6qw/iM3B4nINKbaEnUs=; b=iCuQhi/SQN/iaHYd9Kxl0iJYV09wudfGYBWiXg/tdly/KQhb3Vawt1+lNdELELzKeo/diP0eyGrnLKCkfz6jAZSgSRWwAZ1TYHREHImvZxJxhNThSlrf6mzpgEUs5i5L8/e2N9OdA4wR4e3vxGNWnvvmVTkMWA/lQRNrFIU9Lr4= Received: from KL1P15301MB0006.APCP153.PROD.OUTLOOK.COM (10.170.167.17) by KL1P15301MB0038.APCP153.PROD.OUTLOOK.COM (10.170.167.151) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.609.6; Thu, 15 Mar 2018 17:55:03 +0000 Received: from KL1P15301MB0006.APCP153.PROD.OUTLOOK.COM ([10.170.167.17]) by KL1P15301MB0006.APCP153.PROD.OUTLOOK.COM ([10.170.167.17]) with mapi id 15.20.0609.006; Thu, 15 Mar 2018 17:55:03 +0000 From: Dexuan Cui To: Lorenzo Pieralisi CC: "bhelgaas@google.com" , "linux-pci@vger.kernel.org" , KY Srinivasan , Stephen Hemminger , "linux-kernel@vger.kernel.org" , "driverdev-devel@linuxdriverproject.org" , Haiyang Zhang , "olaf@aepfle.de" , "apw@canonical.com" , "jasowang@redhat.com" , "vkuznets@redhat.com" , "marcelo.cerri@canonical.com" , "Michael Kelley (EOSG)" Subject: RE: [PATCH v4 1/2] PCI: hv: Serialize the present and eject work items Thread-Topic: [PATCH v4 1/2] PCI: hv: Serialize the present and eject work items Thread-Index: AdO8O3aRSVmxMVp3Qwu6AztIm4xINgAQ81WAAAFLJ6A= Date: Thu, 15 Mar 2018 17:55:03 +0000 Message-ID: References: <20180315170131.GA22134@e107981-ln.cambridge.arm.com> In-Reply-To: <20180315170131.GA22134@e107981-ln.cambridge.arm.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Enabled=True; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SiteId=72f988bf-86f1-41af-91ab-2d7cd011db47; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Owner=decui@microsoft.com; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SetDate=2018-03-15T17:55:01.7168159Z; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Name=General; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Application=Microsoft Azure Information Protection; MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Extended_MSFT_Method=Automatic; Sensitivity=General authentication-results: spf=none (sender IP is ) smtp.mailfrom=decui@microsoft.com; x-originating-ip: [2001:4898:80e8::39a] x-ms-publictraffictype: Email x-microsoft-exchange-diagnostics: 1;KL1P15301MB0038;7:QDPwNnmQ0sXh+tvBT/2XF3DEU6cRGZMIFmM0fXqUsv30EimluB8b0xEu/aNH+I13mkcVzJZ2EsshihoFUyoI5dKJGw0i5Yteu2LPkzEH6/CUJqNPYoI4AbHpNf9sEkgnFnI4puPDmXfRK/GTSkker2beEMJ/7L7Akhc9KaPUaTzpNnWQaoHw2k4eFGcklrXH2VBurmIBM7fWToK2BjIz66U6TmLez2esoVP84tC/Kl0bhpVCu1fURSyKcNMxSxAW;20:ZxDdM6OmoE3SfrE1NS/uqCvUFAvL2r0PYHzC7PIiDr7aLLtkuibVKgLvHbDdIz2uZV6n+jf5A5HWjnx/dwFkt4ZuzWrDzR0Y9yF0fbIAbier6n/fBU4gYHM7zMmTj+9DeCRif4B8pUaPoVMT/yUi5AtYDa7ighhLD9w66wObxOI= x-ms-exchange-antispam-srfa-diagnostics: SSOS;SSOR; x-ms-office365-filtering-ht: Tenant x-ms-office365-filtering-correlation-id: 8760c97c-dab4-4f3e-5c4f-08d58a9de147 x-microsoft-antispam: UriScan:;BCL:0;PCL:0;RULEID:(7020095)(4652020)(48565401081)(5600026)(4604075)(3008032)(4534165)(4627221)(201703031133081)(201702281549075)(2017052603328)(7193020);SRVR:KL1P15301MB0038; x-ms-traffictypediagnostic: KL1P15301MB0038: x-microsoft-antispam-prvs: x-exchange-antispam-report-test: UriScan:(180628864354917)(9452136761055); x-exchange-antispam-report-cfa-test: BCL:0;PCL:0;RULEID:(8211001083)(61425038)(6040522)(2401047)(8121501046)(5005006)(93006095)(93001095)(3231221)(944501244)(52105095)(3002001)(10201501046)(6055026)(61426038)(61427038)(6041310)(20161123562045)(20161123564045)(20161123558120)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123560045)(6072148)(201708071742011);SRVR:KL1P15301MB0038;BCL:0;PCL:0;RULEID:;SRVR:KL1P15301MB0038; x-forefront-prvs: 0612E553B4 x-forefront-antispam-report: SFV:NSPM;SFS:(10019020)(396003)(366004)(346002)(39860400002)(39380400002)(376002)(189003)(199004)(14454004)(46003)(6436002)(10290500003)(186003)(478600001)(5660300001)(2900100001)(4326008)(53936002)(10090500001)(86362001)(575784001)(9686003)(2950100002)(6916009)(97736004)(77096007)(55016002)(6246003)(107886003)(106356001)(105586002)(99286004)(229853002)(7736002)(102836004)(6506007)(2906002)(316002)(25786009)(22452003)(8990500004)(74316002)(7416002)(6116002)(68736007)(76176011)(3660700001)(54906003)(33656002)(305945005)(86612001)(7696005)(8936002)(81166006)(81156014)(3280700002)(8676002);DIR:OUT;SFP:1102;SCL:1;SRVR:KL1P15301MB0038;H:KL1P15301MB0006.APCP153.PROD.OUTLOOK.COM;FPR:;SPF:None;PTR:InfoNoRecords;MX:1;A:1;LANG:en; received-spf: None (protection.outlook.com: microsoft.com does not designate permitted sender hosts) x-microsoft-antispam-message-info: XHoDttLnst+rJKMfZQulYSdAASSC5lhuChl12cNc8Y7E6/eN2PeEuN3Y4FQspalSh8saIHYzgGmuhIoGp6sK4Ubt/CeXtfBvGXu6Xn7vkBBFv3PACk7kaNxoEapw9bARaS9fCaNh+rYSCBvmTz32Svcqu5Tde7jM3ZFL/GpqaIFcgPfEEgPWDBOMTyuG8jDxaqWHnHhyNqrRAXI/8J1O/A81JMlu+T9RxqzcBIbNctoiQDTPB3uCGUo9mCqD2J0/HJq1APd9UWPaCwm+n5EH7W0yRJqorN5ZGsnFyhseeuLNeopj3jVaiwWrXuiZtjlKe9zv/B/WrvbzKK1Z97XDqQ== spamdiagnosticoutput: 1:99 spamdiagnosticmetadata: NSPM Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-Network-Message-Id: 8760c97c-dab4-4f3e-5c4f-08d58a9de147 X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Mar 2018 17:55:03.2310 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-Transport-CrossTenantHeadersStamped: KL1P15301MB0038 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > From: Lorenzo Pieralisi > Sent: Thursday, March 15, 2018 10:02 > On Thu, Mar 15, 2018 at 02:20:53PM +0000, Dexuan Cui wrote: > > When we hot-remove the device, we first receive a PCI_EJECT message and > > then receive a PCI_BUS_RELATIONS message with bus_rel->device_count =3D= =3D > 0. > > > > The first message is offloaded to (), and the second > > is offloaded to pci_devices_present_work(). Both the paths can be runni= ng > > list_del(&hpdev->list_entry), causing general protection fault, because > > system_wq can run them concurrently. > > > > The patch eliminates the race condition. > > > > Cc: stable@vger.kernel.org >=20 > I need to know either what commit you are fixing (ie Fixes: tag - which > is preferrable) or you tell me which kernel versions we are targeting > for the stable backport. >=20 > Thanks, > Lorenzo Sorry. Here I was hesitant to add a "Fixes:" because the bug was there the= first day when the driver was introduced. Please use Fixes: 4daace0d8ce8 ("PCI: hv: Add paravirtual PCI front-end for Microsoft = Hyper-V VMs") or=20 Cc: # v4.6+ Thanks, -- Dexuan