mirror of
				https://github.com/torvalds/linux.git
				synced 2025-10-31 08:38:45 +02:00 
			
		
		
		
	 374908a15a
			
		
	
	
		374908a15a
		
	
	
	
	
		
			
			In commit392e34b6bc("kbuild: rust: remove the `alloc` crate and `GlobalAlloc`") we stopped using the upstream `alloc` crate. Thus remove a few leftover mentions treewide. Cc: stable@vger.kernel.org # Also to 6.12.y after the `alloc` backport lands Fixes:392e34b6bc("kbuild: rust: remove the `alloc` crate and `GlobalAlloc`") Reviewed-by: Danilo Krummrich <dakr@kernel.org> Reviewed-by: Andreas Hindborg <a.hindborg@kernel.org> Link: https://lore.kernel.org/r/20250303171030.1081134-1-ojeda@kernel.org Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
		
			
				
	
	
		
			260 lines
		
	
	
	
		
			9.5 KiB
		
	
	
	
		
			Rust
		
	
	
	
	
	
			
		
		
	
	
			260 lines
		
	
	
	
		
			9.5 KiB
		
	
	
	
		
			Rust
		
	
	
	
	
	
| // SPDX-License-Identifier: GPL-2.0
 | |
| 
 | |
| //! Generates KUnit tests from saved `rustdoc`-generated tests.
 | |
| //!
 | |
| //! KUnit passes a context (`struct kunit *`) to each test, which should be forwarded to the other
 | |
| //! KUnit functions and macros.
 | |
| //!
 | |
| //! However, we want to keep this as an implementation detail because:
 | |
| //!
 | |
| //!   - Test code should not care about the implementation.
 | |
| //!
 | |
| //!   - Documentation looks worse if it needs to carry extra details unrelated to the piece
 | |
| //!     being described.
 | |
| //!
 | |
| //!   - Test code should be able to define functions and call them, without having to carry
 | |
| //!     the context.
 | |
| //!
 | |
| //!   - Later on, we may want to be able to test non-kernel code (e.g. `core` or third-party
 | |
| //!     crates) which likely use the standard library `assert*!` macros.
 | |
| //!
 | |
| //! For this reason, instead of the passed context, `kunit_get_current_test()` is used instead
 | |
| //! (i.e. `current->kunit_test`).
 | |
| //!
 | |
| //! Note that this means other threads/tasks potentially spawned by a given test, if failing, will
 | |
| //! report the failure in the kernel log but will not fail the actual test. Saving the pointer in
 | |
| //! e.g. a `static` per test does not fully solve the issue either, because currently KUnit does
 | |
| //! not support assertions (only expectations) from other tasks. Thus leave that feature for
 | |
| //! the future, which simplifies the code here too. We could also simply not allow `assert`s in
 | |
| //! other tasks, but that seems overly constraining, and we do want to support them, eventually.
 | |
| 
 | |
| use std::{
 | |
|     fs,
 | |
|     fs::File,
 | |
|     io::{BufWriter, Read, Write},
 | |
|     path::{Path, PathBuf},
 | |
| };
 | |
| 
 | |
| /// Find the real path to the original file based on the `file` portion of the test name.
 | |
| ///
 | |
| /// `rustdoc` generated `file`s look like `sync_locked_by_rs`. Underscores (except the last one)
 | |
| /// may represent an actual underscore in a directory/file, or a path separator. Thus the actual
 | |
| /// file might be `sync_locked_by.rs`, `sync/locked_by.rs`, `sync_locked/by.rs` or
 | |
| /// `sync/locked/by.rs`. This function walks the file system to determine which is the real one.
 | |
| ///
 | |
| /// This does require that ambiguities do not exist, but that seems fair, especially since this is
 | |
| /// all supposed to be temporary until `rustdoc` gives us proper metadata to build this. If such
 | |
| /// ambiguities are detected, they are diagnosed and the script panics.
 | |
| fn find_real_path<'a>(srctree: &Path, valid_paths: &'a mut Vec<PathBuf>, file: &str) -> &'a str {
 | |
|     valid_paths.clear();
 | |
| 
 | |
|     let potential_components: Vec<&str> = file.strip_suffix("_rs").unwrap().split('_').collect();
 | |
| 
 | |
|     find_candidates(srctree, valid_paths, Path::new(""), &potential_components);
 | |
|     fn find_candidates(
 | |
|         srctree: &Path,
 | |
|         valid_paths: &mut Vec<PathBuf>,
 | |
|         prefix: &Path,
 | |
|         potential_components: &[&str],
 | |
|     ) {
 | |
|         // The base case: check whether all the potential components left, joined by underscores,
 | |
|         // is a file.
 | |
|         let joined_potential_components = potential_components.join("_") + ".rs";
 | |
|         if srctree
 | |
|             .join("rust/kernel")
 | |
|             .join(prefix)
 | |
|             .join(&joined_potential_components)
 | |
|             .is_file()
 | |
|         {
 | |
|             // Avoid `srctree` here in order to keep paths relative to it in the KTAP output.
 | |
|             valid_paths.push(
 | |
|                 Path::new("rust/kernel")
 | |
|                     .join(prefix)
 | |
|                     .join(joined_potential_components),
 | |
|             );
 | |
|         }
 | |
| 
 | |
|         // In addition, check whether each component prefix, joined by underscores, is a directory.
 | |
|         // If not, there is no need to check for combinations with that prefix.
 | |
|         for i in 1..potential_components.len() {
 | |
|             let (components_prefix, components_rest) = potential_components.split_at(i);
 | |
|             let prefix = prefix.join(components_prefix.join("_"));
 | |
|             if srctree.join("rust/kernel").join(&prefix).is_dir() {
 | |
|                 find_candidates(srctree, valid_paths, &prefix, components_rest);
 | |
|             }
 | |
|         }
 | |
|     }
 | |
| 
 | |
|     assert!(
 | |
|         valid_paths.len() > 0,
 | |
|         "No path candidates found. This is likely a bug in the build system, or some files went \
 | |
|         away while compiling."
 | |
|     );
 | |
| 
 | |
|     if valid_paths.len() > 1 {
 | |
|         eprintln!("Several path candidates found:");
 | |
|         for path in valid_paths {
 | |
|             eprintln!("    {path:?}");
 | |
|         }
 | |
|         panic!(
 | |
|             "Several path candidates found, please resolve the ambiguity by renaming a file or \
 | |
|             folder."
 | |
|         );
 | |
|     }
 | |
| 
 | |
|     valid_paths[0].to_str().unwrap()
 | |
| }
 | |
| 
 | |
| fn main() {
 | |
|     let srctree = std::env::var("srctree").unwrap();
 | |
|     let srctree = Path::new(&srctree);
 | |
| 
 | |
|     let mut paths = fs::read_dir("rust/test/doctests/kernel")
 | |
|         .unwrap()
 | |
|         .map(|entry| entry.unwrap().path())
 | |
|         .collect::<Vec<_>>();
 | |
| 
 | |
|     // Sort paths.
 | |
|     paths.sort();
 | |
| 
 | |
|     let mut rust_tests = String::new();
 | |
|     let mut c_test_declarations = String::new();
 | |
|     let mut c_test_cases = String::new();
 | |
|     let mut body = String::new();
 | |
|     let mut last_file = String::new();
 | |
|     let mut number = 0;
 | |
|     let mut valid_paths: Vec<PathBuf> = Vec::new();
 | |
|     let mut real_path: &str = "";
 | |
|     for path in paths {
 | |
|         // The `name` follows the `{file}_{line}_{number}` pattern (see description in
 | |
|         // `scripts/rustdoc_test_builder.rs`). Discard the `number`.
 | |
|         let name = path.file_name().unwrap().to_str().unwrap().to_string();
 | |
| 
 | |
|         // Extract the `file` and the `line`, discarding the `number`.
 | |
|         let (file, line) = name.rsplit_once('_').unwrap().0.rsplit_once('_').unwrap();
 | |
| 
 | |
|         // Generate an ID sequence ("test number") for each one in the file.
 | |
|         if file == last_file {
 | |
|             number += 1;
 | |
|         } else {
 | |
|             number = 0;
 | |
|             last_file = file.to_string();
 | |
| 
 | |
|             // Figure out the real path, only once per file.
 | |
|             real_path = find_real_path(srctree, &mut valid_paths, file);
 | |
|         }
 | |
| 
 | |
|         // Generate a KUnit name (i.e. test name and C symbol) for this test.
 | |
|         //
 | |
|         // We avoid the line number, like `rustdoc` does, to make things slightly more stable for
 | |
|         // bisection purposes. However, to aid developers in mapping back what test failed, we will
 | |
|         // print a diagnostics line in the KTAP report.
 | |
|         let kunit_name = format!("rust_doctest_kernel_{file}_{number}");
 | |
| 
 | |
|         // Read the test's text contents to dump it below.
 | |
|         body.clear();
 | |
|         File::open(path).unwrap().read_to_string(&mut body).unwrap();
 | |
| 
 | |
|         // Calculate how many lines before `main` function (including the `main` function line).
 | |
|         let body_offset = body
 | |
|             .lines()
 | |
|             .take_while(|line| !line.contains("fn main() {"))
 | |
|             .count()
 | |
|             + 1;
 | |
| 
 | |
|         use std::fmt::Write;
 | |
|         write!(
 | |
|             rust_tests,
 | |
|             r#"/// Generated `{name}` KUnit test case from a Rust documentation test.
 | |
| #[no_mangle]
 | |
| pub extern "C" fn {kunit_name}(__kunit_test: *mut kernel::bindings::kunit) {{
 | |
|     /// Overrides the usual [`assert!`] macro with one that calls KUnit instead.
 | |
|     #[allow(unused)]
 | |
|     macro_rules! assert {{
 | |
|         ($cond:expr $(,)?) => {{{{
 | |
|             kernel::kunit_assert!("{kunit_name}", "{real_path}", __DOCTEST_ANCHOR - {line}, $cond);
 | |
|         }}}}
 | |
|     }}
 | |
| 
 | |
|     /// Overrides the usual [`assert_eq!`] macro with one that calls KUnit instead.
 | |
|     #[allow(unused)]
 | |
|     macro_rules! assert_eq {{
 | |
|         ($left:expr, $right:expr $(,)?) => {{{{
 | |
|             kernel::kunit_assert_eq!("{kunit_name}", "{real_path}", __DOCTEST_ANCHOR - {line}, $left, $right);
 | |
|         }}}}
 | |
|     }}
 | |
| 
 | |
|     // Many tests need the prelude, so provide it by default.
 | |
|     #[allow(unused)]
 | |
|     use kernel::prelude::*;
 | |
| 
 | |
|     // Unconditionally print the location of the original doctest (i.e. rather than the location in
 | |
|     // the generated file) so that developers can easily map the test back to the source code.
 | |
|     //
 | |
|     // This information is also printed when assertions fail, but this helps in the successful cases
 | |
|     // when the user is running KUnit manually, or when passing `--raw_output` to `kunit.py`.
 | |
|     //
 | |
|     // This follows the syntax for declaring test metadata in the proposed KTAP v2 spec, which may
 | |
|     // be used for the proposed KUnit test attributes API. Thus hopefully this will make migration
 | |
|     // easier later on.
 | |
|     kernel::kunit::info(format_args!("    # {kunit_name}.location: {real_path}:{line}\n"));
 | |
| 
 | |
|     /// The anchor where the test code body starts.
 | |
|     #[allow(unused)]
 | |
|     static __DOCTEST_ANCHOR: i32 = core::line!() as i32 + {body_offset} + 1;
 | |
|     {{
 | |
|         {body}
 | |
|         main();
 | |
|     }}
 | |
| }}
 | |
| 
 | |
| "#
 | |
|         )
 | |
|         .unwrap();
 | |
| 
 | |
|         write!(c_test_declarations, "void {kunit_name}(struct kunit *);\n").unwrap();
 | |
|         write!(c_test_cases, "    KUNIT_CASE({kunit_name}),\n").unwrap();
 | |
|     }
 | |
| 
 | |
|     let rust_tests = rust_tests.trim();
 | |
|     let c_test_declarations = c_test_declarations.trim();
 | |
|     let c_test_cases = c_test_cases.trim();
 | |
| 
 | |
|     write!(
 | |
|         BufWriter::new(File::create("rust/doctests_kernel_generated.rs").unwrap()),
 | |
|         r#"//! `kernel` crate documentation tests.
 | |
| 
 | |
| const __LOG_PREFIX: &[u8] = b"rust_doctests_kernel\0";
 | |
| 
 | |
| {rust_tests}
 | |
| "#
 | |
|     )
 | |
|     .unwrap();
 | |
| 
 | |
|     write!(
 | |
|         BufWriter::new(File::create("rust/doctests_kernel_generated_kunit.c").unwrap()),
 | |
|         r#"/*
 | |
|  * `kernel` crate documentation tests.
 | |
|  */
 | |
| 
 | |
| #include <kunit/test.h>
 | |
| 
 | |
| {c_test_declarations}
 | |
| 
 | |
| static struct kunit_case test_cases[] = {{
 | |
|     {c_test_cases}
 | |
|     {{ }}
 | |
| }};
 | |
| 
 | |
| static struct kunit_suite test_suite = {{
 | |
|     .name = "rust_doctests_kernel",
 | |
|     .test_cases = test_cases,
 | |
| }};
 | |
| 
 | |
| kunit_test_suite(test_suite);
 | |
| 
 | |
| MODULE_LICENSE("GPL");
 | |
| "#
 | |
|     )
 | |
|     .unwrap();
 | |
| }
 |