* [PATCH] xfs: remove unecessary AIL push wakeups
@ 2012-01-12 17:30 ` JiSheng Zhang
0 siblings, 0 replies; 4+ messages in thread
From: JiSheng Zhang @ 2012-01-12 17:30 UTC (permalink / raw)
To: xfs, elder; +Cc: linux-kernel
In commit 0030807c66("xfs: revert to using a kthread for AIL pushing"),
AIL pushing is revert back to use a kthread. But unfortunately, it
introduced a regression: there are unnecessary AIL push wakeups during
idle, ~20 wakeups/s. This change remove the wakeups by reverting a bit
more: set "tout" value as zero during idle.
Signed-off-by: JiSheng Zhang <jszhang3@mail.ustc.edu.cn>
---
fs/xfs/xfs_trans_ail.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/fs/xfs/xfs_trans_ail.c b/fs/xfs/xfs_trans_ail.c
index ed9252b..2014387 100644
--- a/fs/xfs/xfs_trans_ail.c
+++ b/fs/xfs/xfs_trans_ail.c
@@ -508,7 +508,7 @@ out_done:
ailp->xa_last_pushed_lsn = 0;
ailp->xa_log_flush = 0;
- tout = 50;
+ tout = 0;
} else if (XFS_LSN_CMP(lsn, target) >= 0) {
/*
* We reached the target so wait a bit longer for I/O to
--
1.7.8.3
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [PATCH] xfs: remove unecessary AIL push wakeups
@ 2012-01-12 17:30 ` JiSheng Zhang
0 siblings, 0 replies; 4+ messages in thread
From: JiSheng Zhang @ 2012-01-12 17:30 UTC (permalink / raw)
To: xfs, elder; +Cc: linux-kernel
In commit 0030807c66("xfs: revert to using a kthread for AIL pushing"),
AIL pushing is revert back to use a kthread. But unfortunately, it
introduced a regression: there are unnecessary AIL push wakeups during
idle, ~20 wakeups/s. This change remove the wakeups by reverting a bit
more: set "tout" value as zero during idle.
Signed-off-by: JiSheng Zhang <jszhang3@mail.ustc.edu.cn>
---
fs/xfs/xfs_trans_ail.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/fs/xfs/xfs_trans_ail.c b/fs/xfs/xfs_trans_ail.c
index ed9252b..2014387 100644
--- a/fs/xfs/xfs_trans_ail.c
+++ b/fs/xfs/xfs_trans_ail.c
@@ -508,7 +508,7 @@ out_done:
ailp->xa_last_pushed_lsn = 0;
ailp->xa_log_flush = 0;
- tout = 50;
+ tout = 0;
} else if (XFS_LSN_CMP(lsn, target) >= 0) {
/*
* We reached the target so wait a bit longer for I/O to
--
1.7.8.3
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] xfs: remove unecessary AIL push wakeups
2012-01-12 17:30 ` JiSheng Zhang
@ 2012-01-24 18:02 ` Christoph Hellwig
-1 siblings, 0 replies; 4+ messages in thread
From: Christoph Hellwig @ 2012-01-24 18:02 UTC (permalink / raw)
To: JiSheng Zhang; +Cc: xfs, elder, linux-kernel
On Fri, Jan 13, 2012 at 01:30:28AM +0800, JiSheng Zhang wrote:
>
> In commit 0030807c66("xfs: revert to using a kthread for AIL pushing"),
> AIL pushing is revert back to use a kthread. But unfortunately, it
> introduced a regression: there are unnecessary AIL push wakeups during
> idle, ~20 wakeups/s. This change remove the wakeups by reverting a bit
> more: set "tout" value as zero during idle.
>
> Signed-off-by: JiSheng Zhang <jszhang3@mail.ustc.edu.cn>
Looks fine, although I'm a bit worried that I don't have the setup to
reproduce the issue that the patch addressed any more and we might
regress here.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] xfs: remove unecessary AIL push wakeups
@ 2012-01-24 18:02 ` Christoph Hellwig
0 siblings, 0 replies; 4+ messages in thread
From: Christoph Hellwig @ 2012-01-24 18:02 UTC (permalink / raw)
To: JiSheng Zhang; +Cc: elder, linux-kernel, xfs
On Fri, Jan 13, 2012 at 01:30:28AM +0800, JiSheng Zhang wrote:
>
> In commit 0030807c66("xfs: revert to using a kthread for AIL pushing"),
> AIL pushing is revert back to use a kthread. But unfortunately, it
> introduced a regression: there are unnecessary AIL push wakeups during
> idle, ~20 wakeups/s. This change remove the wakeups by reverting a bit
> more: set "tout" value as zero during idle.
>
> Signed-off-by: JiSheng Zhang <jszhang3@mail.ustc.edu.cn>
Looks fine, although I'm a bit worried that I don't have the setup to
reproduce the issue that the patch addressed any more and we might
regress here.
_______________________________________________
xfs mailing list
xfs@oss.sgi.com
http://oss.sgi.com/mailman/listinfo/xfs
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2012-01-24 18:17 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-01-12 17:30 [PATCH] xfs: remove unecessary AIL push wakeups JiSheng Zhang
2012-01-12 17:30 ` JiSheng Zhang
2012-01-24 18:02 ` Christoph Hellwig
2012-01-24 18:02 ` Christoph Hellwig
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.