Skip to content

Conversation

jmi-reds
Copy link
Contributor

@jmi-reds jmi-reds commented Oct 7, 2025

Add 'writev` syscall to SO3 kernel

Also updated the name of the generated syscall function names to __do_<SYSCALL>

Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
@jmi-reds jmi-reds self-assigned this Oct 7, 2025
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
@jmi-reds jmi-reds changed the title Add 'writev' syscall draft: Add 'writev' & 'readv' syscall Oct 7, 2025
@jmi-reds jmi-reds marked this pull request as draft October 7, 2025 15:13
@jmi-reds jmi-reds changed the title draft: Add 'writev' & 'readv' syscall Add 'writev' & 'readv' syscall Oct 7, 2025
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
@jmi-reds jmi-reds marked this pull request as ready for review October 7, 2025 15:54
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
Copy link
Contributor

@clemdiep clemdiep left a comment

Choose a reason for hiding this comment

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

Small new fix

Copy link
Contributor

@clemdiep clemdiep left a comment

Choose a reason for hiding this comment

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

Just find out that clang-format was not running on those PR since target branch isn't main.

So this list change that needs to be done, you can run the following commands to get them:

git clang-format 50446524cc7257db7c47f03ca2ca9704b4e206f6

Also, can you add 144-support-musl to .github/workflows/style.yml line 13, so this check will be running for futur pull request. (will need to be removed when 144-support-musl will be merge into main)

Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
Signed-off-by: Jean-Pierre Miceli <jean-pierre.miceli@heig-vd.ch>
Copy link
Contributor

@clemdiep clemdiep left a comment

Choose a reason for hiding this comment

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

All good for me!

@jmi-reds jmi-reds merged commit d175143 into 144-support-musl Oct 9, 2025
4 checks passed
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