On Thu, 07 May 2026 11:28, Alice Ryhl <[email protected]> wrote:
On Tue, May 5, 2026 at 10:14 AM Manos Pitsidianakis
<[email protected]> wrote:

Some internal kernel virtio API functions are inline macros, so define
their symbols in a helper file.

Signed-off-by: Manos Pitsidianakis <[email protected]>
---
 MAINTAINERS            |  6 ++++++
 rust/helpers/helpers.c |  3 +++
 rust/helpers/virtio.c  | 35 +++++++++++++++++++++++++++++++++++
 3 files changed, 44 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 
d1cc0e12fe1f004da89b1aa339116908f642e894..48c9c666d90b5a256ab6fae1f42508b789a0ce50
 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -27930,6 +27930,12 @@ F:     include/uapi/linux/virtio_*.h
 F:     net/vmw_vsock/virtio*
 F:     tools/virtio/

+VIRTIO CORE API BINDINGS [RUST]
+M:     Manos Pitsidianakis <[email protected]>
+L:     [email protected]
+S:     Maintained
+F:     rust/helpers/virtio.c
+
 VIRTIO CRYPTO DRIVER
 M:     Gonglei <[email protected]>
 L:     [email protected]
diff --git a/rust/helpers/helpers.c b/rust/helpers/helpers.c
index 
a3c42e51f00a0990bea81ebce6e99bb397ce7533..84b54690d95be37699ef9a9c4d7cedec0bbae6d3
 100644
--- a/rust/helpers/helpers.c
+++ b/rust/helpers/helpers.c
@@ -62,6 +62,9 @@
 #include "uaccess.c"
 #include "usb.c"
 #include "vmalloc.c"
+#if defined(CONFIG_VIRTIO)
+#include "virtio.c"
+#endif /* defined(CONFIG_VIRTIO) */

Please move this ifdef inside the file instead, to match all other
files with similar ifdefs.

Alice

Will do, thanks. Should this also be IS_ENABLED(CONFIG_VIRTIO) btw?

Manos

Reply via email to