Skip to content

Commit

Permalink
mem_slab: enforce minimum alignment on statically allocated slabs
Browse files Browse the repository at this point in the history
There is no point allowing smaller alignments. And on 64-bit systems the
minimum becomes 8 rather than 4, so let's adjust things automatically.

Signed-off-by: Nicolas Pitre <npitre@baylibre.com>
  • Loading branch information
Nicolas Pitre authored and nashif committed Jun 20, 2019
1 parent bc30f4f commit 46cd5a0
Showing 1 changed file with 5 additions and 4 deletions.
9 changes: 5 additions & 4 deletions include/kernel.h
Original file line number Diff line number Diff line change
Expand Up @@ -3912,11 +3912,11 @@ struct k_mem_slab {
* @req K-MSLAB-001
*/
#define K_MEM_SLAB_DEFINE(name, slab_block_size, slab_num_blocks, slab_align) \
char __noinit __aligned(slab_align) \
_k_mem_slab_buf_##name[(slab_num_blocks) * (slab_block_size)]; \
char __noinit __aligned(WB_UP(slab_align)) \
_k_mem_slab_buf_##name[(slab_num_blocks) * WB_UP(slab_block_size)]; \
Z_STRUCT_SECTION_ITERABLE(k_mem_slab, name) = \
_K_MEM_SLAB_INITIALIZER(name, _k_mem_slab_buf_##name, \
slab_block_size, slab_num_blocks)
WB_UP(slab_block_size), slab_num_blocks)

/**
* @brief Initialize a memory slab.
Expand All @@ -3925,7 +3925,8 @@ struct k_mem_slab {
*
* The memory slab's buffer contains @a slab_num_blocks memory blocks
* that are @a slab_block_size bytes long. The buffer must be aligned to an
* N-byte boundary, where N is a power of 2 larger than 2 (i.e. 4, 8, 16, ...).
* N-byte boundary matching a word boundary, where N is a power of 2
* (i.e. 4 on 32-bit systems, 8, 16, ...).
* To ensure that each memory block is similarly aligned to this boundary,
* @a slab_block_size must also be a multiple of N.
*
Expand Down

0 comments on commit 46cd5a0

Please sign in to comment.