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

Ignore non-existing CP15 barrier instructions emulation #3285

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

agners
Copy link
Member

@agners agners commented Apr 3, 2024

CP15 barrier instruction emulation only exists on arm64 architecture. Avoid sysctl writing an error to the journal when the setting doesn't exist by prepending a dash.

CP15 barrier instruction emulation only exists on arm64 architecture.
Avoid sysctl writing an error to the journal when the setting doesn't
exist by prepending a dash.
@agners agners requested a review from sairon April 3, 2024 07:49
@agners agners merged commit 889b561 into home-assistant:dev Apr 3, 2024
2 checks passed
jens-maus added a commit to jens-maus/RaspberryMatic that referenced this pull request Apr 7, 2024
@sairon sairon added the os label Apr 8, 2024
sairon pushed a commit that referenced this pull request Apr 8, 2024
CP15 barrier instruction emulation only exists on arm64 architecture.
Avoid sysctl writing an error to the journal when the setting doesn't
exist by prepending a dash.

(cherry picked from commit 889b561)
jens-maus added a commit to jens-maus/RaspberryMatic that referenced this pull request Jun 18, 2024
this sysctl statement is used. This reverts the changes proposed by
home-assistant/operating-system#3285
@jens-maus
Copy link
Contributor

@agners Sorry, to bring back this already merged PR, but after I applied this change (add '-' to mitigate warning) my users are reporting that the abi.cp15_barrier setting was not set effective, thus not applied. So are you really sure that adding - to the config items in sysctl.conf is a valid approach? (cf. jens-maus/RaspberryMatic@7bc55f2)

@agners
Copy link
Member Author

agners commented Jun 18, 2024

@jens-maus I remember I tested it, but was now a bit perplex when i realized that Busybox doesn't support that syntax (see https://github.com/mirror/busybox/blob/master/procps/sysctl.c).

However, it seems that at least HAOS uses the full sysctl, so for HAOS, this works.

@agners agners deleted the ignore-non-existing-sysctl-path branch June 18, 2024 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants