[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 03/11] pc: check if KVM has enough memory slots
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 03/11] pc: check if KVM has enough memory slots for DIMM devices |
Date: |
Tue, 04 Nov 2014 09:45:15 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 |
On 03/11/2014 20:11, Igor Mammedov wrote:
>> > Understood. But I think -m slots=too_much is not an important
>> > problem. What is problematic is filling those slots, which patch 1
>> > will help with.
> From user's pov it's still error, i.e. he asked on CLI for one amount
> of hotpluggable slots and couldn't use all of them at runtime.
But he cannot even with your patch, if he was off by just a couple
items. So I don't think the improvement is worth the extra complication
of the code and the small layering violation.
Paolo
- Re: [Qemu-devel] [PATCH 03/11] pc: check if KVM has enough memory slots for DIMM devices, Paolo Bonzini, 2014/11/03
- Re: [Qemu-devel] [PATCH 03/11] pc: check if KVM has enough memory slots for DIMM devices, Igor Mammedov, 2014/11/03
- Re: [Qemu-devel] [PATCH 03/11] pc: check if KVM has enough memory slots for DIMM devices, Paolo Bonzini, 2014/11/03
- Re: [Qemu-devel] [PATCH 03/11] pc: check if KVM has enough memory slots for DIMM devices, Igor Mammedov, 2014/11/03
- Re: [Qemu-devel] [PATCH 03/11] pc: check if KVM has enough memory slots for DIMM devices,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH 03/11] pc: check if KVM has enough memory slots for DIMM devices, Igor Mammedov, 2014/11/04
- Re: [Qemu-devel] [PATCH 03/11] pc: check if KVM has enough memory slots for DIMM devices, Paolo Bonzini, 2014/11/04
- Re: [Qemu-devel] [PATCH 03/11] pc: check if KVM has enough memory slots for DIMM devices, Igor Mammedov, 2014/11/04
- Re: [Qemu-devel] [PATCH 03/11] pc: check if KVM has enough memory slots for DIMM devices, Paolo Bonzini, 2014/11/07