[libvirt] [PATCHv2 06/15] Add virtio-related options to interfaces
Pavel Hrdina
phrdina at redhat.com
Wed Jun 7 14:05:37 UTC 2017
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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170607/0e95da21/attachment-0001.sig>
More information about the libvir-list
mailing list