[dm-devel] [dm-6.4 PATCH 6/8] dm bufio: move dm_bufio_client members to avoid spanning cachelines
Mike Snitzer
snitzer at kernel.org
Wed Mar 22 18:18:43 UTC 2023
Movement also consolidates holes in dm_bufio_client struct. But the
overall size of the struct isn't changed.
Signed-off-by: Mike Snitzer <snitzer at kernel.org>
---
drivers/md/dm-bufio.c | 24 +++++++++++++-----------
1 file changed, 13 insertions(+), 11 deletions(-)
diff --git a/drivers/md/dm-bufio.c b/drivers/md/dm-bufio.c
index 56ef160331ce..b418ee8e3853 100644
--- a/drivers/md/dm-bufio.c
+++ b/drivers/md/dm-bufio.c
@@ -952,13 +952,16 @@ static void cache_remove_range(struct buffer_cache *bc,
* context.
*/
struct dm_bufio_client {
- struct mutex lock;
- spinlock_t spinlock;
- bool no_sleep;
-
struct block_device *bdev;
unsigned int block_size;
s8 sectors_per_block_bits;
+
+ bool no_sleep;
+ struct mutex lock;
+ spinlock_t spinlock;
+
+ int async_write_error;
+
void (*alloc_callback)(struct dm_buffer *buf);
void (*write_callback)(struct dm_buffer *buf);
struct kmem_cache *slab_buffer;
@@ -970,23 +973,22 @@ struct dm_bufio_client {
unsigned int minimum_buffers;
- struct buffer_cache cache;
- wait_queue_head_t free_buffer_wait;
-
sector_t start;
- int async_write_error;
-
- struct list_head client_list;
-
struct shrinker shrinker;
struct work_struct shrink_work;
atomic_long_t need_shrink;
+ wait_queue_head_t free_buffer_wait;
+
+ struct list_head client_list;
+
/*
* Used by global_cleanup to sort the clients list.
*/
unsigned long oldest_buffer;
+
+ struct buffer_cache cache;
};
static DEFINE_STATIC_KEY_FALSE(no_sleep_enabled);
--
2.40.0
More information about the dm-devel
mailing list