Re: [PATCH 05/15] xfs: replace dop_low with transaction flag

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jul 31, 2018 at 01:16:59AM -0700, Christoph Hellwig wrote:
> > +	/*
> > +	 * Low free space mode was historically controlled by a dfops field.
> > +	 * This meant that low mode state potentially carried across multiple
> > +	 * transaction rolls. Transfer low mode on a dfops move to preserve
> > +	 * that behavior.
> > +	 */
> > +	dtp->t_flags |= stp->t_flags & XFS_TRANS_LOWMODE;
> 
> Maybe add parentheses here for clarity.
> 

Added, thanks.

Brian

> Except for that:
> 
> Reviewed-by: Christoph Hellwig <hch@xxxxxx>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-xfs" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [XFS Filesystem Development (older mail)]     [Linux Filesystem Development]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux RAID]     [Linux SCSI]


  Powered by Linux