Re: [PATCH v3 0/4] Try several possible default network devices

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Sep 09, 2015 at 02:50:19PM +0200, Andrea Bolognani wrote:
Changes in v3:

 * detect rtl8139 and e1000 availability as well

 * don't try to use virtio-net based on availability alone:
   try rtl8139, e1000 and virtio-net in this order instead

 * patch 4/4 contains an ugly bit we'll be able to get rid of
   once capabilities can be properly mocked in the test suite

Changes in v2[1]:

 * detect virtio-net availability

 * try to user it if available

Cheers.


[1] https://www.redhat.com/archives/libvir-list/2015-September/msg00058.html

Andrea Bolognani (4):
 qemu: Introduce QEMU_CAPS_DEVICE_RTL8139
 qemu: Introduce QEMU_CAPS_DEVICE_E1000
 qemu: Introduce QEMU_CAPS_DEVICE_VIRTIO_NET
 qemu: Try several network devices when looking for a default


ACK to 1-3, 4/4 has it's own conditional ACK in it's own reply.

src/qemu/qemu_capabilities.c                  | 12 +++++++++-
src/qemu/qemu_capabilities.h                  |  3 +++
src/qemu/qemu_domain.c                        | 32 +++++++++++++++++++++++----
tests/qemucapabilitiesdata/caps_1.2.2-1.caps  |  3 +++
tests/qemucapabilitiesdata/caps_1.3.1-1.caps  |  3 +++
tests/qemucapabilitiesdata/caps_1.4.2-1.caps  |  3 +++
tests/qemucapabilitiesdata/caps_1.5.3-1.caps  |  3 +++
tests/qemucapabilitiesdata/caps_1.6.0-1.caps  |  3 +++
tests/qemucapabilitiesdata/caps_1.6.50-1.caps |  3 +++
tests/qemucapabilitiesdata/caps_2.1.1-1.caps  |  3 +++
tests/qemuhelptest.c                          | 30 ++++++++++++++++++++-----
11 files changed, 87 insertions(+), 11 deletions(-)

--
2.4.3

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

Attachment: signature.asc
Description: PGP signature

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list

[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]