Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

196 Potential race condition in read-modify-write PDF vectors in parallel loop in onesource::generatePDF() #206

Closed
wants to merge 12 commits into from
Closed

196 Potential race condition in read-modify-write PDF vectors in parallel loop in onesource::generatePDF() #206

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
7399581
add the missing arrays as private members in the reduction loop
Sep 12, 2024
96ab990
add cmake directive for building with thread sanitizing
Sep 12, 2024
8f06ccf
add cmake directive for building with thread sanitizing
Sep 12, 2024
7aee2ff
fix coding style and remove flags added by mistake
Sep 13, 2024
8221cbd
Merge branch 'main' into 197-reduction-operation-being-executed-in-pa…
johannct Sep 17, 2024
3cefd8f
add cmake directive for building with thread sanitizing
Sep 12, 2024
5f9c2d4
Support address sanitizer builds
Sep 16, 2024
68a11ac
Merge branch '197-reduction-operation-being-executed-in-parallel-with…
Sep 17, 2024
2c06835
Fix comments in CMakeLists.txt
johannct Sep 17, 2024
43e923c
Fix comments in CMakeLists.txt
johannct Sep 17, 2024
d4b956a
Merge branch '197-reduction-operation-being-executed-in-parallel-with…
johannct Sep 17, 2024
5a396b6
Merge pull request #210 from lephare-photoz/johannct-patch-1
johannct Sep 17, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 14 additions & 2 deletions CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,11 +64,23 @@ else()
add_compile_options("-mno-vzeroupper" "-mavx2" "-floop-unroll-and-jam" )
endif()

# only set these flags if testing presence of race conditions :
# https://github.com/google/sanitizers/wiki/ThreadSanitizerCppManual
if(DEFINED ENV{SANITIZE_THREAD})
message(STATUS "Building with thread sanitizing enabled.")
SET(GCC_COVERAGE_COMPILE_FLAGS "-g -O2 -fsanitize=thread")
SET(GCC_COVERAGE_LINK_FLAGS "-fsanitize=thread")
SET( CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${GCC_COVERAGE_COMPILE_FLAGS}" )
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a coding style inconsistency here: space between the parentheses and variable name vs. no space.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agreed, fixed in #7aee2ff

SET( CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} ${GCC_COVERAGE_LINK_FLAGS}" )
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's a coding style inconsistency here: space between the parentheses and variable name vs. no space.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agreed, fixed in #7aee2ff

endif()


# only set these flags if in `INCLUDE_COVERAGE` env var is set (to any value)
# on linux/gcc it seems that one needs to run with LD_PRELOAD=/lib/x86_64-linux-gnu/libtsan.so.0 prepended
if(DEFINED ENV{INCLUDE_COVERAGE})
message(STATUS "Building with code coverage enabled.")
SET(GCC_COVERAGE_COMPILE_FLAGS "-g -O0 -coverage -fprofile-arcs -ftest-coverage")
SET(GCC_COVERAGE_LINK_FLAGS "-coverage -lgcov")
SET(GCC_COVERAGE_COMPILE_FLAGS "-g -O0 -coverage -fprofile-arcs -ftest-coverage -fPIC -pie")
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are explicit -fPIC and -pie really required here ? These flags are platform-dependent.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agreed. Fixed in #7aee2ff

SET(GCC_COVERAGE_LINK_FLAGS "-coverage -lgcov -lpthread")
SET( CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${GCC_COVERAGE_COMPILE_FLAGS}" )
SET( CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS} ${GCC_COVERAGE_LINK_FLAGS}" )
endif()
Expand Down
Loading