Re: [Qemu-devel] [PATCH v2 2/3] target-i386: turn off CPU.l3-cache only for 2.7 and older machine types
On Tue, Sep 20, 2016 at 09:37:36AM +0200, Igor Mammedov wrote: > On Mon, 19 Sep 2016 13:58:48 -0300 > Eduardo Habkost wrote: > > > On Mon, Sep 19, 2016 at 10:32:34AM +0200, Igor Mammedov wrote: > > > commit (14c985cff target-i386: present virtual L3 cache info for vcpus) > > > misplaced compat property putting it in new 2.8 machine type > > > which would effectively to disable feature until 2.9 is released. > > > Intent of commit probably should be to disable feature for 2.7 > > > and older while allowing not yet released 2.8 to have feature > > > enabled by default. > > > > > > Signed-off-by: Igor Mammedov > > > Reviewed-by: Marcel Apfelbaum > > > --- > > > include/hw/i386/pc.h | 7 +++ > > > 1 file changed, 3 insertions(+), 4 deletions(-) > > > > > > diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h > > > index b0a61f3..29a6c9b 100644 > > > --- a/include/hw/i386/pc.h > > > +++ b/include/hw/i386/pc.h > > > @@ -367,16 +367,15 @@ int e820_get_num_entries(void); > > > bool e820_get_entry(int, uint32_t, uint64_t *, uint64_t *); > > > > > > #define PC_COMPAT_2_8 \ > > > > PC_COMPAT_2_8 isn't even used by any code (and shouldn't be used > > until we release QEMU 2.8). Let's just delete it by now? > Since it's already there in the tree, I'd leave it there. I'd remove to avoid causing confusion during the 2.8 cycle. But as it is something independent from this patch: Reviewed-by: Eduardo Habkost > > > > > > + > > > +#define PC_COMPAT_2_7 \ > > > +HW_COMPAT_2_7 \ > > > {\ > > > .driver = TYPE_X86_CPU,\ > > > .property = "l3-cache",\ > > > .value= "off",\ > > > }, > > > > > > - > > > -#define PC_COMPAT_2_7 \ > > > -HW_COMPAT_2_7 > > > - > > > #define PC_COMPAT_2_6 \ > > > HW_COMPAT_2_6 \ > > > {\ > > > -- > > > 2.7.4 > > > > > > -- Eduardo
Re: [Qemu-devel] [PATCH v2 2/3] target-i386: turn off CPU.l3-cache only for 2.7 and older machine types
On Mon, 19 Sep 2016 13:58:48 -0300 Eduardo Habkost wrote: > On Mon, Sep 19, 2016 at 10:32:34AM +0200, Igor Mammedov wrote: > > commit (14c985cff target-i386: present virtual L3 cache info for vcpus) > > misplaced compat property putting it in new 2.8 machine type > > which would effectively to disable feature until 2.9 is released. > > Intent of commit probably should be to disable feature for 2.7 > > and older while allowing not yet released 2.8 to have feature > > enabled by default. > > > > Signed-off-by: Igor Mammedov > > Reviewed-by: Marcel Apfelbaum > > --- > > include/hw/i386/pc.h | 7 +++ > > 1 file changed, 3 insertions(+), 4 deletions(-) > > > > diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h > > index b0a61f3..29a6c9b 100644 > > --- a/include/hw/i386/pc.h > > +++ b/include/hw/i386/pc.h > > @@ -367,16 +367,15 @@ int e820_get_num_entries(void); > > bool e820_get_entry(int, uint32_t, uint64_t *, uint64_t *); > > > > #define PC_COMPAT_2_8 \ > > PC_COMPAT_2_8 isn't even used by any code (and shouldn't be used > until we release QEMU 2.8). Let's just delete it by now? Since it's already there in the tree, I'd leave it there. > > > + > > +#define PC_COMPAT_2_7 \ > > +HW_COMPAT_2_7 \ > > {\ > > .driver = TYPE_X86_CPU,\ > > .property = "l3-cache",\ > > .value= "off",\ > > }, > > > > - > > -#define PC_COMPAT_2_7 \ > > -HW_COMPAT_2_7 > > - > > #define PC_COMPAT_2_6 \ > > HW_COMPAT_2_6 \ > > {\ > > -- > > 2.7.4 > > >
Re: [Qemu-devel] [PATCH v2 2/3] target-i386: turn off CPU.l3-cache only for 2.7 and older machine types
On Mon, Sep 19, 2016 at 01:58:48PM -0300, Eduardo Habkost wrote: > On Mon, Sep 19, 2016 at 10:32:34AM +0200, Igor Mammedov wrote: > > commit (14c985cff target-i386: present virtual L3 cache info for vcpus) > > misplaced compat property putting it in new 2.8 machine type > > which would effectively to disable feature until 2.9 is released. > > Intent of commit probably should be to disable feature for 2.7 > > and older while allowing not yet released 2.8 to have feature > > enabled by default. > > > > Signed-off-by: Igor Mammedov > > Reviewed-by: Marcel Apfelbaum > > --- > > include/hw/i386/pc.h | 7 +++ > > 1 file changed, 3 insertions(+), 4 deletions(-) > > > > diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h > > index b0a61f3..29a6c9b 100644 > > --- a/include/hw/i386/pc.h > > +++ b/include/hw/i386/pc.h > > @@ -367,16 +367,15 @@ int e820_get_num_entries(void); > > bool e820_get_entry(int, uint32_t, uint64_t *, uint64_t *); > > > > #define PC_COMPAT_2_8 \ > > PC_COMPAT_2_8 isn't even used by any code (and shouldn't be used > until we release QEMU 2.8). Let's just delete it by now? it's unrelated to this patch though. > > + > > +#define PC_COMPAT_2_7 \ > > +HW_COMPAT_2_7 \ > > {\ > > .driver = TYPE_X86_CPU,\ > > .property = "l3-cache",\ > > .value= "off",\ > > }, > > > > - > > -#define PC_COMPAT_2_7 \ > > -HW_COMPAT_2_7 > > - > > #define PC_COMPAT_2_6 \ > > HW_COMPAT_2_6 \ > > {\ > > -- > > 2.7.4 > > > > -- > Eduardo
Re: [Qemu-devel] [PATCH v2 2/3] target-i386: turn off CPU.l3-cache only for 2.7 and older machine types
On Mon, Sep 19, 2016 at 10:32:34AM +0200, Igor Mammedov wrote: > commit (14c985cff target-i386: present virtual L3 cache info for vcpus) > misplaced compat property putting it in new 2.8 machine type > which would effectively to disable feature until 2.9 is released. > Intent of commit probably should be to disable feature for 2.7 > and older while allowing not yet released 2.8 to have feature > enabled by default. > > Signed-off-by: Igor Mammedov > Reviewed-by: Marcel Apfelbaum > --- > include/hw/i386/pc.h | 7 +++ > 1 file changed, 3 insertions(+), 4 deletions(-) > > diff --git a/include/hw/i386/pc.h b/include/hw/i386/pc.h > index b0a61f3..29a6c9b 100644 > --- a/include/hw/i386/pc.h > +++ b/include/hw/i386/pc.h > @@ -367,16 +367,15 @@ int e820_get_num_entries(void); > bool e820_get_entry(int, uint32_t, uint64_t *, uint64_t *); > > #define PC_COMPAT_2_8 \ PC_COMPAT_2_8 isn't even used by any code (and shouldn't be used until we release QEMU 2.8). Let's just delete it by now? > + > +#define PC_COMPAT_2_7 \ > +HW_COMPAT_2_7 \ > {\ > .driver = TYPE_X86_CPU,\ > .property = "l3-cache",\ > .value= "off",\ > }, > > - > -#define PC_COMPAT_2_7 \ > -HW_COMPAT_2_7 > - > #define PC_COMPAT_2_6 \ > HW_COMPAT_2_6 \ > {\ > -- > 2.7.4 > -- Eduardo