All of lore.kernel.org
 help / color / mirror / Atom feed
From: Anthony Liguori <anthony@codemonkey.ws>
To: Avi Kivity <avi@qumranet.com>
Cc: Jerone Young <jyoung5@us.ibm.com>,
	kvm@vger.kernel.org, kvm-ppc@vger.kernel.org
Subject: Re: [PATCH 0 of 2] Remove --disable-cpu-emulation for qemu
Date: Wed, 18 Jun 2008 20:56:45 +0000	[thread overview]
Message-ID: <4859768D.3020809@codemonkey.ws> (raw)
In-Reply-To: <48590BA3.9050901@qumranet.com>

Avi Kivity wrote:
> Avi Kivity wrote:
>> Jerone Young wrote:
>>> These patches remove all traces of --disable-cpu-emulation 
>>> configuration flag.
>>>
>>>   
>>
>> I kind of like --disable-cpu-emulation.  It means we have the choice 
>> of kvm-only (low footprint, shuts up the security people), qemu-only 
>> (running cross-arch or on older cpu), and mixed mode (I'm a distro 
>> so  don't know what cpu I'm running on).
>>
>> Well, actually, at least Fedora already supplies both plain qemu and 
>> qemu-kvm, so it seems they don't care much about mixed mode.  I'll 
>> look at applying the patches.
>>
>
> Er, that only makes sense with Anthony's idea of a -kvm subtarget 
> thrown in as well.

The -kvm subtarget is really easy to implement.  If anyone is interested 
in doing this, let me know and I'll point them in the right direction.  
The good thing about a -kvm subtarget is that we can introduce a 
CONFIG_KVM_ONLY #define which we can use to compile out l1_phys_map et 
al.  That will give a much better RSS reduction than not linking in 
TCG.  I doubt --disable-cpu-emulation really reduces RSS size an 
observable amount.

Regards,

Anthony Liguori



WARNING: multiple messages have this Message-ID (diff)
From: Anthony Liguori <anthony@codemonkey.ws>
To: Avi Kivity <avi@qumranet.com>
Cc: Jerone Young <jyoung5@us.ibm.com>,
	kvm@vger.kernel.org, kvm-ppc@vger.kernel.org
Subject: Re: [PATCH 0 of 2] Remove --disable-cpu-emulation for qemu
Date: Wed, 18 Jun 2008 15:56:45 -0500	[thread overview]
Message-ID: <4859768D.3020809@codemonkey.ws> (raw)
In-Reply-To: <48590BA3.9050901@qumranet.com>

Avi Kivity wrote:
> Avi Kivity wrote:
>> Jerone Young wrote:
>>> These patches remove all traces of --disable-cpu-emulation 
>>> configuration flag.
>>>
>>>   
>>
>> I kind of like --disable-cpu-emulation.  It means we have the choice 
>> of kvm-only (low footprint, shuts up the security people), qemu-only 
>> (running cross-arch or on older cpu), and mixed mode (I'm a distro 
>> so  don't know what cpu I'm running on).
>>
>> Well, actually, at least Fedora already supplies both plain qemu and 
>> qemu-kvm, so it seems they don't care much about mixed mode.  I'll 
>> look at applying the patches.
>>
>
> Er, that only makes sense with Anthony's idea of a -kvm subtarget 
> thrown in as well.

The -kvm subtarget is really easy to implement.  If anyone is interested 
in doing this, let me know and I'll point them in the right direction.  
The good thing about a -kvm subtarget is that we can introduce a 
CONFIG_KVM_ONLY #define which we can use to compile out l1_phys_map et 
al.  That will give a much better RSS reduction than not linking in 
TCG.  I doubt --disable-cpu-emulation really reduces RSS size an 
observable amount.

Regards,

Anthony Liguori



  reply	other threads:[~2008-06-18 20:56 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-06-10 18:50 [PATCH 0 of 2] Remove --disable-cpu-emulation for qemu Jerone Young
2008-06-10 18:50 ` Jerone Young
2008-06-10 18:50 ` [PATCH 1 of 2] Remove qemu files and entries associated with Jerone Young
2008-06-10 18:50   ` [PATCH 1 of 2] Remove qemu files and entries associated with --disable-cpu-emulation configure argument Jerone Young
2008-06-10 18:50 ` [PATCH 2 of 2] Create dyngen dependency for exec.o for those who Jerone Young
2008-06-10 18:50   ` [PATCH 2 of 2] Create dyngen dependency for exec.o for those who still have dyngen Jerone Young
2008-06-13 15:31 ` [PATCH 0 of 2] Remove --disable-cpu-emulation for qemu Jerone Young
2008-06-13 15:31   ` Jerone Young
2008-06-13 17:44   ` Anthony Liguori
2008-06-13 17:44     ` Anthony Liguori
2008-06-13 18:01     ` Soren Hansen
2008-06-13 18:01       ` Soren Hansen
2008-06-13 22:13     ` Jerone Young
2008-06-13 22:13       ` Jerone Young
2008-06-17 13:27 ` Anthony Liguori
2008-06-17 13:27   ` Anthony Liguori
2008-06-18 13:19 ` Avi Kivity
2008-06-18 13:19   ` Avi Kivity
2008-06-18 13:20   ` Avi Kivity
2008-06-18 13:20     ` Avi Kivity
2008-06-18 20:56     ` Anthony Liguori [this message]
2008-06-18 20:56       ` Anthony Liguori

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4859768D.3020809@codemonkey.ws \
    --to=anthony@codemonkey.ws \
    --cc=avi@qumranet.com \
    --cc=jyoung5@us.ibm.com \
    --cc=kvm-ppc@vger.kernel.org \
    --cc=kvm@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.