diff options
author | Martin Liska <mliska@suse.cz> | 2022-05-16 14:18:41 +0200 |
---|---|---|
committer | Martin Liska <mliska@suse.cz> | 2022-06-27 10:12:08 +0200 |
commit | 2669cae081c852bc8bde1647d671aa66930cc556 (patch) | |
tree | 7c2cc38e788c80b3a60036a36c3ea39325c7b5f5 /lto-plugin/lto-plugin.c | |
parent | 64d4f27a0ce47e97867512bda7fa5683acf8a134 (diff) | |
download | gcc-2669cae081c852bc8bde1647d671aa66930cc556.zip gcc-2669cae081c852bc8bde1647d671aa66930cc556.tar.gz gcc-2669cae081c852bc8bde1647d671aa66930cc556.tar.bz2 |
lto-plugin: make claim_file_handler thread-safe
lto-plugin/ChangeLog:
* lto-plugin.c (plugin_lock): New lock.
(claim_file_handler): Use mutex for critical section.
(onload): Initialize mutex.
* configure.ac: Check for pthread.h.
* configure: Regenerate.
* config.h.in: Regenerate.
Diffstat (limited to 'lto-plugin/lto-plugin.c')
-rw-r--r-- | lto-plugin/lto-plugin.c | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/lto-plugin/lto-plugin.c b/lto-plugin/lto-plugin.c index 00b7606..635e126 100644 --- a/lto-plugin/lto-plugin.c +++ b/lto-plugin/lto-plugin.c @@ -40,7 +40,11 @@ along with this program; see the file COPYING3. If not see #ifdef HAVE_CONFIG_H #include "config.h" +#if !HAVE_PTHREAD_H +#error POSIX threads are mandatory dependency #endif +#endif + #if HAVE_STDINT_H #include <stdint.h> #endif @@ -55,6 +59,7 @@ along with this program; see the file COPYING3. If not see #include <unistd.h> #include <fcntl.h> #include <sys/types.h> +#include <pthread.h> #ifdef HAVE_SYS_WAIT_H #include <sys/wait.h> #endif @@ -157,6 +162,9 @@ enum symbol_style ss_uscore, /* Underscore prefix all symbols. */ }; +/* Plug-in mutex. */ +static pthread_mutex_t plugin_lock; + static char *arguments_file_name; static ld_plugin_register_claim_file register_claim_file; static ld_plugin_register_all_symbols_read register_all_symbols_read; @@ -1262,15 +1270,18 @@ claim_file_handler (const struct ld_plugin_input_file *file, int *claimed) lto_file.symtab.syms); check (status == LDPS_OK, LDPL_FATAL, "could not add symbols"); + pthread_mutex_lock (&plugin_lock); num_claimed_files++; claimed_files = xrealloc (claimed_files, num_claimed_files * sizeof (struct plugin_file_info)); claimed_files[num_claimed_files - 1] = lto_file; + pthread_mutex_unlock (&plugin_lock); *claimed = 1; } + pthread_mutex_lock (&plugin_lock); if (offload_files == NULL) { /* Add dummy item to the start of the list. */ @@ -1333,11 +1344,14 @@ claim_file_handler (const struct ld_plugin_input_file *file, int *claimed) offload_files_last_lto = ofld; num_offload_files++; } + pthread_mutex_unlock (&plugin_lock); goto cleanup; err: + pthread_mutex_lock (&plugin_lock); non_claimed_files++; + pthread_mutex_unlock (&plugin_lock); free (lto_file.name); cleanup: @@ -1415,6 +1429,12 @@ onload (struct ld_plugin_tv *tv) struct ld_plugin_tv *p; enum ld_plugin_status status; + if (pthread_mutex_init (&plugin_lock, NULL) != 0) + { + fprintf (stderr, "mutex init failed\n"); + abort (); + } + p = tv; while (p->tv_tag) { |