[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 3/7] s390x/cpu: Move some CPU initialization
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH v6 3/7] s390x/cpu: Move some CPU initialization into realize |
Date: |
Mon, 22 Feb 2016 18:35:31 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.6.0 |
Am 22.02.2016 um 18:06 schrieb Matthew Rosato:
> In preparation for hotplug, defer some CPU initialization
> until the device is actually being realized.
>
> Signed-off-by: Matthew Rosato <address@hidden>
> ---
> target-s390x/cpu.c | 9 ++++++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
Looks reasonable on a brief sight,
Reviewed-by: Andreas Färber <address@hidden>
What is env->cpu_num used for? In particular, have you thought about
linux-user creating multiple CPUs and possibly destroying them again?
Regards,
Andreas
--
SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Felix Imendörffer, Jane Smithard, Graham Norton; HRB 21284 (AG Nürnberg)
- [Qemu-devel] [PATCH v6 0/7] Allow hotplug of s390 CPUs, Matthew Rosato, 2016/02/22
- [Qemu-devel] [PATCH v6 3/7] s390x/cpu: Move some CPU initialization into realize, Matthew Rosato, 2016/02/22
- Re: [Qemu-devel] [PATCH v6 3/7] s390x/cpu: Move some CPU initialization into realize,
Andreas Färber <=
- [Qemu-devel] [PATCH v6 2/7] s390x/cpu: Set initial CPU state in common routine, Matthew Rosato, 2016/02/22
- [Qemu-devel] [PATCH v6 5/7] s390/virtio-ccw: Add hotplug handler, Matthew Rosato, 2016/02/22
- [Qemu-devel] [PATCH v6 6/7] cpu: Add a last_cpu macro, Matthew Rosato, 2016/02/22
- [Qemu-devel] [PATCH v6 7/7] s390x/cpu: Allow hotplug of CPUs, Matthew Rosato, 2016/02/22
- [Qemu-devel] [PATCH v6 4/7] s390x/cpu: Add CPU property links, Matthew Rosato, 2016/02/22
- [Qemu-devel] [PATCH v6 1/7] s390x/cpu: Cleanup init in preparation for hotplug, Matthew Rosato, 2016/02/22