Re: [PATCH 04/13] xfs: don't run the sync work if the filesyste is read-only

Mark Tinguely
2012-09-04T16:13:02Z

On 08/30/12 07:00, Dave Chinner wrote:
> From: Dave Chinner<dchinner@redhat.com>
>
> If the filesytem is mounted or remounted read-only, stop the sync
> worker that tries to flush or cover the log if the filesystem is
> dirty. It's read-only, so it isn't dirty. Restart it on a remount,rw
> as necessary. This avoids the need for RO checks in the work.
>
> Similarly, stop the sync work when the filesystem is frozen, and
> start it again when the filesysetm is thawed. This avoids the need
> for special freeze checks in the work.
>
> Signed-off-by: Dave Chinner<dchinner@redhat.com>

Looks good.

Reviewed-by: Mark Tinguely <tinguely@sgi.com>