diff options
author | Mikulas Patocka <mpatocka@redhat.com> | 2013-07-10 18:41:18 -0400 |
---|---|---|
committer | Alasdair G Kergon <agk@redhat.com> | 2013-07-10 18:41:18 -0400 |
commit | 2a7faeb176fb0478bc6a0cb380c658b32179ead0 (patch) | |
tree | 25916d4ccb6887890e5e1d2a18cb1093863d7f32 /drivers/md/dm.c | |
parent | 83d5e5b0af907d46d241a86d9e44003b3f0accbd (diff) |
dm: optimize reorder structure
This reorder actually improves performance by 20% (from 39.1s to 32.8s)
on x86-64 quad core Opteron.
I have no explanation for this, possibly it makes some other entries are
better cache-aligned.
Signed-off-by: Mikulas Patocka <mpatocka@redhat.com>
Signed-off-by: Alasdair G Kergon <agk@redhat.com>
Diffstat (limited to 'drivers/md/dm.c')
-rw-r--r-- | drivers/md/dm.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/drivers/md/dm.c b/drivers/md/dm.c index ecff83f5b53a..9e39d2b64bf8 100644 --- a/drivers/md/dm.c +++ b/drivers/md/dm.c | |||
@@ -133,6 +133,13 @@ struct mapped_device { | |||
133 | atomic_t holders; | 133 | atomic_t holders; |
134 | atomic_t open_count; | 134 | atomic_t open_count; |
135 | 135 | ||
136 | /* | ||
137 | * The current mapping. | ||
138 | * Use dm_get_live_table{_fast} or take suspend_lock for | ||
139 | * dereference. | ||
140 | */ | ||
141 | struct dm_table *map; | ||
142 | |||
136 | unsigned long flags; | 143 | unsigned long flags; |
137 | 144 | ||
138 | struct request_queue *queue; | 145 | struct request_queue *queue; |
@@ -162,13 +169,6 @@ struct mapped_device { | |||
162 | struct workqueue_struct *wq; | 169 | struct workqueue_struct *wq; |
163 | 170 | ||
164 | /* | 171 | /* |
165 | * The current mapping. | ||
166 | * Use dm_get_live_table{_fast} or take suspend_lock for | ||
167 | * dereference. | ||
168 | */ | ||
169 | struct dm_table *map; | ||
170 | |||
171 | /* | ||
172 | * io objects are allocated from here. | 172 | * io objects are allocated from here. |
173 | */ | 173 | */ |
174 | mempool_t *io_pool; | 174 | mempool_t *io_pool; |