Skip to content

Conversation

TimePrinciple
Copy link

Summary of the PR

  • syscall_table: Update to v6.12 kernel
  • CHANGELOG: Update CHANGELOG.md

Before submitting your PR, please make sure you addressed the following
requirements:

  • All commits in this PR have Signed-Off-By trailers (with
    git commit -s), and the commit message has max 60 characters for the
    summary and max 75 characters for each description line.
  • All added/changed functionality has a corresponding unit/integration
    test.
  • All added/changed public-facing functionality has entries in the "Upcoming
    Release" section of CHANGELOG.md (if no such section exists, please create one).
  • Any newly added unsafe code is properly documented.

("clock_nanosleep", 115),
("clock_settime", 112),
("clone", 220),
("accept4", 288),
Copy link
Collaborator

Choose a reason for hiding this comment

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

these are wrong. they look like they're the same syscall numbers as on x86

Copy link
Author

Choose a reason for hiding this comment

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

Sorry I forgot to update this PR, IMHO our scripts used to generate syscall_table no longer works with v6.12 kernel. I'm in the middle of working out a new workflow 🧐

vec![
("accept", 43),
("accept4", 288),
("accept", 43),
Copy link
Collaborator

Choose a reason for hiding this comment

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

looks like the sort you used when building this is different than the one used in the rust vmm container. you could use that container image to run the script

Some of our downstream communities are moving to 6.12 kernel,
regenerated `syscall_table` from 6.12 kernel source to catch up.

Signed-off-by: Ruoqing He <[email protected]>
Update `CHANGELOG.md` and document the update of `syscall_tables` from
v6.12 kernel source.

Signed-off-by: Ruoqing He <[email protected]>
@TimePrinciple
Copy link
Author

Migrated to #78

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.

3 participants