Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 14 Apr 2017 17:20:26 +0000 (UTC)
From:      Andriy Gapon <avg@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-vendor@freebsd.org
Subject:   svn commit: r316872 - vendor/illumos/dist/cmd/ztest
Message-ID:  <201704141720.v3EHKQ5I040080@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: avg
Date: Fri Apr 14 17:20:26 2017
New Revision: 316872
URL: https://svnweb.freebsd.org/changeset/base/316872

Log:
  7502 ztest should run zdb with -G (debug mode)
  
  illumos/illumos-gate@c3c65d17f7b6689bbd6568a1a1fcc0c4a3bac127
  https://github.com/illumos/illumos-gate/commit/c3c65d17f7b6689bbd6568a1a1fcc0c4a3bac127
  
  https://www.illumos.org/issues/7502
    Right now ztest executes zdb without -G, so when it has errors, the messages
    are often not very helpful:
    Executing zdb -bccsv -d -U /rpool/tmp/zpool.cache ztest
    zdb: can't open 'ztest': Operation not supported
    ztest: '/usr/sbin/amd64/zdb -bccsv -d -U /rpool/tmp/zpool.cache ztest' exit
    code 1
    With -G, we'd have:
    /usr/sbin/amd64/zdb -bccsv -d -U /rpool/tmp/zpool.cache -G ztest
    zdb: can't open 'ztest': Operation not supported
  
    ZFS_DBGMSG(zdb):
    spa_open_common: opening ztest
    spa_load(ztest): LOADING
    spa_load(ztest): FAILED: unable to parse config [error=48]
    spa_load(ztest): UNLOADING
    Which indicates where the error came from
  
  Reviewed by: Matthew Ahrens <mahrens@delphix.com>
  Reviewed by: Dan Kimmel <dan.kimmel@delphix.com>
  Approved by: Gordon Ross <gordon.w.ross@gmail.com>
  Author: Pavel Zakharov <pavel.zakharov@delphix.com>

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

Modified: vendor/illumos/dist/cmd/ztest/ztest.c
==============================================================================
--- vendor/illumos/dist/cmd/ztest/ztest.c	Fri Apr 14 17:15:49 2017	(r316871)
+++ vendor/illumos/dist/cmd/ztest/ztest.c	Fri Apr 14 17:20:26 2017	(r316872)
@@ -5262,7 +5262,7 @@ ztest_run_zdb(char *pool)
 	isa = strdup(isa);
 	/* LINTED */
 	(void) sprintf(bin,
-	    "/usr/sbin%.*s/zdb -bcc%s%s -d -U %s %s",
+	    "/usr/sbin%.*s/zdb -bcc%s%s -G -d -U %s %s",
 	    isalen,
 	    isa,
 	    ztest_opts.zo_verbose >= 3 ? "s" : "",



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201704141720.v3EHKQ5I040080>