Re: [patch v4 07/13] [PATCH 07/13] xfs: syncd workqueue is no more

To:	Ben Myers <bpm@xxxxxxx>
Subject:	Re: [patch v4 07/13] [PATCH 07/13] xfs: syncd workqueue is no more
From:	Mark Tinguely <tinguely@xxxxxxx>
Date:	Fri, 05 Oct 2012 12:59:47 -0500
Cc:	Dave Chinner <david@xxxxxxxxxxxxx>, xfs@xxxxxxxxxxx, Dave Chinner <dchinner@xxxxxxxxxx>
In-reply-to:	<20121005171946.545918929@xxxxxxx>
References:	<20121005171853.985930109@xxxxxxx> <20121005171946.545918929@xxxxxxx>
User-agent:	Mozilla/5.0 (X11; FreeBSD amd64; rv:9.0) Gecko/20120122 Thunderbird/9.0

On 10/05/12 12:19, Ben Myers wrote:

From: Dave Chinner<dchinner@xxxxxxxxxx>

With the syncd functions moved to the log and/or removed, the syncd
workqueue is the only remaining bit left. It is used by the log
covering/ail pushing work, as well as by the inode reclaim work.

Given how cheap workqueues are these days, give the log and inode
reclaim work their own work queues and kill the syncd work queue.

Signed-off-by: Dave Chinner<dchinner@xxxxxxxxxx>
---


Looks good.

Reviewed-by: Mark Tinguely <tinguely@xxxxxxx>

<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 [patch v4 13/13] [PATCH 13/13] xfs: remove xfs_iget.c, Ben Myers [patch v4 01/13] [PATCH 01/13] xfs: xfs_syncd_stop must die, Ben Myers

Previous by Date:	Re: [patch v4 06/13] [PATCH 06/13] xfs: xfs_sync_data is redundant., Mark Tinguely
Next by Date:	Re: [patch v4 06/13] [PATCH 06/13] xfs: xfs_sync_data is redundant., Ben Myers
Previous by Thread:	[patch v4 07/13] [PATCH 07/13] xfs: syncd workqueue is no more, Ben Myers
Next by Thread:	[patch v4 08/13] [PATCH 08/13] xfs: xfs_sync_fsdata is redundant, Ben Myers
Indexes:	[Date] [Thread] [Top] [All Lists]