----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/155/#review471 -----------------------------------------------------------
Ummm, what's all of this extra stuff, and what does it have to do with a putback buddy to #5531? - Dan McDonald On Feb. 4, 2015, 2:03 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:03 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 > usr/src/uts/common/fs/nfs/nfs4_srv_readdir.c > 3069a988353701ea05bf0a2b77d5878d806eb2dd > usr/src/uts/common/fs/nfs/nfs4_srv.c > 127d9e3f29fe5d9678c321aea5c2ebbcc59d83f3 > usr/src/uts/common/fs/lookup.c 6819509d004e73b5969c554f85c662cb574094df > usr/src/cmd/mdb/common/modules/zfs/zfs.c > 225cf3dee18e110149ce6c7180fa50a3928893f9 > usr/src/cmd/cpc/common/cpustat.c 8d405bb6255a23295e2f39af3921671f08550ac4 > > Diff: https://reviews.csiden.org/r/155/diff/ > > > Testing > ------- > > ztest on illumos. > > > Thanks, > > Justin Gibbs > >
_______________________________________________ developer mailing list developer@open-zfs.org http://lists.open-zfs.org/mailman/listinfo/developer
