From patchwork Wed Sep 27 18:20:39 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vadim Lomovtsev X-Patchwork-Id: 819271 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=linux-pci-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=CAVIUMNETWORKS.onmicrosoft.com header.i=@CAVIUMNETWORKS.onmicrosoft.com header.b="BpeVcuc8"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3y2R2t0YBmz9t1G for ; Thu, 28 Sep 2017 04:20:54 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751590AbdI0SUw (ORCPT ); Wed, 27 Sep 2017 14:20:52 -0400 Received: from mail-by2nam01on0081.outbound.protection.outlook.com ([104.47.34.81]:16112 "EHLO NAM01-BY2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751016AbdI0SUv (ORCPT ); Wed, 27 Sep 2017 14:20:51 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=EgHqPirP4UmwC4rkkS68Er3NPYt1jpjLirFfFHyRyxE=; b=BpeVcuc8rhxPLkbMp9MS3pOctpTwZW8DAIEvk9jNirvyD3yzq/IE2F2G+lSyb0BqpBzvJv4f6qFZJS/sIdoyYVqcpeAXCGz2l+7Bdn3LMd2X0npaH9eFhenWT/6/zj713dll5gRHMECuG8tLpoQyDE2H0I6PbNJsLtAt3WWwq84= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Vadim.Lomovtsev@cavium.com; Received: from localhost.localdomain.com (50.233.148.156) by CY4PR07MB3000.namprd07.prod.outlook.com (10.172.116.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Wed, 27 Sep 2017 18:20:48 +0000 From: Vadim Lomovtsev To: bhelgaas@google.com, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org Cc: Wilson.Snyder@cavium.com, alex.williamson@redhat.com, Vadim Lomovtsev Subject: [PATCH v6] PCI: quirks: update Cavium ThunderX ACS quirk implementation Date: Wed, 27 Sep 2017 11:20:39 -0700 Message-Id: <1506536439-29318-1-git-send-email-Vadim.Lomovtsev@caviumnetworks.com> X-Mailer: git-send-email 2.4.11 In-Reply-To: <1506344920-24016-1-git-send-email-Vadim.Lomovtsev@caviumnetworks.com> References: <1506344920-24016-1-git-send-email-Vadim.Lomovtsev@caviumnetworks.com> MIME-Version: 1.0 X-Originating-IP: [50.233.148.156] X-ClientProxiedBy: BN6PR11CA0005.namprd11.prod.outlook.com (10.172.17.15) To CY4PR07MB3000.namprd07.prod.outlook.com (10.172.116.14) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 7b361384-85ba-436b-56a4-08d505d47abb X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254152)(2017052603199)(201703131423075)(201703031133081)(201702281549075); SRVR:CY4PR07MB3000; X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB3000; 3:1Ld/Axbp+/6nSFcPEh79n1pf7YP0v0tIzROyyObLsd9FV7kdroiN6S6ApCkop7jbVJh3LJOrke81Ut64L4zdCr/mv7anq6CWCUn0qoZvesl2Ga9Q/WvdjamoMMoMP3Oy7bnR6lDe777qe9MRFebLhp0KZxQzg6EHMhJ19QsYydZ1e8ajP3ZjAW0rooeJRzHrBC4ZJ9LKDvuUz5KbS2f3//dzm2WWtBQ59aqTO1TChVaFoK40WScGl2PO06Vs31xE; 25:awwNEDc2i3Bcsr/9zZdDYltmX8E0dU0HFJfhiwDdamh5Q060I2Lyu/aTLvYLaDpN3b7SHJz2MsBUtiWRkn2hAHLBBO0R3u0uGwV07gJykquA7E9BN4XkxaAF9fhmXJ8r5ABu9xKxO3YpI8O8RD0U5VkFzdOahwzorr688MZho18ASR2O/xoDkJMiplulhqjfIwcV9ZzaZQrM09zcZJyY0r4IVVBCoCMlXbr6UtvDEQTrjNqc3b6XTxbO0GC2WWwmHOjcmIKZKNoFYBTSL9BnNGFtAY33wagWjPbJ0hkCyq1sqqRckcjmhOgrW+u2AX0AM+yeNdwiG0yokDHlhnf/gw==; 31:4rFMFpl1pz1VRnjw0n5AZDD9+42CHe25HTbCgxfXeeI2ne2NT6rXpc0D6bzVYyHQjeHwYMNSzHG7XPhUOgxXmB8p08jypHrDCrN9S/LnFKfotXFnrJWMX+Rwp8H8kULs+9njGXNYp8eZ31UsbjhzVXL9kSXMciXG/TO4woRBzC8n93uMUyl7Ph7oP/NJbHW3Sd+j1GHajQTGCV8S3KAMGMuMJ+NtvIRL1xvgjxIcnTo= X-MS-TrafficTypeDiagnostic: CY4PR07MB3000: X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB3000; 20:Q2x0habjPjNxbSIeYJ2Jw5ieWzedzRgjqwPfpOLyOPjrTTQXtJ63kpjad1Wgxs5nbGsfXZ8KggzVmMxxlXyJ/Skp4Ui+w21rYuO0wbfLYoQIcGrl4qCYzXVARt/NN/M9sBggU8lQcCQDPpjyOzybxeF19GCWSaDLDNXuQmzFb/oDRtTdvMNYb7jXwD7/RzNZesp9rvGjqk+2i8kHAUDZkcxfz1NJtT2FvjtZhzfPZpfc5z4sApItvRBjXja2CnX3fJbC9zVUoBbfKETW/Xr7uSq5FMsdHJFiOVWCjNKhlGp3Xa/aexBuRXt6gkEvEE4Y29avUhgRHQ1G9/3QX11rwZK5O4BXC5uXv8qko/rAa7xtnNG9dR8EQ5IVvLwypkfqnl3iDKF5GwgIyafWCj34dRdTkdaqxU52wK8m95B8Dgnr47tNPgmD664pEytYyhaVmPFrdgChv8sdY/rVhI8mP58FmYpuf9+vNRapbRK8u3E2JZz4gSITMP8gC4Q7kZAvMgAas+jUAPwOdnbzgLdRWhacYDnLlbdzoDZTrpB0+hkrtwVDNhZ82JtD4vyoDgEHCgFtLEbK6ndapSqmBX1UQTRmiQuLfw74FleSOrs0Txk=; 4:1UyzBMkNCsER7DMPclycyDZMDFION1B+o3S7eCZPjEjYf8huPTLaJ9LfKC2bDCEQ/YFLmzINb4K+1oTFu30c1mDeC8mWc3ZMIb7xbD/NT5G6rOg6mIRT20ahfCNsFc0D4t4v5hiegbAOnekPJ7NHLcoaHpdVnAnoR8bb3hmhVv1tVOpnyfjgcy19NavlECDQOopo+5YI9zUZI5QZHlVs8kPctgxdB6hcdXZWv5BXEk7G1h/7lfvvUfUmqbDhSfn7 X-Exchange-Antispam-Report-Test: UriScan:; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(8121501046)(5005006)(93006095)(10201501046)(3002001)(100000703101)(100105400095)(6041248)(20161123560025)(20161123555025)(20161123558100)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123564025)(20161123562025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:CY4PR07MB3000; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:CY4PR07MB3000; X-Forefront-PRVS: 04433051BF X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6009001)(6069001)(376002)(346002)(189002)(199003)(81166006)(101416001)(69596002)(42882006)(478600001)(50226002)(48376002)(6486002)(66066001)(5660300001)(16526017)(2950100002)(106356001)(50466002)(53936002)(4326008)(6506006)(5003940100001)(316002)(105586002)(189998001)(8936002)(68736007)(36756003)(25786009)(76176999)(97736004)(72206003)(81156014)(16586007)(47776003)(6512007)(50986999)(15650500001)(575784001)(3846002)(6666003)(6116002)(107886003)(2906002)(7736002)(305945005)(53416004)(8676002); DIR:OUT; SFP:1101; SCL:1; SRVR:CY4PR07MB3000; H:localhost.localdomain.com; FPR:; SPF:None; PTR:InfoNoRecords; MX:1; A:1; LANG:en; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB3000; 23:fKmMSLUGCxerj49TWnz4z0Aagy11gPgd4PDXO/u+jsnLy5o1ZswHcDuFrU4koNok5n7a1vGoRNvdkWuIwLUxInIogpcpGssgUVEVDOYKyEH2kCLF88vumADbdVCi2XVaguTKlcZ0GqTAyd4oHygDUTZMi/dxhbNjyvoJVJeKLfHtO02oS5ExjXczXZHsHa3X1V/+/wmCxI1OCs8G9T/NR/6EXQ4go41Zb00tyYrAg6xVx/HHl7bwwnhdi8y7NSwbtdOrtp/7w8PpsKKQzlEFx0v7sWbQHvlwXPqc68f3k9H8LsUIVHktl7eD9JRKGRX7nd34idTZs+SnkRJgYv5xnbkaoUDhtNGTJ6W3Ucgl2k3KGdm9p6TVuIY8lLpoON37l8pUq3vVXmllHipsAIEnGUlsc58eIsTFPdFco4O6HuLi77aKwSnfG+sKvoWyl/fwrb/w9m4Tp9qNM0nDZuOugtGefpBwxGuR3aiV0reXRcjbmWBcCjGW48JQNbfh7EEapCKyHqeA9JIOC/iu/RmpwaIhts9D6Q6EOtLCuNpVNOVzZJy7rBzKo6aiC6j2RF2hV4gTo1jt7Ui+eWR5fe8MrJHOZvYjyan5+RsSkLx+2XUmCQJvvQ6bfpbrBj5C3NXgEVFEZKbp6dKinKF4tsXQfeCJqc745T2dts5j7+6m0gIcu9L3myTxjPHiilQZk7JYCSyDj5Gig/EIjEUI0wvuKbK6kzVbIVeYL4UOvPoLh/XGmN1MbuPfj7tkIGzOo1MD9FP6ovp6uuNLXn/eXnYgz+tlyVNs0xVqQhP/dyfhqIKy5E8DBPK+WxRcmUw2FRuugYXJZpbnqomGwvnZCL6XrdfuIwML5j5xAe6U0ctmb0tkau+IhT1vdtncfvQinf8Dlf1bPv09tlvVU5zS6eToWV66wcoMVPTrTtu4ckSzie5K/E9QYYLb9b7nFjDuhdbwvIe/+DjZ6iLLjQqdXdChBvLyt2Q7c2SEMMt2Q2/71mDEPHBBT0pGhVP7gnhuvzeLBFTRmu6XN2PleAoJyXR8+/gUYzIrXdlBuCZ/NNs5WFcO28UmZ1589c7dik4HBSYgUVpCDihWotF/SSNx4Rf6XO43UnniYkfCK2aKMOK9XtB6rT2grIV8zAUJIeQcATSHqhcnwizKwlX4O/WnqumTreCCWEOvHpv6ch3eOfNd5no= X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB3000; 6:Uxgsp4tiauz5EhxlNMaW3C+5LXY24fM40fKIQJUwR0Adhw42dDRmV5WkStvEd1LMmPLnm2031oPh2QJqvAuTV0fzqLogzvsKm0YvvbG83oJHaWJn/MPZSq8w1uU4t75ZTQsv6O6MuMl+ePjg0I65OyQ7I9QgG/GP9aAB+BSumxPGPknTpc7Au3yVE9gU7K3rVGRyVTimr8c0NF5nDT34CJxoaDbodM/Brv+hV94D8hPRFRiyU9rqzRf7e57Y6uUJf5U5f2RZfAmzvGAJgT/wVwD8+hmWsXs/8qUNpHa1XPiw7gy9cLgEK3BTtcvVszRLr7DAmBcg2EMTxhhufteGlg==; 5:yALZUyUw+Z1pIfq38XAjo1mdw50GMnRm6X6w+dp1hcrf3/JS/s9hPSajJRKCZmGd6agCngaKGQyUH3RRAN94A4Z58JDrJf5bhB/63vyWBXSRhpVQZq6K+JbbPjYCBPijOOuhM0LU1Zk15S4YEvNNSg==; 24:ljf/PEar3FZJCZn90jq76Gj/HxAqo5oeU2dQPVTm2FG3KKAPgmcZ5qPRn6EZ2dwi6r/HH+w5YzoJL7pl06bAhZh//uIkBfLcadhmZMLwM6E=; 7:4dV/CB9pdVmV0y6leIbCVapkAkLbs2nTtLj4kmdBzMbNP5rkr+mY6Yh5PUgJn7o0uhlyjq9Tx16LR3q75An2tCbg68AND9//H3VmxgKEo0gMk0iTcr4EtqBubJJYNa7CY7Um9Ln2EMsHt+my3vsB1ChniID8VNNyi60BgqarWNRXCU58c0K1NsRaAcB7xe1sJjHRBU4ZuaNu95hZK/k/IDsISukM7oy3upYE/eQIFh0= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Sep 2017 18:20:48.3096 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR07MB3000 Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org This commit makes Cavium PCI ACS quirk applicable only to Cavium ThunderX (CN8XXX) family PCIE Root Ports which has limited PCI capabilities in terms of no ACS support advertisement. However, the RTL internally implements similar protection as if ACS had completion/request redirection, upstream forwarding and validation features enabled. Current quirk implementation doesn't take into account PCIERCs which also needs to be quirked. So the pci device id check mask is updated and check of device ID moved into separate function. Signed-off-by: Vadim Lomovtsev Reviewed-by: Alex Williamson --- v5 -> v6: comment typo fix: change 0xa00 to 0xa000 drivers/pci/quirks.c | 29 +++++++++++++++++++++-------- 1 file changed, 21 insertions(+), 8 deletions(-) diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index a4d3361..ed6c76d 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c @@ -4211,20 +4211,33 @@ static int pci_quirk_amd_sb_acs(struct pci_dev *dev, u16 acs_flags) #endif } -static int pci_quirk_cavium_acs(struct pci_dev *dev, u16 acs_flags) +/* + * The Cavium downstream ports doesn't advertise their ACS capability registers. + * However, the RTL internally implements similar protection as if + * ACS had completion redirection, forwarding and validation features enabled. + * So by this flags we're asserting that the hardware implements and + * enables equivalent ACS functionality for these flags. + */ +#define CAVIUM_CN8XXX_ACS_FLAGS (PCI_ACS_RR | PCI_ACS_CR | PCI_ACS_SV | PCI_ACS_UF) + +static __inline__ bool pci_quirk_cavium_acs_match(struct pci_dev *dev) { /* - * Cavium devices matching this quirk do not perform peer-to-peer - * with other functions, allowing masking out these bits as if they - * were unimplemented in the ACS capability. + * Effectively selects all downstream ports for whole ThunderX 1 family + * by 0xa000 mask (which represents 8 SoCs), while the lower bits of device ID + * are used to indicate which subdevice is used within the SoC. */ - acs_flags &= ~(PCI_ACS_SV | PCI_ACS_TB | PCI_ACS_RR | - PCI_ACS_CR | PCI_ACS_UF | PCI_ACS_DT); + return (pci_is_pcie(dev) && + (pci_pcie_type(dev) == PCI_EXP_TYPE_ROOT_PORT) && + ((dev->device & 0xf800) == 0xa000)); +} - if (!((dev->device >= 0xa000) && (dev->device <= 0xa0ff))) +static int pci_quirk_cavium_acs(struct pci_dev *dev, u16 acs_flags) +{ + if (!pci_quirk_cavium_acs_match(dev)) return -ENOTTY; - return acs_flags ? 0 : 1; + return acs_flags & ~(CAVIUM_CN8XXX_ACS_FLAGS) ? 0 : 1; } static int pci_quirk_xgene_acs(struct pci_dev *dev, u16 acs_flags) From patchwork Tue Oct 17 12:47:39 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vadim Lomovtsev X-Patchwork-Id: 826951 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Authentication-Results: ozlabs.org; spf=none (mailfrom) smtp.mailfrom=vger.kernel.org (client-ip=209.132.180.67; helo=vger.kernel.org; envelope-from=linux-pci-owner@vger.kernel.org; receiver=) Authentication-Results: ozlabs.org; dkim=pass (1024-bit key; unprotected) header.d=CAVIUMNETWORKS.onmicrosoft.com header.i=@CAVIUMNETWORKS.onmicrosoft.com header.b="mJmLvLQP"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3yGZkB2PW6z9rxl for ; Tue, 17 Oct 2017 23:48:34 +1100 (AEDT) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1761970AbdJQMsc (ORCPT ); Tue, 17 Oct 2017 08:48:32 -0400 Received: from mail-by2nam01on0084.outbound.protection.outlook.com ([104.47.34.84]:30784 "EHLO NAM01-BY2-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1761949AbdJQMs2 (ORCPT ); Tue, 17 Oct 2017 08:48:28 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=CAVIUMNETWORKS.onmicrosoft.com; s=selector1-cavium-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version; bh=8/sKH4bmj0HEgjpQEKTCycN6whNKcf8Dc0dB0CVE71g=; b=mJmLvLQPxPVv+SkA64170zVrKVj3Y3aPkciKB5vB9+v0511gXr/W2lHndEThodttYkZYZwrAo1dQsh4kdEJekUhOgNZ1s3+/ahMNqsdXA5/LOV3HflD5Ap2AdJ7+AxWQ53b3a+EftvTZPaLyTJ0zO5Tb8Dmgssvz8A5cGdQy6FM= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Vadim.Lomovtsev@cavium.com; Received: from localhost.localdomain.com (50.233.148.156) by BN6PR07MB2993.namprd07.prod.outlook.com (10.172.106.11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.77.7; Tue, 17 Oct 2017 12:48:25 +0000 From: Vadim Lomovtsev To: bhelgaas@google.com, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, alex.williamson@redhat.com Cc: Wilson.Snyder@cavium.com, robert.richter@cavium.com, david.daney@cavium.com, mjaggi@caviumnetworks.com, Vadim Lomovtsev Subject: [PATCH v7 2/2] PCI: quirks: Apply Cavium ThunderX ACS quirk only to Root Ports Date: Tue, 17 Oct 2017 05:47:39 -0700 Message-Id: <1508244459-2615-3-git-send-email-Vadim.Lomovtsev@caviumnetworks.com> X-Mailer: git-send-email 2.4.11 In-Reply-To: <1508244459-2615-1-git-send-email-Vadim.Lomovtsev@caviumnetworks.com> References: <1506536439-29318-1-git-send-email-Vadim.Lomovtsev@caviumnetworks.com> <1508244459-2615-1-git-send-email-Vadim.Lomovtsev@caviumnetworks.com> MIME-Version: 1.0 X-Originating-IP: [50.233.148.156] X-ClientProxiedBy: MWHPR15CA0050.namprd15.prod.outlook.com (10.174.254.12) To BN6PR07MB2993.namprd07.prod.outlook.com (10.172.106.11) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 728b242d-3cd0-456d-10e6-08d5155d5c60 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(22001)(2017030254152)(2017052603199)(201703131423075)(201703031133081)(201702281549075); SRVR:BN6PR07MB2993; X-Microsoft-Exchange-Diagnostics: 1; BN6PR07MB2993; 3:co/iE1l6lqNEUWGMc3ptHVOAr+5acoY0QXPAojvrlVVTTNCVv+JU8Tg/WJw/8enFlZ83vgtHGqmsWhjnFDTPzdgkqm7h2rBl3JEDQQxAptw8HikmoD1GjUW1Q5PNklRuTrMHb8OMSYiuplOvS7TKPaW7ZdOEncCHyk+NfcKp5l6yFYA6hM97UZ6CrzhvOpTFfhvkd5uzsib7j9veQCF5bjNhM470vKE2EmxrFP9mENdhu1450HpHO3UF39Npg7Li; 25:COutlW2lCrxiRH59EA7h2/sQHyFIBPv4a5APaRrwRM8/nQp4yJvmFpXzNxISDv2xO1JvTj/mkVqgUowSMHfaqVfBrmOtz9x5Qul+3AdhzJFKEraFP+Up5BuC7UCIbxIc30oMhBZ4VJwxEKXevRjanm6C+VuLqTT/n45CHtDcw3fK3UXn38HV8W/Ad8qjYoXERsu5NGfm4Spzjr7G8uCN+W3A2isx3DuWFQqDu+Z8yst9S/oSWb4QeBgq0opmLkGN1lXeMMhhIJ6lsr0Dq1gGTZIyvuD73bgnZbF7lvfa9GivYC1gIpPTEA60MhrFmU343f+ozmUPehErW2hs6uLOiA==; 31:lcjPYYwgzGWs/zB4dUZh+YqUP2IJQC0zJtylkQLyNNks9t3bPlImOOe6u5c2yaLiS8AXCQ1zXwT/VvmOJ85keK9RQmGCTMkV4kYHFcow3aCZskqSGFQFxhsHUkuauxqAcGxdBGnHYXfufgPeZVrHqJWC2PjrBKCRoGUV9/ahwaFe9DzBn44CgUxPj9fHdZxrrZjDZRwA0ZXBoMcoMr6HhLNL2S8SCN7RopMS8Sw+ziE= X-MS-TrafficTypeDiagnostic: BN6PR07MB2993: X-Microsoft-Exchange-Diagnostics: 1; BN6PR07MB2993; 20:4GTTh8nM8WqH+NPnjnLRpi7m1LqtjhuJ9VID9Ye2E8c/YNcFZl2VgsrDUQ/m2cZqLbahUMmNcMKClWDW6M/B8IX4ePJJPL+S6bJaXQRmsBP8MBIGvQ5ropX4a/rerbyoo+pLMEvy0GbpA55i3TOaftB7rQZe7zi1HtXhy8F4eILovoh0IBwu934ojrwhrSPDVdk79NlQRY3dHeP1KlILwi5hP2OSzwAnd9fi83dDP/qOYh+BhsRtHrE6lgNRSPXzasB72j+2KxjjoT+wgVK/2gdVqqK2HJd9fXePZzM0Jfrx8xopFF7qXFEbw3eRpjkyZi5rvAIJJjlZDnmAe9JuiL9erPQTADmBvIwRAnV3C9OvFXZNwQoehdekLGZSL4B/scW7p/5du/YDt0DJCE0L7tJrHehTet19YP75CEACaMcnrIyzcUhYKVGLMNIgvVlo8Uea25lzS8gbg9r06tDKjz8b8HAnhGyzNBdr6Q+8J5rJzE9XhZhTxGIfQiXd8MvYGNP1J4eSHx4Cd/sl10+eis6qzVTetBXhfLQkbxdVFIjndmv9+UfP+3XV6Vkq8X7ze58K5vwb/KBT/NvZnKEVEDg0h2F/cWQjKIdOe3OljNQ=; 4:8NaP/NFGUZHEh3Jdg+2uF45xd1Gu072l75GTj114IeQxiwaZA4V7PAzWcl3htwxnc8YXzAIfFs/r7NZ/WGpeg+0cnIpymUlpOgB+Qrv+lvkKnTEiwf6At8OUHXecJ3/AkHODsF4kDZvxbRfWjCUTBWxKPENBRO64HD48yiRv8t8TeATbdo24lL7svzICOklTN9rNh0KgVLWGGwCyvmKmSB7gHv3eQb0M9jG0gvmM2sVl6XbX27dNac9RHg5NJW4U X-Exchange-Antispam-Report-Test: UriScan:; X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-CFA-Test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(8121501046)(5005006)(10201501046)(3002001)(100000703101)(100105400095)(93006095)(6041248)(20161123560025)(20161123558100)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123562025)(20161123555025)(20161123564025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:BN6PR07MB2993; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:BN6PR07MB2993; X-Forefront-PRVS: 04631F8F77 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6069001)(6009001)(346002)(376002)(199003)(189002)(50226002)(3846002)(305945005)(4326008)(50466002)(575784001)(105586002)(5003940100001)(53416004)(69596002)(16586007)(72206003)(25786009)(16526018)(53936002)(478600001)(36756003)(97736004)(48376002)(6116002)(6512007)(5660300001)(2906002)(101416001)(2950100002)(68736007)(8676002)(42882006)(6486002)(8936002)(316002)(81166006)(50986999)(47776003)(76176999)(7736002)(106356001)(6506006)(189998001)(107886003)(66066001)(81156014)(6666003); DIR:OUT; SFP:1101; SCL:1; SRVR:BN6PR07MB2993; H:localhost.localdomain.com; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en; Received-SPF: None (protection.outlook.com: cavium.com does not designate permitted sender hosts) X-Microsoft-Exchange-Diagnostics: 1; BN6PR07MB2993; 23:/iU9XYU3bi3gOd8FyeXHlgZcauh12Q8a48cx0yAcz5itTt4nJZQm7e0jeKiEtwF6EI0hJUkI+W+pmYGaIw2zwy2yMCxj4gRXWZx8V42OgJAP3uZLOtJiGx/KVSPgZVSp803g0zL3TgCF5xLWyJr717oC/iw1SiMMZjf/v3ZOWYF2r5oFDsA7UoQdp6j+towGc7k9UhsLj+VAeylcaIHdNwWnMDBuyAY/bLO4QcJQAZpqrKxzI6Y6NT/cT2wbQXfy4aB1qG43EsUN9QerIqpDB7iwju3nQr8+tN/eLyLx9eohHu7Q5S6BQmrGtDxmWHbDULKGwqNtD37tpg7x4QEL4fbpCF83AFu3Dnp+KxEoeb07SYB+H832JMaUtdfSqPd2WEQhywdOUElUtzJvgtLSg0/MNcARBKLtsHi1HzAQUG3XtqYSMbLWF1xgf+7MYn33MtqIYV7BQeMnUxqry/cHh6NwRzdSNksvKARW8eR3LJAXVgAfSV/aJ/eEyAAVi/K/XpK0QGBkM9uFYC0Bp3rFfgWy+ayGQJ5mIScj4YDrxLM7vPy08J/qpVE4HNF33e16CI2oJDI9UZ3K1h+CNTGMokm/lakl6UvMOkGbGF3HttcrDbcGwYX7vShK+xt8iQn2ROZeZ0aTTnSYT7kio7XSA5xH2lzsLe2hRvFqv+yX3iYJ4HVl6CFhjz0MXEpjAxtlGPflhBmgo80ojz6stX3a1t5HCeEwj0uCkyvOT3I3W264RRI0/M3xc6FREJSTyNhtqmVnxs0Rzx+oM4nvhLeVK2d8cI6IuoxpI3pLK35g0OhOqsR/Qs0u96074g9GeODJAXi+BJ5tMay2JJdHNFwIDpcwdhuJZJ267xCLAilYwE4ReFn9E+z6VMGKhT2oxVZcrkwzqAj0hGH2E+s20Ny0/c7OXGKOwlYW/Wj9a4MbNfF8dIfVyOIaMlg09h1iwLtHJsOJ+BV5nc8doV1z2xkWl8T3cvfKYk4WsGgMBs4rmLfRomCFqXuftNhviiRLfL3LBQndo5bwQGY5Z+CyXArrMvfmbK8uv8rKpSDSMnkQxLYBMsqsck2BqRZphuEhK01HpgMOJchc9HJiPqAs4VMN8TdCoHCfPxWUclhFmui9fa8acngWM/P2LiB0Q0yBM6Z8 X-Microsoft-Exchange-Diagnostics: 1; BN6PR07MB2993; 6:LK5YXOrIYmJAeGE5I7P1ggcnaCKiQ4J2WjK+pJwplkwHKJJAyE4ZSTaYePS25N13OPxobmKCfsZ4ty6ledpeRqnTuCEam+Y35WmKSnR1xPbo0yGofo3/bX6EFc4p1KFy1vU/Ka9GjBnr/vspp/75Z5uuZPdiZ+vgqSE4LgX5XIsTN4+pg/ksY4/q9KoHL01v0F9TX75VsAOjzLHO7ES60dUwyO5GltziAy3bY0t/YmGWqhF/ih/AHcjWKa8Xf2PVqotw0cTbfzJdHgFuwF7T3iWgN5uzmibV8A4TWUpn7FBcMrz0iKzTt8MvlaQ97Q3EE4XZMucEcclwIEs2df7idg==; 5:KWsHUe9P/OZZ3VRkZhqlP6T4x8ae+IFosLNQ/L1Tyk3pK4oZBEmc16sGuYzm0g0Opie62Ptyuvp55PkpUMDYnLtcu5oo31hAN1dQKAJFTf3S+jy3WkX+efxS6p36nUvcugYfEg9ia3frjrCxBV8oqQ==; 24:25VVLGDRvFnVv7A5DKcpYNzBf2v2dLw6sO2TBKlpT7Ct0RNpCBaSHOfNmMlWTJKsNE6oLubNdUr7lp5Sl8l3KGllSuME2Bw8t/H6x2ltRK8=; 7:GJYv2MNSVtR2nwHyUDL0YmSl3v17jKg5Ka7N6U5kMh08YwIi+7Kwiu7TWAY/gVb/rdX8j6SOb7bCjJV2tMbRhzWMyRTZBN/lIyMPtcCq2GEHStqIbynRPsEIXOs1dNFU+1ek0DtG7i4ChADtar0MNkp3nTuf5pLLL7DlUIB7stV0uhKUMVsu7lL7+R69RbdD31xTCBxiARg0Ys0yts0gZMsq7lxLO2c6JC/2zM2FYZ0= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 17 Oct 2017 12:48:25.7640 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR07MB2993 Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org From: Vadim Lomovtsev This commit changes the Cavium PCI ACS quirk appliance only to Cavium CN8xxx PICe Root Ports. Current quirk implementation doesn't take into account PCIe Root Ports which needs to be quirked. So the pci device id check mask is updated accordingly to CN8xxx device identification scheme and check of device ID moved into separate function. Signed-off-by: Vadim Lomovtsev --- drivers/pci/quirks.c | 15 ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-) diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index 5e0e83304fda..cfefb2431348 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c @@ -4211,6 +4211,19 @@ static int pci_quirk_amd_sb_acs(struct pci_dev *dev, u16 acs_flags) #endif } +static bool pci_quirk_cavium_acs_match(struct pci_dev *dev) +{ + /* + * Effectively selects all downstream ports for whole ThunderX 1 + * family by 0xf800 mask (which represents 8 SoCs), while the lower + * bits of device ID are used to indicate which subdevice is used + * within the SoC. + */ + return (pci_is_pcie(dev) && + (pci_pcie_type(dev) == PCI_EXP_TYPE_ROOT_PORT) && + ((dev->device & 0xf800) == 0xa000)); +} + static int pci_quirk_cavium_acs(struct pci_dev *dev, u16 acs_flags) { /* @@ -4222,7 +4235,7 @@ static int pci_quirk_cavium_acs(struct pci_dev *dev, u16 acs_flags) */ acs_flags &= ~(PCI_ACS_RR | PCI_ACS_CR | PCI_ACS_SV | PCI_ACS_UF); - if (!((dev->device >= 0xa000) && (dev->device <= 0xa0ff))) + if (!pci_quirk_cavium_acs_match(dev)) return -ENOTTY; return acs_flags ? 0 : 1;