3580 Want zvols to return volblocksize when queried for physical block size

[Christopher Siden]

http://cr.illumos.org/~webrev/csiden/illumos-3580/

Work by George Wilson.

Chris

[Boris Protopopov]

LGTM


On Tue, Nov 26, 2013 at 7:40 PM, Christopher Siden <

Post by Christopher Siden
http://cr.illumos.org/~webrev/csiden/illumos-3580/
Work by George Wilson.
Chris
-------------------------------------------
illumos-zfs
Archives: https://www.listbox.com/member/archive/182191/=now
https://www.listbox.com/member/archive/rss/182191/23052084-8e2408bc
https://www.listbox.com/member/?&
Powered by Listbox: http://www.listbox.com

--
Best regards,

Boris Protopopov
Nexenta Systems

455 El Camino Real, Santa Clara, CA 95050

[d] 408.791.3366 | [c] 978.621.6901
Skype: bprotopopov



-------------------------------------------
illumos-zfs
Archives: https://www.listbox.com/member/archive/182191/=now
RSS Feed: https://www.listbox.com/member/archive/rss/182191/23047029-187a0c8d
Modify Your Subscription: https://www.listbox.com/member/?member_id=23047029&id_secret=23047029-2e85923f
Powered by Listbox: http://www.listbox.com

[Jorgen Lundman]

I don't that much say, but seems fine. Amusingly, we already had to
implement this for OSX.


https://github.com/zfs-osx/zfs/blob/master/module/zfs/zvol.c#L1968

Lund

Post by Boris Protopopov
LGTM
On Tue, Nov 26, 2013 at 7:40 PM, Christopher Siden
http://cr.illumos.org/~webrev/csiden/illumos-3580/
Work by George Wilson.
Chris
-------------------------------------------
illumos-zfs
Archives: https://www.listbox.com/member/archive/182191/=now
https://www.listbox.com/member/archive/rss/182191/23052084-8e2408bc
https://www.listbox.com/member/?member_id=23052084&id_secret=23052084-f99c1c78
Powered by Listbox: http://www.listbox.com
--
Best regards,
Boris Protopopov
Nexenta Systems
455 El Camino Real, Santa Clara, CA 95050
[d] 408.791.3366 | [c] 978.621.6901
Skype: bprotopopov
_______________________________________________
developer mailing list
http://lists.open-zfs.org/mailman/listinfo/developer

[Garrett D'Amore]

LGTM. (Didn't I review this already somewhere?)


On Tue, Nov 26, 2013 at 4:40 PM, Christopher Siden <

Post by Christopher Siden
http://cr.illumos.org/~webrev/csiden/illumos-3580/
Work by George Wilson.
Chris
-------------------------------------------
illumos-zfs
Archives: https://www.listbox.com/member/archive/182191/=now
https://www.listbox.com/member/archive/rss/182191/22035932-85c5d227
https://www.listbox.com/member/?&
Powered by Listbox: http://www.listbox.com

-------------------------------------------
illumos-zfs
Archives: https://www.listbox.com/member/archive/182191/=now
RSS Feed: https://www.listbox.com/member/archive/rss/182191/23047029-187a0c8d
Modify Your Subscription: https://www.listbox.com/member/?member_id=23047029&id_secret=23047029-2e85923f
Powered by Listbox: http://www.listbox.com