From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eduardo Habkost Subject: Re: [PATCH v16 1/3] i386: Fix up the Node id for CPUID_8000_001E Date: Thu, 21 Jun 2018 15:28:41 -0300 Message-ID: <20180621182841.GV7451@localhost.localdomain> References: <1529443919-67509-1-git-send-email-babu.moger@amd.com> <1529443919-67509-2-git-send-email-babu.moger@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: geoff@hostfission.com, kvm@vger.kernel.org, mst@redhat.com, kash@tripleback.net, mtosatti@redhat.com, qemu-devel@nongnu.org, pbonzini@redhat.com, rth@twiddle.net To: Babu Moger Return-path: Content-Disposition: inline In-Reply-To: <1529443919-67509-2-git-send-email-babu.moger@amd.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+gceq-qemu-devel2=m.gmane.org@nongnu.org Sender: "Qemu-devel" List-Id: kvm.vger.kernel.org On Tue, Jun 19, 2018 at 05:31:57PM -0400, Babu Moger wrote: > This is part of topoext support. To keep the compatibility, it is better > we support all the combination of nr_cores and nr_threads currently > supported. By allowing more nr_cores and nr_threads, we might end up with > more nodes than we can actually support with the real hardware. We need to > fix up the node id to make this work. We can achieve this by shifting the > socket_id bits left to address more nodes. > > Signed-off-by: Babu Moger Reviewed-by: Eduardo Habkost Queued, thanks. -- Eduardo