From patchwork Tue Sep 12 11:55:16 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Vadim Lomovtsev X-Patchwork-Id: 812810 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="DS4Ao/4O"; dkim-atps=neutral Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3xs3CS0wTrz9s5L for ; Tue, 12 Sep 2017 21:55:36 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751393AbdILLze (ORCPT ); Tue, 12 Sep 2017 07:55:34 -0400 Received: from mail-dm3nam03on0081.outbound.protection.outlook.com ([104.47.41.81]:45252 "EHLO NAM03-DM3-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751289AbdILLzd (ORCPT ); Tue, 12 Sep 2017 07:55:33 -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=7Gn+0kcF/v5b4tNcI8+o0heGPmRF8CtUE/gAMl9+pP0=; b=DS4Ao/4OgsvFy6tyv6E9rmtey0px2nDGwtOr7Fq8//LJ0ctpl8CzWnunyoHL6Gv6Yzl84NxM5wUvE9n/iYReB3HHkVQgIinHA9/ysSYFmzljNeyA+i8ZdLc1QF1vE2rLCsfYmMmM7I5I5ANfQAK2+gflZNeA3IEzoJiGZvJDI0o= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Vadim.Lomovtsev@cavium.com; Received: from localhost.localdomain.com (50.233.148.156) by CY4PR07MB2997.namprd07.prod.outlook.com (10.172.116.11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.35.12; Tue, 12 Sep 2017 11:55:30 +0000 From: Vadim Lomovtsev To: bhelgaas@google.com, linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org Cc: David.Daney@cavium.com, jcm@redhat.com, Robert.Richter@cavium.com, Wilson.Snyder@cavium.com, Vadim Lomovtsev Subject: [PATCH] PCI: quirks: update cavium ACS quirk implementation Date: Tue, 12 Sep 2017 04:55:16 -0700 Message-Id: <1505217316-15702-1-git-send-email-Vadim.Lomovtsev@caviumnetworks.com> X-Mailer: git-send-email 2.4.11 MIME-Version: 1.0 X-Originating-IP: [50.233.148.156] X-ClientProxiedBy: MWHPR2201CA0001.namprd22.prod.outlook.com (10.174.164.14) To CY4PR07MB2997.namprd07.prod.outlook.com (10.172.116.11) X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 0f37c86d-54eb-47f2-384b-08d4f9d52b10 X-Microsoft-Antispam: UriScan:; BCL:0; PCL:0; RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(300000502095)(300135100095)(22001)(2017030254152)(300000503095)(300135400095)(2017052603199)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095); SRVR:CY4PR07MB2997; X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB2997; 3:nGeCdqNHhXM7p1zd1KaJETScLF66Rm/Gqd5te2+yrHUm7GiKauz1W5L8uRE48dtOPSbIdnx/KJNrG7+iZeqKkMzUG1tBbCw7jOd93FplZbrEUspMZRyyo/3v6vcmT/b8HaeeQeYPOQN2ig54kEPTqt5SQr7G1WUKdE/dUl4z2UO1ctDxEjt37SrOjwJcArXbCh6UpdmVNyvXNKtnkLFb5B1sM31a8+PqdhfVt3pHXb8tRlWjkdELvYrMctYZH5PA; 25:J0UmwqQP/N0X++XgtMHA8SyhQDoFbXiscc2w1GdC+i321KQOi25Mx9dxP3s9YeGvpuwgSdI/gONS8kQxPtZkRHpQE5TqXjLmx0zF7KEBh3l7NPGLe4UgmCPpPahPHGRsJ590Bfmmtl/v0mdzkxgimOtA1627rz/syD4gVSUCdrq+2Cf2oIHl2pxXogNkIldTY1FvdyaeT1CR85uIOz00fdYzqOHcV/e/hFzUiqIAmMhZJgxfYU+4uzVSL/rY6hLwhj35V2sAXUT5lXX2w5/nfrMZLPVeOS1y82AcQsP3Wo8K5Soh3CfwX6MPTFQYkkef+IgqRTPnWZzqea/6q72eBg==; 31:DVLFpqR0NBZ0CaI0rIrj+ECK1zutJxG/E9V//Piaj2QODdM0vWhUCk2ZTu9l7ELQ+OtGHwylx7iri5HkLZ20v/qhfBeCGLhHpLYYq8ITRHVfzbM6c9FTXlMI2ATeJURFCPjyMnRGsV0d2cXDPWQAK+sizHtsHTrPHswI1nL52w7o6kyoypLmzwmdtEQVq6oH/O99H0xm37GqFTAW8LYYrWzebIDSvDRCH27fW8J3G1A= X-MS-TrafficTypeDiagnostic: CY4PR07MB2997: X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB2997; 20:kr4YCEUfVpnB9PX8klLjYcJ0AGshgrrw2frc1fOtrRYUFSwqphNYaQAfqqc1zaOB9DLuj1yPJ3sTr4XdfKl4CUp56Vr+YMRw1WjGTnRI48yRuXb4ggHaK2aS3KMLcDXaIXgTU6ELrhs+8rEEn0cC43skB8b04bDoFzeP/qfCoNELb4fsP6xLSNXydNBuyQDtlYmUzxEK9UwM5m5wcFQzsYCmUNnwy+aqhuD7+QHTO1oYb9l+fTbxnksYZkSjwSQXPEb/Fe/WoSUtbnB9tNMeEY98I8tcejh2EZYirVGkfqKi+X6pk6jXzDNc6vlqHwndcCOFu3q5Jqgg35mxwPzt0uOP2GKGMFP1HmyupIimgXcd49HRwexDtpOp/D5IyYBiyxmdx2c8bp+6bFR2PxX1Yc4U/nOhvwjv7dhpgQrnZ2P5qcHzpsf8H8D9lnVCVjU2XVmSOEM7Aq2jeWAPYnRnmfsDy6oktRgkvg+jK9GhJo34FmLJO1keHGBzXKqL/i6IeoqCeYh7mMvj/lu6GFva/LtuN5Pp1fOqRX7nVZjLeua62zCWEIRs3WshZFRkOSMXdEryu4UlokuWvkRgxXBhB3t4zOX+mJq9GAKhyXlfIUA=; 4:Cp2vOEMYX+RWJIDGfJRrst9sbhrxXtsL6MGJAQSohbcgNknVJwmhPA6b4dEYRvOjM2sNGrr/+Mlivsm4AUlS13xdl2Mt/UptzfoZpCoszDy8Q8nlNGTgfLjSMf3mTPOVrLXJ+pRmTRBrKxA7V2UTGrRXQK28zCte3fR4I/nerQ8cYQ6hj/gsEbHpOLgRInoBYN8B+oyKfeG7ex6GlKcOI/fGV9TwkZLfd+XVeMc+/P4Ys/iLdo9f6oHkJ5u0gWQb 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)(3002001)(10201501046)(100000703101)(100105400095)(6041248)(20161123555025)(20161123560025)(20161123562025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123558100)(20161123564025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:CY4PR07MB2997; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:CY4PR07MB2997; X-Forefront-PRVS: 042857DBB5 X-Forefront-Antispam-Report: SFV:NSPM; SFS:(10009020)(6069001)(6009001)(199003)(189002)(42882006)(575784001)(2906002)(36756003)(6512007)(6506006)(97736004)(53416004)(105586002)(6486002)(53936002)(316002)(106356001)(81166006)(8936002)(7736002)(50986999)(47776003)(66066001)(81156014)(101416001)(8676002)(305945005)(4326008)(50226002)(110136004)(189998001)(6666003)(69596002)(107886003)(25786009)(42186005)(72206003)(68736007)(50466002)(48376002)(478600001)(5660300001)(3846002)(6116002)(5003940100001); DIR:OUT; SFP:1101; SCL:1; SRVR:CY4PR07MB2997; 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; CY4PR07MB2997; 23:Ud5TvRFOQw1r2XuO4o74t//pu/pDYtnousja4CErw8l+61tGZ1+z7aCunx2uq3bq+H4mK6hsmluY/tT1FDKBpYCTfkZDqmxLUAwE0WGOISOO44EiDK76P93GY8uhzzi/nJKnLIzUDTBK3mncYIMQY73BS+HaSUbgAQyA0q4ZaP56gj0/gMx433cTD9RHU2PYx8VaRvFWi3lgZd63FCY+ZCBzBoMPM2IdV+L8J/XowhWru7kXMCu5Ufow7lqjZwX68dLDwFDw1mOtKa7Tb9iZFfSaK6IeK6lsFe6DT2uI9W2BTpDaDsmmw5csfIY6GIpJKYiuju3FFmzMDJhUVoMMakLGURGxo/2YdMMHVYBfY1Z1vAVGsN3jO+nDrXzy9TyMCUzWy4G2RVe04J+JRXBhiQ2HzChzXNMq5tDtGQEQjuDznqlHmqkl4DjiO4arXKBxhaJ2eBHPyKkf9Gz8zq6dfzT10z5aWkH7sXAnB7s1sT3OM7dHoEKNTn5vtZoZHtqNH1PO8BWf9a0hLMFGGiYJu49q0bdRcWS9jkCbLS/hjWddIPFEGeKSSNPbYxosTFZQAxJ070CHdLDeLPfmQ3Qj1JS69LnGkgJtZlV9GrtIytI2VgwBe2BXOYBbsDzolQUdn31Bn2bgfc88PltzAPrKFBFPJYv3fepz2qohA6geT20LdIxulAwuw/+R4vO/rZGwc5axAJO6SXzbChU+8tRohHgmAcoqxq5tQw/X3ShPld4y3048TDw3O+h9f0I05pu8NYNijchvULJDxTJIZhXLJcjhjIaz42D4EvLvjpDfUeBjx9ojFyevIHz6101DbF57dTuxR2ePMogsW6ipRoU12tj8P+TL8hna3GtQ1n22v6bYKujutRBoN8QjvyRFGJDtynoJtJT81u0UoIJec5W0YdQZo5d3kpF/wwED19qqyGjJ8lWT4O/Ee2Y1vlhu2dJXkD5pDQi/aIFqmfr/E4HBmi0BvrI6yAT08x1wyiKOho7qec3HX1YknrrGCd8YKa5waUuCNQIR4thpy2zMsGVzoJ0/8nECGSNWZKijcRbfcNXxJlwSctpZ/E8pfk/iYjuH X-Microsoft-Exchange-Diagnostics: 1; CY4PR07MB2997; 6:67Moznif8qhDQ/tHJROxjJLMBzKQKNcYKzBoHvhBN0yMgivjYPKm2nhGeIUIykzGl10IjowWeploggU3XpUxQY7m472cTLugMsVXE1Pm1luyS7G4wvaE6tGZi69jvcrHHBlUNFAOEKELINX7Ykln7KEmiClelVWtJqO/G6yTiV0ZzIESZ940AXioMvt4KB3wePDB4j+abSne5lsuufNxd++22/O5385GPri0jvKCjCtS2/3Poht1Oph03oRBlLFS/vXffQWUZ5RYdu0jwtPViNwsQT2hoJJ7oCAy76zn/pmm/rQfcQ8xa4mUdSiku0s0p0H5gB1lK8OIp3zOrfPFdQ==; 5:cKXlXGtbN0UHyMOrD7eo9ye/J0mMYjHRPm1HB+BTD07UsU2rAWhcnMb6VNqMmEB/iM2Emcf2EiFTAID1TwV0+qx5phKFd2zbC6oDhikd9ewfowfXKxf5/wQzYsNUxuWjqmPU3wgEOieDIjsYUb+pEA==; 24:f0EYsktre8oTB48wHOcSlDchxD8YG6As0HerYbnVALS8nHP8XcQ9MJv7piTjLuJYkGmBwtl2EfSgCRzDiaBVjo420B1yP8k3ZSSoG15Lgfg=; 7:DdIT19dFDbqMRHaXlWsVcMNKWLDG4872dp62rduQ00wuckPuKAFglqlKAFhg9obMvODGmmcvVcZdG3eV6Ed13y27ba1B3PDnlfMBYzROrD99DuvlgKW1QqIQwY/Pu+5152iMZD4nmFHqpx2A4SIuAJn3WlzUVcQhhNQO3t7sqZo25GFGorOKKlIZ9b8rOUZE0tiCJoNB4n1keNGRj9906f4QkkGtdkK6YgZrJplF21s= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-OriginatorOrg: caviumnetworks.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Sep 2017 11:55:30.7384 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 711e4ccf-2e9b-4bcf-a551-4094005b6194 X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR07MB2997 Sender: linux-pci-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pci@vger.kernel.org This commit makes PIC ACS quirk applicable only to Cavium PCIE devices and Cavium PCIE Root Ports which has limited PCI capabilities in terms of no ACS support. Match function checks for ACS support and exact ACS bits set at the device capabilities. Also by this commit we get rid off device ID range values checkings. Signed-off-by: Vadim Lomovtsev --- drivers/pci/quirks.c | 30 +++++++++++++++++++++++++----- 1 file changed, 25 insertions(+), 5 deletions(-) diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c index a4d3361..11ca951 100644 --- a/drivers/pci/quirks.c +++ b/drivers/pci/quirks.c @@ -4211,6 +4211,29 @@ static int pci_quirk_amd_sb_acs(struct pci_dev *dev, u16 acs_flags) #endif } +#define CAVIUM_ACS_FLAGS (PCI_ACS_SV | PCI_ACS_TB | PCI_ACS_RR | \ + PCI_ACS_CR | PCI_ACS_UF | PCI_ACS_DT) + +static __inline__ bool pci_quirk_cavium_acs_match(struct pci_dev *dev) +{ + int pos = 0; + u32 caps = 0; + + /* Filter out a few obvious non-matches first */ + if (!pci_is_pcie(dev) || pci_pcie_type(dev) != PCI_EXP_TYPE_ROOT_PORT) + return false; + + /* Get the ACS caps offset */ + pos = pci_find_ext_capability(dev, PCI_EXT_CAP_ID_ACS); + if (pos) { + pci_read_config_dword(dev, pos + PCI_ACS_CAP, &caps); + /* If we have no such bits set, then we will need a quirk */ + return ((caps & CAVIUM_ACS_FLAGS) != CAVIUM_ACS_FLAGS); + } + + return true; +} + static int pci_quirk_cavium_acs(struct pci_dev *dev, u16 acs_flags) { /* @@ -4218,13 +4241,10 @@ static int pci_quirk_cavium_acs(struct pci_dev *dev, u16 acs_flags) * with other functions, allowing masking out these bits as if they * were unimplemented in the ACS capability. */ - acs_flags &= ~(PCI_ACS_SV | PCI_ACS_TB | PCI_ACS_RR | - PCI_ACS_CR | PCI_ACS_UF | PCI_ACS_DT); - - if (!((dev->device >= 0xa000) && (dev->device <= 0xa0ff))) + if (!pci_quirk_cavium_acs_match(dev)) return -ENOTTY; - return acs_flags ? 0 : 1; + return acs_flags & ~(CAVIUM_ACS_FLAGS) ? 0 : 1; } static int pci_quirk_xgene_acs(struct pci_dev *dev, u16 acs_flags)