Re: PING-2: [PATCH v5 0/4] qemu: Allow PCI virtio on ARM "virt" machine

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

 



On Tue, Aug 04, 2015 at 02:08:02PM +0300, Pavel Fedin wrote:
Hello! Can anybody tell me what is wrong with this? Ignored for a long time.


I'm not sure why I'm in Cc, it's usually enough to just ping the
series.  Looking at previous mails it looks like you're adding people
because they're getting "removed" from Cc.  That's something the list
is doing, although the mail was delivered to them, but just only once
(originating from you - yes, originating from the list itself - no),
that happens for people who set it up in the config.  And the fact
that the mail is getting deleted from the headers is a mailman bug, I
guess.

Anyway, I think the biggest problem here is that your "ping" is not
reply to any other mail.  If you'd replied with the ping to the cover
letter, that would really ping the whole series (which I had to look
up now) near newer threads.  Of course this depends on MUA
configuration, but I guess most of people use something similar to
what I just explained here.

I'm not the right one to talk to about ARM machines, but I can at
least have a look at it.  Will do in a moment.

Kind regards,
Pavel Fedin
Expert Engineer
Samsung Electronics Research center Russia


-----Original Message-----
From: libvir-list-bounces@xxxxxxxxxx [mailto:libvir-list-bounces@xxxxxxxxxx] On Behalf Of Pavel
Fedin
Sent: Friday, July 17, 2015 2:28 PM
To: libvir-list@xxxxxxxxxx
Cc: Peter Krempa
Subject:  [PATCH v5 0/4] qemu: Allow PCI virtio on ARM "virt" machine

Virt machine in qemu since v2.3.0 has PCI generic host controller, and
can use PCI devices. This provides performance improvement as well as
vhost-net with irqfd support for virtio-net. However libvirt currently
does not allow ARM virt machine to have PCI devices. This patchset adds
the necessary support.

Changes since v4:
- Rebased onto current master
- Added possibility to plug virtio-net-pci adapter directly into PCIe bus.
  This is necessary for irqfds to work in qemu.
Changes since v3:
- Capability is based not on qemu version but on support of "gpex-pcihost"
  device by qemu
- Added a workaround, allowing to pass "make check". The problem is that
  test suite does not build capabilities cache. Unfortunately this means
  that correct unit-test for the new functionality currently cannot be
  written. Test suite framework needs to be improved.
Changes since v2:
Complete rework, use different approach
- Correctly model PCI Express bus on the machine. It is now possible to
  explicitly specify <address-type='pci'> with attributes. This allows to
  attach not only virtio, but any other PCI device to the model.
- Default is not changed and still mmio, for backwards compatibility with
  existing installations. PCI bus has to be explicitly specified.
- Check for the capability in correct place, in v2 it actually did not
  work
Changes since v1:
- Added capability based on qemu version number
- Recognize also "virt-" prefix

Pavel Fedin (4):
  qemu: Introduce QEMU_CAPS_OBJECT_GPEX
  Add PCI-Express root to ARM virt machine
  Build correct command line for PCI NICs on ARM
  Allow to plug virtio-net-pci into PCIe slot

 src/qemu/qemu_capabilities.c | 10 ++++++++++
 src/qemu/qemu_capabilities.h |  1 +
 src/qemu/qemu_command.c      | 11 ++++++++++-
 src/qemu/qemu_domain.c       | 17 +++++++++++++----
 4 files changed, 34 insertions(+), 5 deletions(-)

--
1.9.5.msysgit.0

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

--
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]