From: Luis Chamberlain <mcgrof@kernel.org>
To: Andreas Hindborg <nmi@metaspace.dk>
Cc: Miguel Ojeda <ojeda@kernel.org>,
rust-for-linux@vger.kernel.org, linux-modules@vger.kernel.org,
linux-kernel@vger.kernel.org,
Andreas Hindborg <a.hindborg@samsung.com>,
Adam Bratschi-Kaye <ark.email@gmail.com>,
Sami Tolvanen <samitolvanen@google.com>
Subject: Re: [PATCH] rust: add `module_params` macro
Date: Mon, 8 Jul 2024 14:42:30 -0700 [thread overview]
Message-ID: <ZoxdRjpy2hRndqmc@bombadil.infradead.org> (raw)
In-Reply-To: <20240705111455.142790-1-nmi@metaspace.dk>
On Fri, Jul 05, 2024 at 11:15:11AM +0000, Andreas Hindborg wrote:
> From: Andreas Hindborg <a.hindborg@samsung.com>
>
> This patch includes changes required for Rust kernel modules to utilize
> module parameters. This code implements read only support for integer
> types without `sysfs` support.
>
> This code is a reduced and updated version of code by Adam available in the
> original `rust` branch [1].
>
> [1] https://github.com/Rust-for-Linux/linux/tree/bc22545f38d74473cfef3e9fd65432733435b79f
>
> Cc: Adam Bratschi-Kaye <ark.email@gmail.com>
> Signed-off-by: Andreas Hindborg <a.hindborg@samsung.com>
>
This poses an interesting challenge I'd like to take up with the Rust
community. I'm fine with Rust bindings, however many C maintainers
neither don't speak / write Rust, and in many cases many don't even want
to touch Rust at all. In my case I want to get there.. but just haven't
had time yet. So we have to live with that world. But to help with a
Rust world, clearly we need to allow for some Rust bindings.
As a compromise, I recently suggested for example for the firmware_loader
Rust bindings to be acceptable if and only if we could get the developer
doing those changes to be willing to commit to also being *both* a C and
rust maintainer for the firmware loader.
I'm starting to feel the same way about modules, but modules requires
more work than the firmware loader. And since I also know Andreas has
already a lot on his plate, I'm at a cross roads. My above request for
the firmware loader made sense to the person working on the firmware
loader changes, but who would help on the modules side of things? And
does this request make sense to help scale?
The rationale here is that a rust binding means commitment then also
from fresh blood to help co-maintain review C / Rust for exising code
when there is will / desire to collaborate from an existing C maintainer.
I realize this may be a lot to ask, but I think this is one of the
responsible ways to ask to scale here.
Luis
next prev parent reply other threads:[~2024-07-08 21:42 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-07-05 11:15 [PATCH] rust: add `module_params` macro Andreas Hindborg
2024-07-05 11:33 ` Andreas Hindborg
2024-07-08 21:42 ` Luis Chamberlain [this message]
2024-07-09 6:00 ` nmi
2024-07-09 8:27 ` Greg KH
2024-07-09 9:25 ` nmi
2024-07-09 10:08 ` Miguel Ojeda
2024-07-18 17:15 ` Luis Chamberlain
2024-07-24 17:04 ` Sami Tolvanen
[not found] ` <CGME20240819200056eucas1p1589eddef05f72d97836f1d5be889048b@eucas1p1.samsung.com>
2024-08-19 20:00 ` Daniel Gomez
2024-08-19 21:59 ` Luis Chamberlain
2024-07-29 17:16 ` Benno Lossin
2024-08-01 11:29 ` Andreas Hindborg
2024-08-01 12:25 ` Benno Lossin
2024-08-01 13:40 ` Andreas Hindborg
2024-08-01 14:21 ` Benno Lossin
2024-08-01 15:11 ` Andreas Hindborg
2024-08-01 19:28 ` Benno Lossin
2024-08-02 10:27 ` Andreas Hindborg
2024-08-02 17:11 ` Benno Lossin
2024-08-05 10:55 ` Andreas Hindborg
2024-08-06 8:09 ` Benno Lossin
2024-08-15 13:11 ` Andreas Hindborg
2024-08-15 19:33 ` Benno Lossin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZoxdRjpy2hRndqmc@bombadil.infradead.org \
--to=mcgrof@kernel.org \
--cc=a.hindborg@samsung.com \
--cc=ark.email@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-modules@vger.kernel.org \
--cc=nmi@metaspace.dk \
--cc=ojeda@kernel.org \
--cc=rust-for-linux@vger.kernel.org \
--cc=samitolvanen@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).