xfs
[Top] [All Lists]

[PATCH 35/37] repair: prefetching is turned off unnecessarily

To: xfs@xxxxxxxxxxx
Subject: [PATCH 35/37] repair: prefetching is turned off unnecessarily
From: Dave Chinner <david@xxxxxxxxxxxxx>
Date: Wed, 6 Nov 2013 12:07:21 +1100
Delivered-to: xfs@xxxxxxxxxxx
In-reply-to: <1383700043-32305-1-git-send-email-david@xxxxxxxxxxxxx>
References: <1383700043-32305-1-git-send-email-david@xxxxxxxxxxxxx>
From: Dave Chinner <dchinner@xxxxxxxxxx>

When we have a large filesystem, prefetching is only enabled when
there is a significant amount of RAM available - roughly 16GB RAM
for every 100TB of disk space. For large filesystems, this memory
usage calculation is mostly derived from the memory needed to track
used space rather than inodes. That is, for a 100TB filesystem with
50 million inodes, only 50M * 4 bytes or 200MB of the the required
16GB of RAM is used for tracking inodes. Hence with prefetching
turned off, such a filesystem only uses 230MB of memory to run
repair to completion.

With prefetching turned on, this increases to about 900MB of RAM,
but it is still far, far less than the predicted 16GB of RAM needed
to enable prefetching. Hence we are turning off prefetching when we
really don't need to and hence large filesystems are being checked
slower than they could be.

This patch makes prefetching always be enabled, but adds warnings in
the case that we might not have enough memory to complete
successfully and if it fails to run again with prefetching disabled:

  Memory available for repair (12031MB) may not be sufficient.
  At least 13044MB is needed to repair this filesystem efficiently
  If repair fails due to lack of memory, please
  turn prefetching off (-P) to reduce the memory footprint.

A similar warning is also added when prefetching is disabled and
xfs_repair exhausts memory then more RAM/swap should be added to the
system.

Signed-off-by: Dave Chinner <dchinner@xxxxxxxxxx>
---
 repair/xfs_repair.c | 42 ++++++++++++++++++++++--------------------
 1 file changed, 22 insertions(+), 20 deletions(-)

diff --git a/repair/xfs_repair.c b/repair/xfs_repair.c
index 77a040e..78f8363 100644
--- a/repair/xfs_repair.c
+++ b/repair/xfs_repair.c
@@ -674,34 +674,36 @@ main(int argc, char **argv)
                                mp->m_sb.sb_dblocks >> (10 + 1));
 
                if (max_mem <= mem_used) {
-                       /*
-                        * Turn off prefetch and minimise libxfs cache if
-                        * physical memory is deemed insufficient
-                        */
                        if (max_mem_specified) {
                                do_abort(
        _("Required memory for repair is greater that the maximum specified\n"
          "with the -m option. Please increase it to at least %lu.\n"),
                                        mem_used / 1024);
-                       } else {
-                               do_warn(
-       _("Not enough RAM available for repair to enable prefetching.\n"
-         "This will be _slow_.\n"
-         "You need at least %luMB RAM to run with prefetching enabled.\n"),
-                                       mem_used * 1280 / (1024 * 1024));
                        }
-                       do_prefetch = 0;
-                       libxfs_bhash_size = 64;
-               } else {
-                       max_mem -= mem_used;
-                       if (max_mem >= (1 << 30))
-                               max_mem = 1 << 30;
-                       libxfs_bhash_size = max_mem / (HASH_CACHE_RATIO *
-                                       (mp->m_inode_cluster_size >> 10));
-                       if (libxfs_bhash_size < 512)
-                               libxfs_bhash_size = 512;
+                       do_warn(
+       _("Memory available for repair (%luMB) may not be sufficient.\n"
+         "At least %luMB is needed to repair this filesystem efficiently\n"
+         "If repair fails due to lack of memory, please\n"),
+                               max_mem / 1024, mem_used / 1024);
+                       if (do_prefetch)
+                               do_warn(
+       _("turn prefetching off (-P) to reduce the memory footprint.\n"));
+                       else
+                               do_warn(
+       _("increase system RAM and/or swap space to at least %luMB.\n"),
+                       mem_used * 2 / 1024);
+
+                       max_mem = mem_used;
                }
 
+               max_mem -= mem_used;
+               if (max_mem >= (1 << 30))
+                       max_mem = 1 << 30;
+               libxfs_bhash_size = max_mem / (HASH_CACHE_RATIO *
+                               (mp->m_inode_cluster_size >> 10));
+               if (libxfs_bhash_size < 512)
+                       libxfs_bhash_size = 512;
+
                if (verbose)
                        do_log(_("        - block cache size set to %d 
entries\n"),
                                libxfs_bhash_size * HASH_CACHE_RATIO);
-- 
1.8.4.rc3

<Prev in Thread] Current Thread [Next in Thread>