[PATCH] RISC-V: Update AT_VECTOR_SIZE_ARCH for new AT_MINSIGSTKSZ

Conor Dooley conor at kernel.org
Sat Dec 16 03:45:33 PST 2023


Hey,

A few process issues here unfortunately.

On Fri, Dec 15, 2023 at 11:27:20PM -0500, Victor Isaev wrote:
> "riscv: signal: Report signal frame size to userspace via auxv" (e92f469)
> has added new constant AT_MINSIGSTKSZ but failed to increment the size of
> auxv, keeping AT_VECTOR_SIZE_ARCH at 9.
> This fix correctly increments AT_VECTOR_SIZE_ARCH to 10, following the 
> approach in the commit 94b07c1 ("arm64: signal: Report signal frame size
> to userspace via auxv").

Both of your references to commits do not use the correct style, that
checkpatch would have told you about:
ERROR: Please use git commit description style 'commit <12+ chars of sha1> ("<title line>")' - ie: 'commit 94b07c1f8c39 ("arm64: signal: Report signal frame size to userspace via auxv")'
#97: 
approach in the commit 94b07c1 ("arm64: signal: Report signal frame size
to userspace via auxv").

Also, you should add a Fixes: tag for the first commit you mention here
since it is the source of the bug.

> ---
>  a/arch/riscv/include/uapi/asm/auxvec.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/riscv/include/uapi/asm/auxvec.h b/arch/riscv/include/uapi/asm/auxvec.h
> index 10aaa83db89e..95050ebe9ad0 100644
> --- a/arch/riscv/include/uapi/asm/auxvec.h
> +++ b/arch/riscv/include/uapi/asm/auxvec.h
> @@ -34,7 +34,7 @@
>  #define AT_L3_CACHEGEOMETRY    47
> 
>  /* entries in ARCH_DLINFO */
> -#define AT_VECTOR_SIZE_ARCH    9
> +#define AT_VECTOR_SIZE_ARCH    10
>  #define AT_MINSIGSTKSZ         51

This patch is corrupted with tabs changed to spaces and cannot be
applied.

> 
>  #endif /* _UAPI_ASM_RISCV_AUXVEC_H */
> ---
> 

> Signed-off-by: Victor Isaev <isv at google.com>

This needs to be in the commit message, before the --- line.
Also, you are missing a from header in your message that needs to be
added when the sending email does not match the one that has been
attributed authorship.

> Signed-off-by: Ivan Komarov <ivan.komarov at dfyz.info>

What was Ivan's contribution to the patch?

Cheers,
Conor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-riscv/attachments/20231216/879976ec/attachment.sig>


More information about the linux-riscv mailing list