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 2F8EFC433F5 for ; Thu, 7 Oct 2021 10:37:52 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id E7B5760F02 for ; Thu, 7 Oct 2021 10:37:51 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org E7B5760F02 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=infradead.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org 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=NqCpLxG3j2AT2Rz7mm/VoF90b5KlMRf1XrZhDvtiwJg=; b=2OVfG4g71oLytV uCbN5Km+p6nzoa1QczNdPLw8BFNQGSKCCQOxzRk4AkZzz0gLwEESKeyTnsw3z5AEiM7Huj0JEDdOo YP4U7+Y9rv/krhY/Luj1GPtKGkPRt3tGLshORfVHw23IuyWylIYuwUFOYc3YSGAkNExC7Ar0YCDCj yrKB0ym/ImfkeWk4ibRg/62xcs89i7WO9uTAaQmn7YXhhDmxNBApb/PQQbwXQm4v8lIulcyHEEQQi x+ez/yimzCgROB4pinoAKVNq0pEaO3l+YNGyr/j5lgPOtsxZeAzNz1vlZyurguk/daA3whiyQ+/fL rV0IqxTvSPgGSzadq7vA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mYQl0-00GvJ6-Mv; Thu, 07 Oct 2021 10:36:10 +0000 Received: from casper.infradead.org ([2001:8b0:10b:1236::1]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mYQky-00GvIm-Aa for linux-arm-kernel@bombadil.infradead.org; Thu, 07 Oct 2021 10:36:08 +0000 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=Z0HFg8la3obAWV2GGdEJBj4YtmyT7c3u08WRq35X0fo=; b=iRFJNOzhmcmh/dAkVO9P881YVq 86hGzZgtuBXX8iNGtm5OzclBT6NslnMwl9H3R5sb5/Lwe0supJf8CaF0mM4FDLrf/Eg6yrxbgMHZ3 Py44t3TottmnEdhwKDGGH1VzQn6tLcpASoiodCpI860AsXYGFay7QeKxB6OY4KsALjQP1PQtxaJYu mmvrdvMoltaAxf0eo3YBMycRPBw7X3HhivxTMwtOYPtk48PjrD6hloR4Zg4iP3uu5hT+p+8JchnPb NDhOItF9JFmq9KpBgibmgNtWkzCV2eLmJaRQb8h7hBuEGH/zPR3yjcrC92pwx/adGzsH3tixw0GLr MUYR9FGg==; 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 1mYQkI-001l03-C6; Thu, 07 Oct 2021 10:35:43 +0000 Received: by worktop.programming.kicks-ass.net (Postfix, from userid 1000) id 508D29811BB; Thu, 7 Oct 2021 12:35:26 +0200 (CEST) Date: Thu, 7 Oct 2021 12:35:26 +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: <20211007103526.GS174703@worktop.programming.kicks-ass.net> References: <20210924085104.44806-1-21cnbao@gmail.com> <20210924085104.44806-2-21cnbao@gmail.com> <87o883l9c8.mognet@arm.com> <20211006121858.GI174703@worktop.programming.kicks-ass.net> <20211006135550.GJ174703@worktop.programming.kicks-ass.net> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: 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 Thu, Oct 07, 2021 at 11:30:36PM +1300, Barry Song wrote: > On Thu, Oct 7, 2021 at 2:55 AM Peter Zijlstra wrote: > > > > On Thu, Oct 07, 2021 at 01:50:43AM +1300, Barry Song wrote: > > > On Thu, Oct 7, 2021 at 1:20 AM Peter Zijlstra wrote: > > > > > > > > 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 > > > > > > yes. It is correct for patch 1/3, thanks! > > oops, there is a typo there: > + if (cpuid_topo->cluster_id == cpu_topo->cluster_id && > + cpuid_topo->clister_id != -1) { > > clister should be cluster. Yeah, my bad, typing so hard... :-) Already fixed. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel