[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 05/11] rust/block: Add empty crate
From: |
Zhao Liu |
Subject: |
Re: [PATCH v2 05/11] rust/block: Add empty crate |
Date: |
Wed, 19 Feb 2025 14:46:15 +0800 |
> diff --git a/rust/block/Cargo.toml b/rust/block/Cargo.toml
> new file mode 100644
> index 0000000000..fbc2f2d6ef
> --- /dev/null
> +++ b/rust/block/Cargo.toml
> @@ -0,0 +1,16 @@
> +[package]
> +name = "block"
> +version = "0.1.0"
> +edition = "2021"
> +authors = ["Kevin Wolf <kwolf@redhat.com>"]
> +license = "GPL-2.0-or-later"
> +readme = "README.md"
> +description = "Block backends for QEMU"
> +repository = "https://gitlab.com/qemu-project/qemu/"
> +publish = false
> +keywords = []
> +categories = []
Per the commit f26137893b98 ("rust: remove unnecessary Cargo.toml
metadata"), readme & repository are not necessary. :-)
And rust-version is needed (since the commit 669fab6a1f3e).
> +[dependencies]
> +qemu_api = { path = "../qemu-api" }
> +libc = "0.2.162"
I think it's necessary add the lints section to apply workspace's lint
configurations:
[lints]
workspace = true
> diff --git a/rust/block/README.md b/rust/block/README.md
> new file mode 100644
> index 0000000000..debcc9d815
> --- /dev/null
> +++ b/rust/block/README.md
> @@ -0,0 +1,3 @@
> +# QEMU block backends
> +
> +This library implements block drivers for QEMU.
And doc can be updated in docs/devel/rust.rst, like Paolo's patch:
20250218080455.426383-1-pbonzini@redhat.com/">https://lore.kernel.org/qemu-devel/20250218080455.426383-1-pbonzini@redhat.com/
> diff --git a/rust/block/meson.build b/rust/block/meson.build
> new file mode 100644
> index 0000000000..ca93afd939
> --- /dev/null
> +++ b/rust/block/meson.build
> @@ -0,0 +1,20 @@
> +_block_rs = static_library(
> + 'block',
> + files('src/lib.rs'),
> + override_options: ['rust_std=2021', 'build.rust_std=2021'],
> + rust_abi: 'rust',
> + dependencies: [
> + qemu_api_tools,
> + qemu_api_macros,
I'm unsure without testing, the dependencies here looks different with
the Cargo.toml?
Regards,
Zhao
- [PATCH v2 00/11] rust/block: Add minimal block driver bindings, Kevin Wolf, 2025/02/18
- [PATCH v2 02/11] meson: Add rust_block_ss and link tools with it, Kevin Wolf, 2025/02/18
- [PATCH v2 01/11] rust: Build separate qemu_api_tools and qemu_api_system, Kevin Wolf, 2025/02/18
- [PATCH v2 05/11] rust/block: Add empty crate, Kevin Wolf, 2025/02/18
- Re: [PATCH v2 05/11] rust/block: Add empty crate,
Zhao Liu <=
- [PATCH v2 04/11] rust/qemu-api: Add wrappers to run futures in QEMU, Kevin Wolf, 2025/02/18
- [PATCH v2 06/11] rust/block: Add I/O buffer traits, Kevin Wolf, 2025/02/18
- [PATCH v2 08/11] rust/block: Add driver module, Kevin Wolf, 2025/02/18
- [PATCH v2 09/11] rust/block: Add read support for block drivers, Kevin Wolf, 2025/02/18