Re: [PATCH v2] rust_binder: Downgrade startup warning to info

From: Alice Ryhl

Date: Fri Mar 27 2026 - 05:37:05 EST


On Fri, Mar 27, 2026 at 07:24:35AM +0100, Greg KH wrote:
> On Thu, Mar 26, 2026 at 11:07:12PM +0000, Carlos Llamas wrote:
> > From: Pedro Montes Alcalde <pedro.montes.alcalde@xxxxxxxxx>
> >
> > The "Loaded Rust Binder." message is logged during normal
> > initialization and does not indicate an error/warning condition.
> >
> > Logging it as a warning creates unnecessary noise and may
> > mislead developers when inspecting logs. Log it as info instead
> >
> > Signed-off-by: Pedro Montes Alcalde <pedro.montes.alcalde@xxxxxxxxx>
> > Acked-by: Carlos Llamas <cmllamas@xxxxxxxxxx>
> > Signed-off-by: Carlos Llamas <cmllamas@xxxxxxxxxx>
> > ---
> > v2: fixed patch formatting, added tags.
> >
> > drivers/android/binder/rust_binder_main.rs | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/android/binder/rust_binder_main.rs b/drivers/android/binder/rust_binder_main.rs
> > index aa5f2a75adb4..3907291a33c9 100644
> > --- a/drivers/android/binder/rust_binder_main.rs
> > +++ b/drivers/android/binder/rust_binder_main.rs
> > @@ -292,7 +292,7 @@ fn init(_module: &'static kernel::ThisModule) -> Result<Self> {
> > // SAFETY: The module initializer never runs twice, so we only call this once.
> > unsafe { crate::context::CONTEXTS.init() };
> >
> > - pr_warn!("Loaded Rust Binder.");
> > + pr_info!("Loaded Rust Binder.");
>
> When drivers work properly, they are quiet, so why print anything out at
> all here?

Yeah let's just delete this. It's not needed anymore.

Alice