Author: delphij
Date: Thu Oct  9 05:48:06 2014
New Revision: 272802
URL: https://svnweb.freebsd.org/changeset/base/272802

Log:
  5169 zdb should limit its ARC size
  5170 zdb -c should create more scrub i/os by default
  5171 zdb should print status while loading metaslabs for leak detection
  
  Reviewed by: Christopher Siden <christopher.si...@delphix.com>
  Reviewed by: George Wilson <george.wil...@delphix.com>
  Reviewed by: Paul Dagnelie <paul.dagne...@delphix.com>
  Reviewed by: Bayard Bell <bayard.b...@nexenta.com>
  Approved by: Robert Mustacchi <r...@joyent.com>
  Author: Matthew Ahrens <mahr...@delphix.com>
  
  illumos/illumos-gate@06be98028b8a84e7f327188613dd09a708294d1a

Modified:
  vendor/illumos/dist/cmd/zdb/zdb.c

Modified: vendor/illumos/dist/cmd/zdb/zdb.c
==============================================================================
--- vendor/illumos/dist/cmd/zdb/zdb.c   Thu Oct  9 05:42:07 2014        
(r272801)
+++ vendor/illumos/dist/cmd/zdb/zdb.c   Thu Oct  9 05:48:06 2014        
(r272802)
@@ -76,8 +76,10 @@
 
 #ifndef lint
 extern boolean_t zfs_recover;
+extern uint64_t zfs_arc_max, zfs_arc_meta_limit;
 #else
 boolean_t zfs_recover;
+uint64_t zfs_arc_max, zfs_arc_meta_limit;
 #endif
 
 const char cmdname[] = "zdb";
@@ -89,7 +91,7 @@ extern void dump_intent_log(zilog_t *);
 uint64_t *zopt_object = NULL;
 int zopt_objects = 0;
 libzfs_handle_t *g_zfs;
-uint64_t max_inflight = 200;
+uint64_t max_inflight = 1000;
 
 /*
  * These libumem hooks provide a reasonable set of defaults for the allocator's
@@ -2353,7 +2355,7 @@ zdb_blkptr_cb(spa_t *spa, zilog_t *zilog
 
        zcb->zcb_readfails = 0;
 
-       if (dump_opt['b'] < 5 && isatty(STDERR_FILENO) &&
+       if (dump_opt['b'] < 5 &&
            gethrtime() > zcb->zcb_lastprint + NANOSEC) {
                uint64_t now = gethrtime();
                char buf[10];
@@ -2438,9 +2440,9 @@ zdb_leak_init(spa_t *spa, zdb_cb_t *zcb)
 
        if (!dump_opt['L']) {
                vdev_t *rvd = spa->spa_root_vdev;
-               for (int c = 0; c < rvd->vdev_children; c++) {
+               for (uint64_t c = 0; c < rvd->vdev_children; c++) {
                        vdev_t *vd = rvd->vdev_child[c];
-                       for (int m = 0; m < vd->vdev_ms_count; m++) {
+                       for (uint64_t m = 0; m < vd->vdev_ms_count; m++) {
                                metaslab_t *msp = vd->vdev_ms[m];
                                mutex_enter(&msp->ms_lock);
                                metaslab_unload(msp);
@@ -2453,6 +2455,15 @@ zdb_leak_init(spa_t *spa, zdb_cb_t *zcb)
                                 * interfaces.
                                 */
                                if (msp->ms_sm != NULL) {
+                                       (void) fprintf(stderr,
+                                           "\rloading space map for "
+                                           "vdev %llu of %llu, "
+                                           "metaslab %llu of %llu ...",
+                                           (longlong_t)c,
+                                           (longlong_t)rvd->vdev_children,
+                                           (longlong_t)m,
+                                           (longlong_t)vd->vdev_ms_count);
+
                                        msp->ms_ops = &zdb_metaslab_ops;
                                        VERIFY0(space_map_load(msp->ms_sm,
                                            msp->ms_tree, SM_ALLOC));
@@ -2461,6 +2472,7 @@ zdb_leak_init(spa_t *spa, zdb_cb_t *zcb)
                                mutex_exit(&msp->ms_lock);
                        }
                }
+               (void) fprintf(stderr, "\n");
        }
 
        spa_config_enter(spa, SCL_CONFIG, FTAG, RW_READER);
@@ -3460,6 +3472,12 @@ main(int argc, char **argv)
                usage();
        }
 
+       /*
+        * ZDB does not typically re-read blocks; therefore limit the ARC
+        * to 256 MB, which can be used entirely for metadata.
+        */
+       zfs_arc_max = zfs_arc_meta_limit = 256 * 1024 * 1024;
+
        kernel_init(FREAD);
        g_zfs = libzfs_init();
        ASSERT(g_zfs != NULL);
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to