Re: [patch v4 04/13] [PATCH 04/13] xfs: sync work is now only periodic l

xfs

Re: [patch v4 04/13] [PATCH 04/13] xfs: sync work is now only periodic l

To:	Christoph Hellwig <hch@xxxxxxxxxxxxx>
Subject:	Re: [patch v4 04/13] [PATCH 04/13] xfs: sync work is now only periodic log work
From:	Mark Tinguely <tinguely@xxxxxxx>
Date:	Fri, 05 Oct 2012 13:31:40 -0500
Cc:	Ben Myers <bpm@xxxxxxx>, Dave Chinner <dchinner@xxxxxxxxxx>, xfs@xxxxxxxxxxx
In-reply-to:	<20121005181645.GB1056@xxxxxxxxxxxxx>
References:	<20121005171853.985930109@xxxxxxx> <20121005171945.888412779@xxxxxxx> <20121005181645.GB1056@xxxxxxxxxxxxx>
User-agent:	Mozilla/5.0 (X11; FreeBSD amd64; rv:9.0) Gecko/20120122 Thunderbird/9.0

On 10/05/12 13:16, Christoph Hellwig wrote:

On Fri, Oct 05, 2012 at 12:18:57PM -0500, Ben Myers wrote:

                ASSERT(mp->m_flags&  XFS_MOUNT_RDONLY);
        }

+       xfs_log_work_queue(mp);
+
        return error;


I still think queueing the work item here if we return a failure is
the wrong thing to do.

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs

You are correct. I did not see that it was not moved when mentioned inthe first series.


--Mark.

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[patch v4 00/13] xfs: remove the xfssyncd mess, Ben Myers [patch v4 06/13] [PATCH 06/13] xfs: xfs_sync_data is redundant., Ben Myers Re: [patch v4 06/13] [PATCH 06/13] xfs: xfs_sync_data is redundant., Mark Tinguely Re: [patch v4 06/13] [PATCH 06/13] xfs: xfs_sync_data is redundant., Ben Myers Re: [patch v4 06/13] [PATCH 06/13] xfs: xfs_sync_data is redundant., Christoph Hellwig Re: [patch v4 06/13] [PATCH 06/13] xfs: xfs_sync_data is redundant., Mark Tinguely [patch v4 04/13] [PATCH 04/13] xfs: sync work is now only periodic log work, Ben Myers Re: [patch v4 04/13] [PATCH 04/13] xfs: sync work is now only periodic log work, Christoph Hellwig Re: [patch v4 04/13] [PATCH 04/13] xfs: sync work is now only periodic log work, Mark Tinguely <= [patch v4 07/13] [PATCH 07/13] xfs: syncd workqueue is no more, Ben Myers Re: [patch v4 07/13] [PATCH 07/13] xfs: syncd workqueue is no more, Mark Tinguely [patch v4 08/13] [PATCH 08/13] xfs: xfs_sync_fsdata is redundant, Ben Myers [patch v4 10/13] [PATCH 10/13] xfs: xfs_quiesce_attr() should quiesce the log like, Ben Myers [patch v4 05/13] [PATCH 05/13] xfs: Bring some sanity to log unmounting, Ben Myers [patch v4 09/13] [PATCH 09/13] xfs: move xfs_quiesce_attr() into xfs_super.c, Ben Myers [patch v4 11/13] [PATCH 11/13] xfs: rename xfs_sync.[ch] to xfs_icache.[ch], Ben Myers [patch v4 02/13] [PATCH 02/13] xfs: rationalise xfs_mount_wq users, Ben Myers [patch v4 03/13] [PATCH 03/13] xfs: dont run the sync work if the filesystem is, Ben Myers [patch v4 12/13] [PATCH 12/13] xfs: move inode locking functions to xfs_inode.c, Ben Myers

Previous by Date:	Re: [patch v4 04/13] [PATCH 04/13] xfs: sync work is now only periodic log work, Christoph Hellwig
Next by Date:	Re: INTERNATIONAL SWIFT ATM CARD in your name, UNITED PARCEL SERVICE
Previous by Thread:	Re: [patch v4 04/13] [PATCH 04/13] xfs: sync work is now only periodic log work, Christoph Hellwig
Next by Thread:	[patch v4 07/13] [PATCH 07/13] xfs: syncd workqueue is no more, Ben Myers
Indexes:	[Date] [Thread] [Top] [All Lists]