Skip to content

add per-syscall grates#99

Open
drapl0n wants to merge 1 commit intomainfrom
per-syscall-grate
Open

add per-syscall grates#99
drapl0n wants to merge 1 commit intomainfrom
per-syscall-grate

Conversation

@drapl0n
Copy link
Copy Markdown
Contributor

@drapl0n drapl0n commented Apr 26, 2026

  • adds geteuid, getuid, getpid, and close grate

- adds geteuid, getuid, getpid, and close grate
@stupendoussuperpowers
Copy link
Copy Markdown
Contributor

geteuid already exists right?

Also, can these just be replaced by using the testing-grate? Instead of needing to have individual grates for all of these single syscalls?

@Yaxuan-w
Copy link
Copy Markdown
Member

Can we auto-generate these grates?

@stupendoussuperpowers
Copy link
Copy Markdown
Contributor

Can we auto-generate these grates?

Agree to this, or like I pointed out in my comment -
the testing-grate can be configured for these use cases I think since it just takes runtime arguments and maps those with the return values we need.

@rennergade
Copy link
Copy Markdown
Contributor

We can just not merge this branch. I think it's useful to have for now since these are used for the policy graph

Copy link
Copy Markdown
Contributor

@stupendoussuperpowers stupendoussuperpowers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's useful to have for now since these are used for the policy graph

In that case I am not opposed to merging these.

Few things:

  1. Maybe we can change the folder name from "per-syscall-grates" to "single-syscall-grates" or "passthrough-syscall-grates" or something clearer/better worded.

  2. build.conf files are not needed anymore so we can remove them.

  3. All of these grates just pass the syscall through without any logging/parameter modifications, not sure if we should be doing at least something here. If these are being used for benchmarking, it might be a little unrepresentative of what a typical single-syscall grate is used for.

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.

4 participants