2021-07-03 14:56:17 +00:00
|
|
|
// SPDX-License-Identifier: GPL-2.0
|
|
|
|
|
rust: add #[export] macro
Rust has two different tools for generating function declarations to
call across the FFI boundary:
* bindgen. Generates Rust declarations from a C header.
* cbindgen. Generates C headers from Rust declarations.
However, we only use bindgen in the kernel. This means that when C code
calls a Rust function by name, its signature must be duplicated in both
Rust code and a C header, and the signature needs to be kept in sync
manually.
Introducing cbindgen as a mandatory dependency to build the kernel would
be a rather complex and large change, so we do not consider that at this
time. Instead, to eliminate this manual checking, introduce a new macro
that verifies at compile time that the two function declarations use the
same signature. The idea is to run the C declaration through bindgen,
and then have rustc verify that the function pointers have the same
type.
The signature must still be written twice, but at least you can no
longer get it wrong. If the signatures don't match, you will get errors
that look like this:
error[E0308]: `if` and `else` have incompatible types
--> <linux>/rust/kernel/print.rs:22:22
|
21 | #[export]
| --------- expected because of this
22 | unsafe extern "C" fn rust_fmt_argument(
| ^^^^^^^^^^^^^^^^^ expected `u8`, found `i8`
|
= note: expected fn item `unsafe extern "C" fn(*mut u8, *mut u8, *mut c_void) -> *mut u8 {bindings::rust_fmt_argument}`
found fn item `unsafe extern "C" fn(*mut i8, *mut i8, *const c_void) -> *mut i8 {print::rust_fmt_argument}`
It is unfortunate that the error message starts out by saying "`if` and
`else` have incompatible types", but I believe the rest of the error
message is reasonably clear and not too confusing.
Reviewed-by: Tamir Duberstein <tamird@gmail.com>
Reviewed-by: Andreas Hindborg <a.hindborg@kernel.org>
Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Alice Ryhl <aliceryhl@google.com>
Link: https://lore.kernel.org/r/20250303-export-macro-v3-3-41fbad85a27f@google.com
Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
2025-03-03 08:45:14 +00:00
|
|
|
use proc_macro::{token_stream, Group, Ident, TokenStream, TokenTree};
|
2021-07-03 14:56:17 +00:00
|
|
|
|
|
|
|
pub(crate) fn try_ident(it: &mut token_stream::IntoIter) -> Option<String> {
|
|
|
|
if let Some(TokenTree::Ident(ident)) = it.next() {
|
|
|
|
Some(ident.to_string())
|
|
|
|
} else {
|
|
|
|
None
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
pub(crate) fn try_literal(it: &mut token_stream::IntoIter) -> Option<String> {
|
|
|
|
if let Some(TokenTree::Literal(literal)) = it.next() {
|
|
|
|
Some(literal.to_string())
|
|
|
|
} else {
|
|
|
|
None
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2022-11-10 16:41:19 +00:00
|
|
|
pub(crate) fn try_string(it: &mut token_stream::IntoIter) -> Option<String> {
|
|
|
|
try_literal(it).and_then(|string| {
|
|
|
|
if string.starts_with('\"') && string.ends_with('\"') {
|
|
|
|
let content = &string[1..string.len() - 1];
|
|
|
|
if content.contains('\\') {
|
|
|
|
panic!("Escape sequences in string literals not yet handled");
|
|
|
|
}
|
|
|
|
Some(content.to_string())
|
|
|
|
} else if string.starts_with("r\"") {
|
|
|
|
panic!("Raw string literals are not yet handled");
|
2021-07-03 14:56:17 +00:00
|
|
|
} else {
|
|
|
|
None
|
|
|
|
}
|
|
|
|
})
|
|
|
|
}
|
|
|
|
|
|
|
|
pub(crate) fn expect_ident(it: &mut token_stream::IntoIter) -> String {
|
|
|
|
try_ident(it).expect("Expected Ident")
|
|
|
|
}
|
|
|
|
|
|
|
|
pub(crate) fn expect_punct(it: &mut token_stream::IntoIter) -> char {
|
|
|
|
if let TokenTree::Punct(punct) = it.next().expect("Reached end of token stream for Punct") {
|
|
|
|
punct.as_char()
|
|
|
|
} else {
|
|
|
|
panic!("Expected Punct");
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2022-11-10 16:41:19 +00:00
|
|
|
pub(crate) fn expect_string(it: &mut token_stream::IntoIter) -> String {
|
|
|
|
try_string(it).expect("Expected string")
|
|
|
|
}
|
|
|
|
|
|
|
|
pub(crate) fn expect_string_ascii(it: &mut token_stream::IntoIter) -> String {
|
|
|
|
let string = try_string(it).expect("Expected string");
|
|
|
|
assert!(string.is_ascii(), "Expected ASCII string");
|
|
|
|
string
|
2021-07-03 14:56:17 +00:00
|
|
|
}
|
|
|
|
|
2023-03-23 12:35:10 +00:00
|
|
|
pub(crate) fn expect_group(it: &mut token_stream::IntoIter) -> Group {
|
|
|
|
if let TokenTree::Group(group) = it.next().expect("Reached end of token stream for Group") {
|
|
|
|
group
|
|
|
|
} else {
|
|
|
|
panic!("Expected Group");
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
2021-07-03 14:56:17 +00:00
|
|
|
pub(crate) fn expect_end(it: &mut token_stream::IntoIter) {
|
|
|
|
if it.next().is_some() {
|
|
|
|
panic!("Expected end");
|
|
|
|
}
|
|
|
|
}
|
2023-04-24 08:11:38 +00:00
|
|
|
|
rust: add #[export] macro
Rust has two different tools for generating function declarations to
call across the FFI boundary:
* bindgen. Generates Rust declarations from a C header.
* cbindgen. Generates C headers from Rust declarations.
However, we only use bindgen in the kernel. This means that when C code
calls a Rust function by name, its signature must be duplicated in both
Rust code and a C header, and the signature needs to be kept in sync
manually.
Introducing cbindgen as a mandatory dependency to build the kernel would
be a rather complex and large change, so we do not consider that at this
time. Instead, to eliminate this manual checking, introduce a new macro
that verifies at compile time that the two function declarations use the
same signature. The idea is to run the C declaration through bindgen,
and then have rustc verify that the function pointers have the same
type.
The signature must still be written twice, but at least you can no
longer get it wrong. If the signatures don't match, you will get errors
that look like this:
error[E0308]: `if` and `else` have incompatible types
--> <linux>/rust/kernel/print.rs:22:22
|
21 | #[export]
| --------- expected because of this
22 | unsafe extern "C" fn rust_fmt_argument(
| ^^^^^^^^^^^^^^^^^ expected `u8`, found `i8`
|
= note: expected fn item `unsafe extern "C" fn(*mut u8, *mut u8, *mut c_void) -> *mut u8 {bindings::rust_fmt_argument}`
found fn item `unsafe extern "C" fn(*mut i8, *mut i8, *const c_void) -> *mut i8 {print::rust_fmt_argument}`
It is unfortunate that the error message starts out by saying "`if` and
`else` have incompatible types", but I believe the rest of the error
message is reasonably clear and not too confusing.
Reviewed-by: Tamir Duberstein <tamird@gmail.com>
Reviewed-by: Andreas Hindborg <a.hindborg@kernel.org>
Acked-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Alice Ryhl <aliceryhl@google.com>
Link: https://lore.kernel.org/r/20250303-export-macro-v3-3-41fbad85a27f@google.com
Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
2025-03-03 08:45:14 +00:00
|
|
|
/// Given a function declaration, finds the name of the function.
|
|
|
|
pub(crate) fn function_name(input: TokenStream) -> Option<Ident> {
|
|
|
|
let mut input = input.into_iter();
|
|
|
|
while let Some(token) = input.next() {
|
|
|
|
match token {
|
|
|
|
TokenTree::Ident(i) if i.to_string() == "fn" => {
|
|
|
|
if let Some(TokenTree::Ident(i)) = input.next() {
|
|
|
|
return Some(i);
|
|
|
|
}
|
|
|
|
return None;
|
|
|
|
}
|
|
|
|
_ => continue,
|
|
|
|
}
|
|
|
|
}
|
|
|
|
None
|
|
|
|
}
|