storage: cleanup: do away with 'createFile'

Have storage building functions be definitions of
virStorageBackendBuildVolFrom: we will need to do this in the future anyways
if we ever support the flags attribute.

diff --git a/src/storage_backend.c b/src/storage_backend.c
index 936e06bf872185cebc8b673d839612aea7aaa1b9..f4493c16027bdffbcb8b2e4c97fe9d2c914a4c60 100644 (file)
--- a/src/storage_backend.c
+++ b/src/storage_backend.c
@@ -106,7 +106,8 @@ enum {
 int
 virStorageBackendCreateRaw(virConnectPtr conn,
                            virStorageVolDefPtr vol,
-                           virStorageVolDefPtr inputvol)
+                           virStorageVolDefPtr inputvol,
+                           unsigned int flags ATTRIBUTE_UNUSED)
 {
     int fd = -1;
     int inputfd = -1;
@@ -257,7 +258,8 @@ cleanup:
 static int
 virStorageBackendCreateQemuImg(virConnectPtr conn,
                                virStorageVolDefPtr vol,
-                               virStorageVolDefPtr inputvol)
+                               virStorageVolDefPtr inputvol,
+                               unsigned int flags ATTRIBUTE_UNUSED)
 {
     char size[100];
     char *create_tool;
@@ -395,7 +397,8 @@ virStorageBackendCreateQemuImg(virConnectPtr conn,
 static int
 virStorageBackendCreateQcowCreate(virConnectPtr conn,
                                   virStorageVolDefPtr vol,
-                                  virStorageVolDefPtr inputvol)
+                                  virStorageVolDefPtr inputvol,
+                                  unsigned int flags ATTRIBUTE_UNUSED)
 {
     char size[100];
     const char *imgargv[4];
@@ -438,7 +441,7 @@ virStorageBackendCreateQcowCreate(virConnectPtr conn,
     return 0;
 }
 
-createFile
+virStorageBackendBuildVolFrom
 virStorageBackendFSImageToolTypeToFunc(virConnectPtr conn, int tool_type)
 {
     switch (tool_type) {
@@ -478,7 +481,7 @@ virStorageBackendFindFSImageTool(char **tool)
     return tool_type;
 }
 
-createFile
+virStorageBackendBuildVolFrom
 virStorageBackendGetBuildVolFromFunction(virConnectPtr conn,
                                          virStorageVolDefPtr vol,
                                          virStorageVolDefPtr inputvol)

diff --git a/src/storage_backend.h b/src/storage_backend.h
index 54478f7e08405356c513030ff03d212d386bfac5..e80b05d07abe65ad9fb45211550b376a47bdb2a4 100644 (file)
--- a/src/storage_backend.h
+++ b/src/storage_backend.h
@@ -40,21 +40,19 @@ typedef int (*virStorageBackendRefreshVol)(virConnectPtr conn, virStoragePoolObj
 typedef int (*virStorageBackendDeleteVol)(virConnectPtr conn, virStoragePoolObjPtr pool, virStorageVolDefPtr vol, unsigned int flags);
 typedef int (*virStorageBackendBuildVolFrom)(virConnectPtr conn, virStorageVolDefPtr origvol, virStorageVolDefPtr newvol, unsigned int flags);
 
-typedef int (*createFile)(virConnectPtr conn,
-                          virStorageVolDefPtr vol,
-                          virStorageVolDefPtr inputvol);
-
 /* File creation/cloning functions used for cloning between backends */
 int virStorageBackendCreateRaw(virConnectPtr conn,
                                virStorageVolDefPtr vol,
-                               virStorageVolDefPtr inputvol);
-createFile
+                               virStorageVolDefPtr inputvol,
+                               unsigned int flags);
+virStorageBackendBuildVolFrom
 virStorageBackendGetBuildVolFromFunction(virConnectPtr conn,
                                          virStorageVolDefPtr vol,
                                          virStorageVolDefPtr inputvol);
 int virStorageBackendFindFSImageTool(char **tool);
-createFile virStorageBackendFSImageToolTypeToFunc(virConnectPtr conn,
-                                                  int tool_type);
+virStorageBackendBuildVolFrom
+virStorageBackendFSImageToolTypeToFunc(virConnectPtr conn,
+                                       int tool_type);
 
 
 typedef struct _virStorageBackend virStorageBackend;

diff --git a/src/storage_backend_fs.c b/src/storage_backend_fs.c
index 3b2af596e9f24670e3d310f8d2bb00896fe367c0..8cfc462f29222ff2ae8909aebda63d4c1676bc68 100644 (file)
--- a/src/storage_backend_fs.c
+++ b/src/storage_backend_fs.c
@@ -1008,7 +1008,8 @@ virStorageBackendFileSystemVolCreate(virConnectPtr conn,
 
 static int createFileDir(virConnectPtr conn,
                          virStorageVolDefPtr vol,
-                         virStorageVolDefPtr inputvol) {
+                         virStorageVolDefPtr inputvol,
+                         unsigned int flags ATTRIBUTE_UNUSED) {
     if (inputvol) {
         virStorageReportError(conn, VIR_ERR_INTERNAL_ERROR,
                               "%s",
@@ -1032,7 +1033,7 @@ _virStorageBackendFileSystemVolBuild(virConnectPtr conn,
                                      virStorageVolDefPtr inputvol)
 {
     int fd;
-    createFile create_func;
+    virStorageBackendBuildVolFrom create_func;
     int tool_type;
 
     if (inputvol) {
@@ -1056,7 +1057,7 @@ _virStorageBackendFileSystemVolBuild(virConnectPtr conn,
         return -1;
     }
 
-    if (create_func(conn, vol, inputvol) < 0)
+    if (create_func(conn, vol, inputvol, 0) < 0)
         return -1;
 
     if ((fd = open(vol->target.path, O_RDONLY)) < 0) {