keyctl
keyctl copied to clipboard
Add support for arm64 targets
Add syscall IDs for arm64.
Ping
Well, I've accidentally pushed the same changes (closed already). So basically it works fair enough, here's my log:
/ # uname -a
Linux key-test 5.10.186-179.751.amzn2.aarch64 #1 SMP Tue Aug 1 20:51:46 UTC 2023 aarch64 Linux
/ # keyctl show
Session Keyring
954719602 --alswrv 0 0 keyring: _ses.4b0d344fee412fdbc258b6615332aa864b94d6d3f86ac39bd42bb44f44e84320
276939416 --alswrv 0 0 \_ user: test
/ # ./app
test
/ # keyctl print 276939416
test
@jsipprell looks like several sites are using forks with this patch. Could you please merge this?
@rzimmerman @tarvitz @arrestle Given that I assume each of us have a fork of this project with this patch, I wonder if we need to consider forking to a new community repo.
@hallyn I don't have a proper answer for it. We required to have arm64 support, but I don't see any potential for any other architectures on our side. So @rzimmerman and mine changes (I made them accidentally because I missed this pr, but since they are trivial they look the same :) just support the current lib with a minor enhancement.
Since the mine fork is used in a client-side app which is updated from time to time, I assume I'll continue to support it (in the case if the original/this repository remains silent) from the following perspectives:
- Security (using govulncheck for the dependencies).
- Golang new versions compatibilities (supporting deprecations and so on).