Skip to content

Conversation

@andreittr
Copy link
Contributor

@andreittr andreittr commented Dec 5, 2025

Description of changes

This changeset adds a ukfs driver for virtiofs, on top of virtio FS devices.

Depends on the following PR, and has it included as squashed commit:
- #1741

PLEASE DO NOT MERGE UNTIL REBASE

Prerequisite checklist

  • Read the contribution guidelines regarding submitting new changes to the project;
  • Tested your changes against relevant architectures and platforms;
  • Ran the checkpatch.uk on your commit series before opening this PR;
  • Updated relevant documentation.

Base target

  • Architecture(s): [e.g. x86_64 or N/A]
  • Platform(s): [e.g. kvm, xen or N/A]
  • Application(s): [e.g. app-python3 or N/A]

Additional configuration

N/A

@github-actions github-actions bot added area/lib Internal Unikraft Microlibrary lang/c Issues or PRs to do with C/C++ labels Dec 5, 2025
Copy link
Member

@mogasergiu mogasergiu left a comment

Choose a reason for hiding this comment

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

Had a first look. Just a couple of comments.

@andreittr
Copy link
Contributor Author

Pushed update addressing comments.

@andreittr andreittr requested a review from mogasergiu December 8, 2025 15:06
Copy link
Member

@mogasergiu mogasergiu left a comment

Choose a reason for hiding this comment

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

Reviewed-by: Sergiu Moga [email protected]

This commit imports the Linux UAPI fuse.h header, defining opcodes and
input/output structs for FUSE operations used by virtiofs.

Source of import:
Repository: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
Tag: v6.11
Path: include/uapi/linux/fuse.h

Checkpatch-Ignore: BLOCK_COMMENT_STYLE
Checkpatch-Ignore: BIT_MACRO
Checkpatch-Ignore: PREFER_KERNEL_TYPES
Checkpatch-Ignore: LONG_LINE_COMMENT
Checkpatch-Ignore: COMMIT_LOG_LONG_LINE
Checkpatch-Ignore: COMMIT_LOG_USE_LINK
Signed-off-by: Andrei Tatar <[email protected]>
This change introduces a ukfs driver for VirtioFS filesystems, exposed
under the "virtiofs" fstype.

Checkpatch-Ignore: MACRO_ARG_REUSE
Checkpatch-Ignore: CAMELCASE
Checkpatch-Ignore: LINE_SPACING
Checkpatch-Ignore: OPEN_ENDED_LINE
Checkpatch-Ignore: PREFER_KERNEL_TYPES
Checkpatch-Ignore: PARENTHESIS_ALIGNMENT
Checkpatch-Ignore: ENOSYS
Signed-off-by: Andrei Tatar <[email protected]>
@andreittr
Copy link
Contributor Author

Pushed rebase to current staging following merging of #1741 .

Copy link
Member

@michpappas michpappas left a comment

Choose a reason for hiding this comment

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

Approved-by: Michalis Pappas [email protected]

@michpappas michpappas added the merge Label to trigger merge action label Dec 9, 2025
@unikraft-bot unikraft-bot changed the base branch from staging to staging-1742 December 9, 2025 10:11
@unikraft-bot unikraft-bot merged commit d08b58e into unikraft:staging-1742 Dec 9, 2025
11 of 12 checks passed
unikraft-bot pushed a commit that referenced this pull request Dec 9, 2025
This commit imports the Linux UAPI fuse.h header, defining opcodes and
input/output structs for FUSE operations used by virtiofs.

Source of import:
Repository: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
Tag: v6.11
Path: include/uapi/linux/fuse.h

Checkpatch-Ignore: BLOCK_COMMENT_STYLE
Checkpatch-Ignore: BIT_MACRO
Checkpatch-Ignore: PREFER_KERNEL_TYPES
Checkpatch-Ignore: LONG_LINE_COMMENT
Checkpatch-Ignore: COMMIT_LOG_LONG_LINE
Checkpatch-Ignore: COMMIT_LOG_USE_LINK
Signed-off-by: Andrei Tatar <[email protected]>
Reviewed-by: Sergiu Moga <[email protected]>
Approved-by: Michalis Pappas <[email protected]>
GitHub-Closes: #1742
unikraft-bot pushed a commit that referenced this pull request Dec 9, 2025
This change introduces a ukfs driver for VirtioFS filesystems, exposed
under the "virtiofs" fstype.

Checkpatch-Ignore: MACRO_ARG_REUSE
Checkpatch-Ignore: CAMELCASE
Checkpatch-Ignore: LINE_SPACING
Checkpatch-Ignore: OPEN_ENDED_LINE
Checkpatch-Ignore: PREFER_KERNEL_TYPES
Checkpatch-Ignore: PARENTHESIS_ALIGNMENT
Checkpatch-Ignore: ENOSYS
Signed-off-by: Andrei Tatar <[email protected]>
Reviewed-by: Sergiu Moga <[email protected]>
Approved-by: Michalis Pappas <[email protected]>
GitHub-Closes: #1742
@unikraft-bot unikraft-bot added ci/merged Merged by CI and removed merge Label to trigger merge action labels Dec 9, 2025
@github-project-automation github-project-automation bot moved this from 🧊 Icebox to ✅ Done! in Unikraft Core Roadmap Dec 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/lib Internal Unikraft Microlibrary ci/merged Merged by CI lang/c Issues or PRs to do with C/C++

Projects

Status: Done!

Development

Successfully merging this pull request may close these issues.

4 participants