From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753364AbcBWOOi (ORCPT ); Tue, 23 Feb 2016 09:14:38 -0500 Received: from mail-bn1on0063.outbound.protection.outlook.com ([157.56.110.63]:2848 "EHLO na01-bn1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753185AbcBWOOg (ORCPT ); Tue, 23 Feb 2016 09:14:36 -0500 Authentication-Results: 8bytes.org; dkim=none (message not signed) header.d=none;8bytes.org; dmarc=none action=none header.from=amd.com; From: Suravee Suthikulpanit To: , , , , CC: , , , , , "Suravee Suthikulpanit" Subject: [PATCH V5 07/10] perf/amd/iommu: Clean up get_next_available_iommu_bnk_cntr Date: Tue, 23 Feb 2016 08:12:41 -0600 Message-ID: <1456236764-1569-8-git-send-email-Suravee.Suthikulpanit@amd.com> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1456236764-1569-1-git-send-email-Suravee.Suthikulpanit@amd.com> References: <1456236764-1569-1-git-send-email-Suravee.Suthikulpanit@amd.com> MIME-Version: 1.0 Content-Type: text/plain X-Originating-IP: [124.121.8.20] X-ClientProxiedBy: HKNPR06CA0045.apcprd06.prod.outlook.com (10.141.16.35) To BLUPR12MB0436.namprd12.prod.outlook.com (25.162.92.141) X-MS-Office365-Filtering-Correlation-Id: d9c30b51-0349-4600-5307-08d33c5ba7cf X-Microsoft-Exchange-Diagnostics: 1;BLUPR12MB0436;2:rx+tE0nW5CwVeidFjn6IITmyq3C6yXc86/rTms3zcAbH/LKPpTq2O6vlHanAp8VGIRpKqAyxe3yumW9/qvWNvLwuxCaH6Je0E07bTxMRSF3XTSpFiBtV4UDr3XRJl4rSvZDf9MJ4/73i9xbEJ7Tep9Rw8gu7LRbVf0MBHjICsZ9PmQVvIQ4tWE7khei6XtND;3:ZmPFtGNqv11bQvoTMfc/AT6Mo6rrVwmR4Dd6VW7eucgGHN+trC/sEvzzILK9sIJe5cjpGMg6x3onVU+k+CAI0Ox/wK4AqxbiJw95orMZiGAqppPWTg7yLMigRkGkQdyb;25:P56TP+ABDKfgNBzkPCiu7xUIwJK/7t3FoUOPkHUoU2UChW/38HceWjFMJA59LKFpjcMaY2jH7euQhcfPLYHZnyLnhcSX2BzzXtGXMq4km1LnLJoTzIe0+83WzjirrTSlydtrLvCFXwCOVATyRQpCxtfyV2DrkmOnxsxt3uuT08B/nqR1V3sY9n8Mvjor11/oL8iwI8SvdpfJnaOepiwesaQanqjLLctvOFNdT14oRJFuZ8jxJqmNoPclKUciJGlXJ0SmpObCi/dn8avZVRPo3lcPmBrZeASv0uH8Tx65nTLxybJdrOx77HHpoMHS/iGj+wG+gKHkK6jrWs37aXA9zhutIjr1N64+f56EdZDd8Yc= X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:;SRVR:BLUPR12MB0436; X-Microsoft-Exchange-Diagnostics: 1;BLUPR12MB0436;20:Lu4BRQyuddgJlKeg3qB8iPXli8ZW95LH2yV/YvhoNOiHCQ05LWYOyfqA9UkTpRr5VACavrN9mmocoFUSE6Oj7pqf1/Ikac98XwbT4RYsHAFT7qGGTXAM+rMk/IAA4tlvjw3RQI6HRBdTi2e1djtBY9n/SbUoD+KkM2IbU2G7PYbyvOl2z72ixVLcmAztPctFLjabqH0pI4a1BtZJ6avOa4s8s0r+0KGc4gYh4hhylU90MewESFP9mXayEqx4kPLiH3CwMTQKbAyyRer1/e2mWMZ3sCXbXB2CTS8g8K8QEKV/7CIdSZ6bIevP1PiPAVHT2mtpux++anKwMLBgn/siqSya22Z9tALQ7ZItYM3wrPYdRQiz2P/ZqFBMP12s91fICa4a/hHYtexLs3exgVJb7nZtMbmc1kH6kyynAgSPZNY/Wi+wEsMHlBediJ0T3mkt1D/nxsO4qcnzbb2MRLeOfb2yjFNsvUQj2URVQEUuV7jxbyf1I3oPbj8PspIq6uDH;4:MVz4zEufSmSph57QDd/qQHDMq1U08JlGVDsq99HoM9tstbLl/FsDaquR2Xy28JYeIlLr9MQUycMO53bG79Na9g9PbkZ2fl9L7uzO+zjYYAlhqQfc2/rAG4R5bi4IryuBHWCkbZLpvqCMvIriRG0Y7gO1xsjYU0eJWQkT8gJ+1gcUJFJxQTF3r1dchq0ryzH1uerwIzE/QvSDmZZ3PxaxF+W9S70ktjr3P4y2sHZTKQtbs+InrxrfsFa7XRHq8eFDeQDLrgGhGNkjA85Itr7xy1//BpRaAzAZ5mWGtPQ8FxX5u9qNFpmvz9TuXXUC4GkP+lSotvJCkUgCbng3Kowk43hiB9AOnPjAvuowtdagx93Sm0jkDEtaz2/JtYDFy3IUS+zlEoH/BhrwcoKSo4xSJ2wnC6Co5zzPGPMC7VfE7PI= X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:(767451399110); X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(601004)(2401047)(5005006)(8121501046)(10201501046)(3002001);SRVR:BLUPR12MB0436;BCL:0;PCL:0;RULEID:;SRVR:BLUPR12MB0436; X-Forefront-PRVS: 08617F610C X-Forefront-Antispam-Report: SFV:NSPM;SFS:(10009020)(4630300001)(6009001)(6069001)(5004730100002)(48376002)(77096005)(4326007)(2950100001)(2906002)(3846002)(6116002)(42186005)(5008740100001)(19580405001)(1096002)(19580395003)(86362001)(40100003)(50466002)(586003)(122386002)(36756003)(76176999)(5001960100002)(47776003)(66066001)(50986999)(5001770100001)(189998001)(229853001)(50226001)(87976001)(92566002)(2201001);DIR:OUT;SFP:1101;SCL:1;SRVR:BLUPR12MB0436;H:localhost.localdomain;FPR:;SPF:None;MLV:sfv;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;BLUPR12MB0436;23:jZZXRjQ5d7OYF4oYRBYKYGGqMcrlAYSm3WwqpufYvC8/K9OL9QbfLnUl6/kgcbvWXaaI1dLpNBo1uF8AcVI7NreOohLsCEMN72JNUtFxfd5bQLP39qyqiuM9l0UOT97LSdGPiVK8kJjvBvNfSe0fnTkEo3hHDcPRsloUuqh8jH73Z1nf/3Jso6RdFMjmeLGJusFOxan0UOgfRBGu+ymotv+jGYrBRBF/008OebZQ9m0rqNIVa6tAtSxfAKrIHJNYBFQ08FOdZEBPSDwiZ0Ip86L+hnkoNwW5Kx5V432A7NQUCCwNAjj1memagQIedezLgF6VfmHklZt5E92i0q5+DPHCAD/RKjN2M8adQTrs7Yhwiw3TGHbjjg0wIXPkCRc/+e1UVQpgWNLfO+7XVw6kDZXNs3lnnNEzeZFZnVHYn84/eXfVEu2eVtVlPRaChzXz1NtS/ldAaxLXXTv6heLEMCsNy+NUwM6NKzo7Df1+I5QO9blFGom7s/28S2xY1BjBGuNSgIDuP0cnIihHvl/dFaneB1U4KqIx2vSf/ZdkqD7aYCZUf/t8TrxCB/ZwkVhaLIu/2uX2mO0LslpS9sB8YmkvlAsF7apyIMrogBt7ypSYkKsjf82VAWdFfFPM/t4oKa62sKwg8Nz2hGX+WyBLHRgOBOdMMLT5aVxxWDDaYxdX9uNQbso7PrE7ErN8PkuQdkMV14Nlqqq/lndg9s7YyHxGOMlZ9gA4bysDInfjLkocbt0aPZLXuczYJsaL288LGvUwInIoZmVcqozcLqi+d3bbyYPSnmFfOws9esy6aSIZtddBAWP79ekEE0S4Tw3c1Z8jHW4qny23gEd9zaKQP+9WVy0GV8WSHJfR/5/UGmYZv7+VqKtI+kLHfCYISBYPFxXsLgCTicD91XP9lKtX+Va3ye73V8gTZ5GW1zGJV/7+DBVqejzZaiOdwYaZZfpw X-Microsoft-Exchange-Diagnostics: 1;BLUPR12MB0436;5:5R50teRzHlQkOCwV46D8RD71CP7h+kNZo/ObzkGJ3JVmUb9rp3yzXruOCXA25IWkv/3FENhrU+kViWcl907SmOMtkE9LKtsbC2iKcs1Nal/+O4ADU3QN21jm9cXwtTQgrDVJBSHHaQDWgoUXGV8JcQ==;24:GRRNKgPrc9cq/nHcJsY0eXLy7efSXsovmw4NXteajmdOcPcPrOpxN+FOGSSVP7jED2dsYHlwNp0FfWmnYWpbK6brTSvpfKTAMB2ICQBXN4E=;20:M/iLtVzY+PcCxQS/lIIdI8laCAp7AT3adyGqEueEn+/OWi3zCub7r1iusl9WwEro7je8kYmpgHSEQQslg90nYwJ2oz/iFwQnVMegUYymWku/OnALFzmpT4Ua2xVZ1E9F9HVeL0lk6DTQZqmoP8FoR2VsKFaGjmqUdj10Q2oTyOBawuVNkBLFizvXmFmvjnS9sJT+VWTarXpi3ntGq2pzqc3ClANWKaNAlURvMDT18CxvKK0ZEXh+bMmvVRFP4/HY X-OriginatorOrg: amd.com X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Feb 2016 14:14:30.5224 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-Transport-CrossTenantHeadersStamped: BLUPR12MB0436 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This patch cleans up the coding style of this function. This should not affect the logic and functionality. Cc: Borislav Petkov Signed-off-by: Suravee Suthikulpanit --- arch/x86/events/amd/iommu.c | 18 ++++++++---------- 1 file changed, 8 insertions(+), 10 deletions(-) diff --git a/arch/x86/events/amd/iommu.c b/arch/x86/events/amd/iommu.c index 1a678b9..d5e4d39 100644 --- a/arch/x86/events/amd/iommu.c +++ b/arch/x86/events/amd/iommu.c @@ -150,25 +150,23 @@ static struct attribute_group amd_iommu_cpumask_group = { static int get_next_avail_iommu_bnk_cntr(struct perf_amd_iommu *perf_iommu) { unsigned long flags; - int shift, bank, cntr, retval; - int max_banks = perf_iommu->max_banks; - int max_cntrs = perf_iommu->max_counters; + int bank, cntr, retval = -ENOSPC; raw_spin_lock_irqsave(&perf_iommu->lock, flags); - for (bank = 0, shift = 0; bank < max_banks; bank++) { - for (cntr = 0; cntr < max_cntrs; cntr++) { - shift = bank + (bank*3) + cntr; - if (perf_iommu->cntr_assign_mask & (1ULL<max_banks; bank++) { + for (cntr = 0; cntr < perf_iommu->max_counters; cntr++) { + int shift = (perf_iommu->max_counters * bank) + cntr; + + if (perf_iommu->cntr_assign_mask & (1ULL << shift)) { continue; } else { - perf_iommu->cntr_assign_mask |= (1ULL<cntr_assign_mask |= (1ULL << shift); + retval = ((u16)((u16)bank << 8) | (u8)(cntr)); goto out; } } } - retval = -ENOSPC; out: raw_spin_unlock_irqrestore(&perf_iommu->lock, flags); return retval; -- 1.9.1