> On Feb. 4, 2015, 2:05 a.m., Dan McDonald wrote: > > Ummm, what's all of this extra stuff, and what does it have to do with a > > putback buddy to #5531?
I believe the "extra stuff" was due to a 'git pull --rebase' I did prior to starting my build and illumos ZFS test suite run. rbt decided to pull in those additional commits into this review. :-( I noticed it just after hitting publish, but fixed it too late for you not to notice the mistake. - Justin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/155/#review471 ----------------------------------------------------------- On Feb. 4, 2015, 2:06 a.m., Justin Gibbs wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.csiden.org/r/155/ > ----------------------------------------------------------- > > (Updated Feb. 4, 2015, 2:06 a.m.) > > > Review request for OpenZFS Developer Mailing List. > > > Bugs: 5531 > https://www.illumos.org/projects/illumos-gate//issues/5531 > > > Repository: illumos-gate > > > Description > ------- > > Follow on to /r/153. > > uts/common/fs/zfs/dsl_dataset.c: > Close race in dsl_dataset_try_addref() where the dsl_dataset_t > still points to a valid bonus buffer, but that bonus buffer is > held by a new dsl_dataset_t instance for this dataset. Allow > the addref operation only if the passed in dsl_dataset_t owns > it's referenced bonus buffer - the dbuf user for the bonus > buffer is this dsl_dataset_t. > > > Diffs > ----- > > usr/src/uts/common/fs/zfs/dsl_dataset.c > ba9c766c65f64289b4e5e8c799a04ab3fde65e8e > > Diff: https://reviews.csiden.org/r/155/diff/ > > > Testing > ------- > > ztest on illumos. > > > Thanks, > > Justin Gibbs > >
_______________________________________________ developer mailing list [email protected] http://lists.open-zfs.org/mailman/listinfo/developer
