Date: Tue, 25 Jul 2017 07:56:47 +0200
From: Borislav Petkov <bp@suse.de>
To: Suravee Suthikulpanit <Suravee.Suthikulpanit@amd.com>
Cc: linux-kernel@vger.kernel.org, x86@kernel.org, tglx@linutronix.de,
        mingo@redhat.com, hpa@zytor.com, peterz@infradead.org,
        Yazen.Ghannam@amd.com, Andreas Herrmann <aherrmann@suse.com>
Subject: Re: [PATCH v4 2/2] x86/amd: Fixup cpu_core_id for family17h downcore
 configuration
Message-ID: <20170725055647.GB21822@nazgul.tnic>
References: <1500888165-2345-1-git-send-email-suravee.suthikulpanit@amd.com>
 <1500888165-2345-3-git-send-email-suravee.suthikulpanit@amd.com>
 <20170724111408.GC28024@nazgul.tnic>
 <c1ed46c7-de8d-07f4-0e0b-68128be8a256@amd.com>
 <20170724144445.GA31300@nazgul.tnic>
 <8e0f6094-f527-3ce7-3652-12df0a8f1ac3@amd.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <8e0f6094-f527-3ce7-3652-12df0a8f1ac3@amd.com>
User-Agent: Mutt/1.6.0 (2016-04-01)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 879
Lines: 24

On Tue, Jul 25, 2017 at 12:51:53PM +0700, Suravee Suthikulpanit wrote:
> Ok. Sure, it doesn't need be contiguous. But at least the cpu_core_id should
> represent an ID that make some sense since it is used in the
> arch/x86/kernel/smpboot.c: match_smt() and some other places. So, if it's
> invalid for the downcore configuration (i.e. duplicated where it should not
> be), we should at least clean this up.

Ah right, we do use it for the SMT siblings. So yes, it should be
correct for them. And I'm pretty sure the numbers we derive from the
initial APIC ID are already good enough for that.

> I will update the patch to only limit the fixup to pre-family17h.

Yeah, give that a try. Make sure to check the topology masks are
correct.

Thanks.

-- 
Regards/Gruss,
    Boris.

SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton, HRB 21284 (AG Nürnberg)
--