| Message ID | 20231123185821.2272504-1-christoph.muellner@vrull.eu (mailing list archive) |
|---|---|
| Headers | show |
| Series | tools: selftests: riscv: Fix compiler warnings | expand |
On Thu, Nov 23, 2023 at 07:58:16PM +0100, Christoph Muellner wrote: > From: Christoph Müllner <christoph.muellner@vrull.eu> > > When building the RISC-V selftests with a riscv32 compiler I ran into > a couple of compiler warnings. While riscv32 support for these tests is > questionable, the fixes are so trivial that it is probably best to simply > apply them. > > Note that the missing-include patch and some format string warnings > are also relevant for riscv64. I also posted [1] a couple days ago for the format warnings, but, as this series also includes rv32 fixes, then we can drop [1] in favor of this. [1] https://lore.kernel.org/all/20231122171821.130854-2-ajones@ventanamicro.com/ For the series, Reviewed-by: Andrew Jones <ajones@ventanamicro.com> Thanks, drew
Ping. It would be great to have these compiler warnings fixed before the release. On Fri, Nov 24, 2023 at 10:40 AM Andrew Jones <ajones@ventanamicro.com> wrote: > > On Thu, Nov 23, 2023 at 07:58:16PM +0100, Christoph Muellner wrote: > > From: Christoph Müllner <christoph.muellner@vrull.eu> > > > > When building the RISC-V selftests with a riscv32 compiler I ran into > > a couple of compiler warnings. While riscv32 support for these tests is > > questionable, the fixes are so trivial that it is probably best to simply > > apply them. > > > > Note that the missing-include patch and some format string warnings > > are also relevant for riscv64. > > I also posted [1] a couple days ago for the format warnings, but, as this > series also includes rv32 fixes, then we can drop [1] in favor of this. > > [1] https://lore.kernel.org/all/20231122171821.130854-2-ajones@ventanamicro.com/ > > For the series, > > Reviewed-by: Andrew Jones <ajones@ventanamicro.com> > > Thanks, > drew
Hello: This series was applied to riscv/linux.git (for-next) by Palmer Dabbelt <palmer@rivosinc.com>: On Thu, 23 Nov 2023 19:58:16 +0100 you wrote: > From: Christoph Müllner <christoph.muellner@vrull.eu> > > When building the RISC-V selftests with a riscv32 compiler I ran into > a couple of compiler warnings. While riscv32 support for these tests is > questionable, the fixes are so trivial that it is probably best to simply > apply them. > > [...] Here is the summary with links: - [1/5] tools: selftests: riscv: Fix compile warnings in hwprobe https://git.kernel.org/riscv/c/b91c26fdb0e8 - [2/5] tools: selftests: riscv: Fix compile warnings in cbo https://git.kernel.org/riscv/c/ac7b2a02d62f - [3/5] tools: selftests: riscv: Add missing include for vector test https://git.kernel.org/riscv/c/b250c9089841 - [4/5] tools: selftests: riscv: Fix compile warnings in vector tests https://git.kernel.org/riscv/c/e1baf5e68ed1 - [5/5] tools: selftests: riscv: Fix compile warnings in mm tests https://git.kernel.org/riscv/c/12c16919652b You are awesome, thank you!
From: Christoph Müllner <christoph.muellner@vrull.eu> When building the RISC-V selftests with a riscv32 compiler I ran into a couple of compiler warnings. While riscv32 support for these tests is questionable, the fixes are so trivial that it is probably best to simply apply them. Note that the missing-include patch and some format string warnings are also relevant for riscv64. Christoph Müllner (5): tools: selftests: riscv: Fix compile warnings in hwprobe tools: selftests: riscv: Fix compile warnings in cbo tools: selftests: riscv: Add missing include for vector test tools: selftests: riscv: Fix compile warnings in vector tests tools: selftests: riscv: Fix compile warnings in mm tests tools/testing/selftests/riscv/hwprobe/cbo.c | 6 +++--- tools/testing/selftests/riscv/hwprobe/hwprobe.c | 4 ++-- tools/testing/selftests/riscv/mm/mmap_test.h | 3 +++ tools/testing/selftests/riscv/vector/v_initval_nolibc.c | 2 +- tools/testing/selftests/riscv/vector/vstate_exec_nolibc.c | 3 +++ tools/testing/selftests/riscv/vector/vstate_prctl.c | 4 ++-- 6 files changed, 14 insertions(+), 8 deletions(-)