On Feb 15, 2012, at 8:24 AM, Gordon Ross wrote:

> A recent version of the "ashift" patch can be found here:
>  http://yalms.org/cr/zfs-blksize/

This is a good start, but I'd like to see some different changes, let's discuss.
My list:

        1. separate the changes for format and zpool into two different 
changesets

        2. change the keyword "blocksize" to "physical-blocksize" to help avoid 
confusion
           around the overloaded term, "blocksize"

        3. need to include grub tests (see earlier discussion on grub fixes for 
different asize)

        4. need to explore implications of ashift > 11, is the practical limit 
ashift=13?

        5. should we hardware the slogs to minimum 4KB anyway?

        6. should we set un->un_sys_blocksize to 4KB for ISSSD() drives in sd?

> 
> There are also changes there for the format command to make it do
> megabyte aligned partitions.  With both changes in place, one can
> ensure 4k-aligned access on an illumos partition.

Don't really need a patch, a simple mdb script to change the 
un->un_phy_blocksize
(or similar) prior to pool creation works. To some degree, this is what BSD 
offers with 
the geom utilities. IMNSHO, it would be nice to have a better UI to changing sd 
un
parameters than "vi /kernel/drv/sd.conf; reboot"
 -- richard

> 
> Gordon
> 
> On Wed, Feb 15, 2012 at 8:22 AM, Alasdair Lumsden <[email protected]> 
> wrote:
>> Hi Roy,
>> 
>>> Any idea how much code this is? Or is it the old patch that was rejected a 
>>> year ago or so?
>> 
>> It is indeed that patch.
>> 
>> Illumos wasn't prepared to integrate it, but potentially OI could, as a 
>> workaround until Illumos provide whatever solution they had in mind.
>> 
>> I'll try and get hold of the code.
>> 
>> Regards,
>> 
>> Alasdair
>> 
>> -------------------------------------------
>> illumos-discuss
>> Archives: https://www.listbox.com/member/archive/182180/=now
>> RSS Feed: https://www.listbox.com/member/archive/rss/182180/21175550-b1877313
>> Modify Your Subscription: https://www.listbox.com/member/?&;
>> Powered by Listbox: http://www.listbox.com
> 
> 
> -------------------------------------------
> illumos-discuss
> Archives: https://www.listbox.com/member/archive/182180/=now
> RSS Feed: https://www.listbox.com/member/archive/rss/182180/21175743-23d1427b
> Modify Your Subscription: https://www.listbox.com/member/?&;
> Powered by Listbox: http://www.listbox.com

--
DTrace Conference, April 3, 2012, 
http://wiki.smartos.org/display/DOC/dtrace.conf
ZFS Performance and Training
[email protected]
+1-760-896-4422






-------------------------------------------
illumos-discuss
Archives: https://www.listbox.com/member/archive/182180/=now
RSS Feed: https://www.listbox.com/member/archive/rss/182180/21175430-2e6923be
Modify Your Subscription: 
https://www.listbox.com/member/?member_id=21175430&id_secret=21175430-6a77cda4
Powered by Listbox: http://www.listbox.com

Reply via email to