Skip to content

[PATCH v2] linux-gen: time: remove memory clobber from aarch64 cntvct_el0 read#2341

Merged
MatiasElo merged 1 commit intoOpenDataPlane:masterfrom
MatiasElo:dev/arm-time-opt
Apr 28, 2026
Merged

[PATCH v2] linux-gen: time: remove memory clobber from aarch64 cntvct_el0 read#2341
MatiasElo merged 1 commit intoOpenDataPlane:masterfrom
MatiasElo:dev/arm-time-opt

Conversation

@MatiasElo
Copy link
Copy Markdown
Collaborator

Remove unnecessary compiler memory barrier from aarch64 cntvct_el0 read in _odp_time_cpu_global() function, which is used to implement non-strict odp_cpu_cycles(), odp_time_local(), and odp_time_global() API functions. It's enough to have the memory clobber in _odp_time_cpu_global_strict(), which is used to implement the strict functions variants.

@odpbuild odpbuild changed the title linux-gen: time: remove memory clobber from aarch64 cntvct_el0 read [PATCH v1] linux-gen: time: remove memory clobber from aarch64 cntvct_el0 read Apr 23, 2026
Remove unnecessary compiler memory barrier from aarch64 cntvct_el0 read in
_odp_time_cpu_global() function, which is used to implement non-strict
odp_cpu_cycles(), odp_time_local(), and odp_time_global() API functions.
It's enough to have the memory clobber in _odp_time_cpu_global_strict(),
which is used to implement the strict functions variants.

Signed-off-by: Matias Elo <matias.elo@nokia.com>
Reviewed-by: Janne Peltonen <janne.peltonen@nokia.com>
@odpbuild odpbuild changed the title [PATCH v1] linux-gen: time: remove memory clobber from aarch64 cntvct_el0 read [PATCH v2] linux-gen: time: remove memory clobber from aarch64 cntvct_el0 read Apr 28, 2026
@MatiasElo MatiasElo enabled auto-merge (rebase) April 28, 2026 06:16
@MatiasElo MatiasElo merged commit 71df202 into OpenDataPlane:master Apr 28, 2026
255 of 256 checks passed
@MatiasElo MatiasElo deleted the dev/arm-time-opt branch April 28, 2026 07:06
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