diff options
author | Vivek Goyal <vgoyal@redhat.com> | 2011-05-23 13:35:04 -0400 |
---|---|---|
committer | Jens Axboe <jaxboe@fusionio.com> | 2011-05-23 13:35:04 -0400 |
commit | 4cbadbd16e2fb727f6926597e0a580829e6222f1 (patch) | |
tree | d3482b68a6743609cedcd3a5165a749b0d9ca990 /block | |
parent | 7e69723fef8771a9d57bd27d36281d756130b4b5 (diff) |
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>
Diffstat (limited to 'block')
-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 b791022beef3..c898049dafd5 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) | |||
96 | INIT_RADIX_TREE(&ret->radix_root, GFP_ATOMIC | __GFP_HIGH); | 96 | INIT_RADIX_TREE(&ret->radix_root, GFP_ATOMIC | __GFP_HIGH); |
97 | INIT_HLIST_HEAD(&ret->cic_list); | 97 | INIT_HLIST_HEAD(&ret->cic_list); |
98 | ret->ioc_data = NULL; | 98 | ret->ioc_data = NULL; |
99 | #if defined(CONFIG_BLK_CGROUP) || defined(CONFIG_BLK_CGROUP_MODULE) | ||
100 | ret->cgroup_changed = 0; | ||
101 | #endif | ||
99 | } | 102 | } |
100 | 103 | ||
101 | return ret; | 104 | return ret; |