Re: [OpenZFS Developer] Review Request 163: 5630 stale bonus buffer in recycled dnode_t leads to data corruption

2015-02-22 Thread Justin Gibbs
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/163/ --- (Updated Feb. 23, 2015, 5:13 a.m.) Review request for OpenZFS Developer Mailing L

Re: [OpenZFS Developer] Review Request 163: 5630 stale bonus buffer in recycled dnode_t leads to data corruption

2015-02-22 Thread Justin Gibbs
> On Feb. 22, 2015, 9:51 p.m., Will Andrews wrote: > > I'd add a dnode_evict_bonus() to share the bonus removal with > > dnode_evict_dbufs(). But as is, LGTM. Done. - Justin --- This is an automatically generated e-mail. To reply, vis

Re: [OpenZFS Developer] Review Request 163: 5630 stale bonus buffer in recycled dnode_t leads to data corruption

2015-02-22 Thread Will Andrews
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/163/#review531 --- Ship it! I'd add a dnode_evict_bonus() to share the bonus removal with

Re: [OpenZFS Developer] Review Request 163: 5630 stale bonus buffer in recycled dnode_t leads to data corruption

2015-02-22 Thread George Wilson
--- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/163/#review530 --- Ship it! Ship It! - George Wilson On Feb. 21, 2015, 5:34 a.m., Justi