On Sun, 2010-09-05 at 21:44 -0400, Christoph Hellwig wrote: > In xfs_qm_dqflush we know that q_blkno must be initialized already from a > previous xfs_qm_dqread. So instead of calling xfs_qm_dqtobp we can > simply read the quota buffer directly. This also saves us from a duplicate > xfs_qm_dqcheck call check and allows xfs_qm_dqtobp to be simplified now > that it is always called for a newly initialized inode. In addition to > that properly unwind all locks in xfs_qm_dqflush when xfs_qm_dqcheck > fails. Looks good. Reviewed-by: Alex Elder <aelder@xxxxxxx> > This mirrors a similar cleanup in the inode lookup done earlier. > > Signed-off-by: Christoph Hellwig <hch@xxxxxx> > . . . _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
