2011-05-31 01:12:52

by Kyungmin Park

Subject: [PATCH] CFQ: Fix typo and remove unnecessary semicolon

From: Kyungmin Park <[email protected]>

Fix comment typo and remove unnecessary semicolon at macro

Signed-off-by: Kyungmin Park <[email protected]>
---
diff --git a/block/cfq-iosched.c b/block/cfq-iosched.c
index 7c52d68..8a02c95 100644
--- a/block/cfq-iosched.c
+++ b/block/cfq-iosched.c
@@ -185,7 +185,7 @@ struct cfq_group {
int nr_cfqq;

/*
- * Per group busy queus average. Useful for workload slice calc. We
+ * Per group busy queues average. Useful for workload slice calc. We
* create the array for each prio class but at run time it is used
* only for RT and BE class and slot for IDLE class remains unused.
* This is primarily done to avoid confusion and a gcc warning.
@@ -369,16 +369,16 @@ CFQ_CFQQ_FNS(wait_busy);
#define cfq_log_cfqq(cfqd, cfqq, fmt, args...) \
blk_add_trace_msg((cfqd)->queue, "cfq%d%c %s " fmt, (cfqq)->pid, \
cfq_cfqq_sync((cfqq)) ? 'S' : 'A', \
- blkg_path(&(cfqq)->cfqg->blkg), ##args);
+ blkg_path(&(cfqq)->cfqg->blkg), ##args)

#define cfq_log_cfqg(cfqd, cfqg, fmt, args...) \
blk_add_trace_msg((cfqd)->queue, "%s " fmt, \
- blkg_path(&(cfqg)->blkg), ##args); \
+ blkg_path(&(cfqg)->blkg), ##args) \

#else
#define cfq_log_cfqq(cfqd, cfqq, fmt, args...) \
blk_add_trace_msg((cfqd)->queue, "cfq%d " fmt, (cfqq)->pid, ##args)
-#define cfq_log_cfqg(cfqd, cfqg, fmt, args...) do {} while (0);
+#define cfq_log_cfqg(cfqd, cfqg, fmt, args...) do {} while (0)
#endif
#define cfq_log(cfqd, fmt, args...) \
blk_add_trace_msg((cfqd)->queue, "cfq " fmt, ##args)

2011-05-31 08:05:07

by Jens Axboe

Subject: Re: [PATCH] CFQ: Fix typo and remove unnecessary semicolon

On 2011-05-31 03:12, Kyungmin Park wrote:
> From: Kyungmin Park <[email protected]>
>
> Fix comment typo and remove unnecessary semicolon at macro

Thanks, applied.

--
Jens Axboe