Re: [PATCH v3 0/5] linux-user/riscv: add vector state to signal context

[Nicholas Piggin <npiggin@gmail.com>]

On Wed, Mar 25, 2026 at 02:45:24PM +1000, Alistair Francis wrote:
> On Sun, Mar 22, 2026 at 12:16 AM Nicholas Piggin <npiggin@gmail.com> wrote:
> >
> > Changes in v3:
> > - Fixed up a bunch of good review comments that Richard made against
> >   the v1 series here https://lore.kernel.org/all/06afcbdc-28e2-4d13-81d4-26fd257ed8a3@linaro.org/#r
> >   in particular:
> >   - Restoring vl/vtype state using CPU accessors rather than stuffing the
> >     values into the registers directly.
> >   - Fix ABI types.
> >   - Fix some confusion with host/guest pointer mapping.
> >   - Fix error handling to SIGSEGV.
> >   - Style fixes.
> > - Fixed build issue without riscv intrinsics header.
> > - Add some more test cases.
> >
> > Changes in v2:
> > - Fix 32-bit compile
> > - Moved sizeof_rt_sigframe change to its correct patch (patch 2 is the
> >   one that extends the size of rt_sigframe, patch 3 is adding new
> >   extended state to the frame which is added dynamically).
> >
> > This series adds vector state to the linux-user signal handler,
> > and adds a basic signal handling test case. As a sanity check, I
> > also verified the signal handling test works in the same way when
> > run under a real Linux kernel.
> >
> > Thanks,
> > Nick
> >
> > Nicholas Piggin (5):
> >   tests/tcg/riscv64: Add a user signal handling test
> >   linux-user/riscv: Allow restore_sigcontext to return error
> >   linux-user/riscv: Add extended state to sigcontext
> >   linux-user/riscv: Add vector state to signal context
> >   tests/tcg/riscv64: Add vector state to signal test
> 
> Can you rebase this on the latest master branch?

Will do. I'll give a few more days then send out updates. Thanks
for the reviews.

Thanks,
Nick