Date: Fri, 07 Nov 2014 13:31:07 +0000 From: Steven Hartland <killing@multiplay.co.uk> To: Andriy Gapon <avg@FreeBSD.org>, Borja Marcos <borjam@sarenet.es> Cc: freebsd-fs@FreeBSD.org Subject: Re: ZFS bug: was creating ZIL ignores vfs.zfs.min_auto_ashift, should be ZIL sets improper ashift with AHCI controllers Message-ID: <545CC99B.1010302@multiplay.co.uk> In-Reply-To: <545CB43E.6040901@FreeBSD.org> References: <B731A922-3F83-4D8E-A4EA-22C5CA8A3850@sarenet.es> <9C91F97841BC4347910F206618BAA3BB9AF327D1@PAIMAIL.pai.local> <545B76EF.6050709@multiplay.co.uk> <21D2A3A9-B6C1-458F-B17F-480251E999AE@sarenet.es> <545BE3E0.4030203@multiplay.co.uk> <757B9039-8DF0-4E6A-A036-82A2143D7F45@sarenet.es> <545C9552.30207@multiplay.co.uk> <545CB43E.6040901@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On 07/11/2014 11:59, Andriy Gapon wrote:
> On 07/11/2014 11:48, Steven Hartland wrote:
>> On 07/11/2014 07:31, Borja Marcos wrote:
>>> snip...
>>>
>>> You are right!
>>>
>>> This time I had not altered min_auto_ashift but I had added a quirk to
>>> ata_da.c. So it
>>> correctly used the quirk sector size to adjust ashift.
>>>
>>> Seems the bug should be "improper cache file generation for a pool used for
>>> boot". I'll
>>> see if I can run some more tests.
>>>
>> I believe the attached is the correct fix, but I've asked upstream as this test
>> was explicitly added ages ago.
> It could be that this problem is of no interest / relevance to the illumos
> upstream as they have more restrictions on how a root pool can be configured. I
> am not sure if they allow slog devices for root pools.
>
illumos does indeed have the following restrictions for root pools
according to zfs_ioc_vdev_add
* Concatenated devices are not supported
* Intent log device is not supported
So your right but I'm still curious if there are any reasons why we
shouldn't sync the zpool.cache file for the root pool as that seems
independent to this? The only thing I can think of is that it was an
optimisation but this would still break things like a reguid.
Regards
Steve
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?545CC99B.1010302>