Prev: Bugzilla # 16395: Regression in 2.6.35-rc4 since commit 0b28bac5aef7bd1ab213723df031e61db9ff151a on HP Mini 110 Netbook
Next: [PATCH 13/18] xstat: AFS: Use d_automount() rather than abusing follow_link() [ver #6]
From: Gui Jianfeng on 14 Jul 2010 22:10 Delay searching or allocating a cfq group at the very point that we really need it. This change would save some cycles every time we're getting a cfq queue. Signed-off-by: Gui Jianfeng <guijianfeng(a)cn.fujitsu.com> --- block/cfq-iosched.c | 5 +++-- 1 files changed, 3 insertions(+), 2 deletions(-) diff --git a/block/cfq-iosched.c b/block/cfq-iosched.c index eb4086f..9b0cf13 100644 --- a/block/cfq-iosched.c +++ b/block/cfq-iosched.c @@ -2820,10 +2820,8 @@ cfq_find_alloc_queue(struct cfq_data *cfqd, bool is_sync, { struct cfq_queue *cfqq, *new_cfqq = NULL; struct cfq_io_context *cic; - struct cfq_group *cfqg; retry: - cfqg = cfq_get_cfqg(cfqd, 1); cic = cfq_cic_lookup(cfqd, ioc); /* cic always exists here */ cfqq = cic_to_cfqq(cic, is_sync); @@ -2852,6 +2850,9 @@ retry: } if (cfqq) { + struct cfq_group *cfqg; + + cfqg = cfq_get_cfqg(cfqd, 1); cfq_init_cfqq(cfqd, cfqq, current->pid, is_sync); cfq_init_prio_data(cfqq, ioc); cfq_link_cfqq_cfqg(cfqq, cfqg); -- 1.5.4.rc3 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ |