From 0cfb8077c520ed8e188fb88687bcd1000014147f Mon Sep 17 00:00:00 2001 From: Adam Rutkowski Date: Fri, 26 Aug 2022 06:47:26 +0200 Subject: [PATCH] allocate fixed map status alongside request struct It is wastefull to allocate a full 1B to store 1 bit of alock status per cacheline. Fixed allocation of 128 bits seems more reasonable. Signed-off-by: Adam Rutkowski --- src/ocf_request.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/src/ocf_request.c b/src/ocf_request.c index bf7d8a9..cb82561 100644 --- a/src/ocf_request.c +++ b/src/ocf_request.c @@ -59,8 +59,13 @@ static inline size_t ocf_req_sizeof_alock_status(uint32_t lines) int ocf_req_allocator_init(struct ocf_ctx *ocf_ctx) { - ocf_ctx->resources.req = env_mpool_create(sizeof(struct ocf_request), - sizeof(struct ocf_map_info) + sizeof(uint8_t), ENV_MEM_NORMAL, ocf_req_size_128, + enum ocf_req_size max_req_size = ocf_req_size_128; + size_t alock_status_size = ocf_req_sizeof_alock_status( + (1U << (unsigned)max_req_size)); + size_t header_size = sizeof(struct ocf_request) + alock_status_size; + + ocf_ctx->resources.req = env_mpool_create(header_size, + sizeof(struct ocf_map_info), ENV_MEM_NORMAL, max_req_size, false, NULL, "ocf_req", true); if (ocf_ctx->resources.req == NULL)