[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC 12/42] pc: initialize legacy hotplug only for 2.6
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types |
Date: |
Wed, 11 May 2016 20:28:59 -0300 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On Thu, May 12, 2016 at 12:51:55AM +0300, Michael S. Tsirkin wrote:
> On Wed, May 11, 2016 at 03:50:39PM +0200, Igor Mammedov wrote:
> > On Tue, 10 May 2016 17:24:14 -0300
> > Eduardo Habkost <address@hidden> wrote:
> >
> > > On Mon, May 02, 2016 at 02:33:21PM +0200, Igor Mammedov wrote:
> > > > on old machine types CPU hotplug was uncondtionally
> > > > enabled since it was introduced, consuming IO ports
> > > > and providing AML regardles of whether it was actually
> > > > in use or not. Keep it so for 2.6 and older machines.
> > > >
> > > > New machine types will have an option to turn CPU
> > > > hotplug on if it's needed while by default it stays
> > > > disabled not consuming extra RAM/IO resources.
> > > >
> > > > Signed-off-by: Igor Mammedov <address@hidden>
> > >
> > > What if people are using "-machine pc -smp N,max_cpus=M"?
> > > Shouldn't we at least warning about missing CPU hotplug support
> > > when using just "max_cpus" with no "cpu-hotplug=on" with pc-2.7
> > > and newer?
> > Yep, I'll add it on next respin.
> > Would hard error better than warning?
>
> Most people don't need cpu hotplug, attempts
> to hotplug fail, should be enough.
People who don't need CPU hotplug shouldn't be using the max_cpus
option. I believe we should at least warn people (early, during
initialization) that their configuration don't make sense
anymore.
--
Eduardo
- [Qemu-devel] [RFC 01/42] acpi: add aml_debug(), (continued)
- [Qemu-devel] [RFC 01/42] acpi: add aml_debug(), Igor Mammedov, 2016/05/02
- [Qemu-devel] [RFC 06/42] pc: acpi: cpu-hotplug: make AML CPU_foo defines local to cpu_hotplug_acpi_table.c, Igor Mammedov, 2016/05/02
- [Qemu-devel] [RFC 07/42] pc: acpi: mark current CPU hotplug functions as legacy, Igor Mammedov, 2016/05/02
- [Qemu-devel] [RFC 09/42] pc: acpi: simplify build_legacy_cpu_hotplug_aml() signature, Igor Mammedov, 2016/05/02
- [Qemu-devel] [RFC 11/42] pc: add 2.7 machine, Igor Mammedov, 2016/05/02
- [Qemu-devel] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types, Igor Mammedov, 2016/05/02
- Re: [Qemu-devel] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types, Eduardo Habkost, 2016/05/10
- Re: [Qemu-devel] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types, Igor Mammedov, 2016/05/11
- Re: [Qemu-devel] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types, Michael S. Tsirkin, 2016/05/11
- Re: [Qemu-devel] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types,
Eduardo Habkost <=
- Re: [Qemu-devel] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types, Michael S. Tsirkin, 2016/05/12
- Re: [Qemu-devel] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types, Igor Mammedov, 2016/05/12
- Re: [Qemu-devel] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types, Eduardo Habkost, 2016/05/12
- Re: [Qemu-devel] [libvirt] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types, Peter Krempa, 2016/05/12
- Re: [Qemu-devel] [libvirt] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types, Igor Mammedov, 2016/05/12
- Re: [Qemu-devel] [libvirt] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types, Peter Krempa, 2016/05/12
- Re: [Qemu-devel] [libvirt] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types, Igor Mammedov, 2016/05/12
- Re: [Qemu-devel] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types, Eduardo Habkost, 2016/05/11
- Re: [Qemu-devel] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types, Michael S. Tsirkin, 2016/05/12
- Re: [Qemu-devel] [RFC 12/42] pc: initialize legacy hotplug only for 2.6 and older machine types, Igor Mammedov, 2016/05/12