Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-07-04 Thread Igor Mammedov
vel@nongnu.org, "Igor Mammedov" , "Paolo Bonzini" Sent: Friday, June 8, 2012 2:36:53 PM Subject: Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n] On 2012-06-08 14:34, Andreas Färber wrote: Am 08.06.2012 14:05, schrieb Igor Mammedov: On Fri, Jun 08, 2012 at

Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-06-08 Thread Q (Igor Mammedov)
>>> , qemu-devel@nongnu.org, "Igor >>> Mammedov" , "Paolo Bonzini" >>> Sent: Friday, June 8, 2012 2:36:53 PM >>> Subject: Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as       >>> /machine/cpu[n] >>> >>> O

Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-06-08 Thread Andreas Färber
> , qemu-devel@nongnu.org, "Igor >>> Mammedov" , "Paolo Bonzini" >>> Sent: Friday, June 8, 2012 2:36:53 PM >>> Subject: Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as >>> /machine/cpu[n] >>> >>> O

Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-06-08 Thread Jan Kiszka
; , "Paolo Bonzini" >> Sent: Friday, June 8, 2012 2:36:53 PM >> Subject: Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as >> /machine/cpu[n] >> >> On 2012-06-08 14:34, Andreas Färber wrote: >>> Am 08.06.2012 14:05, schrieb Igor Mammedov: >

Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-06-08 Thread Igor Mammedov
- Original Message - > From: "Jan Kiszka" > To: "Andreas Färber" > Cc: "Igor Mammedov" , "Anthony Liguori" > , qemu-devel@nongnu.org, "Igor > Mammedov" , "Paolo Bonzini" > Sent: Friday, June 8, 201

Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-06-08 Thread Jan Kiszka
On 2012-06-08 14:34, Andreas Färber wrote: > Am 08.06.2012 14:05, schrieb Igor Mammedov: >> On Fri, Jun 08, 2012 at 11:11:11AM +0200, Andreas Färber wrote: >>> Another factor that is making this slightly difficult is that there are >>> three APIC subclasses. Currently they all have an instance_size

Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-06-08 Thread Andreas Färber
Am 08.06.2012 14:05, schrieb Igor Mammedov: > On Fri, Jun 08, 2012 at 11:11:11AM +0200, Andreas Färber wrote: >> Another factor that is making this slightly difficult is that there are >> three APIC subclasses. Currently they all have an instance_size of >> sizeof(APICCommonState) so it could be cr

Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-06-08 Thread Andreas Färber
Am 08.06.2012 13:36, schrieb Igor Mammedov: > On Fri, Jun 08, 2012 at 12:36:18PM +0200, Andreas Färber wrote: >> Am 08.06.2012 12:21, schrieb Jan Kiszka: >>> On 2012-06-08 11:11, Andreas Färber wrote: >From what I understand about the x86 modeling, the only case this matters is if you hot

Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-06-08 Thread Igor Mammedov
On Fri, Jun 08, 2012 at 11:11:11AM +0200, Andreas Färber wrote: > Am 08.06.2012 10:20, schrieb Igor Mammedov: > > On Wed, May 23, 2012 at 05:07:27AM +0200, Andreas Färber wrote: > > This is touching on a core QOM question: Can we link<> during initfn? > That's what you're trying to do for the APIC

Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-06-08 Thread Igor Mammedov
On Fri, Jun 08, 2012 at 12:36:18PM +0200, Andreas Färber wrote: > Am 08.06.2012 12:21, schrieb Jan Kiszka: > > On 2012-06-08 11:11, Andreas Färber wrote: > >> >From what I understand about the x86 modeling, the only case this > >> matters is if you hot-unplug CPU 0, right? Question is, what happens

Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-06-08 Thread Andreas Färber
Am 08.06.2012 12:36, schrieb Andreas Färber: > Am 08.06.2012 12:21, schrieb Jan Kiszka: >> On 2012-06-08 11:11, Andreas Färber wrote: >>> >From what I understand about the x86 modeling, the only case this >>> matters is if you hot-unplug CPU 0, right? Question is, what happens >>> with the APIC the

Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-06-08 Thread Andreas Färber
Am 08.06.2012 12:21, schrieb Jan Kiszka: > On 2012-06-08 11:11, Andreas Färber wrote: >> >From what I understand about the x86 modeling, the only case this >> matters is if you hot-unplug CPU 0, right? Question is, what happens >> with the APIC then? I would guess the remaining n-1 CPUs still want

Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-06-08 Thread Jan Kiszka
On 2012-06-08 11:11, Andreas Färber wrote: >>From what I understand about the x86 modeling, the only case this > matters is if you hot-unplug CPU 0, right? Question is, what happens > with the APIC then? I would guess the remaining n-1 CPUs still want to > access the APIC APICs are per-CPU, each h

Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-06-08 Thread Andreas Färber
Am 08.06.2012 10:20, schrieb Igor Mammedov: > On Wed, May 23, 2012 at 05:07:27AM +0200, Andreas Färber wrote: >> Using the cpu_index, give the X86CPU a canonical path. >> This must be done before initializing the APIC. >> >> Signed-off-by: Igor Mammedov >> Signed-off-by: Andreas Färber >> --- >>

Re: [Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-06-08 Thread Igor Mammedov
On Wed, May 23, 2012 at 05:07:27AM +0200, Andreas Färber wrote: > Using the cpu_index, give the X86CPU a canonical path. > This must be done before initializing the APIC. > > Signed-off-by: Igor Mammedov > Signed-off-by: Andreas Färber > --- > hw/pc.c | 12 > 1 files changed, 12

[Qemu-devel] [PATCH qom-next 04/59] pc: Add CPU as /machine/cpu[n]

2012-05-22 Thread Andreas Färber
Using the cpu_index, give the X86CPU a canonical path. This must be done before initializing the APIC. Signed-off-by: Igor Mammedov Signed-off-by: Andreas Färber --- hw/pc.c | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/hw/pc.c b/hw/pc.c index 4167782..e9d