Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 10 Aug 2015 21:19:17 +0000 (UTC)
From:      Alexander Motin <mav@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-vendor@freebsd.org
Subject:   svn commit: r286599 - vendor-sys/illumos/dist/uts/common/fs/zfs
Message-ID:  <201508102119.t7ALJHaO042891@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: mav
Date: Mon Aug 10 21:19:16 2015
New Revision: 286599
URL: https://svnweb.freebsd.org/changeset/base/286599

Log:
  5808 spa_check_logs is not necessary on readonly pools
  
  Reviewed by: George Wilson <george@delphix.com>
  Reviewed by: Paul Dagnelie <paul.dagnelie@delphix.com>
  Reviewed by: Simon Klinkert <simon.klinkert@gmail.com>
  Reviewed by: Will Andrews <will@freebsd.org>
  Approved by: Gordon Ross <gwr@nexenta.com>
  Author: Matthew Ahrens <mahrens@delphix.com>
  
  illumos/illumos-gate@23367a2f2caec1ccb4d918bdd0f2fc2c9cadcd06

Modified:
  vendor-sys/illumos/dist/uts/common/fs/zfs/spa.c

Modified: vendor-sys/illumos/dist/uts/common/fs/zfs/spa.c
==============================================================================
--- vendor-sys/illumos/dist/uts/common/fs/zfs/spa.c	Mon Aug 10 21:13:59 2015	(r286598)
+++ vendor-sys/illumos/dist/uts/common/fs/zfs/spa.c	Mon Aug 10 21:19:16 2015	(r286599)
@@ -2687,7 +2687,7 @@ spa_load_impl(spa_t *spa, uint64_t pool_
 		if (rvd->vdev_state <= VDEV_STATE_CANT_OPEN)
 			return (SET_ERROR(ENXIO));
 
-		if (spa_check_logs(spa)) {
+		if (spa_writeable(spa) && spa_check_logs(spa)) {
 			*ereport = FM_EREPORT_ZFS_LOG_REPLAY;
 			return (spa_vdev_err(rvd, VDEV_AUX_BAD_LOG, ENXIO));
 		}



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