[Libguestfs] [PATCH v2 12/17] v2v: drop redundant umount_all() and shutdown()

Roman Kagan rkagan at virtuozzo.com
Tue Aug 11 17:00:31 UTC 2015


umount_all() and shutdown() are performed by guestfs automatically on
close(), so drop explicit calls to them right before close().

Also umount_all() in the middle of processing doesn't look justified so
drop it, too.
(The only step following it is do_fstrim() which does mounting/umounting
on its own).

Signed-off-by: Roman Kagan <rkagan at virtuozzo.com>
---
 v2v/v2v.ml | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/v2v/v2v.ml b/v2v/v2v.ml
index 554b30d..9a21abc 100644
--- a/v2v/v2v.ml
+++ b/v2v/v2v.ml
@@ -889,8 +889,6 @@ let main () =
   let keep_serial_console = output#keep_serial_console in
   let guestcaps = do_convert g inspect source keep_serial_console in
 
-  g#umount_all ();
-
   if no_trim <> ["*"] && (do_copy || debug_overlays) then (
     (* Doing fstrim on all the filesystems reduces the transfer size
      * because unused blocks are marked in the overlay and thus do
@@ -901,8 +899,6 @@ let main () =
   );
 
   message (f_"Closing the overlay");
-  g#umount_all ();
-  g#shutdown ();
   g#close ();
 
   let target_firmware = get_target_firmware inspect guestcaps source output in
-- 
2.4.3




More information about the Libguestfs mailing list