From: Vladislav Shpilevoy <v.shpilevoy@tarantool.org> To: tarantool-patches@dev.tarantool.org, korablev@tarantool.org, tsafin@tarantool.org, gorcunov@gmail.com Subject: [Tarantool-patches] [PATCH small 0/2] Aligned lsregion Date: Fri, 15 May 2020 01:31:02 +0200 [thread overview] Message-ID: <cover.1589498880.git.v.shpilevoy@tarantool.org> (raw) The patchset introduces a new function - lsregion_aligned_alloc(). This activity goes in scope of the big issue about lots of 'undefined behaviour' cases among Tarantool sources. One and the biggest of which is unaligned memory access. Currently lsregion is used in vinyl for statements and BPS tree blocks. All of them are not aligned. Addresses are absolutely arbitrary, sometimes even odd, like ending with 1, 9, etc. Not counting UB, this is also slower, may be much slower sometimes. Branch: http://github.com/tarantool/small/tree/gerold103/aligned-lsregion Issue: https://github.com/tarantool/tarantool/issues/4609 Vladislav Shpilevoy (2): lsregion: introduce lsregion_reserve() lsregion: provide aligned version of alloc small/lsregion.c | 12 ++----- small/lsregion.h | 78 +++++++++++++++++++++++++++++++++--------- test/lsregion.c | 88 +++++++++++++++++++++++++++++++++++++++++++++++- 3 files changed, 152 insertions(+), 26 deletions(-) -- 2.21.1 (Apple Git-122.3)
next reply other threads:[~2020-05-14 23:31 UTC|newest] Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-05-14 23:31 Vladislav Shpilevoy [this message] 2020-05-14 23:31 ` [Tarantool-patches] [PATCH small 1/2] lsregion: introduce lsregion_reserve() Vladislav Shpilevoy 2020-05-15 12:35 ` Aleksandr Lyapunov 2020-05-14 23:31 ` [Tarantool-patches] [PATCH small 2/2] lsregion: provide aligned version of alloc Vladislav Shpilevoy 2020-05-15 13:03 ` Aleksandr Lyapunov 2020-05-15 23:24 ` Vladislav Shpilevoy 2020-05-15 12:26 ` [Tarantool-patches] [PATCH small 0/2] Aligned lsregion Aleksandr Lyapunov 2020-05-15 23:22 ` Vladislav Shpilevoy 2020-05-16 19:09 ` Aleksandr Lyapunov 2020-05-17 13:56 ` Vladislav Shpilevoy
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=cover.1589498880.git.v.shpilevoy@tarantool.org \ --to=v.shpilevoy@tarantool.org \ --cc=gorcunov@gmail.com \ --cc=korablev@tarantool.org \ --cc=tarantool-patches@dev.tarantool.org \ --cc=tsafin@tarantool.org \ --subject='Re: [Tarantool-patches] [PATCH small 0/2] Aligned lsregion' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox