All of lore.kernel.org
 help / color / mirror / Atom feed
From: Gerd Hoffmann <kraxel@redhat.com>
To: Andrea Bolognani <abologna@redhat.com>
Cc: qemu-devel@nongnu.org, marcel@redhat.com, drjones@redhat.com
Subject: Re: [Qemu-devel] [PATCH v2] q35: Improve sample configuration files
Date: Thu, 02 Feb 2017 12:49:04 +0100	[thread overview]
Message-ID: <1486036144.20628.40.camel@redhat.com> (raw)
In-Reply-To: <1486031975.3484.17.camel@redhat.com>

  Hi,

> > If you add a video card manually (via -device or config file entry) qemu
> > will not add the default video card.
> > 
> > Likewise, if you add a NIC manually, qemu will not add the default
> > ethernet device (and IIRC creating the default netdev will be skipped
> > too).
> 
> This has the potential to give you some unexpected results

Indeed.

> but I'd say it's still entirely reasonable.

Well, makes it easy for users to get started, just "qemu -cdrom $live"
gives you a usable virtual machine without having to worry too much
about what devices you want plug in.

But that kind of magic -- while convenient at times -- can also easily
get into the way.  Guess why the -nodefaults switch exists ...

> So what do you think, should q35-emulated.cfg recommend the
> use of -nodefaults the same way q35-virtio-*.cfg do and add
> the video card (cirrus-vga?) and Ethernet adapter explicitly?

I think that would be good, for consistency across all q35*.cfg files.

But please don't use cirrus[1], it isn't the default any more for good
reasons.  I'd suggest to go with our new default, which is stdvga
(-device VGA), as we don't have intel graphics emulation.

> The advantage in doing so would be that we would provide
> maybe slightly more documentation, and we would be able to
> plug the Ethernet adapter at 19.0 instead of 02.0, which as
> I understand it matches real hardware better.

Yes.

cheers,
  Gerd

[1]
https://www.kraxel.org/blog/2014/10/qemu-using-cirrus-considered-harmful/

  reply	other threads:[~2017-02-02 11:49 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-01 17:20 [Qemu-devel] [PATCH v2] q35: Improve sample configuration files Andrea Bolognani
2017-02-02  7:49 ` Gerd Hoffmann
2017-02-02 10:39   ` Andrea Bolognani
2017-02-02 11:49     ` Gerd Hoffmann [this message]
2017-02-02 14:42 ` Marcel Apfelbaum
2017-02-02 15:20   ` Andrea Bolognani
2017-02-02 20:38     ` Marcel Apfelbaum
2017-02-02 22:13     ` Gerd Hoffmann
2017-02-02 22:21       ` Marcel Apfelbaum
2017-02-03  8:39       ` Andrea Bolognani

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=1486036144.20628.40.camel@redhat.com \
    --to=kraxel@redhat.com \
    --cc=abologna@redhat.com \
    --cc=drjones@redhat.com \
    --cc=marcel@redhat.com \
    --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 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.