svn commit: r216230 - head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs

[Ivan Voras]

On 6 December 2010 23:36, Andriy Gapon <avg at freebsd.org> wrote:
> on 07/12/2010 00:00 John Baldwin said the following:
>> It is probably the 4K logical sector size that needs to
>> come up with a new field, not vice versa.
>
> Just expressing my overall confusion - 4K would be the physical size and 512
> would be the logical one?  My thinking: on a platter it's a 4K sector, but drive
> supports data addressing with 512 byte granularity.

Yes, the whole thing is overly complicated by the need for 512/4096
kludge; The cleanest would be to simply go with One True Sector Size
(4 KiB) for the drive which support it but that would break
compatibility all over the place (including booting).

Upthread I've advocated adding an "physsectorsize" field for this, but
basically I'm fine either way.