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

aarch64: Use architecture specific header to general syscalls #235

Closed

Conversation

alexmurray
Copy link
Contributor

This ensures clone3, getrlimit and setrlimit are defined for aarch64 - also
update syscalls.csv from 5.6.0 to pick up these new syscalls

@coveralls
Copy link

coveralls commented May 11, 2020

Coverage Status

Coverage remained the same at 92.806% when pulling fff41e9 on alexmurray:fix-missing-aarch64-syscalls into ff2a538 on seccomp:master.

@jdstrand
Copy link

FYI, this is to address https://launchpad.net/bugs/1877633

@drakenclimber
Copy link
Member

The changes look good to me. I tested out the getrlimit() and setrlimit() changes on an arm64 box I had laying around. I didn't test the clone3() change, but I see no issues there.

@alexmurray - could you please update the patch with your signed-off-by? Thanks!

Reviewed-by: Tom Hromatka <tom.hromatka@oracle.com>

@pcmoore, I originally tagged this as v2.5, but I'm now second guessing myself and wondering if this deserves a v2.4.4 release of its own. Thoughts?

@pcmoore
Copy link
Member

pcmoore commented May 12, 2020

Thanks @alexmurray for the fix, but as @drakenclimber already pointed out, please include your sign-off in the patch. We've written up a guide to submitting patches in the CONTRIBUTING.md doc if that helps.

@drakenclimber, yes, I think we need this both for v2.5.0 and v.2.4.x. Do you want to create a new issue for this for the v2.4.4 milestone? We obviously can't backport this fix.

@drakenclimber
Copy link
Member

Sure. I can take care of the v2.4.4 side of things.

This ensures clone3, getrlimit and setrlimit are defined for aarch64 - also
update syscalls.csv from 5.6.0 to pick up these new syscalls

Signed-off-by: Alex Murray <alex.murray@canonical.com>
Reviewed-by: Tom Hromatka <tom.hromatka@oracle.com>
@alexmurray alexmurray force-pushed the fix-missing-aarch64-syscalls branch from 94ef9ca to fff41e9 Compare May 13, 2020 02:20
@alexmurray
Copy link
Contributor Author

Apologies for missing CONTRIBUTING.md - have added my signed-off-by and Tom's reviewed-by. @drakenclimber I can try take a look at 2.4.4 if you need - let me know.

@drakenclimber
Copy link
Member

Apologies for missing CONTRIBUTING.md - have added my signed-off-by and Tom's reviewed-by. @drakenclimber I can try take a look at 2.4.4 if you need - let me know.

That would be really helpful :)! Thanks.

@pcmoore
Copy link
Member

pcmoore commented May 19, 2020

Looks good to me, merged via 7780b08 - thanks everyone!

@pcmoore pcmoore closed this May 19, 2020
gstrauss added a commit to gstrauss/lighttpd1.4 that referenced this pull request Oct 18, 2020
modify use of getrlimit() to not be fatal if an error occurs

Some container configurations might cause getrlimit() and setrlimit() to
fail.  lighttpd will issue error trace if this occurs, but will now
proceed.

x-ref:
  "aarch64: Use architecture specific header to general syscalls"
  seccomp/libseccomp#235
  "Rasperry Docker PiHole / "Starting lighttpd" Hanging"
  pi-hole/docker-pi-hole#593
  "Pi-hole runs on podman"
  https://discourse.pi-hole.net/t/pi-hole-runs-on-podman/34719/5
lighttpd-git pushed a commit to lighttpd/lighttpd1.4 that referenced this pull request Oct 20, 2020
modify use of getrlimit() to not be fatal if an error occurs

Some container configurations might cause getrlimit() and setrlimit() to
fail.  lighttpd will issue error trace if this occurs, but will now
proceed.

x-ref:
  "aarch64: Use architecture specific header to general syscalls"
  seccomp/libseccomp#235
  "Rasperry Docker PiHole / "Starting lighttpd" Hanging"
  pi-hole/docker-pi-hole#593
  "Pi-hole runs on podman"
  https://discourse.pi-hole.net/t/pi-hole-runs-on-podman/34719/5
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.

None yet

5 participants