I assume that this is just a preliminary review, and before pushing you'll
be adding code to detect if the device is a SSD and set vdev_rotation_rate
to VDEV_RATE_NON_ROTATING?

--matt


On Wed, Nov 20, 2013 at 4:23 PM, Steven Hartland <[email protected]>wrote:

> Webrev updated as per feedback:
> http://cr.illumos.org/~webrev/steveh/illumos-4322/
>
> Changes:
> * Renamed lastoffset vars and methods so they are
>  less like the new lastoffset vars added in the
>  queuing rework.
> * Removed static declaration on tunables.
> * Corrected vdev_queue_length calculation to be
>  compatible with latest multi queue logic.
> * Added requested comments.
>
>
>    Regards
>    Steve
>
>
> ================================================
> 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 [email protected].
>
>
>
> -------------------------------------------
> illumos-zfs
> Archives: https://www.listbox.com/member/archive/182191/=now
> RSS Feed: https://www.listbox.com/member/archive/rss/182191/
> 21635000-ebd1d460
> Modify Your Subscription: https://www.listbox.com/
> member/?member_id=21635000&id_secret=21635000-73dc201a
> Powered by Listbox: http://www.listbox.com
>
_______________________________________________
developer mailing list
[email protected]
http://lists.open-zfs.org/mailman/listinfo/developer

Reply via email to