Re: [ovs-dev] [PATCHv2] fatal-signal: Fix clang error due to lock.

2020-03-24 Thread William Tu
On Tue, Mar 24, 2020 at 03:38:31PM +0100, Ilya Maximets wrote: > On 3/24/20 3:27 PM, Ilya Maximets wrote: > > On 3/24/20 3:17 PM, William Tu wrote: > >> Due to not acquiring lock, clang reports: > >> lib/vlog.c:618:12: error: reading variable 'log_fd' requires holding > >> mutex > >> 'log_file

Re: [ovs-dev] [PATCHv2] fatal-signal: Fix clang error due to lock.

2020-03-24 Thread Ilya Maximets
On 3/24/20 3:27 PM, Ilya Maximets wrote: > On 3/24/20 3:17 PM, William Tu wrote: >> Due to not acquiring lock, clang reports: >> lib/vlog.c:618:12: error: reading variable 'log_fd' requires holding mutex >> 'log_file_mutex' [-Werror,-Wthread-safety-analysis] >> return log_fd; >> >> The patch

Re: [ovs-dev] [PATCHv2] fatal-signal: Fix clang error due to lock.

2020-03-24 Thread Ilya Maximets
On 3/24/20 3:17 PM, William Tu wrote: > Due to not acquiring lock, clang reports: > lib/vlog.c:618:12: error: reading variable 'log_fd' requires holding mutex > 'log_file_mutex' [-Werror,-Wthread-safety-analysis] > return log_fd; > > The patch fixes it by creating a function in vlog.c to wri

[ovs-dev] [PATCHv2] fatal-signal: Fix clang error due to lock.

2020-03-24 Thread William Tu
Due to not acquiring lock, clang reports: lib/vlog.c:618:12: error: reading variable 'log_fd' requires holding mutex 'log_file_mutex' [-Werror,-Wthread-safety-analysis] return log_fd; The patch fixes it by creating a function in vlog.c to write directly to log file unsafely. Tested-at: http