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 1F1B4C433EF for ; Fri, 8 Jul 2022 14:02:12 +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=fm6EFYfMtfgWpR6CxB7onJhm3Rs3hcQgECqgOwnN0tU=; b=xGhrC2CfiEU+8U YZ5N5dWEMRejIfX7VpUzB6hotrV9k8b8CT/YaZx/Y8/1L3HwI6toqbm954TOTDFWhHjUnczHYl/gL 95m4WdS4Uv7SK4+FxZ52QI390WLbBDSY5q3SY/MZyrg8y0SuAlUq/3+Xosujky9ZlXRLqtwpRV/mc 8oHCm639oScvxGQq7vfIaZtf9yCmUxxIoLExM+c8hQFOA3S4vMGEEXZe64pcHRQoPdDD+mWB90lkG QhQSK+OksEmDa0VAGRe8Hi6zwC4eyKbGkGAtQIwDSTPrKz0Lx39tk9AX+IyF0vmKrhY3J/IP7Rdbq RLVPECBON8/KCwigQzaA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1o9oXS-0042Ei-Jc; Fri, 08 Jul 2022 14:00:58 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1o9oXJ-0042B4-50; Fri, 08 Jul 2022 14:00:51 +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 7CCE0106F; Fri, 8 Jul 2022 07:00:47 -0700 (PDT) Received: from bogus (unknown [10.57.39.193]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id AC1E33F66F; Fri, 8 Jul 2022 07:00:42 -0700 (PDT) Date: Fri, 8 Jul 2022 14:59:32 +0100 From: Sudeep Holla To: Conor.Dooley@microchip.com Cc: gregkh@linuxfoundation.org, geert@linux-m68k.org, paul.walmsley@sifive.com, palmer@dabbelt.com, palmer@rivosinc.com, aou@eecs.berkeley.edu, catalin.marinas@arm.com, will@kernel.org, rafael@kernel.org, Daire.McNamara@microchip.com, niklas.cassel@wdc.com, damien.lemoal@opensource.wdc.com, zong.li@sifive.com, kernel@esmil.dk, hahnjo@hahnjo.de, guoren@kernel.org, anup@brainfault.org, atishp@atishpatra.org, changbin.du@intel.com, heiko@sntech.de, philipp.tomsich@vrull.eu, robh@kernel.org, maz@kernel.org, viresh.kumar@linaro.org, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Brice.Goglin@inria.fr Subject: Re: [RFC 2/4] arch-topology: add a default implementation of store_cpu_topology() Message-ID: <20220708135932.7srpzia53jbpmitg@bogus> References: <20220707220436.4105443-1-mail@conchuod.ie> <20220707220436.4105443-3-mail@conchuod.ie> <20220708082443.azoqvuj7afrg7ox7@bogus> <473e6b17-465b-3d14-b04d-01b187390e66@microchip.com> <20220708092100.c6mgmnt7e2k7u634@bogus> <20220708094710.rxk6flrueegdsggr@bogus> <20220708113915.ui77mgqckjzalwlh@bogus> <9f8d4e4d-f24b-faf3-687f-90370939c72f@microchip.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <9f8d4e4d-f24b-faf3-687f-90370939c72f@microchip.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220708_070049_270799_5CFB7917 X-CRM114-Status: GOOD ( 17.65 ) 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, Jul 08, 2022 at 11:57:05AM +0000, Conor.Dooley@microchip.com wrote: > > I did a little bit of poking in the git history. > The last code touching the arm implementation was: > 3102bc0e6ac7 ("arm64: topology: Stop using MPIDR for topology information") > on Fri Oct 2 12:01:41 2020 +0100 > > The introduction of arch-topology stuff to RISC-V was: > 03f11f03dbfe ("RISC-V: Parse cpu topology during boot.") > on Thu Jun 27 12:53:00 2019 -0700 > > Backporting as far as v5.10 should be no real effort and I don't think > to v5.4 that should be meaninfully harder. If 3102bc0e6ac7 hasn't been > backported already, maybe it should be since it appears to have been > fixing a problem too. > Thanks for doing the research and sorry for the noise earlier. > Based on that, I think doing this the straightforward way in the first > place is a better idea. > > I'll respin the series as: > patch 1: Move arm64 to the generic implementation I don't think the mpidr check we have there is of much use IMO. You can drop that and see if arm64 maintainers and/or others agree. As you have already figured, since 3102bc0e6ac7 we are not using MPIDR and the one check we have is optional IMO. So you can either drop it or keep it as in your RFC and then post updates. -- Regards, Sudeep _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel