From: Vladislav Shpilevoy <v.shpilevoy@tarantool.org> To: tarantool-patches@dev.tarantool.org, korablev@tarantool.org, tsafin@tarantool.org, alyapunov@tarantool.org, gorcunov@gmail.com Subject: [Tarantool-patches] [PATCH 04/10] crc32: disable align sanitizer Date: Thu, 21 May 2020 22:37:27 +0200 [thread overview] Message-ID: <c5a809ba0d4331c597fd55e30dcf8f012fdbd68f.1590093222.git.v.shpilevoy@tarantool.org> (raw) In-Reply-To: <cover.1590093222.git.v.shpilevoy@tarantool.org> There is some assembly working with a byte array like with an array of unsigned long values. Better allow it to continue working like that with disabled sanitizer, than accidentally break or slow down something here. Part of #4609 --- I am sure there is a way to fix it instead of muting, but I don't know how. Probably there is someone, who understands what is written in this assembly code, and can help. src/cpu_feature.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/cpu_feature.c b/src/cpu_feature.c index 98567ccb3..09b6c84ee 100644 --- a/src/cpu_feature.c +++ b/src/cpu_feature.c @@ -29,6 +29,7 @@ * SUCH DAMAGE. */ #include "trivia/config.h" +#include "trivia/util.h" #include <sys/types.h> #include <errno.h> #include <stdlib.h> @@ -65,7 +66,7 @@ crc32c_hw_byte(uint32_t crc, unsigned char const *data, unsigned int length) } -uint32_t +NOSANITIZE_ALIGN uint32_t crc32c_hw(uint32_t crc, const char *buf, unsigned int len) { unsigned int iquotient = len / SCALE_F; -- 2.21.1 (Apple Git-122.3)
next prev parent reply other threads:[~2020-05-21 20:37 UTC|newest] Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-05-21 20:37 [Tarantool-patches] [PATCH 00/10] Sanitize unaligned access Vladislav Shpilevoy 2020-05-21 20:37 ` [Tarantool-patches] [PATCH 01/10] small: sanitized rlist and new region API Vladislav Shpilevoy 2020-06-08 12:17 ` Cyrill Gorcunov 2020-05-21 20:37 ` [Tarantool-patches] [PATCH 10/10] xrow: use unaligned store operation in xrow_to_iovec() Vladislav Shpilevoy 2020-06-08 12:26 ` Cyrill Gorcunov 2020-05-21 20:37 ` [Tarantool-patches] [PATCH 02/10] cmake: ignore warnings on alignof() and offsetof() Vladislav Shpilevoy 2020-06-08 12:52 ` Cyrill Gorcunov 2020-05-21 20:37 ` [Tarantool-patches] [PATCH 03/10] cmake: add option ENABLE_UB_SANITIZER Vladislav Shpilevoy 2020-06-08 12:53 ` Cyrill Gorcunov 2020-05-21 20:37 ` Vladislav Shpilevoy [this message] 2020-06-08 13:58 ` [Tarantool-patches] [PATCH 04/10] crc32: disable align sanitizer Cyrill Gorcunov 2020-05-21 20:37 ` [Tarantool-patches] [PATCH 05/10] sql: make BtCursor's memory aligned Vladislav Shpilevoy 2020-06-08 13:58 ` Cyrill Gorcunov 2020-05-21 20:37 ` [Tarantool-patches] [PATCH 06/10] region: use aligned allocations where necessary Vladislav Shpilevoy 2020-06-08 14:00 ` Cyrill Gorcunov 2020-05-21 20:37 ` [Tarantool-patches] [PATCH 07/10] vinyl: align statements and bps tree extents Vladislav Shpilevoy 2020-06-08 14:02 ` Cyrill Gorcunov 2020-05-21 20:37 ` [Tarantool-patches] [PATCH 08/10] tuple: use unaligned store-load for field map Vladislav Shpilevoy 2020-06-08 14:04 ` Cyrill Gorcunov 2020-05-21 20:37 ` [Tarantool-patches] [PATCH 09/10] port: make port_c_entry not PACKED Vladislav Shpilevoy 2020-06-08 14:04 ` Cyrill Gorcunov 2020-05-21 22:25 ` [Tarantool-patches] [PATCH 00/10] Sanitize unaligned access Sergey Bronnikov 2020-05-27 23:33 ` 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=c5a809ba0d4331c597fd55e30dcf8f012fdbd68f.1590093222.git.v.shpilevoy@tarantool.org \ --to=v.shpilevoy@tarantool.org \ --cc=alyapunov@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 04/10] crc32: disable align sanitizer' \ /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