diff options
author | Jens Axboe <jaxboe@fusionio.com> | 2010-06-09 04:42:09 -0400 |
---|---|---|
committer | Jens Axboe <jaxboe@fusionio.com> | 2010-08-07 12:13:00 -0400 |
commit | e2e1a148bc45855816ae6b4692ce29d0020fa22e (patch) | |
tree | fd5ec8a580d4333b471acfe50f6f92b4cc880087 /block/blk-sysfs.c | |
parent | 841fdffdd382722d33579a6aa1487e8a4e526dbd (diff) |
block: add sysfs knob for turning off disk entropy contributions
There are two reasons for doing this:
- On SSD disks, the completion times aren't as random as they
are for rotational drives. So it's questionable whether they
should contribute to the random pool in the first place.
- Calling add_disk_randomness() has a lot of overhead.
This adds /sys/block/<dev>/queue/add_random that will allow you to
switch off on a per-device basis. The default setting is on, so there
should be no functional changes from this patch.
Signed-off-by: Jens Axboe <jaxboe@fusionio.com>
Diffstat (limited to 'block/blk-sysfs.c')
-rw-r--r-- | block/blk-sysfs.c | 28 |
1 files changed, 28 insertions, 0 deletions
diff --git a/block/blk-sysfs.c b/block/blk-sysfs.c index 306759bbdf1b..58b53c354c2c 100644 --- a/block/blk-sysfs.c +++ b/block/blk-sysfs.c | |||
@@ -250,6 +250,27 @@ queue_rq_affinity_store(struct request_queue *q, const char *page, size_t count) | |||
250 | return ret; | 250 | return ret; |
251 | } | 251 | } |
252 | 252 | ||
253 | static ssize_t queue_random_show(struct request_queue *q, char *page) | ||
254 | { | ||
255 | return queue_var_show(blk_queue_add_random(q), page); | ||
256 | } | ||
257 | |||
258 | static ssize_t queue_random_store(struct request_queue *q, const char *page, | ||
259 | size_t count) | ||
260 | { | ||
261 | unsigned long val; | ||
262 | ssize_t ret = queue_var_store(&val, page, count); | ||
263 | |||
264 | spin_lock_irq(q->queue_lock); | ||
265 | if (val) | ||
266 | queue_flag_set(QUEUE_FLAG_ADD_RANDOM, q); | ||
267 | else | ||
268 | queue_flag_clear(QUEUE_FLAG_ADD_RANDOM, q); | ||
269 | spin_unlock_irq(q->queue_lock); | ||
270 | |||
271 | return ret; | ||
272 | } | ||
273 | |||
253 | static ssize_t queue_iostats_show(struct request_queue *q, char *page) | 274 | static ssize_t queue_iostats_show(struct request_queue *q, char *page) |
254 | { | 275 | { |
255 | return queue_var_show(blk_queue_io_stat(q), page); | 276 | return queue_var_show(blk_queue_io_stat(q), page); |
@@ -374,6 +395,12 @@ static struct queue_sysfs_entry queue_iostats_entry = { | |||
374 | .store = queue_iostats_store, | 395 | .store = queue_iostats_store, |
375 | }; | 396 | }; |
376 | 397 | ||
398 | static struct queue_sysfs_entry queue_random_entry = { | ||
399 | .attr = {.name = "add_random", .mode = S_IRUGO | S_IWUSR }, | ||
400 | .show = queue_random_show, | ||
401 | .store = queue_random_store, | ||
402 | }; | ||
403 | |||
377 | static struct attribute *default_attrs[] = { | 404 | static struct attribute *default_attrs[] = { |
378 | &queue_requests_entry.attr, | 405 | &queue_requests_entry.attr, |
379 | &queue_ra_entry.attr, | 406 | &queue_ra_entry.attr, |
@@ -394,6 +421,7 @@ static struct attribute *default_attrs[] = { | |||
394 | &queue_nomerges_entry.attr, | 421 | &queue_nomerges_entry.attr, |
395 | &queue_rq_affinity_entry.attr, | 422 | &queue_rq_affinity_entry.attr, |
396 | &queue_iostats_entry.attr, | 423 | &queue_iostats_entry.attr, |
424 | &queue_random_entry.attr, | ||
397 | NULL, | 425 | NULL, |
398 | }; | 426 | }; |
399 | 427 | ||