Re: [PATCHv2 06/15] Add virtio-related options to interfaces

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

 



On Wed, Jun 07, 2017 at 03:56:44PM +0200, Ján Tomko wrote:
> On Wed, Jun 07, 2017 at 03:43:53PM +0200, Pavel Hrdina wrote:
> >On Tue, Jun 06, 2017 at 01:36:20PM +0200, Ján Tomko wrote:
> >> <interface type='user'>
> >>   <mac address='52:54:56:5a:5c:5e'/>
> >>   <model type='virtio'/>
> >>   <driver iommu_platform='on' ats='on'/>
> >> </interface>
> >>
> >> https://bugzilla.redhat.com/show_bug.cgi?id=1283251
> >> ---
> >>  docs/formatdomain.html.in                          | 19 +++++++
> >>  docs/schemas/domaincommon.rng                      | 12 +++++
> >>  src/conf/domain_conf.c                             | 63 ++++++++++++++++++++++
> >>  src/conf/domain_conf.h                             | 19 +++++++
> >>  .../qemuxml2argv-virtio-options.xml                |  2 +
> >>  .../qemuxml2xmlout-virtio-options.xml              |  2 +
> >>  6 files changed, 117 insertions(+)
> >
> >I would spit this patch into two patches, one that introduces the virtio
> >related options and second one that introduces this element to the
> >interfaces.
> 
> That can be done for the schema and documentation changes, but code with
> an unused static function would not compile.

Oh, right, I didn't realize that, so ignore that comment :).

Pavel

Attachment: signature.asc
Description: Digital 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]
  Powered by Linux