External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-09-30 11:25:47 +00:00
Code Issues Packages Projects Releases Wiki Activity

xml: nodedev: make pci capability class element optional

Browse Source 
Commit 3bd4ed46 introduced this element as required which
breaks backcompat for test driver. Let's make the element optional.

Reviewed-by: Erik Skultety <eskultet@redhat.com>
Signed-off-by: Nikolay Shirokovskiy <nshirokovskiy@virtuozzo.com>
This commit is contained in:
Nikolay Shirokovskiy 2019-03-21 10:12:40 +03:00
parent 01e11ebcb6
commit 1193d9737b
6 changed files with 23 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
docs/formatnode.html.in
View File

@ -71,7 +71,7 @@
include: include:
<dl> <dl>
<dt><code>class</code></dt> <dt><code>class</code></dt>
<dd>Combined class, subclass and <dd>Optional element for combined class, subclass and
programming interface codes as 6-digit hexadecimal number. programming interface codes as 6-digit hexadecimal number.
<span class="since">Since 5.2.0</span></dd> <span class="since">Since 5.2.0</span></dd>
<dt><code>domain</code></dt> <dt><code>domain</code></dt>

2
docs/schemas/nodedev.rng
View File

@ -133,11 +133,13 @@
<value>pci</value> <value>pci</value>
</attribute> </attribute>
<optional>
<element name='class'> <element name='class'>
<data type="string"> <data type="string">
<param name="pattern">0x[0-9a-fA-F]{6}</param> <param name="pattern">0x[0-9a-fA-F]{6}</param>
</data> </data>
</element> </element>
</optional>
<element name='domain'> <element name='domain'>
<ref name='unsignedLong'/> <ref name='unsignedLong'/>
</element> </element>

15
src/conf/node_device_conf.c
View File

@ -208,6 +208,7 @@ virNodeDeviceCapPCIDefFormat(virBufferPtr buf,
{ {
size_t i; size_t i;
if (data->pci_dev.klass >= 0)
virBufferAsprintf(buf, "<class>0x%.6x</class>\n", data->pci_dev.klass); virBufferAsprintf(buf, "<class>0x%.6x</class>\n", data->pci_dev.klass);
virBufferAsprintf(buf, "<domain>%d</domain>\n", virBufferAsprintf(buf, "<domain>%d</domain>\n",
data->pci_dev.domain); data->pci_dev.domain);
@ -1645,17 +1646,17 @@ virNodeDevCapPCIDevParseXML(xmlXPathContextPtr ctxt,
orignode = ctxt->node; orignode = ctxt->node;
ctxt->node = node; ctxt->node = node;
if (virNodeDevCapsDefParseHexId("string(./class[1])", ctxt, if ((tmp = virXPathString("string(./class[1])", ctxt))) {
&pci_dev->klass, def, if (virStrToLong_i(tmp, NULL, 16, &pci_dev->klass) < 0 ||
_("no PCI class supplied for '%s'"), pci_dev->klass > 0xffffff) {
_("invalid PCI class supplied for '%s'")) < 0)
goto out;
if (pci_dev->klass > 0xffffff) {
virReportError(VIR_ERR_INTERNAL_ERROR, virReportError(VIR_ERR_INTERNAL_ERROR,
_("invalid PCI class supplied for '%s'"), def->name); _("invalid PCI class supplied for '%s'"), def->name);
goto out; goto out;
} }
VIR_FREE(tmp);
} else {
pci_dev->klass = -1;
}
if (virNodeDevCapsDefParseULong("number(./domain[1])", ctxt, if (virNodeDevCapsDefParseULong("number(./domain[1])", ctxt,
&pci_dev->domain, def, &pci_dev->domain, def,

2
src/conf/node_device_conf.h
View File

@ -153,7 +153,7 @@ struct _virNodeDevCapPCIDev {
unsigned int function; unsigned int function;
unsigned int product; unsigned int product;
unsigned int vendor; unsigned int vendor;
unsigned int klass; int klass;
char *product_name; char *product_name;
char *vendor_name; char *vendor_name;
virPCIDeviceAddressPtr physical_function; virPCIDeviceAddressPtr physical_function;

3
src/node_device/node_device_udev.c
View File

@ -402,7 +402,8 @@ udevProcessPCI(struct udev_device *device,
privileged = driver->privileged; privileged = driver->privileged;
nodeDeviceUnlock(); nodeDeviceUnlock();
if (udevGetUintProperty(device, "PCI_CLASS", &pci_dev->klass, 16) < 0) pci_dev->klass = -1;
if (udevGetIntProperty(device, "PCI_CLASS", &pci_dev->klass, 16) < 0)
goto cleanup; goto cleanup;
if ((p = strrchr(def->sysfs_path, '/')) == NULL || if ((p = strrchr(def->sysfs_path, '/')) == NULL ||

1
tests/nodedevschemadata/pci_0000_02_10_7_sriov_pf_vfs_all.xml
View File

@ -2,7 +2,6 @@
<name>pci_0000_02_10_7</name> <name>pci_0000_02_10_7</name>
<parent>pci_0000_00_04_0</parent> <parent>pci_0000_00_04_0</parent>
<capability type='pci'> <capability type='pci'>
<class>0xffffff</class>
<domain>0</domain> <domain>0</domain>
<bus>2</bus> <bus>2</bus>
<slot>16</slot> <slot>16</slot>