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.

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

Fix compiling error when ARM_MTE=1 iso_alloc_internal.h #230

Merged
merged 1 commit into from
Jun 29, 2024

Conversation

rrbranco
Copy link
Contributor

When ARM_MTE=1, MEMORY_TAGGING has to be 0. Therefore, the TAGGED_PTR_MAX (used by both) has to be defined if MEMORY_TAGGING is defined or when ARM_MTE is.

When ARM_MTE=1, MEMORY_TAGGING has to be 0.  Therefore, the TAGGED_PTR_MAX (used by both) has to be defined if MEMORY_TAGGING is defined or when ARM_MTE is.
@struct
Copy link
Owner

struct commented Jun 20, 2024

I think TAGGED_PTR_MASK is only used for the software memory tagging implementation?

@rrbranco
Copy link
Contributor Author

In src/iso_alloc_search.c, line 17: there is this assignment:
n = (void *) ((uintptr_t) n & TAGGED_PTR_MASK);

It is inside a #if MEMORY_TAGGING || (ARM_MTE == 1)

So either this || (ARM_MTE == 1) has to be removed, or it has to be added to the #define of the TAGGED_PTR_MASK.

@struct struct merged commit df5d71c into struct:master Jun 29, 2024
8 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants