Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 14 Oct 2016 16:18:39 +0300
From:      Andriy Gapon <avg@FreeBSD.org>
To:        Alexander Motin <mav@FreeBSD.org>, src-committers@FreeBSD.org, svn-src-all@FreeBSD.org, svn-src-head@FreeBSD.org
Subject:   Re: svn commit: r307317 - in head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs: . sys
Message-ID:  <030747b3-a83c-8b16-d9b1-3d5d3b7ae142@FreeBSD.org>
In-Reply-To: <65265e09-a704-86c0-8e8a-ed423c941d40@FreeBSD.org>
References:  <201610141201.u9EC1X2O073182@repo.freebsd.org> <0e5b9464-e039-1c00-9805-6110f154f9f0@FreeBSD.org> <65265e09-a704-86c0-8e8a-ed423c941d40@FreeBSD.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On 14/10/2016 15:22, Alexander Motin wrote:
> On 14.10.2016 15:11, Andriy Gapon wrote:
>> On 14/10/2016 15:01, Alexander Motin wrote:
>>> Author: mav
>>> Date: Fri Oct 14 12:01:33 2016
>>> New Revision: 307317
>>> URL: https://svnweb.freebsd.org/changeset/base/307317
>>>
>>> Log:
>>>   MFV r307313:
>>>   5120 zfs should allow large block/gzip/raidz boot pool (loader project)
>>>   
>>>   Reviewed by: George Wilson <george.wilson@delphix.com>
>>>   Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com>
>>>   Reviewed by: Andrew Stormont <andyjstormont@gmail.com>
>>>   Reviewed by: Matthew Ahrens <mahrens@delphix.com>
>>>   Approved by: Robert Mustacchi <rm@joyent.com>
>>>   Author: Toomas Soome <tsoome@me.com>
>>>   
>>>   openzfs/openzfs@c8811bd3e2427dddbac6c05a59cfe117d8fea370
>>>   
>>>   FreeBSD still does not support booting from gzip-compressed datasets,
>>>   so keep one chunk of this commit out.
>>
>> Does FreeBSD boot code support large blocks? Just to be sure.
> 
> According to r304321 commit message -- it does now.

Okay.

>> And maybe this is worth some additional notice because users sometimes forget to
>> upgrade gptzfsboot / zfsboot.
> 
> Where would you like to see it to be noticeable?

That's a good question for which I do not have a good answer.
Ideally 'zfs set recordsize=' would warn when setting a large block size on a
boot filesystem and 'zpool set bootfs=' would warn when switching the boot
filesystem to a filesystem that already has a large block size.
But not sure if that's doable without kludges.

-- 
Andriy Gapon



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?030747b3-a83c-8b16-d9b1-3d5d3b7ae142>