Skip to content

Fix atomic build on riscv64 #433

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

Closed
LocutusOfBorg wants to merge 3 commits into from
Closed

Fix atomic build on riscv64 #433

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
559aa84
Add CheckAtomic.cmake from cmake-extra-modules package
LocutusOfBorg Jan 10, 2023
0d18b4f
Revert "Add CheckAtomic.cmake from cmake-extra-modules package"
LocutusOfBorg Jan 18, 2023
8d10933
Change atomic bool to atomic int, this way riscv64 builds fine withou…
LocutusOfBorg Jan 18, 2023
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
6 changes: 3 additions & 3 deletions src/lib/dlt_user.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@
#endif /* DLT_FATAL_LOG_RESET_ENABLE */

static DltUser dlt_user;
static atomic_bool dlt_user_initialised = false;
static atomic_int dlt_user_initialised = false;
static int dlt_user_freeing = 0;
static bool dlt_user_file_reach_max = false;

Expand Down Expand Up @@ -451,7 +451,7 @@ DltReturnValue dlt_init(void)
pass fully. The other one will immediately return, because when it executes the atomic function
'dlt_user_initialised' will be for sure already set to true.
*/
bool expected = false;
int expected = false;
if (!(atomic_compare_exchange_strong(&dlt_user_initialised, &expected, true)))
return DLT_RETURN_OK;

Expand Down Expand Up @@ -563,7 +563,7 @@ DltReturnValue dlt_init_file(const char *name)
pass fully. The other one will immediately return, because when it executes the atomic function
'dlt_user_initialised' will be for sure already set to true.
*/
bool expected = false;
int expected = false;
if (!(atomic_compare_exchange_strong(&dlt_user_initialised, &expected, true)))
return DLT_RETURN_OK;

Expand Down