Re: [Qemu-devel] [PATCH v7 1/9] i386: Helpers to encode cache information consistently

2018-05-08 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Tuesday, May 8, 2018 2:08 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: ge...@hostfission.com; k...@vger.kernel.org; m...@redhat.com; > k...@tripleback.net; mtosa...@redhat.co

Re: [Qemu-devel] [PATCH v8 3/8] i386: Add new property to control cache info

2018-05-14 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Friday, May 11, 2018 3:48 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: ge...@hostfission.com; k...@vger.kernel.org; m...@redhat.com; > k...@tripleback.net; mtosa...@redhat.co

Re: [Qemu-devel] [PATCH v6 7/9] i386: Add support for CPUID_8000_001E for AMD

2018-05-14 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, May 14, 2018 4:12 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; mar...@redhat.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; ge

Re: [Qemu-devel] [PATCH v9 3/7] i386: Populate AMD Processor Cache Information for cpuid 0x8000001D

2018-05-14 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, May 14, 2018 2:47 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qe

Re: [Qemu-devel] [PATCH v7 1/9] i386: Helpers to encode cache information consistently

2018-05-07 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, May 7, 2018 4:27 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: ge...@hostfission.com; k...@vger.kernel.org; m...@redhat.com; > k...@tripleback.net; mtosa...@redhat.co

Re: [Qemu-devel] [PATCH v7 5/9] i386: Use the statically loaded cache definitions

2018-05-07 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, May 7, 2018 2:38 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; mar...@redhat.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; ge

Re: [Qemu-devel] [PATCH v7 4/9] i386: Add new property to control cache info

2018-05-07 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, May 7, 2018 2:14 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; mar...@redhat.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; ge

Re: [Qemu-devel] [PATCH v7 2/9] i386: Add cache information in X86CPUDefinition

2018-05-07 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, May 7, 2018 2:10 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; mar...@redhat.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; ge

Re: [Qemu-devel] [PATCH v7 5/9] i386: Use the statically loaded cache definitions

2018-05-07 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, May 7, 2018 2:16 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; mar...@redhat.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; ge

Re: [Qemu-devel] [PATCH v7 1/9] i386: Helpers to encode cache information consistently

2018-05-07 Thread Moger, Babu
Eduardo, Thanks for all the comments. Will respond to each one separately. > -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, May 7, 2018 2:05 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; mar...@redhat.com

Re: [Qemu-devel] [PATCH v7 7/9] i386: Add support for CPUID_8000_001E for AMD

2018-05-07 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, May 7, 2018 2:40 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; mar...@redhat.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; ge

Re: [Qemu-devel] [PATCH v7 7/9] i386: Add support for CPUID_8000_001E for AMD

2018-05-11 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Friday, May 11, 2018 9:12 AM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; mar...@redhat.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; ge

Re: [Qemu-devel] [PATCH v8 3/8] i386: Add new property to control cache info

2018-05-11 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Friday, May 11, 2018 2:22 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qe

Re: [Qemu-devel] [PATCH v8 7/8] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-05-11 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Friday, May 11, 2018 3:47 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.co

Re: [Qemu-devel] [PATCH v7 7/9] i386: Add support for CPUID_8000_001E for AMD

2018-05-08 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Tuesday, May 8, 2018 9:17 AM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; mar...@redhat.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; ge

Re: [Qemu-devel] [PATCH v7 6/9] i386: Populate AMD Processor Cache Information for cpuid 0x8000001D

2018-05-08 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, May 7, 2018 4:07 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; mar...@redhat.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; ge

Re: [Qemu-devel] [PATCH v7 5/9] i386: Use the statically loaded cache definitions

2018-05-08 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Tuesday, May 8, 2018 9:12 AM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; mar...@redhat.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; ge

Re: [Qemu-devel] [PATCH v7 4/9] i386: Add new property to control cache info

2018-05-08 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Tuesday, May 8, 2018 9:26 AM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; mar...@redhat.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qe

Re: [Qemu-devel] [PATCH v7 1/9] i386: Helpers to encode cache information consistently

2018-05-08 Thread Moger, Babu
Hi Eduardo, One more comment below. > -Original Message- > From: kvm-ow...@vger.kernel.org [mailto:kvm-ow...@vger.kernel.org] > On Behalf Of Moger, Babu > Sent: Monday, May 7, 2018 5:48 PM > To: Eduardo Habkost <ehabk...@redhat.com> > Cc: ge...@hostfission.com;

Re: [Qemu-devel] [PATCH v7 4/9] i386: Add new property to control cache info

2018-05-08 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Tuesday, May 8, 2018 1:34 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; mar...@redhat.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qe

Re: [Qemu-devel] [PATCH v10 2/5] i386: Populate AMD Processor Cache Information for cpuid 0x8000001D

2018-05-22 Thread Moger, Babu
> -Original Message- > From: Duran, Leo > Sent: Monday, May 21, 2018 8:32 PM > To: Moger, Babu <babu.mo...@amd.com>; m...@redhat.com; > marcel.apfelb...@gmail.com; pbonz...@redhat.com; r...@twiddle.net; > ehabk...@redhat.com; mtosa...@redhat.com > C

Re: [Qemu-devel] [PATCH v11 0/5] i386: Enable TOPOEXT to support hyperthreading on AMD CPU

2018-05-24 Thread Moger, Babu
> -Original Message- > From: Kash Pande [mailto:k...@tripleback.net] > Sent: Thursday, May 24, 2018 10:51 AM > To: Moger, Babu <babu.mo...@amd.com>; m...@redhat.com; > marcel.apfelb...@gmail.com; pbonz...@redhat.com; r...@twiddle.net; > ehabk...@redhat.com; mtosa.

Re: [Qemu-devel] [PATCH v9 3/7] i386: Populate AMD Processor Cache Information for cpuid 0x8000001D

2018-05-16 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Wednesday, May 16, 2018 7:52 AM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.co

Re: [Qemu-devel] [PATCH v10 2/5] i386: Populate AMD Processor Cache Information for cpuid 0x8000001D

2018-05-23 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Tuesday, May 22, 2018 9:04 AM > To: Moger, Babu <babu.mo...@amd.com> > Cc: Duran, Leo <leo.du...@amd.com>; m...@redhat.com; > marcel.apfelb...@gmail.com; pbonz...@redhat.

Re: [Qemu-devel] [PATCH v10 2/5] i386: Populate AMD Processor Cache Information for cpuid 0x8000001D

2018-05-23 Thread Moger, Babu
Hi Eduardo, Please see my comments below. > -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Tuesday, May 22, 2018 8:54 AM > To: Moger, Babu <babu.mo...@amd.com> > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.co

Re: [Qemu-devel] [PATCH v12 2/4] i386: Verify if topoext feature can be supported

2018-06-07 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Wednesday, June 6, 2018 5:06 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-dev

Re: [Qemu-devel] [PATCH v12 3/4] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-08 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Friday, June 8, 2018 2:50 PM > To: Moger, Babu > Cc: ge...@hostfission.com; k...@vger.kernel.org; m...@redhat.com; > k...@tripleback.net; mtosa...@redhat.com; xiaoguangr...@tencent

Re: [Qemu-devel] [PATCH v12 3/4] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-08 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Friday, June 8, 2018 2:23 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-devel@nongnu.org; >

Re: [Qemu-devel] [PATCH v12 1/4] i386: Add support for CPUID_8000_001E for AMD

2018-06-08 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Wednesday, June 6, 2018 4:26 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-dev

Re: [Qemu-devel] [PATCH v12 3/4] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-08 Thread Moger, Babu
Hi Eduardo, Sorry for the late response. Got pulled into something else. > -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Wednesday, June 6, 2018 5:40 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@

Re: [Qemu-devel] [PATCH v12 3/4] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-08 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Friday, June 8, 2018 2:50 PM > To: Moger, Babu > Cc: ge...@hostfission.com; k...@vger.kernel.org; m...@redhat.com; > k...@tripleback.net; mtosa...@redhat.com; xiaoguangr...@tencent

Re: [Qemu-devel] [PATCH v14 1/6] i386: Set TOPOEXT unconditionally for comapatibility

2018-06-14 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Wednesday, June 13, 2018 9:22 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-dev

Re: [Qemu-devel] [PATCH v13 3/5] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-12 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Tuesday, June 12, 2018 12:40 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-dev

Re: [Qemu-devel] [PATCH v13 3/5] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-13 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Wednesday, June 13, 2018 1:18 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-dev

Re: [Qemu-devel] [PATCH v13 3/5] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-13 Thread Moger, Babu
..@twiddle.net; mtosa...@redhat.com; qemu-devel@nongnu.org; > k...@vger.kernel.org; k...@tripleback.net; ge...@hostfission.com; Jiri > Denemark > Subject: Re: [PATCH v13 3/5] i386: Enable TOPOEXT feature on AMD EPYC > CPU > > > > On 06/12/2018 02:05 PM, Eduardo Habkost wrote: > &

Re: [Qemu-devel] [PATCH v13 3/5] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-13 Thread Moger, Babu
> -Original Message- > From: kvm-ow...@vger.kernel.org [mailto:kvm-ow...@vger.kernel.org] > On Behalf Of Moger, Babu > Sent: Wednesday, June 13, 2018 1:11 PM > To: Eduardo Habkost > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r.

Re: [Qemu-devel] [PATCH v13 3/5] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-13 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Wednesday, June 13, 2018 12:18 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-dev

Re: [Qemu-devel] [PATCH v14 5/6] i386: Disable TOPOEXT feature if it cannot be supported

2018-06-15 Thread Moger, Babu
> -Original Message- > From: Moger, Babu > Sent: Thursday, June 14, 2018 6:09 PM > To: Moger, Babu ; Eduardo Habkost > > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-devel@nongnu.org; >

Re: [Qemu-devel] [PATCH v14 2/6] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-14 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Thursday, June 14, 2018 1:40 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-dev

Re: [Qemu-devel] [PATCH v14 3/6] i386: Disable TOPOEXT feature on pc-2.12

2018-06-14 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Thursday, June 14, 2018 1:41 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-dev

Re: [Qemu-devel] [PATCH v14 5/6] i386: Disable TOPOEXT feature if it cannot be supported

2018-06-14 Thread Moger, Babu
> -Original Message- > From: kvm-ow...@vger.kernel.org [mailto:kvm-ow...@vger.kernel.org] > On Behalf Of Moger, Babu > Sent: Thursday, June 14, 2018 5:19 PM > To: Eduardo Habkost > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r.

Re: [Qemu-devel] [PATCH v14 2/6] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-14 Thread Moger, Babu
> -Original Message- > From: kvm-ow...@vger.kernel.org [mailto:kvm-ow...@vger.kernel.org] > On Behalf Of Moger, Babu > Sent: Thursday, June 14, 2018 3:41 PM > To: Eduardo Habkost > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r.

Re: [Qemu-devel] [PATCH v15 1/3] i386: Fix up the Node id for CPUID_8000_001E

2018-06-15 Thread Moger, Babu
m > Cc: qemu-devel@nongnu.org; mtosa...@redhat.com; k...@vger.kernel.org; > k...@tripleback.net; ge...@hostfission.com; Moger, Babu > > Subject: [PATCH v15 1/3] i386: Fix up the Node id for CPUID_8000_001E > > This is part of topoext support. To keep the compatibility, we n

Re: [Qemu-devel] [PATCH v13 1/5] i386: Add support for CPUID_8000_001E for AMD

2018-06-11 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, June 11, 2018 3:55 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-dev

Re: [Qemu-devel] [PATCH v13 2/5] i386: Introduce auto_topoext bit to manage topoext

2018-06-11 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, June 11, 2018 3:46 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-dev

Re: [Qemu-devel] [PATCH v13 4/5] i386: Verify and enable topoext feature if supported

2018-06-11 Thread Moger, Babu
> -Original Message- > From: kvm-ow...@vger.kernel.org [mailto:kvm-ow...@vger.kernel.org] > On Behalf Of Eduardo Habkost > Sent: Monday, June 11, 2018 3:52 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r.

Re: [Qemu-devel] [PATCH v13 2/5] i386: Introduce auto_topoext bit to manage topoext

2018-06-11 Thread Moger, Babu
> -Original Message- > From: kvm-ow...@vger.kernel.org [mailto:kvm-ow...@vger.kernel.org] > On Behalf Of Eduardo Habkost > Sent: Monday, June 11, 2018 4:05 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r.

Re: [Qemu-devel] [PATCH v13 2/5] i386: Introduce auto_topoext bit to manage topoext

2018-06-11 Thread Moger, Babu
er > Sent: Friday, June 8, 2018 5:56 PM > To: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; ehabk...@redhat.com; mtosa...@redhat.com > Cc: qemu-devel@nongnu.org; k...@vger.kernel.org; Moger, Babu > ; k...@tripleback.net; ge...@hostfission.com &

Re: [Qemu-devel] [PATCH v13 3/5] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-11 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, June 11, 2018 3:51 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-dev

Re: [Qemu-devel] [PATCH v13 3/5] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-13 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Wednesday, June 13, 2018 1:49 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-dev

Re: [Qemu-devel] [PATCH v13 3/5] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-12 Thread Moger, Babu
> -Original Message- > From: kvm-ow...@vger.kernel.org [mailto:kvm-ow...@vger.kernel.org] > On Behalf Of Eduardo Habkost > Sent: Monday, June 11, 2018 4:10 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r.

Re: [Qemu-devel] [PATCH v14 5/6] i386: Disable TOPOEXT feature if it cannot be supported

2018-06-14 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Thursday, June 14, 2018 2:13 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; qemu-dev

Re: [Qemu-devel] [PATCH v11 3/5] i386: Add support for CPUID_8000_001E for AMD

2018-06-05 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, June 4, 2018 9:46 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; ge...@hostfission.com; >

Re: [Qemu-devel] [PATCH v11 2/5] i386: Populate AMD Processor Cache Information for cpuid 0x8000001D

2018-06-05 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, June 4, 2018 8:59 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; ge...@hostfission.com; >

Re: [Qemu-devel] [PATCH v11 4/5] i386: Enable TOPOEXT feature on AMD EPYC CPU

2018-06-05 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, June 4, 2018 9:57 PM > To: Moger, Babu > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r...@twiddle.net; mtosa...@redhat.com; ge...@hostfission.com; >

Re: [Qemu-devel] [PATCH v11 3/5] i386: Add support for CPUID_8000_001E for AMD

2018-06-05 Thread Moger, Babu
> -Original Message- > From: kvm-ow...@vger.kernel.org [mailto:kvm-ow...@vger.kernel.org] > On Behalf Of Moger, Babu > Sent: Tuesday, June 5, 2018 10:49 AM > To: Eduardo Habkost > Cc: m...@redhat.com; marcel.apfelb...@gmail.com; pbonz...@redhat.com; > r.

Re: [Qemu-devel] [PATCH 0/5] Enable TOPOEXT to support hyperthreading on AMD CPU

2018-02-20 Thread Moger, Babu
.@redhat.com; r...@twiddle.net; ehabk...@redhat.com; > p...@polepetko.eu > Cc: mtosa...@redhat.com; Moger, Babu <babu.mo...@amd.com>; qemu- > de...@nongnu.org; k...@vger.kernel.org > Subject: [PATCH 0/5] Enable TOPOEXT to support hyperthreading on AMD > CPU > > Thes

Re: [Qemu-devel] [PATCH] i386: Disable TOPOEXT by default on "-cpu host"

2018-08-13 Thread Moger, Babu
Looks good. Did some basic testing. Reviewed-by: Babu Moger > -Original Message- > From: Richard W.M. Jones > Sent: Friday, August 10, 2018 2:41 AM > To: Eduardo Habkost > Cc: qemu-devel@nongnu.org; Paolo Bonzini ; > Richard Henderson ; Moger, Babu > > S

Re: [Qemu-devel] [PATCH for-3.0] i386: Rename enum CacheType members

2018-07-18 Thread Moger, Babu
> -Original Message- > From: Aleksandar Markovic [mailto:amarko...@wavecomp.com] > Sent: Wednesday, July 18, 2018 8:35 AM > To: Philippe Mathieu-Daudé ; Eduardo Habkost > ; qemu-devel@nongnu.org > Cc: Moger, Babu ; Paolo Bonzini > ; Aurelien Jarno ; Richard >

Re: [Qemu-devel] [PATCH] pc: Fix typo on PC_COMPAT_2_12

2018-07-09 Thread Moger, Babu
Looks good. thanks > -Original Message- > From: Eduardo Habkost [mailto:ehabk...@redhat.com] > Sent: Monday, July 2, 2018 8:10 PM > To: qemu-devel@nongnu.org > Cc: Eduardo Habkost ; Paolo Bonzini > ; Moger, Babu ; Michael > S. Tsirkin ; Igor Mammedov > Subje

Re: [Qemu-devel] [PATCH v2 1/5] target/i386: Fix a minor typo found while reviwing

2018-02-28 Thread Moger, Babu
> -Original Message- > From: Radim Krčmář [mailto:rkrc...@redhat.com] > Sent: Wednesday, February 28, 2018 11:38 AM > To: Moger, Babu <babu.mo...@amd.com> > Cc: pbonz...@redhat.com; r...@twiddle.net; ehabk...@redhat.com; > mtosa...@redhat.com; qemu-devel@nongnu.o

Re: [Qemu-devel] [PATCH v2 1/5] target/i386: Fix a minor typo found while reviwing

2018-02-28 Thread Moger, Babu
> -Original Message- > From: Eric Blake [mailto:ebl...@redhat.com] > Sent: Wednesday, February 28, 2018 12:49 PM > To: Radim Krčmář <rkrc...@redhat.com>; Moger, Babu > <babu.mo...@amd.com> > Cc: ehabk...@redhat.com; k...@vger.kernel.org; mtosa...@re

Re: [Qemu-devel] [PATCH v2 3/5] target/i386: Add support for CPUID_8000_001E for AMD

2018-02-28 Thread Moger, Babu
> -Original Message- > From: Radim Krčmář [mailto:rkrc...@redhat.com] > Sent: Wednesday, February 28, 2018 12:24 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: pbonz...@redhat.com; r...@twiddle.net; ehabk...@redhat.com; > mtosa...@redhat.com; qemu-devel@nongnu.o

Re: [Qemu-devel] [PATCH v2 2/5] target/i386: Populate AMD Processor Cache Information

2018-03-01 Thread Moger, Babu
Radim, Thanks for your comments. I am working on the changes. But, I need few clarifications on your comments. Please see inline. > -Original Message- > From: Radim Krčmář [mailto:rkrc...@redhat.com] > Sent: Wednesday, February 28, 2018 12:09 PM > To: Moger, Babu <bab

Re: [Qemu-devel] [PATCH v5 0/9] i386: Enable TOPOEXT to support hyperthreading on AMD CPU

2018-04-08 Thread Moger, Babu
Ping... Checking to see if there are any further comments on these patch series. Thanks > -Original Message- > From: Moger, Babu > Sent: Tuesday, March 27, 2018 4:31 PM > To: m...@redhat.com; mar...@redhat.com; pbonz...@redhat.com; > r...@twiddle.net; ehabk...@re

Re: [Qemu-devel] [PATCH v5 4/9] i386: Add new property to control cache info

2018-04-09 Thread Moger, Babu
> -Original Message- > From: Alexandr Iarygin <alexandr.iary...@profitbricks.com> > Sent: Monday, April 9, 2018 12:00 PM > To: Moger, Babu <babu.mo...@amd.com>; m...@redhat.com; > mar...@redhat.com; pbonz...@redhat.com; r...@twiddle.net; > ehabk...@redhat

Re: [Qemu-devel] [PATCH v4 2/5] target/i386: Populate AMD Processor Cache Information

2018-03-27 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost <ehabk...@redhat.com> > Sent: Wednesday, March 21, 2018 3:30 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: pbonz...@redhat.com; r...@twiddle.net; rkrc...@redhat.com; > Lendacky, Thomas <thomas.lenda...@amd.c

Re: [Qemu-devel] [PATCH v4 2/5] target/i386: Populate AMD Processor Cache Information

2018-03-21 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost <ehabk...@redhat.com> > Sent: Wednesday, March 21, 2018 1:15 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: pbonz...@redhat.com; r...@twiddle.net; rkrc...@redhat.com; > Lendacky, Thomas <thomas.lenda...@amd.c

Re: [Qemu-devel] [PATCH for-2.13] i386: Helpers to encode cache information consistently

2018-03-19 Thread Moger, Babu
@amd.com>; Paolo Bonzini > <pbonz...@redhat.com>; Moger, Babu <babu.mo...@amd.com>; > Lendacky, Thomas <thomas.lenda...@amd.com>; Igor Mammedov > <imamm...@redhat.com>; Hook, Gary <gary.h...@amd.com>; Eduardo > Habkost <ehabk...@redhat.com>; Radi

Re: [Qemu-devel] [PATCH v4 2/5] target/i386: Populate AMD Processor Cache Information

2018-03-21 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost <ehabk...@redhat.com> > Sent: Wednesday, March 21, 2018 12:10 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: pbonz...@redhat.com; r...@twiddle.net; rkrc...@redhat.com; > Lendacky, Thomas <thomas.lenda...@amd.c

Re: [Qemu-devel] [PATCH v4 2/5] target/i386: Populate AMD Processor Cache Information

2018-03-21 Thread Moger, Babu
Hi Eduardo, > -Original Message- > From: Eduardo Habkost <ehabk...@redhat.com> > Sent: Tuesday, March 20, 2018 12:54 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: pbonz...@redhat.com; r...@twiddle.net; rkrc...@redhat.com; > Lendacky, Thomas <thomas.

Re: [Qemu-devel] [PATCH v4 2/5] target/i386: Populate AMD Processor Cache Information

2018-03-20 Thread Moger, Babu
Hi Eduardo, Thanks for the comments. Please see the response inline. > -Original Message- > From: Eduardo Habkost <ehabk...@redhat.com> > Sent: Friday, March 16, 2018 1:00 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: pbonz...@redhat.com; r...@twiddle.net; rkr

Re: [Qemu-devel] [PATCH v4 2/5] target/i386: Populate AMD Processor Cache Information

2018-03-20 Thread Moger, Babu
> -Original Message- > From: Eduardo Habkost <ehabk...@redhat.com> > Sent: Tuesday, March 20, 2018 12:54 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: pbonz...@redhat.com; r...@twiddle.net; rkrc...@redhat.com; > Lendacky, Thomas <thomas.lenda...@amd.c

Re: [Qemu-devel] [PATCH v2 3/5] target/i386: Add support for CPUID_8000_001E for AMD

2018-03-02 Thread Moger, Babu
> -Original Message- > From: Radim Krčmář [mailto:rkrc...@redhat.com] > Sent: Thursday, March 1, 2018 1:57 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: pbonz...@redhat.com; r...@twiddle.net; ehabk...@redhat.com; > mtosa...@redhat.com; qemu-devel@nongnu.org; k

Re: [Qemu-devel] [PATCH v2 2/5] target/i386: Populate AMD Processor Cache Information

2018-03-02 Thread Moger, Babu
> -Original Message- > From: Radim Krčmář [mailto:rkrc...@redhat.com] > Sent: Thursday, March 1, 2018 1:56 PM > To: Moger, Babu <babu.mo...@amd.com> > Cc: pbonz...@redhat.com; r...@twiddle.net; ehabk...@redhat.com; > mtosa...@redhat.com; qemu-devel@nongnu.org; k

[Qemu-devel] [RFC 2 PATCH 03/16] hw/i386: Introduce X86CPUTopoInfo to contain topology info

2019-09-06 Thread Moger, Babu
This is an effort to re-arrange few data structure for better readability. Add X86CPUTopoInfo which will have all the topology informations required to build the cpu topology. There is no functional changes. Signed-off-by: Babu Moger --- hw/i386/pc.c | 40

[Qemu-devel] [RFC 2 PATCH 07/16] hw/386: Add new epyc mode topology decoding functions

2019-09-06 Thread Moger, Babu
These functions add support for building new epyc mode topology given smp details like numa nodes, cores, threads and sockets. Subsequent patches will use these functions to build the topology. The topology details are available in Processor Programming Reference (PPR) for AMD Family 17h Model

[Qemu-devel] [RFC 2 PATCH 04/16] machine: Add SMP Sockets in CpuTopology

2019-09-06 Thread Moger, Babu
Store the smp Sockets in CpuTopology. Socket information is required to build the cpu topology in new epyc mode. Signed-off-by: Babu Moger --- hw/core/machine.c |1 + hw/i386/pc.c|1 + include/hw/boards.h |2 ++ vl.c|1 + 4 files changed, 5

[Qemu-devel] [RFC 2 PATCH 13/16] machine: Add new epyc property in PCMachineState

2019-09-06 Thread Moger, Babu
Adds new epyc property in PCMachineState and also in MachineState. This property will be used to initialize the mode specific handlers to generate apic ids. Signed-off-by: Babu Moger --- hw/i386/pc.c | 23 +++ include/hw/boards.h |2 ++ include/hw/i386/pc.h |

[Qemu-devel] [RFC 2 PATCH 14/16] hw/i386: Introduce epyc mode function handlers

2019-09-06 Thread Moger, Babu
Introduce following handlers for new epyc mode. x86_apicid_from_cpu_idx_epyc: Generate apicid from cpu index. x86_topo_ids_from_apicid_epyc: Generate topo ids from apic id. x86_apicid_from_topo_ids_epyci: Generate apicid from topo ids. Signed-off-by: Babu Moger --- hw/i386/pc.c |5 + 1

[Qemu-devel] [RFC 2 PATCH 12/16] hw/i386: Introduce apic_id_from_topo_ids handler in PCMachineState

2019-09-06 Thread Moger, Babu
Add function pointer apic_id_from_topo_ids in PCMachineState. Initialize with correct handler based on the mode selected. Also rename the handler apicid_from_topo_ids to x86_apicid_from_topo_ids for consistency. x86_apicid_from_topo_ids will be the default handler. Signed-off-by: Babu Moger ---

[Qemu-devel] [RFC 2 PATCH 08/16] i386: Cleanup and use the new epyc mode topology functions

2019-09-06 Thread Moger, Babu
Use the new epyc mode functions and delete the unused code. Signed-off-by: Babu Moger --- target/i386/cpu.c | 171 +++-- 1 file changed, 48 insertions(+), 123 deletions(-) diff --git a/target/i386/cpu.c b/target/i386/cpu.c index

[Qemu-devel] [RFC 2 PATCH 09/16] hw/i386: Introduce initialize_topo_info function

2019-09-06 Thread Moger, Babu
Introduce initialize_topo_info to initialize X86CPUTopoInfo data structure to build the topology. No functional change. Signed-off-by: Babu Moger --- hw/i386/pc.c | 29 + 1 file changed, 17 insertions(+), 12 deletions(-) diff --git a/hw/i386/pc.c b/hw/i386/pc.c

[Qemu-devel] [RFC 2 PATCH 06/16] hw/core: Add core complex id in X86CPU topology

2019-09-06 Thread Moger, Babu
Introduce cpu core complex id(ccx_id) in x86CPU topology. Each CCX can have upto 4 cores and share same L3 cache. This information is required to build the topology in new apyc mode. Signed-off-by: Babu Moger --- hw/core/machine-hmp-cmds.c |3 +++ hw/core/machine.c | 13

[Qemu-devel] [RFC 2 PATCH 11/16] Introduce-topo_ids_from_apicid-handler

2019-09-06 Thread Moger, Babu
hw/i386: Introduce topo_ids_from_apicid handler PCMachineState Add function pointer topo_ids_from_apicid in PCMachineState. Initialize with correct handler based on mode selected. x86_apicid_from_cpu_idx will be the default handler. Signed-off-by: Babu Moger --- hw/i386/pc.c | 13

[Qemu-devel] [RFC 2 PATCH 10/16] hw/i386: Introduce apicid_from_cpu_idx in PCMachineState

2019-09-06 Thread Moger, Babu
Add function pointers in PCMachineState to handle apic id specific functionalities. This will be used to initialize with correct handlers based on mode selected. x86_apicid_from_cpu_idx will be default handler. Signed-off-by: Babu Moger --- hw/i386/pc.c |5 -

[Qemu-devel] [RFC 2 PATCH 16/16] hw/core: Fix up the machine_set_cpu_numa_node for epyc

2019-09-06 Thread Moger, Babu
Current topology id match will not work for epyc mode when setting the node id. In epyc mode, ids like smt_id, thread_id, core_id, ccx_id, socket_id can be same for more than one CPUs with across two numa nodes. For example, we can have two CPUs with following ids on two different node. 1.

[Qemu-devel] [RFC 2 PATCH 00/16] APIC ID fixes for AMD EPYC CPU models

2019-09-06 Thread Moger, Babu
These series fixes the problems encoding APIC ID for AMD EPYC cpu models. https://bugzilla.redhat.com/show_bug.cgi?id=1728166 This is the second pass to give an idea of the changes required to address the issue. First pass is availabe at https://patchwork.kernel.org/cover/11069785/ Currently,

[Qemu-devel] [RFC 2 PATCH 01/16] numa: Split the numa functionality

2019-09-06 Thread Moger, Babu
To support new epyc mode, we need to know the number of numa nodes in advance to generate apic id correctly. So, split the numa initialization into two. The function parse_numa initializes numa_info and updates nb_numa_nodes. And then parse_numa_node does the numa node initialization.

[Qemu-devel] [RFC 2 PATCH 02/16] hw/i386: Rename X86CPUTopoInfo structure to X86CPUTopoIDs

2019-09-06 Thread Moger, Babu
Rename few data structures related to X86 topology. X86CPUTopoIDs will have individual arch ids. Next patch introduces X86CPUTopoInfo which will have all topology information(like cores, threads etc..). Adds node_id and ccx_id. This will be required to support new epyc mode mode. There is no

Re: [Qemu-devel] [RFC 2 PATCH 06/16] hw/core: Add core complex id in X86CPU topology

2019-09-06 Thread Moger, Babu
On 9/6/19 2:20 PM, Eric Blake wrote: > On 9/6/19 2:12 PM, Moger, Babu wrote: >> Introduce cpu core complex id(ccx_id) in x86CPU topology. >> Each CCX can have upto 4 cores and share same L3 cache. >> This information is required to build the topology in >> new

[Qemu-devel] [RFC 2 PATCH 05/16] hw/i386: Simplify topology Offset/width Calculation

2019-09-06 Thread Moger, Babu
Some parameters are unnecessarily passed for offset/width calculation. Remove those parameters from function prototypes. No functional change. Signed-off-by: Babu Moger --- include/hw/i386/topology.h | 45 ++-- target/i386/cpu.c | 12

[Qemu-devel] [RFC 2 PATCH 15/16] i386: Fix pkg_id offset for epyc mode

2019-09-06 Thread Moger, Babu
Signed-off-by: Babu Moger --- target/i386/cpu.c | 24 +++- 1 file changed, 19 insertions(+), 5 deletions(-) diff --git a/target/i386/cpu.c b/target/i386/cpu.c index f25491a029..f8b1fc5c07 100644 --- a/target/i386/cpu.c +++ b/target/i386/cpu.c @@ -4094,9 +4094,10 @@ void

[Qemu-devel] [RFC PATCH 2/5] hw/i386: Add AMD EPYC topology encoding

2019-07-31 Thread Moger, Babu
Currently, the apicid is a sequential number in x86 cpu models. This works fine for most of the cases. But, in certain cases this will result into cpu topology inconsistency. This problem was observed in AMD EPYC cpu models. To address that we need to build apicid as per the hardware

[Qemu-devel] [RFC PATCH 3/5] i386: Use topology functions from topology.h

2019-07-31 Thread Moger, Babu
Use the functions defined in topology.h and remove the old code. Signed-off-by: Babu Moger --- target/i386/cpu.c | 146 +- 1 file changed, 27 insertions(+), 119 deletions(-) diff --git a/target/i386/cpu.c b/target/i386/cpu.c index

[Qemu-devel] [RFC PATCH 4/5] hw/i386: Generate apicid based on cpu_type

2019-07-31 Thread Moger, Babu
Check the cpu_type before calling the apicid functions from topology.h. Signed-off-by: Babu Moger --- hw/i386/pc.c | 81 +--- 1 file changed, 70 insertions(+), 11 deletions(-) diff --git a/hw/i386/pc.c b/hw/i386/pc.c index ef39463fd5..dad55c940f

[Qemu-devel] [RFC PATCH 0/5] APIC ID fixes for AMD EPYC CPU models

2019-07-31 Thread Moger, Babu
These series fixes the problems encoding APIC ID for AMD EPYC cpu models. https://bugzilla.redhat.com/show_bug.cgi?id=1728166 This is the first pass to give an idea of the changes required to address the issue. Please feel free to comment. Currently, apic id is decoded based on

[Qemu-devel] [RFC PATCH 5/5] i386: Fix pkg_id offset EPYC

2019-07-31 Thread Moger, Babu
Per Processor Programming Reference (PPR) for AMD Family 17h Models, the pkg_id offset in apicid is 6. Fix the offset based on EPYC models. Signed-off-by: Babu Moger --- target/i386/cpu.c | 14 +- 1 file changed, 9 insertions(+), 5 deletions(-) diff --git a/target/i386/cpu.c

[Qemu-devel] [RFC PATCH 1/5] hw/boards: Add sockets in CpuTopology structure

2019-07-31 Thread Moger, Babu
Add sockets in CpuTopology. This is required when building the CPU topology. Signed-off-by: Babu Moger --- hw/core/machine.c | 1 + hw/i386/pc.c| 1 + include/hw/boards.h | 2 ++ vl.c| 1 + 4 files changed, 5 insertions(+) diff --git a/hw/core/machine.c

Re: [Qemu-devel] [RFC PATCH 4/5] hw/i386: Generate apicid based on cpu_type

2019-08-01 Thread Moger, Babu
Hi Eduardo, Thanks for the quick comments. I will look into your comments closely and will let you know if I have questions. > -Original Message- > From: Eduardo Habkost > Sent: Thursday, August 1, 2019 2:29 PM > To: Moger, Babu > Cc: marcel.apfelb...@gmail.com; m...@re

  1   2   >