[PATCH v2 00/19] rust: replace `kernel::c_str!` with C-Strings

Tamir Duberstein posted 19 patches 4 months, 2 weeks ago
There is a newer version of this series
drivers/block/rnull.rs                |  2 +-
drivers/cpufreq/rcpufreq_dt.rs        |  5 ++---
drivers/gpu/drm/nova/driver.rs        | 10 +++++-----
drivers/gpu/nova-core/driver.rs       |  6 +++---
drivers/net/phy/ax88796b_rust.rs      |  7 +++----
drivers/net/phy/qt2025.rs             |  5 ++---
rust/kernel/clk.rs                    |  6 ++----
rust/kernel/configfs.rs               |  9 +++++----
rust/kernel/cpufreq.rs                |  3 +--
rust/kernel/device.rs                 |  4 +---
rust/kernel/device/property.rs        |  6 +++---
rust/kernel/firmware.rs               |  6 +++---
rust/kernel/io/mem.rs                 |  7 +++----
rust/kernel/kunit.rs                  | 11 ++++-------
rust/kernel/net/phy.rs                |  6 ++----
rust/kernel/platform.rs               |  6 +++---
rust/kernel/regulator.rs              |  9 +++------
rust/kernel/seq_file.rs               |  4 ++--
rust/kernel/str.rs                    |  5 ++---
rust/kernel/sync.rs                   |  5 ++---
rust/kernel/sync/completion.rs        |  2 +-
rust/kernel/workqueue.rs              |  8 ++++----
rust/macros/kunit.rs                  | 10 +++++-----
rust/macros/module.rs                 |  2 +-
samples/rust/rust_configfs.rs         |  5 ++---
samples/rust/rust_driver_auxiliary.rs |  4 ++--
samples/rust/rust_driver_faux.rs      |  4 ++--
samples/rust/rust_driver_pci.rs       |  4 ++--
samples/rust/rust_driver_platform.rs  | 30 ++++++++++++++----------------
samples/rust/rust_misc_device.rs      |  3 +--
scripts/rustdoc_test_gen.rs           |  4 ++--
31 files changed, 88 insertions(+), 110 deletions(-)
[PATCH v2 00/19] rust: replace `kernel::c_str!` with C-Strings
Posted by Tamir Duberstein 4 months, 2 weeks ago
This series depends on step 3[0].

Subsystem maintainers: I would appreciate your `Acked-by`s so that this
can be taken through Miguel's tree (where the previous series must go).

Link: https://lore.kernel.org/all/20250925-cstr-core-v16-0-5cdcb3470ec2@gmail.com/ [0]

Signed-off-by: Tamir Duberstein <tamird@gmail.com>
---
Changes in v2:
- Rebase.
- Add two patches to address new code.
- Drop incorrectly applied Acked-by tags from Danilo.
- Link to v1: https://lore.kernel.org/r/20250710-core-cstr-cstrings-v1-0-027420ea799e@gmail.com

---
Tamir Duberstein (19):
      drivers: net: replace `kernel::c_str!` with C-Strings
      gpu: nova-core: replace `kernel::c_str!` with C-Strings
      rust: auxiliary: replace `kernel::c_str!` with C-Strings
      rust: clk: replace `kernel::c_str!` with C-Strings
      rust: configfs: replace `kernel::c_str!` with C-Strings
      rust: cpufreq: replace `kernel::c_str!` with C-Strings
      rust: device: replace `kernel::c_str!` with C-Strings
      rust: firmware: replace `kernel::c_str!` with C-Strings
      rust: kunit: replace `kernel::c_str!` with C-Strings
      rust: macros: replace `kernel::c_str!` with C-Strings
      rust: miscdevice: replace `kernel::c_str!` with C-Strings
      rust: net: replace `kernel::c_str!` with C-Strings
      rust: pci: replace `kernel::c_str!` with C-Strings
      rust: platform: replace `kernel::c_str!` with C-Strings
      rust: seq_file: replace `kernel::c_str!` with C-Strings
      rust: str: replace `kernel::c_str!` with C-Strings
      rust: sync: replace `kernel::c_str!` with C-Strings
      rust: io: replace `kernel::c_str!` with C-Strings
      rust: regulator: replace `kernel::c_str!` with C-Strings

 drivers/block/rnull.rs                |  2 +-
 drivers/cpufreq/rcpufreq_dt.rs        |  5 ++---
 drivers/gpu/drm/nova/driver.rs        | 10 +++++-----
 drivers/gpu/nova-core/driver.rs       |  6 +++---
 drivers/net/phy/ax88796b_rust.rs      |  7 +++----
 drivers/net/phy/qt2025.rs             |  5 ++---
 rust/kernel/clk.rs                    |  6 ++----
 rust/kernel/configfs.rs               |  9 +++++----
 rust/kernel/cpufreq.rs                |  3 +--
 rust/kernel/device.rs                 |  4 +---
 rust/kernel/device/property.rs        |  6 +++---
 rust/kernel/firmware.rs               |  6 +++---
 rust/kernel/io/mem.rs                 |  7 +++----
 rust/kernel/kunit.rs                  | 11 ++++-------
 rust/kernel/net/phy.rs                |  6 ++----
 rust/kernel/platform.rs               |  6 +++---
 rust/kernel/regulator.rs              |  9 +++------
 rust/kernel/seq_file.rs               |  4 ++--
 rust/kernel/str.rs                    |  5 ++---
 rust/kernel/sync.rs                   |  5 ++---
 rust/kernel/sync/completion.rs        |  2 +-
 rust/kernel/workqueue.rs              |  8 ++++----
 rust/macros/kunit.rs                  | 10 +++++-----
 rust/macros/module.rs                 |  2 +-
 samples/rust/rust_configfs.rs         |  5 ++---
 samples/rust/rust_driver_auxiliary.rs |  4 ++--
 samples/rust/rust_driver_faux.rs      |  4 ++--
 samples/rust/rust_driver_pci.rs       |  4 ++--
 samples/rust/rust_driver_platform.rs  | 30 ++++++++++++++----------------
 samples/rust/rust_misc_device.rs      |  3 +--
 scripts/rustdoc_test_gen.rs           |  4 ++--
 31 files changed, 88 insertions(+), 110 deletions(-)
---
base-commit: f3f6b3664302e16ef1c6b91034a72df5564d6b8a
change-id: 20250710-core-cstr-cstrings-1faaa632f0fd
prerequisite-change-id: 20250201-cstr-core-d4b9b69120cf:v16
prerequisite-patch-id: e0ca756f740ab0ce7478bbf6510948ba89529a2f
prerequisite-patch-id: 6d8dbdf864f79fc0c2820e702a7cb87753649ca0
prerequisite-patch-id: 7d4d1d036043a85dcbaf0d09ea85768120efe094

Best regards,
--  
Tamir Duberstein <tamird@gmail.com>
Re: [PATCH v2 00/19] rust: replace `kernel::c_str!` with C-Strings
Posted by Mark Brown 4 months, 2 weeks ago
On Thu, Sep 25, 2025 at 09:53:48AM -0400, Tamir Duberstein wrote:
> This series depends on step 3[0].
> 
> Subsystem maintainers: I would appreciate your `Acked-by`s so that this
> can be taken through Miguel's tree (where the previous series must go).

Given that we're almost at the merge window isn't it likely that these
will get applied once the current rust tree is in mainline?
Re: [PATCH v2 00/19] rust: replace `kernel::c_str!` with C-Strings
Posted by Miguel Ojeda 4 months, 2 weeks ago
On Thu, Sep 25, 2025 at 4:01 PM Mark Brown <broonie@debian.org> wrote:
>
> Given that we're almost at the merge window isn't it likely that these
> will get applied once the current rust tree is in mainline?

Yeah, I am submitting the PR to Linus very soon anyway.

Cheers,
Miguel
Re: [PATCH v2 00/19] rust: replace `kernel::c_str!` with C-Strings
Posted by Miguel Ojeda 4 months, 1 week ago
On Thu, Sep 25, 2025 at 3:54 PM Tamir Duberstein <tamird@gmail.com> wrote:
>
> Changes in v2:

For future reference, this is v3.

Cheers,
Miguel
Re: (subset) [PATCH v2 00/19] rust: replace `kernel::c_str!` with C-Strings
Posted by Mark Brown 1 month, 2 weeks ago
On Thu, 25 Sep 2025 09:53:48 -0400, Tamir Duberstein wrote:
> This series depends on step 3[0].
> 
> Subsystem maintainers: I would appreciate your `Acked-by`s so that this
> can be taken through Miguel's tree (where the previous series must go).
> 
> Link: https://lore.kernel.org/all/20250925-cstr-core-v16-0-5cdcb3470ec2@gmail.com/ [0]
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/regulator.git for-next

Thanks!

[19/19] rust: regulator: replace `kernel::c_str!` with C-Strings
        commit: b0655377aa5a410df02d89170c20141a1a5bbc28

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark