All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Bonzini <pbonzini@redhat.com>
To: Michal Novotny <minovotn@redhat.com>
Cc: "'xen-devel@lists.xensource.com'" <xen-devel@lists.xensource.com>
Subject: Re: [PATCH] Make NIC model fallback to default when specified model is not supported
Date: Mon, 20 Sep 2010 14:07:19 +0200	[thread overview]
Message-ID: <4C974E77.4030201@redhat.com> (raw)
In-Reply-To: <4C9735C0.5000806@redhat.com>

On 09/20/2010 12:21 PM, Michal Novotny wrote:
> Hi,
> this is the patch to introduce a NIC model fallback to default when model
> specified is not supported. It's been tested on x86_64 host with current
> version of qemu-dm using the RHEL-5 i386 virtual machine and by trying to
> setup the invalid (unsupported) model of NIC device. Also, the new constant
> in the net.h called the DEFAULT_NIC_MODEL has been introduced to be able to
> change the default NIC model easily. This variable is being used to set
> the default NIC model when necessary.
>
> It's been tested by running a guest with NIC model set to be "eee" which
> is not supported and when the guest was started I tried to issue "lspci"
> to list all the PCI devices and there was Realtek RTL-8239 (AS) card
> which is basically the ne2k_pci card (default for Xen-4.1 qemu-dm) so
> the old default has been preserved and it's being used as a fallback.

I don't think qemu-dm and upstream QEMU should diverge again, so this 
patch does not sound like a good thing for Xen's ioemu codebase.

Paolo

  reply	other threads:[~2010-09-20 12:07 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-20 10:21 [PATCH] Make NIC model fallback to default when specified model is not supported Michal Novotny
2010-09-20 12:07 ` Paolo Bonzini [this message]
2010-09-20 14:53 ` Ian Jackson

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=4C974E77.4030201@redhat.com \
    --to=pbonzini@redhat.com \
    --cc=minovotn@redhat.com \
    --cc=xen-devel@lists.xensource.com \
    /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.