From: Richard Jones <rjones@redhat.com>
Date: Fri, 30 Apr 2010 12:56:11 +0000 (+0100)
Subject: Document that guestfs_mount implies -o sync and performance problem (RHBZ#587582).
X-Git-Tag: 1.2.5~1
X-Git-Url: http://git.annexia.org/?a=commitdiff_plain;h=e6d09ffe07d18565e39adfd6a23c360f4dec919c;p=libguestfs.git

Document that guestfs_mount implies -o sync and performance problem (RHBZ#587582).
(cherry picked from commit fd2d10d6ed91f417a9b99bb61526b45bb7ed3e4e)
---

diff --git a/src/generator.ml b/src/generator.ml
index fc3dc34..ed29caa 100755
--- a/src/generator.ml
+++ b/src/generator.ml
@@ -953,8 +953,15 @@ exist.
 The mounted filesystem is writable, if we have sufficient permissions
 on the underlying device.
 
-The filesystem options C<sync> and C<noatime> are set with this
-call, in order to improve reliability.");
+B<Important note:>
+When you use this call, the filesystem options C<sync> and C<noatime>
+are set implicitly.  This was originally done because we thought it
+would improve reliability, but it turns out that I<-o sync> has a
+very large negative performance impact and negligible effect on
+reliability.  Therefore we recommend that you avoid using
+C<guestfs_mount> in any code that needs performance, and instead
+use C<guestfs_mount_options> (use an empty string for the first
+parameter if you don't want any options).");
 
   ("sync", (RErr, []), 2, [],
    [ InitEmpty, Always, TestRun [["sync"]]],
@@ -2079,7 +2086,11 @@ mounts the filesystem with the read-only (I<-o ro>) flag.");
    "\
 This is the same as the C<guestfs_mount> command, but it
 allows you to set the mount options as for the
-L<mount(8)> I<-o> flag.");
+L<mount(8)> I<-o> flag.
+
+If the C<options> parameter is an empty string, then
+no options are passed (all options default to whatever
+the filesystem uses).");
 
   ("mount_vfs", (RErr, [String "options"; String "vfstype"; Device "device"; String "mountpoint"]), 75, [],
    [],