Re: [OpenZFS Developer] Review Request 259: ZFS indirection tables prefetch

2015-10-23 Thread Matthew Ahrens
> On Oct. 14, 2015, 5:36 a.m., Matthew Ahrens wrote: > > usr/src/uts/common/fs/zfs/dmu.c, lines 444-447 > > > > > > I don't see that DNODE_IS_CACHEABLE has ever existed in illumos. > > Alexander Motin wrote: > I've

Re: [OpenZFS Developer] Review Request 259: ZFS indirection tables prefetch

2015-10-14 Thread Alexander Motin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/259/ --- (Updated Oct. 14, 2015, 6:35 a.m.) Review request for OpenZFS Developer Mailing

Re: [OpenZFS Developer] Review Request 259: ZFS indirection tables prefetch

2015-10-13 Thread Matthew Ahrens
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/259/#review833 --- usr/src/uts/common/fs/zfs/dmu.c (lines 444 - 447)

[OpenZFS Developer] Review Request 259: ZFS indirection tables prefetch

2015-10-12 Thread Alexander Motin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/259/ --- Review request for OpenZFS Developer Mailing List and Christopher Siden. Bugs:

Re: [OpenZFS Developer] Review Request 259: ZFS indirection tables prefetch

2015-10-12 Thread Steven Hartland
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/259/#review827 --- Ship it! Looks good to me. - Steven Hartland On Oct. 12, 2015, 4:33