From mboxrd@z Thu Jan 1 00:00:00 1970 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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 2C677CDB482 for ; Fri, 13 Oct 2023 11:08:45 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=IRc3IHx0l4tyIJ4NPTGVmaY4rdbk26nrH/2EITwFghE=; b=N/6i9oSauhuZaf GfHEGKGXFEIc9HtT8SfYkTj/NOKy5Ko/euzGRb3i5ncuX2qUBFZLAsu6169Xzyz57d7YN8YrWuuET RoeqXncjElyJPJUF3+n7a4BJyaVGiWhUJb5yMfCNdctNd00aApM+RGsh4VndeWh4YMH4fpnHcuMmP QUAsVxePltW5Qx2SUGNo+gmm7eA3yPJeRUTMhSe5v6bVvhurngULxBjOrWtOHdzZVEVBSW1NusvAw s4tM5uVG1n6KBT7W0V/wvHR6whBi37X4w0ZIkHC5ty54xvcBtCd8axWEttNtrJ1PCaT1ftoB9/Eus 40GSs3AR6TkvtLCP1FAA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qrG1m-003Hfn-2n; Fri, 13 Oct 2023 11:08:22 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qrG1j-003Hf2-0e for linux-arm-kernel@lists.infradead.org; Fri, 13 Oct 2023 11:08:21 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 10BAC11FB; Fri, 13 Oct 2023 04:08:56 -0700 (PDT) Received: from FVFF77S0Q05N (unknown [10.57.81.31]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 80AB83F7A6; Fri, 13 Oct 2023 04:08:14 -0700 (PDT) Date: Fri, 13 Oct 2023 12:08:11 +0100 From: Mark Rutland To: Anshuman Khandual Cc: linux-arm-kernel@lists.infradead.org, James Clark , Robin Murphy , Will Deacon , linux-kernel@vger.kernel.org Subject: Re: [PATCH V2] drivers: perf: arm_pmuv3: Drop some unused arguments from armv8_pmu_init() Message-ID: References: <20231013040307.1296109-1-anshuman.khandual@arm.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20231013040307.1296109-1-anshuman.khandual@arm.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231013_040819_334052_138A9D76 X-CRM114-Status: GOOD ( 26.26 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, Oct 13, 2023 at 09:33:07AM +0530, Anshuman Khandual wrote: > There is just a single call site remaining for armv8_pmu_init(), passing on > NULL pointers for all custom 'struct attribute_group's. These arguments are > not really getting used, and hence can be dropped. Afterwards the function > armv8_pmu_init_nogroups() itself becomes redundant, and can also be dropped > as well. > > The commit e424b1798526 ("arm64: perf: Refactor PMU init callbacks") wanted > to preserve the notion that non-default sysfs attributes could be used some > time in the future and hence armv8_pmu_init_nogroups() stayed on but now it > can be dropped to remove some redundant indirection, simplifying the code. If you can make that commit message: | All the PMU init functions want the default sysfs attribute groups, and so | these all call armv8_pmu_init_nogroups(), with none calling armv8_pmu_init() | directly. When we introduced armv8_pmu_init_nogroups() in commit: | | e424b1798526 ("arm64: perf: Refactor PMU init callbacks") | | ... we thought that we might need custom attribute groups in future, but as we | evidently haven't, we can remove the option. | | This patch folds armv8_pmu_init_nogroups() into armv8_pmu_init(), removing the | ability to use custome attribute groups and simplifying the code. Then: Acked-by: Mark Rutland Mark. > CC: James Clark > Cc: Robin Murphy > Cc: Will Deacon > Cc: Mark Rutland > Cc: linux-arm-kernel@lists.infradead.org > Cc: linux-kernel@vger.kernel.org > Signed-off-by: Anshuman Khandual > --- > This applies on v6.6-rc5 > > Changes in V2: > > - Dropped the helper armv8_pmu_init_nogroups() > - Updated the commit message > > Changes in V1: > > https://lore.kernel.org/all/20231009035638.165270-1-anshuman.khandual@arm.com/ > > drivers/perf/arm_pmuv3.c | 44 +++++++++++----------------------------- > 1 file changed, 12 insertions(+), 32 deletions(-) > > diff --git a/drivers/perf/arm_pmuv3.c b/drivers/perf/arm_pmuv3.c > index 8fcaa26f0f8a..c093b6e823f6 100644 > --- a/drivers/perf/arm_pmuv3.c > +++ b/drivers/perf/arm_pmuv3.c > @@ -1187,10 +1187,7 @@ static void armv8_pmu_register_sysctl_table(void) > } > > static int armv8_pmu_init(struct arm_pmu *cpu_pmu, char *name, > - int (*map_event)(struct perf_event *event), > - const struct attribute_group *events, > - const struct attribute_group *format, > - const struct attribute_group *caps) > + int (*map_event)(struct perf_event *event)) > { > int ret = armv8pmu_probe_pmu(cpu_pmu); > if (ret) > @@ -1212,27 +1209,17 @@ static int armv8_pmu_init(struct arm_pmu *cpu_pmu, char *name, > > cpu_pmu->name = name; > cpu_pmu->map_event = map_event; > - cpu_pmu->attr_groups[ARMPMU_ATTR_GROUP_EVENTS] = events ? > - events : &armv8_pmuv3_events_attr_group; > - cpu_pmu->attr_groups[ARMPMU_ATTR_GROUP_FORMATS] = format ? > - format : &armv8_pmuv3_format_attr_group; > - cpu_pmu->attr_groups[ARMPMU_ATTR_GROUP_CAPS] = caps ? > - caps : &armv8_pmuv3_caps_attr_group; > - > + cpu_pmu->attr_groups[ARMPMU_ATTR_GROUP_EVENTS] = &armv8_pmuv3_events_attr_group; > + cpu_pmu->attr_groups[ARMPMU_ATTR_GROUP_FORMATS] = &armv8_pmuv3_format_attr_group; > + cpu_pmu->attr_groups[ARMPMU_ATTR_GROUP_CAPS] = &armv8_pmuv3_caps_attr_group; > armv8_pmu_register_sysctl_table(); > return 0; > } > > -static int armv8_pmu_init_nogroups(struct arm_pmu *cpu_pmu, char *name, > - int (*map_event)(struct perf_event *event)) > -{ > - return armv8_pmu_init(cpu_pmu, name, map_event, NULL, NULL, NULL); > -} > - > #define PMUV3_INIT_SIMPLE(name) \ > static int name##_pmu_init(struct arm_pmu *cpu_pmu) \ > { \ > - return armv8_pmu_init_nogroups(cpu_pmu, #name, armv8_pmuv3_map_event);\ > + return armv8_pmu_init(cpu_pmu, #name, armv8_pmuv3_map_event); \ > } > > PMUV3_INIT_SIMPLE(armv8_pmuv3) > @@ -1263,44 +1250,37 @@ PMUV3_INIT_SIMPLE(armv8_nvidia_denver) > > static int armv8_a35_pmu_init(struct arm_pmu *cpu_pmu) > { > - return armv8_pmu_init_nogroups(cpu_pmu, "armv8_cortex_a35", > - armv8_a53_map_event); > + return armv8_pmu_init(cpu_pmu, "armv8_cortex_a35", armv8_a53_map_event); > } > > static int armv8_a53_pmu_init(struct arm_pmu *cpu_pmu) > { > - return armv8_pmu_init_nogroups(cpu_pmu, "armv8_cortex_a53", > - armv8_a53_map_event); > + return armv8_pmu_init(cpu_pmu, "armv8_cortex_a53", armv8_a53_map_event); > } > > static int armv8_a57_pmu_init(struct arm_pmu *cpu_pmu) > { > - return armv8_pmu_init_nogroups(cpu_pmu, "armv8_cortex_a57", > - armv8_a57_map_event); > + return armv8_pmu_init(cpu_pmu, "armv8_cortex_a57", armv8_a57_map_event); > } > > static int armv8_a72_pmu_init(struct arm_pmu *cpu_pmu) > { > - return armv8_pmu_init_nogroups(cpu_pmu, "armv8_cortex_a72", > - armv8_a57_map_event); > + return armv8_pmu_init(cpu_pmu, "armv8_cortex_a72", armv8_a57_map_event); > } > > static int armv8_a73_pmu_init(struct arm_pmu *cpu_pmu) > { > - return armv8_pmu_init_nogroups(cpu_pmu, "armv8_cortex_a73", > - armv8_a73_map_event); > + return armv8_pmu_init(cpu_pmu, "armv8_cortex_a73", armv8_a73_map_event); > } > > static int armv8_thunder_pmu_init(struct arm_pmu *cpu_pmu) > { > - return armv8_pmu_init_nogroups(cpu_pmu, "armv8_cavium_thunder", > - armv8_thunder_map_event); > + return armv8_pmu_init(cpu_pmu, "armv8_cavium_thunder", armv8_thunder_map_event); > } > > static int armv8_vulcan_pmu_init(struct arm_pmu *cpu_pmu) > { > - return armv8_pmu_init_nogroups(cpu_pmu, "armv8_brcm_vulcan", > - armv8_vulcan_map_event); > + return armv8_pmu_init(cpu_pmu, "armv8_brcm_vulcan", armv8_vulcan_map_event); > } > > static const struct of_device_id armv8_pmu_of_device_ids[] = { > -- > 2.25.1 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel