Was out of town, not sure if you got a review or not. The fix looks alright to me.
jim Pavel Filipensky wrote: > Anyone interested in reviewing this fix for a surprising bug? > > Thanks, > Pavel > > On 04/09/09 17:08, Pavel Filipensky wrote: > >> Hi, >> >> I am asking for a code review of my fix for 6822384. >> >> Webrev: >> http://cr.opensolaris.org/~pavelf/6822384 >> >> >> There is a general problem with locking strategy for sv_name and >> sv_dfh. See "6827805 insufficient locking around svnode_t::sv_name >> can panic NFSv4 client" for details. This fix is not addressing the >> locking problems, it will be addressed separately in 6827805. >> >> Thanks, >> Pavel >> >> _______________________________________________ >> nfs-discuss mailing list >> nfs-discuss at opensolaris.org > > > _______________________________________________ > nfs-discuss mailing list > nfs-discuss at opensolaris.org
