Page MenuHomeFreeBSD

add virtio file system device support
Needs ReviewPublic

Authored by emil_etsalapatis.com on Aug 14 2024, 3:53 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Oct 11, 1:40 AM
Unknown Object (File)
Sat, Oct 4, 6:18 AM
Unknown Object (File)
Wed, Oct 1, 6:56 PM
Unknown Object (File)
Wed, Oct 1, 3:45 PM
Unknown Object (File)
Sat, Sep 27, 11:31 PM
Unknown Object (File)
Fri, Sep 26, 5:31 AM
Unknown Object (File)
Sep 15 2025, 10:03 PM
Unknown Object (File)
Sep 10 2025, 7:08 AM

Details

Summary

Supporting the virtiofs requires a device driver for the virtio fs device. Add the virtio file system driver before introducing the file system itself.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 64544
Build 61428: arc lint + arc unit

Event Timeline

lwhsu added inline comments.
sys/dev/virtio/fs/virtio_fs.h
5

We need a copyright holder here, and may drop "all rights reserved."

asomers requested changes to this revision.Aug 21 2024, 7:30 PM
asomers added inline comments.
sys/dev/virtio/fs/virtio_fs.c
59

This line is redundant with the one in virtio_fs.h

297
551–552
sys/dev/virtio/fs/virtio_fs_internal.h
11

If these messages are just for debugging purposes, shouldn't there be a way to turn them off?

This revision now requires changes to proceed.Aug 21 2024, 7:30 PM
  • sys/dev/virtio/fs: drain virtqueues one at a time
sys/dev/virtio/fs/virtio_fs.h
3

BSD-2-Clause is fine, no need -FreeBSD suffix

  • sys/dev/virtio/fs: use regular BSD-2-Clause license