diff options
| author | Vivek Goyal <vgoyal@redhat.com> | 2011-05-23 19:35:04 +0200 |
|---|---|---|
| committer | Jens Axboe <jaxboe@fusionio.com> | 2011-05-23 19:35:04 +0200 |
| commit | 4cbadbd16e2fb727f6926597e0a580829e6222f1 (patch) | |
| tree | d3482b68a6743609cedcd3a5165a749b0d9ca990 | |
| parent | 7e69723fef8771a9d57bd27d36281d756130b4b5 (diff) | |
| download | op-kernel-dev-4cbadbd16e2fb727f6926597e0a580829e6222f1.zip op-kernel-dev-4cbadbd16e2fb727f6926597e0a580829e6222f1.tar.gz | |
blk-cgroup: Initialize ioc->cgroup_changed at ioc creation time
If we don't explicitly initialize it to zero, CFQ might think that
cgroup of ioc has changed and it generates lots of unnecessary calls
to call_for_each_cic(changed_cgroup). Fix it.
cfq_get_io_context()
cfq_ioc_set_cgroup()
call_for_each_cic(ioc, changed_cgroup)
Signed-off-by: Vivek Goyal <vgoyal@redhat.com>
Signed-off-by: Jens Axboe <jaxboe@fusionio.com>
| -rw-r--r-- | block/blk-ioc.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/block/blk-ioc.c b/block/blk-ioc.c index b791022..c898049 100644 --- a/block/blk-ioc.c +++ b/block/blk-ioc.c @@ -96,6 +96,9 @@ struct io_context *alloc_io_context(gfp_t gfp_flags, int node) INIT_RADIX_TREE(&ret->radix_root, GFP_ATOMIC | __GFP_HIGH); INIT_HLIST_HEAD(&ret->cic_list); ret->ioc_data = NULL; +#if defined(CONFIG_BLK_CGROUP) || defined(CONFIG_BLK_CGROUP_MODULE) + ret->cgroup_changed = 0; +#endif } return ret; |
