----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.csiden.org/r/264/#review842 -----------------------------------------------------------
Ship it! Ship It! - Matthew Ahrens On Oct. 21, 2015, 9:30 p.m., Saso Kiselkov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.csiden.org/r/264/ > ----------------------------------------------------------- > > (Updated Oct. 21, 2015, 9:30 p.m.) > > > Review request for OpenZFS Developer Mailing List and Christopher Siden. > > > Repository: illumos-gate > > > Description > ------- > > If spa_config_tryenter fails to acquire a particular lock, all locks that > have been acquired in the mean time should be released. Unfortunately, the > existing arithmetic in that case doesn't do that, so some locks can be left > acquired. > Fix from Nexenta, originally reviewed internally by Alek Pinchuk and Josef > 'Jeff' Sipek. > > > Diffs > ----- > > usr/src/uts/common/fs/zfs/spa_misc.c > 9148a03433167aebf03f3ebc44771e4ec9c50dd7 > > Diff: https://reviews.csiden.org/r/264/diff/ > > > Testing > ------- > > This is required for ZFS TRIM/UNMAP support, otherwise the code panicks when > a "zpool trim <poolname>" command is issued before the last on-demand TRIM > has finished. > > > Thanks, > > Saso Kiselkov > >
_______________________________________________ developer mailing list developer@open-zfs.org http://lists.open-zfs.org/mailman/listinfo/developer
