Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Thu, 3 Dec 2020 10:28:31 +0100
From:   Peter Zijlstra <peterz@infradead.org>
To:     Valentin Schneider <valentin.schneider@arm.com>
Cc:     Barry Song <song.bao.hua@hisilicon.com>, catalin.marinas@arm.com,
        will@kernel.org, rjw@rjwysocki.net, lenb@kernel.org,
        gregkh@linuxfoundation.org, Jonathan.Cameron@huawei.com,
        mingo@redhat.com, juri.lelli@redhat.com,
        vincent.guittot@linaro.org, dietmar.eggemann@arm.com,
        rostedt@goodmis.org, bsegall@google.com, mgorman@suse.de,
        mark.rutland@arm.com, linux-arm-kernel@lists.infradead.org,
        linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org,
        linuxarm@huawei.com, xuwei5@huawei.com, prime.zeng@hisilicon.com
Subject: Re: [RFC PATCH v2 2/2] scheduler: add scheduler level for clusters
Message-ID: <20201203092831.GH2414@hirez.programming.kicks-ass.net>
References: <20201201025944.18260-1-song.bao.hua@hisilicon.com>
 <20201201025944.18260-3-song.bao.hua@hisilicon.com>
 <jhj1rg9v7gr.mognet@arm.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <jhj1rg9v7gr.mognet@arm.com>
Precedence: bulk

On Tue, Dec 01, 2020 at 04:04:04PM +0000, Valentin Schneider wrote:
> 
> Gating this behind this new config only leveraged by arm64 doesn't make it
> very generic. Note that powerpc also has this newish "CACHE" level which
> seems to overlap in function with your "CLUSTER" one (both are arch
> specific, though).
> 
> I think what you are after here is an SD_SHARE_PKG_RESOURCES domain walk,
> i.e. scan CPUs by increasing cache "distance". We already have it in some
> form, as we scan SMT & LLC domains; AFAICT LLC always maps to MC, except
> for said powerpc's CACHE thingie.

There's some intel chips with a smaller L2, but I don't think we ever
bothered.

There's also the extended topology stuff from Intel: SMT, Core, Module,
Tile, Die, of which we've only partially used Die I think.

Whatever we do, it might make sense to not all use different names.

Also, I think Mel said he was cooking something for
select_idle_balance().

Also, I've previously posted patches that fold all the iterations into
one, so it might make sense to revisit some of that if Mel also already
didn.t