Re: [libvirt] [PATCH v2 08/11] conf: add PCI controllers

2013-04-18 Thread Eric Blake
On 04/17/2013 10:38 PM, Laine Stump wrote: It occurred to me that we should only allow model='pci-root|pci-bridge' if type='pci', and when I went back to look at the rng directly (instead of just the diff), I noticed that there is already a separate set of attributes/values allowable for

[libvirt] [PATCH v2 08/11] conf: add PCI controllers

2013-04-17 Thread Ján Tomko
Add new controller type 'pci' with models 'pci-root' and 'pci-bridge'. --- docs/schemas/domaincommon.rng | 3 +++ src/conf/domain_conf.c| 21 - src/conf/domain_conf.h| 9 + 3 files changed, 32 insertions(+), 1 deletion(-) diff --git

Re: [libvirt] [PATCH v2 08/11] conf: add PCI controllers

2013-04-17 Thread Eric Blake
On 04/17/2013 01:00 PM, Ján Tomko wrote: Add new controller type 'pci' with models 'pci-root' and 'pci-bridge'. --- docs/schemas/domaincommon.rng | 3 +++ src/conf/domain_conf.c| 21 - src/conf/domain_conf.h| 9 + 3 files changed, 32

Re: [libvirt] [PATCH v2 08/11] conf: add PCI controllers

2013-04-17 Thread Eric Blake
On 04/17/2013 05:12 PM, Eric Blake wrote: On 04/17/2013 01:00 PM, Ján Tomko wrote: Add new controller type 'pci' with models 'pci-root' and 'pci-bridge'. --- docs/schemas/domaincommon.rng | 3 +++ src/conf/domain_conf.c| 21 - src/conf/domain_conf.h| 9

Re: [libvirt] [PATCH v2 08/11] conf: add PCI controllers

2013-04-17 Thread Laine Stump
On 04/17/2013 03:00 PM, Ján Tomko wrote: Add new controller type 'pci' with models 'pci-root' and 'pci-bridge'. --- docs/schemas/domaincommon.rng | 3 +++ src/conf/domain_conf.c| 21 - src/conf/domain_conf.h| 9 + 3 files changed, 32

Re: [libvirt] [PATCH v2 08/11] conf: add PCI controllers

2013-04-17 Thread Laine Stump
On 04/17/2013 03:00 PM, Ján Tomko wrote: Add new controller type 'pci' with models 'pci-root' and 'pci-bridge'. --- docs/schemas/domaincommon.rng | 3 +++ src/conf/domain_conf.c| 21 - src/conf/domain_conf.h| 9 + 3 files changed, 32