From nobody Sun Apr 19 12:16:51 2026 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0F0A0C433FE for ; Thu, 24 Nov 2022 12:03:10 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230262AbiKXMDJ (ORCPT ); Thu, 24 Nov 2022 07:03:09 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40762 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229745AbiKXMCy (ORCPT ); Thu, 24 Nov 2022 07:02:54 -0500 Received: from galois.linutronix.de (Galois.linutronix.de [IPv6:2a0a:51c0:0:12e:550::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 09E9753EC5; Thu, 24 Nov 2022 04:02:53 -0800 (PST) Date: Thu, 24 Nov 2022 12:02:50 -0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020; t=1669291371; h=from:from:sender:sender:reply-to:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=50RKIHAAxW5+CnOS1oKtRt9//pFKNSA1UbDaxKJhB3w=; b=o4Zo01jzGrhftZLn5Fdb3YBBnRi48eGdECwWVB2JNXDFLHGGUI8hqguH/llUniRCFOiJvQ +kMr+CcS/UUCSn0Pm0VOD8Zsdiv8XRqfJWl20jvgstJkDfWpWXy6BJgcaSdNUfC71MwyDg IbLMfpYFoQkW79AUJlNt3sp/yQkP7C1dk2u7/trWXGg86tTV9VYrdxOeku3pVTHBix0yiu 2ZEKiMHZ0Kv90CgHpEhl7h/C9TsAXkUEpMxlvY9Lra1cjh7Gcdmwy/aCbQqRnfqvxFJkwE HCweiBcCF5JumWv9gnyl4dNbN97sLssiQ9LpHpRxepRDDCZdd8Yd7zk/YYTsQA== DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=linutronix.de; s=2020e; t=1669291371; h=from:from:sender:sender:reply-to:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=50RKIHAAxW5+CnOS1oKtRt9//pFKNSA1UbDaxKJhB3w=; b=vTgfOPTpEXAWegjM6anggIHV1xEo9MpVwKazShPm32837SDIpbRC1LauI5x5E6cGcBgAZ3 k/ZcsNIrZI46dlAQ== From: "tip-bot2 for Alexander Antonov" Sender: tip-bot2@linutronix.de Reply-to: linux-kernel@vger.kernel.org To: linux-tip-commits@vger.kernel.org Subject: [tip: perf/core] perf/x86/intel/uncore: Make set_mapping() procedure void Cc: Alexander Antonov , "Peter Zijlstra (Intel)" , Kan Liang , x86@kernel.org, linux-kernel@vger.kernel.org In-Reply-To: <20221117122833.3103580-12-alexander.antonov@linux.intel.com> References: <20221117122833.3103580-12-alexander.antonov@linux.intel.com> MIME-Version: 1.0 Message-ID: <166929137027.4906.15651683522712919440.tip-bot2@tip-bot2> Robot-ID: Robot-Unsubscribe: Contact to get blacklisted from these emails Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org The following commit has been merged into the perf/core branch of tip: Commit-ID: d5b73506b5b1b4a6e675c54b7977ea08e64cba19 Gitweb: https://git.kernel.org/tip/d5b73506b5b1b4a6e675c54b7977ea08e= 64cba19 Author: Alexander Antonov AuthorDate: Thu, 17 Nov 2022 12:28:33=20 Committer: Peter Zijlstra CommitterDate: Thu, 24 Nov 2022 11:09:24 +01:00 perf/x86/intel/uncore: Make set_mapping() procedure void Return value of set_mapping() is not needed to be checked anymore. So, make this procedure void. Signed-off-by: Alexander Antonov Signed-off-by: Peter Zijlstra (Intel) Reviewed-by: Kan Liang Link: https://lore.kernel.org/r/20221117122833.3103580-12-alexander.antonov= @linux.intel.com --- arch/x86/events/intel/uncore.h | 2 +- arch/x86/events/intel/uncore_snbep.c | 41 ++++++++++++--------------- 2 files changed, 20 insertions(+), 23 deletions(-) diff --git a/arch/x86/events/intel/uncore.h b/arch/x86/events/intel/uncore.h index fac3612..e278e2e 100644 --- a/arch/x86/events/intel/uncore.h +++ b/arch/x86/events/intel/uncore.h @@ -94,7 +94,7 @@ struct intel_uncore_type { * Optional callbacks for managing mapping of Uncore units to PMONs */ int (*get_topology)(struct intel_uncore_type *type); - int (*set_mapping)(struct intel_uncore_type *type); + void (*set_mapping)(struct intel_uncore_type *type); void (*cleanup_mapping)(struct intel_uncore_type *type); }; =20 diff --git a/arch/x86/events/intel/uncore_snbep.c b/arch/x86/events/intel/u= ncore_snbep.c index 4c2d5b5..b8f9bd1 100644 --- a/arch/x86/events/intel/uncore_snbep.c +++ b/arch/x86/events/intel/uncore_snbep.c @@ -3924,7 +3924,7 @@ static void pmu_clear_mapping_attr(const struct attri= bute_group **groups, } } =20 -static int +static void pmu_set_mapping(struct intel_uncore_type *type, struct attribute_group *ag, ssize_t (*show)(struct device*, struct device_attribute*, char*), int topology_type) @@ -3943,8 +3943,6 @@ pmu_set_mapping(struct intel_uncore_type *type, struc= t attribute_group *ag, if (ret < 0) goto clear_topology; =20 - ret =3D -ENOMEM; - /* One more for NULL. */ attrs =3D kcalloc((uncore_max_dies() + 1), sizeof(*attrs), GFP_KERNEL); if (!attrs) @@ -3968,7 +3966,7 @@ pmu_set_mapping(struct intel_uncore_type *type, struc= t attribute_group *ag, } ag->attrs =3D attrs; =20 - return 0; + return; err: for (; die >=3D 0; die--) kfree(eas[die].attr.attr.name); @@ -3979,7 +3977,6 @@ clear_topology: pmu_free_topology(type); clear_attr_update: pmu_clear_mapping_attr(type->attr_update, ag); - return ret; } =20 static void @@ -3998,15 +3995,15 @@ pmu_cleanup_mapping(struct intel_uncore_type *type,= struct attribute_group *ag) pmu_free_topology(type); } =20 -static int +static void pmu_iio_set_mapping(struct intel_uncore_type *type, struct attribute_group= *ag) { - return pmu_set_mapping(type, ag, skx_iio_mapping_show, IIO_TOPOLOGY_TYPE); + pmu_set_mapping(type, ag, skx_iio_mapping_show, IIO_TOPOLOGY_TYPE); } =20 -static int skx_iio_set_mapping(struct intel_uncore_type *type) +static void skx_iio_set_mapping(struct intel_uncore_type *type) { - return pmu_iio_set_mapping(type, &skx_iio_mapping_group); + pmu_iio_set_mapping(type, &skx_iio_mapping_group); } =20 static void skx_iio_cleanup_mapping(struct intel_uncore_type *type) @@ -4382,15 +4379,15 @@ static const struct attribute_group *skx_upi_attr_u= pdate[] =3D { NULL }; =20 -static int +static void pmu_upi_set_mapping(struct intel_uncore_type *type, struct attribute_group= *ag) { - return pmu_set_mapping(type, ag, skx_upi_mapping_show, UPI_TOPOLOGY_TYPE); + pmu_set_mapping(type, ag, skx_upi_mapping_show, UPI_TOPOLOGY_TYPE); } =20 -static int skx_upi_set_mapping(struct intel_uncore_type *type) +static void skx_upi_set_mapping(struct intel_uncore_type *type) { - return pmu_upi_set_mapping(type, &skx_upi_mapping_group); + pmu_upi_set_mapping(type, &skx_upi_mapping_group); } =20 static void skx_upi_cleanup_mapping(struct intel_uncore_type *type) @@ -4773,9 +4770,9 @@ static int snr_iio_get_topology(struct intel_uncore_t= ype *type) return sad_cfg_iio_topology(type, snr_sad_pmon_mapping); } =20 -static int snr_iio_set_mapping(struct intel_uncore_type *type) +static void snr_iio_set_mapping(struct intel_uncore_type *type) { - return pmu_iio_set_mapping(type, &snr_iio_mapping_group); + pmu_iio_set_mapping(type, &snr_iio_mapping_group); } =20 static void snr_iio_cleanup_mapping(struct intel_uncore_type *type) @@ -5391,14 +5388,14 @@ static int icx_iio_get_topology(struct intel_uncore= _type *type) return sad_cfg_iio_topology(type, icx_sad_pmon_mapping); } =20 -static int icx_iio_set_mapping(struct intel_uncore_type *type) +static void icx_iio_set_mapping(struct intel_uncore_type *type) { /* Detect ICX-D system. This case is not supported */ if (boot_cpu_data.x86_model =3D=3D INTEL_FAM6_ICELAKE_D) { pmu_clear_mapping_attr(type->attr_update, &icx_iio_mapping_group); - return -EPERM; + return; } - return pmu_iio_set_mapping(type, &icx_iio_mapping_group); + pmu_iio_set_mapping(type, &icx_iio_mapping_group); } =20 static void icx_iio_cleanup_mapping(struct intel_uncore_type *type) @@ -5656,9 +5653,9 @@ static const struct attribute_group *icx_upi_attr_upd= ate[] =3D { NULL }; =20 -static int icx_upi_set_mapping(struct intel_uncore_type *type) +static void icx_upi_set_mapping(struct intel_uncore_type *type) { - return pmu_upi_set_mapping(type, &icx_upi_mapping_group); + pmu_upi_set_mapping(type, &icx_upi_mapping_group); } =20 static void icx_upi_cleanup_mapping(struct intel_uncore_type *type) @@ -6125,9 +6122,9 @@ static const struct attribute_group *spr_upi_attr_upd= ate[] =3D { =20 #define SPR_UPI_REGS_ADDR_DEVICE_LINK0 0x01 =20 -static int spr_upi_set_mapping(struct intel_uncore_type *type) +static void spr_upi_set_mapping(struct intel_uncore_type *type) { - return pmu_upi_set_mapping(type, &spr_upi_mapping_group); + pmu_upi_set_mapping(type, &spr_upi_mapping_group); } =20 static void spr_upi_cleanup_mapping(struct intel_uncore_type *type)