aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVivek Goyal2010-10-22 02:48:43 -0500
committerJens Axboe2010-10-22 02:48:43 -0500
commitb4627321e18582dcbdeb45d77df29d3177107c65 (patch)
treef494dae7f39f219d73aa9752dfd3844293d65f50 /block/cfq-iosched.c
parent11a691bea48887c27425cc40bf291e74c922df25 (diff)
downloadkernel-common-b4627321e18582dcbdeb45d77df29d3177107c65.tar.gz
kernel-common-b4627321e18582dcbdeb45d77df29d3177107c65.tar.xz
kernel-common-b4627321e18582dcbdeb45d77df29d3177107c65.zip
cfq-iosched: Fix a gcc 4.5 warning and put some comments
- Andi encountedred following warning with gcc 4.5 linux/block/cfq-iosched.c: In function ‘cfq_dispatch_requests’: linux/block/cfq-iosched.c:2156:3: warning: array subscript is above array bounds - Warning happens due to following code. slice = group_slice * count / max_t(unsigned, cfqg->busy_queues_avg[cfqd->serving_prio], cfq_group_busy_queues_wl(cfqd->serving_prio, cfqd, cfqg)); gcc is complaining about cfqg->busy_queues_avg[] being indexed by CFQ prio classes (RT, BE and IDLE) while the array size is only 2. - At run time, we never access cfqg->busy_queues_avg[IDLE] and return from function before this code hits. - To fix warning increase the array size though it will remain unused. This patch also puts some comments to clarify some of the confusions. - I have taken Jens's patch and modified it a bit. - Compile tested with gcc 4.4 and boot tested. I don't have gcc 4.5 running, Andi can you please test it with gcc 4.5 to make sure it worked. Reported-by: Andi Kleen <ak@linux.intel.com> Signed-off-by: Vivek Goyal <vgoyal@redhat.com> Acked-by: Jeff Moyer <jmoyer@redhat.com> Signed-off-by: Jens Axboe <jaxboe@fusionio.com>
Diffstat (limited to 'block/cfq-iosched.c')
-rw-r--r--block/cfq-iosched.c16
1 files changed, 13 insertions, 3 deletions
diff --git a/block/cfq-iosched.c b/block/cfq-iosched.c
index 86338d5d4d0..dfceb6386bd 100644
--- a/block/cfq-iosched.c
+++ b/block/cfq-iosched.c
@@ -157,6 +157,7 @@ enum wl_prio_t {
157 BE_WORKLOAD = 0, 157 BE_WORKLOAD = 0,
158 RT_WORKLOAD = 1, 158 RT_WORKLOAD = 1,
159 IDLE_WORKLOAD = 2, 159 IDLE_WORKLOAD = 2,
160 CFQ_PRIO_NR,
160}; 161};
161 162
162/* 163/*
@@ -181,10 +182,19 @@ struct cfq_group {
181 /* number of cfqq currently on this group */ 182 /* number of cfqq currently on this group */
182 int nr_cfqq; 183 int nr_cfqq;
183 184
184 /* Per group busy queus average. Useful for workload slice calc. */
185 unsigned int busy_queues_avg[2];
186 /* 185 /*
187 * rr lists of queues with requests, onle rr for each priority class. 186 * Per group busy queus average. Useful for workload slice calc. We
187 * create the array for each prio class but at run time it is used
188 * only for RT and BE class and slot for IDLE class remains unused.
189 * This is primarily done to avoid confusion and a gcc warning.
190 */
191 unsigned int busy_queues_avg[CFQ_PRIO_NR];
192 /*
193 * rr lists of queues with requests. We maintain service trees for
194 * RT and BE classes. These trees are subdivided in subclasses
195 * of SYNC, SYNC_NOIDLE and ASYNC based on workload type. For IDLE
196 * class there is no subclassification and all the cfq queues go on
197 * a single tree service_tree_idle.
188 * Counts are embedded in the cfq_rb_root 198 * Counts are embedded in the cfq_rb_root
189 */ 199 */
190 struct cfq_rb_root service_trees[2][3]; 200 struct cfq_rb_root service_trees[2][3];