From owner-svn-src-all@FreeBSD.ORG Thu Jul 18 00:34:23 2013 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id B25A0DD3; Thu, 18 Jul 2013 00:34:23 +0000 (UTC) (envelope-from prvs=1911771df7=killing@multiplay.co.uk) Received: from mail1.multiplay.co.uk (mail1.multiplay.co.uk [85.236.96.23]) by mx1.freebsd.org (Postfix) with ESMTP id C7FF47BF; Thu, 18 Jul 2013 00:34:22 +0000 (UTC) Received: from r2d2 ([82.69.141.170]) by mail1.multiplay.co.uk (mail1.multiplay.co.uk [85.236.96.23]) (MDaemon PRO v10.0.4) with ESMTP id md50005038579.msg; Thu, 18 Jul 2013 01:34:20 +0100 X-Spam-Processed: mail1.multiplay.co.uk, Thu, 18 Jul 2013 01:34:20 +0100 (not processed: message from valid local sender) X-MDDKIM-Result: neutral (mail1.multiplay.co.uk) X-MDRemoteIP: 82.69.141.170 X-Return-Path: prvs=1911771df7=killing@multiplay.co.uk X-Envelope-From: killing@multiplay.co.uk Message-ID: <251C370CF0FF4EE686D571A1235D4C90@multiplay.co.uk> From: "Steven Hartland" To: "Xin LI" , , , References: <201307180022.r6I0MgeS094364@svn.freebsd.org> Subject: Re: svn commit: r253441 - in head: cddl/contrib/opensolaris/cmd/zpool sys/cddl/contrib/opensolaris/uts/common/fs/zfs Date: Thu, 18 Jul 2013 01:34:43 +0100 MIME-Version: 1.0 Content-Type: text/plain; format=flowed; charset="UTF-8"; reply-type=original Content-Transfer-Encoding: 7bit X-Priority: 3 X-MSMail-Priority: Normal X-Mailer: Microsoft Outlook Express 6.00.2900.5931 X-MimeOLE: Produced By Microsoft MimeOLE V6.00.2900.6157 X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 18 Jul 2013 00:34:23 -0000 This is an interesting change, could this not cause serious issues when we try to read / write to a disk with an incompatible block size? Regards Steve ----- Original Message ----- From: "Xin LI" To: ; ; Sent: Thursday, July 18, 2013 1:22 AM Subject: svn commit: r253441 - in head: cddl/contrib/opensolaris/cmd/zpool sys/cddl/contrib/opensolaris/uts/common/fs/zfs > Author: delphij > Date: Thu Jul 18 00:22:42 2013 > New Revision: 253441 > URL: http://svnweb.freebsd.org/changeset/base/253441 > > Log: > Manually merge part of vendor import r238583 from Illumos. > > Illumos changeset: 13680:2bd022a765e2 > Illumos ZFS issue: > > 2671 zpool import should not fail if vdev ashift has increased > > MFC after: 3 days > > Modified: > head/cddl/contrib/opensolaris/cmd/zpool/zpool_main.c > head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev.c > > Modified: head/cddl/contrib/opensolaris/cmd/zpool/zpool_main.c > ============================================================================== > --- head/cddl/contrib/opensolaris/cmd/zpool/zpool_main.c Wed Jul 17 23:37:33 2013 (r253440) > +++ head/cddl/contrib/opensolaris/cmd/zpool/zpool_main.c Thu Jul 18 00:22:42 2013 (r253441) > @@ -3997,7 +3997,7 @@ print_dedup_stats(nvlist_t *config) > > /* > * If the pool was faulted then we may not have been able to > - * obtain the config. Otherwise, if have anything in the dedup > + * obtain the config. Otherwise, if we have anything in the dedup > * table continue processing the stats. > */ > if (nvlist_lookup_uint64_array(config, ZPOOL_CONFIG_DDT_OBJ_STATS, > > Modified: head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev.c > ============================================================================== > --- head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev.c Wed Jul 17 23:37:33 2013 (r253440) > +++ head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev.c Thu Jul 18 00:22:42 2013 (r253441) > @@ -1258,12 +1258,16 @@ vdev_open(vdev_t *vd) > vd->vdev_ashift = MAX(ashift, vd->vdev_ashift); > } else { > /* > - * Make sure the alignment requirement hasn't increased. > + * Detect if the alignment requirement has increased. > + * We don't want to make the pool unavailable, just > + * issue a warning instead. > */ > - if (ashift > vd->vdev_top->vdev_ashift) { > - vdev_set_state(vd, B_TRUE, VDEV_STATE_CANT_OPEN, > - VDEV_AUX_BAD_LABEL); > - return (EINVAL); > + if (ashift > vd->vdev_top->vdev_ashift && > + vd->vdev_ops->vdev_op_leaf) { > + cmn_err(CE_WARN, > + "Disk, '%s', has a block alignment that is " > + "larger than the pool's alignment\n", > + vd->vdev_path); > } > vd->vdev_max_asize = max_asize; > } > ================================================ This e.mail is private and confidential between Multiplay (UK) Ltd. and the person or entity to whom it is addressed. In the event of misdirection, the recipient is prohibited from using, copying, printing or otherwise disseminating it or any information contained in it. In the event of misdirection, illegible or incomplete transmission please telephone +44 845 868 1337 or return the E.mail to postmaster@multiplay.co.uk.