diff options
author | Jens Axboe <jens.axboe@oracle.com> | 2008-02-08 06:41:03 -0500 |
---|---|---|
committer | Jens Axboe <jens.axboe@oracle.com> | 2008-02-08 06:41:03 -0500 |
commit | 63a7138671c50a6f2c27bbd1a308dc75967062a3 (patch) | |
tree | 6d8b0d9cbc05bce2848299b61528cabc6d04d267 /include/linux/blkdev.h | |
parent | a4ffc0a0b240a29cbe489f6db9dae112a49ef1c1 (diff) |
block: fixup rq_init() a bit
Rearrange fields in cache order and initialize some fields that
we didn't previously init. Remove init of ->completion_data, it's
part of a union with ->hash. Luckily clearing the rb node is the same
as setting it to null!
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
Diffstat (limited to 'include/linux/blkdev.h')
-rw-r--r-- | include/linux/blkdev.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/include/linux/blkdev.h b/include/linux/blkdev.h index 90392a9d7a9c..e1888cc5b8ae 100644 --- a/include/linux/blkdev.h +++ b/include/linux/blkdev.h | |||
@@ -137,7 +137,9 @@ enum rq_flag_bits { | |||
137 | #define BLK_MAX_CDB 16 | 137 | #define BLK_MAX_CDB 16 |
138 | 138 | ||
139 | /* | 139 | /* |
140 | * try to put the fields that are referenced together in the same cacheline | 140 | * try to put the fields that are referenced together in the same cacheline. |
141 | * if you modify this structure, be sure to check block/blk-core.c:rq_init() | ||
142 | * as well! | ||
141 | */ | 143 | */ |
142 | struct request { | 144 | struct request { |
143 | struct list_head queuelist; | 145 | struct list_head queuelist; |