[libvirt] [PATCH 01/10] storage: move driver registration back to end of the file

Daniel P. Berrangé berrange at redhat.com
Fri Jan 26 13:35:28 UTC 2018


By convention the last thing in the driver.c files should be the driver
callback table and function to register it.

Signed-off-by: Daniel P. Berrangé <berrange at redhat.com>
---
 src/storage/storage_driver.c | 172 +++++++++++++++++++++----------------------
 1 file changed, 86 insertions(+), 86 deletions(-)

diff --git a/src/storage/storage_driver.c b/src/storage/storage_driver.c
index 3b66d51719..f68acc75be 100644
--- a/src/storage/storage_driver.c
+++ b/src/storage/storage_driver.c
@@ -2687,92 +2687,6 @@ storageConnectStoragePoolEventDeregisterAny(virConnectPtr conn,
 }
 
 
-
-static virStorageDriver storageDriver = {
-    .name = "storage",
-    .connectNumOfStoragePools = storageConnectNumOfStoragePools, /* 0.4.0 */
-    .connectListStoragePools = storageConnectListStoragePools, /* 0.4.0 */
-    .connectNumOfDefinedStoragePools = storageConnectNumOfDefinedStoragePools, /* 0.4.0 */
-    .connectListDefinedStoragePools = storageConnectListDefinedStoragePools, /* 0.4.0 */
-    .connectListAllStoragePools = storageConnectListAllStoragePools, /* 0.10.2 */
-    .connectStoragePoolEventRegisterAny = storageConnectStoragePoolEventRegisterAny, /* 2.0.0 */
-    .connectStoragePoolEventDeregisterAny = storageConnectStoragePoolEventDeregisterAny, /* 2.0.0 */
-    .connectFindStoragePoolSources = storageConnectFindStoragePoolSources, /* 0.4.0 */
-    .storagePoolLookupByName = storagePoolLookupByName, /* 0.4.0 */
-    .storagePoolLookupByUUID = storagePoolLookupByUUID, /* 0.4.0 */
-    .storagePoolLookupByVolume = storagePoolLookupByVolume, /* 0.4.0 */
-    .storagePoolCreateXML = storagePoolCreateXML, /* 0.4.0 */
-    .storagePoolDefineXML = storagePoolDefineXML, /* 0.4.0 */
-    .storagePoolBuild = storagePoolBuild, /* 0.4.0 */
-    .storagePoolUndefine = storagePoolUndefine, /* 0.4.0 */
-    .storagePoolCreate = storagePoolCreate, /* 0.4.0 */
-    .storagePoolDestroy = storagePoolDestroy, /* 0.4.0 */
-    .storagePoolDelete = storagePoolDelete, /* 0.4.0 */
-    .storagePoolRefresh = storagePoolRefresh, /* 0.4.0 */
-    .storagePoolGetInfo = storagePoolGetInfo, /* 0.4.0 */
-    .storagePoolGetXMLDesc = storagePoolGetXMLDesc, /* 0.4.0 */
-    .storagePoolGetAutostart = storagePoolGetAutostart, /* 0.4.0 */
-    .storagePoolSetAutostart = storagePoolSetAutostart, /* 0.4.0 */
-    .storagePoolNumOfVolumes = storagePoolNumOfVolumes, /* 0.4.0 */
-    .storagePoolListVolumes = storagePoolListVolumes, /* 0.4.0 */
-    .storagePoolListAllVolumes = storagePoolListAllVolumes, /* 0.10.2 */
-
-    .storageVolLookupByName = storageVolLookupByName, /* 0.4.0 */
-    .storageVolLookupByKey = storageVolLookupByKey, /* 0.4.0 */
-    .storageVolLookupByPath = storageVolLookupByPath, /* 0.4.0 */
-    .storageVolCreateXML = storageVolCreateXML, /* 0.4.0 */
-    .storageVolCreateXMLFrom = storageVolCreateXMLFrom, /* 0.6.4 */
-    .storageVolDownload = storageVolDownload, /* 0.9.0 */
-    .storageVolUpload = storageVolUpload, /* 0.9.0 */
-    .storageVolDelete = storageVolDelete, /* 0.4.0 */
-    .storageVolWipe = storageVolWipe, /* 0.8.0 */
-    .storageVolWipePattern = storageVolWipePattern, /* 0.9.10 */
-    .storageVolGetInfo = storageVolGetInfo, /* 0.4.0 */
-    .storageVolGetInfoFlags = storageVolGetInfoFlags, /* 3.0.0 */
-    .storageVolGetXMLDesc = storageVolGetXMLDesc, /* 0.4.0 */
-    .storageVolGetPath = storageVolGetPath, /* 0.4.0 */
-    .storageVolResize = storageVolResize, /* 0.9.10 */
-
-    .storagePoolIsActive = storagePoolIsActive, /* 0.7.3 */
-    .storagePoolIsPersistent = storagePoolIsPersistent, /* 0.7.3 */
-};
-
-
-static virStateDriver stateDriver = {
-    .name = "storage",
-    .stateInitialize = storageStateInitialize,
-    .stateAutoStart = storageStateAutoStart,
-    .stateCleanup = storageStateCleanup,
-    .stateReload = storageStateReload,
-};
-
-static int
-storageRegisterFull(bool allbackends)
-{
-    if (virStorageBackendDriversRegister(allbackends) < 0)
-        return -1;
-    if (virSetSharedStorageDriver(&storageDriver) < 0)
-        return -1;
-    if (virRegisterStateDriver(&stateDriver) < 0)
-        return -1;
-    return 0;
-}
-
-
-int
-storageRegister(void)
-{
-    return storageRegisterFull(false);
-}
-
-
-int
-storageRegisterAll(void)
-{
-    return storageRegisterFull(true);
-}
-
-
 static int
 virStorageAddISCSIPoolSourceHost(virDomainDiskDefPtr def,
                                  virStoragePoolDefPtr pooldef)
@@ -3065,3 +2979,89 @@ virStoragePoolObjBuildTempFilePath(virStoragePoolObjPtr obj,
                              driver->stateDir, def->name, voldef->name));
     return tmp;
 }
+
+
+static virStorageDriver storageDriver = {
+    .name = "storage",
+    .connectNumOfStoragePools = storageConnectNumOfStoragePools, /* 0.4.0 */
+    .connectListStoragePools = storageConnectListStoragePools, /* 0.4.0 */
+    .connectNumOfDefinedStoragePools = storageConnectNumOfDefinedStoragePools, /* 0.4.0 */
+    .connectListDefinedStoragePools = storageConnectListDefinedStoragePools, /* 0.4.0 */
+    .connectListAllStoragePools = storageConnectListAllStoragePools, /* 0.10.2 */
+    .connectStoragePoolEventRegisterAny = storageConnectStoragePoolEventRegisterAny, /* 2.0.0 */
+    .connectStoragePoolEventDeregisterAny = storageConnectStoragePoolEventDeregisterAny, /* 2.0.0 */
+    .connectFindStoragePoolSources = storageConnectFindStoragePoolSources, /* 0.4.0 */
+    .storagePoolLookupByName = storagePoolLookupByName, /* 0.4.0 */
+    .storagePoolLookupByUUID = storagePoolLookupByUUID, /* 0.4.0 */
+    .storagePoolLookupByVolume = storagePoolLookupByVolume, /* 0.4.0 */
+    .storagePoolCreateXML = storagePoolCreateXML, /* 0.4.0 */
+    .storagePoolDefineXML = storagePoolDefineXML, /* 0.4.0 */
+    .storagePoolBuild = storagePoolBuild, /* 0.4.0 */
+    .storagePoolUndefine = storagePoolUndefine, /* 0.4.0 */
+    .storagePoolCreate = storagePoolCreate, /* 0.4.0 */
+    .storagePoolDestroy = storagePoolDestroy, /* 0.4.0 */
+    .storagePoolDelete = storagePoolDelete, /* 0.4.0 */
+    .storagePoolRefresh = storagePoolRefresh, /* 0.4.0 */
+    .storagePoolGetInfo = storagePoolGetInfo, /* 0.4.0 */
+    .storagePoolGetXMLDesc = storagePoolGetXMLDesc, /* 0.4.0 */
+    .storagePoolGetAutostart = storagePoolGetAutostart, /* 0.4.0 */
+    .storagePoolSetAutostart = storagePoolSetAutostart, /* 0.4.0 */
+    .storagePoolNumOfVolumes = storagePoolNumOfVolumes, /* 0.4.0 */
+    .storagePoolListVolumes = storagePoolListVolumes, /* 0.4.0 */
+    .storagePoolListAllVolumes = storagePoolListAllVolumes, /* 0.10.2 */
+
+    .storageVolLookupByName = storageVolLookupByName, /* 0.4.0 */
+    .storageVolLookupByKey = storageVolLookupByKey, /* 0.4.0 */
+    .storageVolLookupByPath = storageVolLookupByPath, /* 0.4.0 */
+    .storageVolCreateXML = storageVolCreateXML, /* 0.4.0 */
+    .storageVolCreateXMLFrom = storageVolCreateXMLFrom, /* 0.6.4 */
+    .storageVolDownload = storageVolDownload, /* 0.9.0 */
+    .storageVolUpload = storageVolUpload, /* 0.9.0 */
+    .storageVolDelete = storageVolDelete, /* 0.4.0 */
+    .storageVolWipe = storageVolWipe, /* 0.8.0 */
+    .storageVolWipePattern = storageVolWipePattern, /* 0.9.10 */
+    .storageVolGetInfo = storageVolGetInfo, /* 0.4.0 */
+    .storageVolGetInfoFlags = storageVolGetInfoFlags, /* 3.0.0 */
+    .storageVolGetXMLDesc = storageVolGetXMLDesc, /* 0.4.0 */
+    .storageVolGetPath = storageVolGetPath, /* 0.4.0 */
+    .storageVolResize = storageVolResize, /* 0.9.10 */
+
+    .storagePoolIsActive = storagePoolIsActive, /* 0.7.3 */
+    .storagePoolIsPersistent = storagePoolIsPersistent, /* 0.7.3 */
+};
+
+
+static virStateDriver stateDriver = {
+    .name = "storage",
+    .stateInitialize = storageStateInitialize,
+    .stateAutoStart = storageStateAutoStart,
+    .stateCleanup = storageStateCleanup,
+    .stateReload = storageStateReload,
+};
+
+
+static int
+storageRegisterFull(bool allbackends)
+{
+    if (virStorageBackendDriversRegister(allbackends) < 0)
+        return -1;
+    if (virSetSharedStorageDriver(&storageDriver) < 0)
+        return -1;
+    if (virRegisterStateDriver(&stateDriver) < 0)
+        return -1;
+    return 0;
+}
+
+
+int
+storageRegister(void)
+{
+    return storageRegisterFull(false);
+}
+
+
+int
+storageRegisterAll(void)
+{
+    return storageRegisterFull(true);
+}
-- 
2.14.3




More information about the libvir-list mailing list