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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 85996C433EF for ; Wed, 6 Oct 2021 12:23:06 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 68964610CC for ; Wed, 6 Oct 2021 12:23:06 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238492AbhJFMY5 (ORCPT ); Wed, 6 Oct 2021 08:24:57 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36410 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238149AbhJFMYy (ORCPT ); Wed, 6 Oct 2021 08:24:54 -0400 Received: from casper.infradead.org (casper.infradead.org [IPv6:2001:8b0:10b:1236::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 5F9C5C061749; Wed, 6 Oct 2021 05:23:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=casper.20170209; h=In-Reply-To:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:Sender:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description; bh=jGRGjTwTIkRqG9rPpRJmFYnotEHM8b7eyzqywKaCxzU=; b=e4FNKRxyiGEAXwkdgHz8fve8RK pTXJgmqkKrRVo77srt1xypzk6uJzHyGsIa7O50LwzUHJX5AxIvDU5j59EcU84NTur6tFr/1YOo7+l 1Zi9ezn1g7so5+alZ8Lceq+yUBE3csRU0KgRwz5VBvo4NzopRiICWJLIOgRNp8OAwiqMnHHA2Oo1u yiieYHiQY9eQR81IyVVLyq5jIMk1CU3aj/TARdgToQOcf/uUi6nTWM+a9UFdB9d+dENK9VkKctC1f 9sRmYEFuP9tld8135eG7crvMrhnu0JCodWDIKpLOxjm/EPoPNYjpp9OEeVYkKVKBAW4Kr5kVyYNIr DrlzLxwg==; Received: from j217100.upc-j.chello.nl ([24.132.217.100] helo=worktop.programming.kicks-ass.net) by casper.infradead.org with esmtpsa (Exim 4.94.2 #2 (Red Hat Linux)) id 1mY5sw-000rWW-2l; Wed, 06 Oct 2021 12:19:06 +0000 Received: by worktop.programming.kicks-ass.net (Postfix, from userid 1000) id 1F97298623A; Wed, 6 Oct 2021 14:18:58 +0200 (CEST) Date: Wed, 6 Oct 2021 14:18:58 +0200 From: Peter Zijlstra To: Barry Song <21cnbao@gmail.com> Cc: Valentin Schneider , Dietmar Eggemann , LKML , Ingo Molnar , Vincent Guittot , Aubrey Li , Borislav Petkov , Daniel Bristot de Oliveira , Ben Segall , Catalin Marinas , Greg Kroah-Hartman , Guodong Xu , "H. Peter Anvin" , Jonathan Cameron , Juri Lelli , "Cc: Len Brown" , ACPI Devel Maling List , LAK , Linuxarm , Mark Rutland , Mel Gorman , msys.mizuma@gmail.com, "Zengtao (B)" , "Rafael J. Wysocki" , Steven Rostedt , Barry Song , Sudeep Holla , Thomas Gleixner , "Rafael J. Wysocki" , Tim Chen , Will Deacon , x86 , yangyicong , Tian Tao Subject: Re: [PATCH RESEND 1/3] topology: Represent clusters of CPUs within a die Message-ID: <20211006121858.GI174703@worktop.programming.kicks-ass.net> References: <20210924085104.44806-1-21cnbao@gmail.com> <20210924085104.44806-2-21cnbao@gmail.com> <87o883l9c8.mognet@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Oct 06, 2021 at 11:50:35PM +1300, Barry Song wrote: > > diff --git a/drivers/base/arch_topology.c b/drivers/base/arch_topology.c > > index 7cb31d959f33..fc0836f460fb 100644 > > --- a/drivers/base/arch_topology.c > > +++ b/drivers/base/arch_topology.c > > @@ -622,7 +622,8 @@ void update_siblings_masks(unsigned int cpuid) > > if (cpuid_topo->package_id != cpu_topo->package_id) > > continue; > > > > - if (cpuid_topo->cluster_id == cpu_topo->cluster_id) { > > + if (cpuid_topo->cluster_id == cpu_topo->cluster_id && > > + cpuid_topo->cluster_id != -1) { > > cpumask_set_cpu(cpu, &cpuid_topo->cluster_sibling); > > cpumask_set_cpu(cpuid, &cpu_topo->cluster_sibling); > > } > > > > Hi Peter, > Would you like to change this line in your tree? Can you please double check: https://git.kernel.org/pub/scm/linux/kernel/git/peterz/queue.git/log/?h=sched/next