[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line o
From: |
Jan Kiszka |
Subject: |
Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option |
Date: |
Fri, 29 Jun 2012 16:30:19 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686 (x86_64); de; rv:1.8.1.12) Gecko/20080226 SUSE/2.0.0.12-1.1 Thunderbird/2.0.0.12 Mnenhy/0.7.5.666 |
On 2012-06-29 16:25, Anthony Liguori wrote:
> On 06/29/2012 09:18 AM, Andreas Färber wrote:
>> Am 25.06.2012 22:39, schrieb Luiz Capitulino:
>>> On Mon, 25 Jun 2012 22:26:58 +0200
>>> Jan Kiszka<address@hidden> wrote:
>>>
>>>> On 2012-06-25 18:55, Luiz Capitulino wrote:
>>>>> Allow for disabling memory merge support (KSM on Linux), which is
>>>>> enabled by default otherwise.
>>>>
>>>> -machine mem_merge=on|off?
>>>
>>> That's possible. But if we do this, then I think that the
>>> set-memory-merge QMP
>>> command should be dropped in favor of doing the same thing via machine
>>> properties, which should be possible once we convert machine types to
>>> QOM?
>>
>> Machine QOM'ification has been requested to be postponed by Anthony, so
>> that we can do it really cleanly when we do it.
>>
>> I don't think we have any official guidelines for QOM naming, but it
>> seemed to me an unwritten rule that we use the dash for separating name
>> components whereas command line options coming from KVM seem to use
>> underscore. For the CPU those were written in stone already so I
>> manually "translated" '_' to '-' there. Might be worth thinking about
>> for this and future command line / property additions.
>
> Yes, it would be better to use '-'s instead of '_'s.
Either let machine options accept both styles or keep the current '_'.
No mixture please.
Jan
--
Siemens AG, Corporate Technology, CT T DE IT 1
Corporate Competence Center Embedded Linux
- [Qemu-devel] [RFC 0/3]: allow to enable/disable KSM per VM, Luiz Capitulino, 2012/06/25
- [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option, Luiz Capitulino, 2012/06/25
- Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option, Jan Kiszka, 2012/06/25
- Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option, Luiz Capitulino, 2012/06/25
- Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option, Andreas Färber, 2012/06/29
- Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option, Luiz Capitulino, 2012/06/29
- Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option, Anthony Liguori, 2012/06/29
- Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option,
Jan Kiszka <=
- Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option, Luiz Capitulino, 2012/06/29
- Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option, Luiz Capitulino, 2012/06/29
- Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option, Anthony Liguori, 2012/06/25
- Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option, Jan Kiszka, 2012/06/26
- Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option, Avi Kivity, 2012/06/26
- Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option, Luiz Capitulino, 2012/06/26
- Re: [Qemu-devel] [RFC 1/3] memory: add -disable-mem-merge command-line option, Jan Kiszka, 2012/06/26
[Qemu-devel] [RFC 3/3] memory/qmp: add set-memory-merge command, Luiz Capitulino, 2012/06/25