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 | |
| 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>
| -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; |
