[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 10/16] dimm: add busy slot check and slot auto-a
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 10/16] dimm: add busy slot check and slot auto-allocation |
Date: |
Tue, 23 Jul 2013 19:09:26 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7 |
Il 23/07/2013 18:23, Igor Mammedov ha scritto:
> - if slot property is not specified on -device/device_add command,
> treat default value as request for assigning DimmDevice to
> the first free slot.
Even with "-m" instead of "-numa mem", I think this is problematic
because we still need to separate the host and guest parts of the DIMM
device. "-numa mem" (or the QMP command that Wanlong added) will be
necessary to allocate memory on the host side before adding a DIMM.
So slots will have three states: free (created with "-m"), allocated (a
free slot moves to this state with "-numa mem...,populated=no" when
migrating, or with the QMP command for regular hotplug), populated (an
allocated slot moves to this state with "-device dimm").
You would be able to plug a DIMM only into an allocated slot, and the
size will be specified on the slot rather than the DIMM device.
In general, I don't think free slots should be managed by the DimmBus,
and host vs. guest separation should be there even if we accept your
"-m" extension (doesn't look bad at all, I must say).
Paolo
- [Qemu-devel] [PATCH 04/16] qapi: make visit_type_size fallback to type_int, (continued)
- [Qemu-devel] [PATCH 04/16] qapi: make visit_type_size fallback to type_int, Igor Mammedov, 2013/07/23
- [Qemu-devel] [PATCH 05/16] qdev: Add SIZE type to qdev properties, Igor Mammedov, 2013/07/23
- [Qemu-devel] [PATCH 06/16] dimm: implement dimm device abstraction, Igor Mammedov, 2013/07/23
- [Qemu-devel] [PATCH 07/16] dimm: map DimmDevice into DimBus provided address space, Igor Mammedov, 2013/07/23
- [Qemu-devel] [PATCH 08/16] pc: piix: make hotplug memory gap in high memory, Igor Mammedov, 2013/07/23
- [Qemu-devel] [PATCH 09/16] pc: i440fx: add DimmBus to chipset and map it into hotplug memory region, Igor Mammedov, 2013/07/23
- [Qemu-devel] [PATCH 10/16] dimm: add busy slot check and slot auto-allocation, Igor Mammedov, 2013/07/23
- Re: [Qemu-devel] [PATCH 10/16] dimm: add busy slot check and slot auto-allocation,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH 10/16] dimm: add busy slot check and slot auto-allocation, Igor Mammedov, 2013/07/24
- Re: [Qemu-devel] [PATCH 10/16] dimm: add busy slot check and slot auto-allocation, Paolo Bonzini, 2013/07/24
- Re: [Qemu-devel] [PATCH 10/16] dimm: add busy slot check and slot auto-allocation, Igor Mammedov, 2013/07/24
- Re: [Qemu-devel] [PATCH 10/16] dimm: add busy slot check and slot auto-allocation, Paolo Bonzini, 2013/07/24
- Re: [Qemu-devel] [PATCH 10/16] dimm: add busy slot check and slot auto-allocation, Igor Mammedov, 2013/07/26
- Re: [Qemu-devel] [PATCH 10/16] dimm: add busy slot check and slot auto-allocation, Paolo Bonzini, 2013/07/26
- Re: [Qemu-devel] [PATCH 10/16] dimm: add busy slot check and slot auto-allocation, Igor Mammedov, 2013/07/26
- Re: [Qemu-devel] [PATCH 10/16] dimm: add busy slot check and slot auto-allocation, Paolo Bonzini, 2013/07/26
[Qemu-devel] [PATCH 11/16] dimm: add busy address check and address auto-allocation, Igor Mammedov, 2013/07/23
[Qemu-devel] [PATCH 12/16] dimm: introduce memory added notifier, Igor Mammedov, 2013/07/23