Re: [PATCH] riscv: Check the return value of reloc handlers

From: Markus Elfring

Date: Sun Mar 22 2026 - 03:27:38 EST


> Currently, process_accumulated_relocations() ignores the return values
> from both reloc_handler() and accumulate_handler().

Were any source code analysis tools involved here?



> So we need to check the return values of the handlers to propagate the
> error, and fall back to the cleanup mode.

See also once more:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v7.0-rc4#n94


> Fixes: 8fd6c5142395 ("riscv: Add remaining module relocations")

See also once more:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/stable-kernel-rules.rst?h=v7.0-rc4#n34


> Signed-off-by: Zishun Yi <vulab@xxxxxxxxxxx>

Do multiple personal names fit really to the same email address according to
the Developer's Certificate of Origin?
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v7.0-rc4#n396

Were code review results presented from an “university laboratory” so far?
https://github.com/ISCAS-Vulab/
https://english.is.cas.cn/

Regards,
Markus