summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJonathon Jongsma <jjongsma@redhat.com>2021-05-26 10:40:26 -0500
committerMichal Privoznik <mprivozn@redhat.com>2021-05-27 15:38:28 +0200
commit70f53b1c04cfed8529c87c7be8ca4c76d1123a30 (patch)
tree391ee37c4b76b07631a0150f23d7281eb8db25dd
parent3e0f552daba4b038152f7e7847ae5eb64a51cede (diff)
downloadlibvirt-7.4.0-rc2.tar.gz
nodedev: Revert auto-start property for mdevsv7.4.0-rc2
We supported autostart of node devices via an xml element, but this is not consistent with other libvirt objects which use an explicit API for setting autostart status. So revert this and implement it as an official API in a future commit. The initial support was refactored after merging, so this commit reverts both of those previous commits. Revert "virNodeDevCapMdevParseXML: Use virXMLPropEnum() for ./start/@type" This reverts commit 9d4cd1d1cda84aa15b77a506f2ad6362a74edf1a. Revert "nodedev: support auto-start property for mdevs" This reverts commit 42a558549935336cbdb7cbfe8b239ffb0e3442e3. Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Signed-off-by: Michal Privoznik <mprivozn@redhat.com> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> Reviewed-by: Michal Privoznik <mprivozn@redhat.com>
-rw-r--r--docs/formatnode.html.in10
-rw-r--r--docs/schemas/nodedev.rng11
-rw-r--r--src/conf/node_device_conf.c21
-rw-r--r--src/conf/node_device_conf.h12
-rw-r--r--src/libvirt_private.syms2
-rw-r--r--src/node_device/node_device_driver.c7
-rw-r--r--tests/nodedevmdevctldata/mdevctl-list-multiple.out.xml4
l---------[-rw-r--r--]tests/nodedevxml2xmlout/mdev_3627463d_b7f0_4fea_b468_f1da537d301b.xml10
-rw-r--r--tests/nodedevxml2xmlout/mdev_d069d019_36ea_4111_8f0a_8c9a70e21366.xml1
-rw-r--r--tests/nodedevxml2xmlout/mdev_d2441d39_495e_4243_ad9f_beb3f14c23d9.xml1
l---------[-rw-r--r--]tests/nodedevxml2xmlout/mdev_ee0b88c4_f554_4dc1_809d_b2a01e8e48ad.xml11
-rw-r--r--tests/nodedevxml2xmlout/mdev_fedc4916_1ca8_49ac_b176_871d16c13076.xml1
12 files changed, 4 insertions, 87 deletions
diff --git a/docs/formatnode.html.in b/docs/formatnode.html.in
index 9a505f0fe9..3b3c3105d4 100644
--- a/docs/formatnode.html.in
+++ b/docs/formatnode.html.in
@@ -388,16 +388,6 @@
<dd>
This element represents the UUID of the mediated device.
</dd>
- <dt><code>start</code></dt>
- <dd>
- This element represents the start policy for the device. It
- has a required attribute <code>type</code>, which can have a
- value of <code>auto</code> or <code>manual</code>. Mediated
- devices with an <code>auto</code> start type will be started
- automatically by the host when the parent device becomes
- available (either on boot, or when the parent device is
- attached). Otherwise the device must be started manually.
- </dd>
</dl>
</dd>
<dt><code>ccw</code></dt>
diff --git a/docs/schemas/nodedev.rng b/docs/schemas/nodedev.rng
index 660cbda3be..777227c38a 100644
--- a/docs/schemas/nodedev.rng
+++ b/docs/schemas/nodedev.rng
@@ -623,17 +623,6 @@
</element>
</optional>
<optional>
- <element name="start">
- <attribute name="type">
- <choice>
- <value>manual</value>
- <value>auto</value>
- </choice>
- </attribute>
- <empty/>
- </element>
- </optional>
- <optional>
<element name="uuid">
<ref name="UUID"/>
</element>
diff --git a/src/conf/node_device_conf.c b/src/conf/node_device_conf.c
index af72e160a5..332b12f997 100644
--- a/src/conf/node_device_conf.c
+++ b/src/conf/node_device_conf.c
@@ -85,12 +85,6 @@ VIR_ENUM_IMPL(virNodeDevDRM,
"render",
);
-VIR_ENUM_IMPL(virNodeDevMdevStart,
- VIR_NODE_DEV_MDEV_START_LAST,
- "manual",
- "auto",
-);
-
static int
virNodeDevCapsDefParseString(const char *xpath,
xmlXPathContextPtr ctxt,
@@ -534,8 +528,6 @@ virNodeDeviceCapMdevDefFormat(virBuffer *buf,
virBufferEscapeString(buf, "<type id='%s'/>\n", data->mdev.type);
virBufferEscapeString(buf, "<uuid>%s</uuid>\n", data->mdev.uuid);
- virBufferEscapeString(buf, "<start type='%s'/>\n",
- virNodeDevMdevStartTypeToString(data->mdev.start));
virBufferAsprintf(buf, "<iommuGroup number='%u'/>\n",
data->mdev.iommuGroupNumber);
@@ -1157,6 +1149,7 @@ virNodeDevCapStorageParseXML(xmlXPathContextPtr ctxt,
return -1;
}
storage->removable_media_size = val;
+
ctxt->node = orignode2;
} else {
virReportError(VIR_ERR_INTERNAL_ERROR,
@@ -1919,7 +1912,6 @@ virNodeDevCapMdevParseXML(xmlXPathContextPtr ctxt,
g_autofree xmlNodePtr *attrs = NULL;
size_t i;
g_autofree char *uuidstr = NULL;
- xmlNodePtr startNode = NULL;
ctxt->node = node;
@@ -1941,17 +1933,6 @@ virNodeDevCapMdevParseXML(xmlXPathContextPtr ctxt,
virUUIDFormat(uuidbuf, mdev->uuid);
}
- mdev->start = VIR_NODE_DEV_MDEV_START_MANUAL;
-
- startNode = virXPathNode("./start[1]", ctxt);
- if (startNode &&
- virXMLPropEnumDefault(startNode, "type",
- virNodeDevMdevStartTypeFromString,
- VIR_XML_PROP_NONE, &mdev->start,
- VIR_NODE_DEV_MDEV_START_MANUAL) < 0) {
- return -1;
- }
-
/* 'iommuGroup' is optional, only report an error if the supplied value is
* invalid (-2), not if it's missing (-1) */
if (virXPathUInt("number(./iommuGroup[1]/@number)",
diff --git a/src/conf/node_device_conf.h b/src/conf/node_device_conf.h
index 1a31133c4c..a60562e4fe 100644
--- a/src/conf/node_device_conf.h
+++ b/src/conf/node_device_conf.h
@@ -124,17 +124,6 @@ typedef enum {
VIR_ENUM_DECL(virNodeDevDRM);
-typedef enum {
- /* Keep in sync with VIR_ENUM_IMPL in node_device_conf.c */
- VIR_NODE_DEV_MDEV_START_MANUAL,
- VIR_NODE_DEV_MDEV_START_AUTO,
-
- VIR_NODE_DEV_MDEV_START_LAST
-} virNodeDevMdevStartType;
-
-VIR_ENUM_DECL(virNodeDevMdevStart);
-
-
typedef struct _virNodeDevCapSystemHardware virNodeDevCapSystemHardware;
struct _virNodeDevCapSystemHardware {
char *vendor_name;
@@ -162,7 +151,6 @@ struct _virNodeDevCapMdev {
char *type;
unsigned int iommuGroupNumber;
char *uuid;
- virNodeDevMdevStartType start;
virMediatedDeviceAttr **attributes;
size_t nattributes;
};
diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms
index 333cca0b41..0ced2a7990 100644
--- a/src/libvirt_private.syms
+++ b/src/libvirt_private.syms
@@ -848,8 +848,6 @@ virNodeDeviceGetSCSIHostCaps;
virNodeDeviceGetSCSITargetCaps;
virNodeDeviceGetWWNs;
virNodeDeviceUpdateCaps;
-virNodeDevMdevStartTypeFromString;
-virNodeDevMdevStartTypeToString;
# conf/node_device_event.h
diff --git a/src/node_device/node_device_driver.c b/src/node_device/node_device_driver.c
index a9f605715b..8a0a2c3847 100644
--- a/src/node_device/node_device_driver.c
+++ b/src/node_device/node_device_driver.c
@@ -615,8 +615,7 @@ nodeDeviceDefToMdevctlConfig(virNodeDeviceDef *def, char **buf)
if (virJSONValueObjectAppendString(json, "mdev_type", mdev->type) < 0)
return -1;
- if (virJSONValueObjectAppendString(json, "start",
- virNodeDevMdevStartTypeToString(mdev->start)) < 0)
+ if (virJSONValueObjectAppendString(json, "start", "manual") < 0)
return -1;
if (mdev->attributes) {
@@ -1015,8 +1014,6 @@ nodeDeviceParseMdevctlChildDevice(const char *parent,
mdev->uuid = g_strdup(uuid);
mdev->type =
g_strdup(virJSONValueObjectGetString(props, "mdev_type"));
- mdev->start =
- virNodeDevMdevStartTypeFromString(virJSONValueObjectGetString(props, "start"));
attrs = virJSONValueObjectGet(props, "attrs");
@@ -1686,8 +1683,6 @@ nodeDeviceDefCopyFromMdevctl(virNodeDeviceDef *dst,
dstmdev->uuid = g_strdup(srcmdev->uuid);
}
- dstmdev->start = srcmdev->start;
-
if (virMediatedDeviceAttrsCopy(dstmdev, srcmdev))
ret = true;
diff --git a/tests/nodedevmdevctldata/mdevctl-list-multiple.out.xml b/tests/nodedevmdevctldata/mdevctl-list-multiple.out.xml
index 83a3010455..cf7e966256 100644
--- a/tests/nodedevmdevctldata/mdevctl-list-multiple.out.xml
+++ b/tests/nodedevmdevctldata/mdevctl-list-multiple.out.xml
@@ -4,7 +4,6 @@
<capability type='mdev'>
<type id='i915-GVTg_V5_4'/>
<uuid>200f228a-c80a-4d50-bfb7-f5a0e4e34045</uuid>
- <start type='manual'/>
<iommuGroup number='0'/>
</capability>
</device>
@@ -14,7 +13,6 @@
<capability type='mdev'>
<type id='i915-GVTg_V5_4'/>
<uuid>de807ffc-1923-4d5f-b6c9-b20ecebc6d4b</uuid>
- <start type='auto'/>
<iommuGroup number='0'/>
</capability>
</device>
@@ -24,7 +22,6 @@
<capability type='mdev'>
<type id='i915-GVTg_V5_8'/>
<uuid>435722ea-5f43-468a-874f-da34f1217f13</uuid>
- <start type='manual'/>
<iommuGroup number='0'/>
<attr name='testattr' value='42'/>
</capability>
@@ -35,7 +32,6 @@
<capability type='mdev'>
<type id='vfio_ap-passthrough'/>
<uuid>783e6dbb-ea0e-411f-94e2-717eaad438bf</uuid>
- <start type='manual'/>
<iommuGroup number='0'/>
<attr name='assign_adapter' value='5'/>
<attr name='assign_adapter' value='6'/>
diff --git a/tests/nodedevxml2xmlout/mdev_3627463d_b7f0_4fea_b468_f1da537d301b.xml b/tests/nodedevxml2xmlout/mdev_3627463d_b7f0_4fea_b468_f1da537d301b.xml
index 30dda60e87..cbc883e48c 100644..120000
--- a/tests/nodedevxml2xmlout/mdev_3627463d_b7f0_4fea_b468_f1da537d301b.xml
+++ b/tests/nodedevxml2xmlout/mdev_3627463d_b7f0_4fea_b468_f1da537d301b.xml
@@ -1,9 +1 @@
-<device>
- <name>mdev_3627463d_b7f0_4fea_b468_f1da537d301b</name>
- <parent>computer</parent>
- <capability type='mdev'>
- <type id='mtty-1'/>
- <start type='manual'/>
- <iommuGroup number='12'/>
- </capability>
-</device>
+../nodedevschemadata/mdev_3627463d_b7f0_4fea_b468_f1da537d301b.xml \ No newline at end of file
diff --git a/tests/nodedevxml2xmlout/mdev_d069d019_36ea_4111_8f0a_8c9a70e21366.xml b/tests/nodedevxml2xmlout/mdev_d069d019_36ea_4111_8f0a_8c9a70e21366.xml
index bff95b015b..4f9d491981 100644
--- a/tests/nodedevxml2xmlout/mdev_d069d019_36ea_4111_8f0a_8c9a70e21366.xml
+++ b/tests/nodedevxml2xmlout/mdev_d069d019_36ea_4111_8f0a_8c9a70e21366.xml
@@ -4,7 +4,6 @@
<capability type='mdev'>
<type id='i915-GVTg_V5_8'/>
<uuid>d069d019-36ea-4111-8f0a-8c9a70e21366</uuid>
- <start type='manual'/>
<iommuGroup number='0'/>
</capability>
</device>
diff --git a/tests/nodedevxml2xmlout/mdev_d2441d39_495e_4243_ad9f_beb3f14c23d9.xml b/tests/nodedevxml2xmlout/mdev_d2441d39_495e_4243_ad9f_beb3f14c23d9.xml
index beb25b1989..1aa0afe085 100644
--- a/tests/nodedevxml2xmlout/mdev_d2441d39_495e_4243_ad9f_beb3f14c23d9.xml
+++ b/tests/nodedevxml2xmlout/mdev_d2441d39_495e_4243_ad9f_beb3f14c23d9.xml
@@ -3,7 +3,6 @@
<parent>pci_0000_00_02_0</parent>
<capability type='mdev'>
<type id='i915-GVTg_V5_8'/>
- <start type='manual'/>
<iommuGroup number='0'/>
<attr name='example-attribute-1' value='attribute-value-1'/>
<attr name='example-attribute-2' value='attribute-value-2'/>
diff --git a/tests/nodedevxml2xmlout/mdev_ee0b88c4_f554_4dc1_809d_b2a01e8e48ad.xml b/tests/nodedevxml2xmlout/mdev_ee0b88c4_f554_4dc1_809d_b2a01e8e48ad.xml
index 1988ffa679..238bfb4dfd 100644..120000
--- a/tests/nodedevxml2xmlout/mdev_ee0b88c4_f554_4dc1_809d_b2a01e8e48ad.xml
+++ b/tests/nodedevxml2xmlout/mdev_ee0b88c4_f554_4dc1_809d_b2a01e8e48ad.xml
@@ -1,10 +1 @@
-<device>
- <name>mdev_ee0b88c4-f554-4dc1-809d-b2a01e8e48ad</name>
- <path>/sys/devices/vfio_ap/matrix/mdev_ee0b88c4-f554-4dc1-809d-b2a01e8e48ad</path>
- <parent>ap_matrix</parent>
- <capability type='mdev'>
- <type id='vfio_ap-passthrough'/>
- <start type='manual'/>
- <iommuGroup number='0'/>
- </capability>
-</device>
+../nodedevschemadata/mdev_ee0b88c4_f554_4dc1_809d_b2a01e8e48ad.xml \ No newline at end of file
diff --git a/tests/nodedevxml2xmlout/mdev_fedc4916_1ca8_49ac_b176_871d16c13076.xml b/tests/nodedevxml2xmlout/mdev_fedc4916_1ca8_49ac_b176_871d16c13076.xml
index 1d6dbb6118..da2a702304 100644
--- a/tests/nodedevxml2xmlout/mdev_fedc4916_1ca8_49ac_b176_871d16c13076.xml
+++ b/tests/nodedevxml2xmlout/mdev_fedc4916_1ca8_49ac_b176_871d16c13076.xml
@@ -3,7 +3,6 @@
<parent>pci_0000_00_02_0</parent>
<capability type='mdev'>
<type id='i915-GVTg_V5_8'/>
- <start type='manual'/>
<iommuGroup number='0'/>
<attr name='example-attribute' value='attribute-value'/>
</capability>