Re: [PATCH] riscv: hwprobe: Register unaligned probes before usermode

Re: [PATCH] riscv: hwprobe: Register unaligned probes before usermode

[Geert Uytterhoeven]

Hi Paul,

On Tue, 11 Aug 2026 at 10:45, Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> On Sat, 8 Aug 2026 at 01:30, Paul Walmsley <pjw@kernel.org> wrote:
> > On Fri, 7 Aug 2026, Nam Cao wrote:
> > > Paul Walmsley <pjw@kernel.org> writes:
> > > > Now that we've queued a patch to get rid of the async hwprobe
> > > > infrastructure, is this patch still needed?
> > >
> > > Yes. Removing async probe only ensures probe is finished before the
> > > kernel is done booting.
> > >
> > > But there are usermode helpers which run while the kernel is still booting.
> > > This patch ensures probe is done before that.
> >
> > Okay, thanks for taking a look.
> >
> > Thanks Rui Qi for the patch; queued for v7.2-rc with a cc: to stable.
>
> This commit is now commit 994dad686e755477 in riscv/fixes.
> However, riscv/for-next contains the similar but empty commit
> acf3fcb5e9199cc9.  Presumably this is the result of a bad rebasing of
> for-next on top of fixes?

The empty commit was rebased, and is now commit 4e864f1c3d8d147b.
Please drop it (it breaks my scripting ;-).

Thanks!

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Re: [PATCH] riscv: hwprobe: Register unaligned probes before usermode

[Paul Walmsley]

Hi Geert,

On Fri, 14 Aug 2026, Geert Uytterhoeven wrote:

> On Tue, 11 Aug 2026 at 10:45, Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> > On Sat, 8 Aug 2026 at 01:30, Paul Walmsley <pjw@kernel.org> wrote:
> > > On Fri, 7 Aug 2026, Nam Cao wrote:
> > > > Paul Walmsley <pjw@kernel.org> writes:
> > > > > Now that we've queued a patch to get rid of the async hwprobe
> > > > > infrastructure, is this patch still needed?
> > > >
> > > > Yes. Removing async probe only ensures probe is finished before the
> > > > kernel is done booting.
> > > >
> > > > But there are usermode helpers which run while the kernel is still booting.
> > > > This patch ensures probe is done before that.
> > >
> > > Okay, thanks for taking a look.
> > >
> > > Thanks Rui Qi for the patch; queued for v7.2-rc with a cc: to stable.
> >
> > This commit is now commit 994dad686e755477 in riscv/fixes.
> > However, riscv/for-next contains the similar but empty commit
> > acf3fcb5e9199cc9.  Presumably this is the result of a bad rebasing of
> > for-next on top of fixes?
> 
> The empty commit was rebased, and is now commit 4e864f1c3d8d147b.
> Please drop it (it breaks my scripting ;-).

Fixed!  Sorry about that,


- Paul