Re: Configure with thread sanitizer fails the thread test

Поиск
Список
Период
Сортировка
От Alvaro Herrera
Тема Re: Configure with thread sanitizer fails the thread test
Дата
Msg-id 202107231735.yhuvz7hfgkun@alvherre.pgsql
обсуждение исходный текст
Ответ на Configure with thread sanitizer fails the thread test  (Ewan Higgs <ewan_higgs@yahoo.co.uk>)
Ответы Re: Configure with thread sanitizer fails the thread test
Список pgsql-hackers
On 2021-Jul-23, Mikhail Matrosov wrote:

> I am not sure what is the proper fix for the issue, but I at least suggest
> to disable this test when building with thread sanitizer:
>
https://github.com/conan-io/conan-center-index/pull/6472/files#diff-b8639f81e30f36c8ba29a0878f1ef4d9f1552293bc9098ebb9b429ddb1f0935f

Here's the proposed patch.  Patches posted to the mailing list by their
authors proposed for inclusion are considered to be under the PostgreSQL
license, but this patch hasn't been posted by the author so let's assume
they're not authorizing them to use it.  (Otherwise, why wouldn't they
just post it here instead of doing the absurdly convoluted dance of a
github PR?)

diff --git a/src/test/thread/thread_test.c b/src/test/thread/thread_test.c
index e1bec01..e4ffd78 100644
--- a/src/test/thread/thread_test.c
+++ b/src/test/thread/thread_test.c
@@ -61,6 +61,14 @@ main(int argc, char *argv[])
     fprintf(stderr, "Perhaps rerun 'configure' using '--enable-thread-safety'.\n");
     return 1;
 }
+
+#elif __has_feature(thread_sanitizer)
+int
+main(int argc, char *argv[])
+{
+    printf("Thread safety check is skipped since it does not work with thread sanitizer.\n");
+    return 0;
+}
 #else
 
 /* This must be down here because this is the code that uses threads. */

-- 
Álvaro Herrera



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Stephen Frost
Дата:
Сообщение: Re: Delegating superuser tasks to new security roles (Was: Granting control of SUSET gucs to non-superusers)
Следующее
От: Alvaro Herrera
Дата:
Сообщение: Re: Configure with thread sanitizer fails the thread test