qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Avi Kivity <avi@redhat.com>
To: Lennart Sorensen <lsorense@csclub.uwaterloo.ca>
Cc: Anthony Liguori <aliguori@us.ibm.com>, qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 1/1] Make the e1000 the default network	adapter for the pc target.
Date: Tue, 25 Aug 2009 20:13:32 +0300	[thread overview]
Message-ID: <4A941BBC.4020504@redhat.com> (raw)
In-Reply-To: <20090825171107.GE23700@csclub.uwaterloo.ca>

On 08/25/2009 08:11 PM, Lennart Sorensen wrote:
> On Tue, Aug 25, 2009 at 07:56:50PM +0300, Avi Kivity wrote:
>    
>> Our users will hate us if we do this.  We should look at XP's actual
>> market share, not formal statements from Microsoft.  If market share
>> starts dropping, we can scale back support in qemu.
>>
>> Providing less than optimal performance is a lot better than starting up
>> with no networking.  In any case optimal performance requires a
>> non-default selection (virtio).
>>      
> Well if e1000 gives 95% of the virtio performance, and 8139 gives 50%,
> then I know which I will be using.  Personally I don't care which one
> is default, since I will use e1000 with bridging anyhow.
>    

if (0) is usually optimized away by the compiler.

The default only matters to unmanaged systems.  I think we should aim 
first for working, and later for performance.

-- 
error compiling committee.c: too many arguments to function

  reply	other threads:[~2009-08-25 17:13 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-14 16:23 [Qemu-devel] [PATCH 1/1] Make the e1000 the default network adapter for the pc target Anthony Liguori
2009-08-15 21:58 ` [Qemu-devel] Re: [PATCH 1/1] Make the e1000 the default network adapterfor " Sebastian Herbszt
2009-08-17 10:34 ` [Qemu-devel] [PATCH 1/1] Make the e1000 the default network adapter for " Mark McLoughlin
2009-08-24 15:12   ` Anthony Liguori
2009-08-24 15:36     ` Gerd Hoffmann
2009-08-24 16:14       ` Anthony Liguori
2009-08-25 10:45 ` Avi Kivity
2009-08-25 12:39   ` Anthony Liguori
2009-08-25 15:55     ` Avi Kivity
2009-08-25 13:23   ` Lennart Sorensen
2009-08-25 13:34     ` Anthony Liguori
2009-08-25 14:23       ` Lennart Sorensen
2009-08-25 16:00       ` Avi Kivity
2009-08-25 15:58     ` Avi Kivity
2009-08-25 16:30       ` Lennart Sorensen
2009-08-25 16:39         ` Anthony Liguori
2009-08-25 16:56           ` Avi Kivity
2009-08-25 17:11             ` Lennart Sorensen
2009-08-25 17:13               ` Avi Kivity [this message]
2009-08-25 18:19                 ` Anthony Liguori
2009-08-26  8:32                   ` Avi Kivity
2009-08-25 17:28               ` Reimar Döffinger
2009-08-26  5:47           ` Amit Shah
2009-08-25 20:20       ` [Qemu-devel] " Bjørn Mork
2009-08-26  9:57         ` Avi Kivity
2009-08-25 13:51   ` [Qemu-devel] " Stefan Weil

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=4A941BBC.4020504@redhat.com \
    --to=avi@redhat.com \
    --cc=aliguori@us.ibm.com \
    --cc=lsorense@csclub.uwaterloo.ca \
    --cc=qemu-devel@nongnu.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).