Skip to content

[PW_SID:955600] Move duplicated instructions macros into asm/insn.h#325

Closed
linux-riscv-bot wants to merge 4 commits into
workflow__riscv__fixesfrom
pw955600
Closed

[PW_SID:955600] Move duplicated instructions macros into asm/insn.h#325
linux-riscv-bot wants to merge 4 commits into
workflow__riscv__fixesfrom
pw955600

Conversation

@linux-riscv-bot
Copy link
Copy Markdown

@linux-riscv-bot linux-riscv-bot commented Apr 22, 2025

PR for series 955600 applied to workflow__riscv__fixes

Name: Move duplicated instructions macros into asm/insn.h
URL: https://patchwork.kernel.org/project/linux-riscv/list/?series=955600
Version: 1

Linux RISC-V bot and others added 4 commits April 16, 2025 18:23
Adding CI files
6c2725a
riscv: Fix typo EXRACT -> EXTRACT
b1543b0 
Simply fix a typo.

Signed-off-by: Alexandre Ghiti <alexghiti@rivosinc.com>
Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
riscv: Strengthen duplicate and inconsistent definition of RV_X()
e25c49c 
RV_X() macro is defined in two different ways which is error prone.

So harmonize its first definition and add another macro RV_X_mask() for
the second one.

Signed-off-by: Alexandre Ghiti <alexghiti@rivosinc.com>
Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
riscv: Move all duplicate insn parsing macros into asm/insn.h
d515ccd 
kernel/traps_misaligned.c and kvm/vcpu_insn.c define the same macros to
extract information from the instructions.

Let's move the definitions into asm/insn.h to avoid this duplication.

Signed-off-by: Alexandre Ghiti <alexghiti@rivosinc.com>
Signed-off-by: Linux RISC-V bot <linux.riscv.bot@gmail.com>
@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 1: "[1/3] riscv: Fix typo EXRACT -> EXTRACT"
build-rv32-defconfig
Desc: Builds riscv32 defconfig
Duration: 102.09 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 1: "[1/3] riscv: Fix typo EXRACT -> EXTRACT"
build-rv64-clang-allmodconfig
Desc: Builds riscv64 allmodconfig with Clang, and checks for errors and added warnings
Duration: 977.06 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 1: "[1/3] riscv: Fix typo EXRACT -> EXTRACT"
build-rv64-gcc-allmodconfig
Desc: Builds riscv64 allmodconfig with GCC, and checks for errors and added warnings
Duration: 1293.45 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 1: "[1/3] riscv: Fix typo EXRACT -> EXTRACT"
build-rv64-nommu-k210-defconfig
Desc: Builds riscv64 defconfig with NOMMU for K210
Duration: 19.77 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 1: "[1/3] riscv: Fix typo EXRACT -> EXTRACT"
build-rv64-nommu-k210-virt
Desc: Builds riscv64 defconfig with NOMMU for the virt platform
Duration: 21.02 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 1: "[1/3] riscv: Fix typo EXRACT -> EXTRACT"
checkpatch
Desc: Runs checkpatch.pl on the patch
Duration: 0.93 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 1: "[1/3] riscv: Fix typo EXRACT -> EXTRACT"
dtb-warn-rv64
Desc: Checks for Device Tree warnings/errors
Duration: 38.79 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 1: "[1/3] riscv: Fix typo EXRACT -> EXTRACT"
header-inline
Desc: Detects static functions without inline keyword in header files
Duration: 0.23 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 1: "[1/3] riscv: Fix typo EXRACT -> EXTRACT"
kdoc
Desc: Detects for kdoc errors
Duration: 0.86 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 1: "[1/3] riscv: Fix typo EXRACT -> EXTRACT"
module-param
Desc: Detect module_param changes
Duration: 0.24 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 1: "[1/3] riscv: Fix typo EXRACT -> EXTRACT"
verify-fixes
Desc: Verifies that the Fixes: tags exist
Duration: 0.21 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 1: "[1/3] riscv: Fix typo EXRACT -> EXTRACT"
verify-signedoff
Desc: Verifies that Signed-off-by: tags are correct
Duration: 0.28 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 2: "[2/3] riscv: Strengthen duplicate and inconsistent definition of RV_X()"
build-rv32-defconfig
Desc: Builds riscv32 defconfig
Duration: 88.85 seconds
Result: ERROR
Output:

Full log:
W: Support for running offline not available (unshare: unshare failed: Operation not permitted)
tuxmake --download-all-korg-gcc-toolchains --target-arch=riscv --kconfig=rv32_defconfig --toolchain=llvm --wrapper=ccache --environment=KBUILD_BUILD_TIMESTAMP=@1621270510 --environment=KBUILD_BUILD_USER=tuxmake --environment=KBUILD_BUILD_HOST=tuxmake --environment=KCFLAGS=-ffile-prefix-map=/build/tmp.Hq76sLxdqI/build/= --runtime=null --image=docker.io/tuxmake/riscv_clang CROSS_COMPILE=riscv64-linux- config default kernel xipkernel modules dtbs dtbs-legacy debugkernel headers
make --silent --keep-going --jobs=48 O=/build/tmp.Hq76sLxdqI/build ARCH=riscv CROSS_COMPILE=riscv64-linux- LLVM=1 'CC=ccache clang' 'HOSTCC=ccache clang' rv32_defconfig
make --silent --keep-going --jobs=48 O=/build/tmp.Hq76sLxdqI/build ARCH=riscv CROSS_COMPILE=riscv64-linux- LLVM=1 'CC=ccache clang' 'HOSTCC=ccache clang'
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:549:10: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  549 |                 insn = RVC_RS2S(insn) << SH_RD;
      |                        ^
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:110:30: note: expanded from macro 'RVC_RS2S'
  110 | #define RVC_RS2S(insn)          (8 + RV_X(insn, SH_RS2C, 3))
      |                                      ^
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:664:12: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  664 |                 data32 = GET_RS2S(insn, &vcpu->arch.guest_context);
      |                          ^
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:130:40: note: expanded from macro 'GET_RS2S'
  130 | #define GET_RS2S(insn, regs)    (*REG_PTR(RVC_RS2S(insn), 0, regs))
      |                                           ^
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:110:30: note: expanded from macro 'RVC_RS2S'
  110 | #define RVC_RS2S(insn)          (8 + RV_X(insn, SH_RS2C, 3))
      |                                      ^
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:640:9: warning: shift count is negative [-Wshift-count-negative]
  640 |         data = GET_RS2(insn, &vcpu->arch.guest_context);
      |                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:128:31: note: expanded from macro 'GET_RS2'
  128 | #define GET_RS2(insn, regs)     (*REG_PTR(insn, SH_RS2, regs))
      |                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:123:29: note: expanded from macro 'REG_PTR'
  123 |         ((ulong *)((ulong)(regs) + REG_OFFSET(insn, pos)))
      |                                    ^~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:120:3: note: expanded from macro 'REG_OFFSET'
  120 |         (SHIFT_RIGHT((insn), (pos) - LOG_REGBYTES) & REG_MASK)
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:114:18: note: expanded from macro 'SHIFT_RIGHT'
  114 |         ((y) < 0 ? ((x) << -(y)) : ((x) >> (y)))
      |                         ^  ~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:664:12: warning: shift count is negative [-Wshift-count-negative]
  664 |                 data32 = GET_RS2S(insn, &vcpu->arch.guest_context);
      |                          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:130:32: note: expanded from macro 'GET_RS2S'
  130 | #define GET_RS2S(insn, regs)    (*REG_PTR(RVC_RS2S(insn), 0, regs))
      |                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:123:29: note: expanded from macro 'REG_PTR'
  123 |         ((ulong *)((ulong)(regs) + REG_OFFSET(insn, pos)))
      |                                    ^~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:120:3: note: expanded from macro 'REG_OFFSET'
  120 |         (SHIFT_RIGHT((insn), (pos) - LOG_REGBYTES) & REG_MASK)
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:114:34: note: expanded from macro 'SHIFT_RIGHT'
  114 |         ((y) < 0 ? ((x) << -(y)) : ((x) >> (y)))
      |                                         ^  ~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:754:3: warning: shift count is negative [-Wshift-count-negative]
  754 |                 SET_RD(insn, &vcpu->arch.guest_context,
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  755 |                         (ulong)data8 << shift >> shift);
      |                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:133:35: note: expanded from macro 'SET_RD'
  133 | #define SET_RD(insn, regs, val) (*REG_PTR(insn, SH_RD, regs) = (val))
      |                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:123:29: note: expanded from macro 'REG_PTR'
  123 |         ((ulong *)((ulong)(regs) + REG_OFFSET(insn, pos)))
      |                                    ^~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:120:3: note: expanded from macro 'REG_OFFSET'
  120 |         (SHIFT_RIGHT((insn), (pos) - LOG_REGBYTES) & REG_MASK)
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:114:18: note: expanded from macro 'SHIFT_RIGHT'
  114 |         ((y) < 0 ? ((x) << -(y)) : ((x) >> (y)))
      |                         ^  ~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:759:3: warning: shift count is negative [-Wshift-count-negative]
  759 |                 SET_RD(insn, &vcpu->arch.guest_context,
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  760 |                         (ulong)data16 << shift >> shift);
      |                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:133:35: note: expanded from macro 'SET_RD'
  133 | #define SET_RD(insn, regs, val) (*REG_PTR(insn, SH_RD, regs) = (val))
      |                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:123:29: note: expanded from macro 'REG_PTR'
  123 |         ((ulong *)((ulong)(regs) + REG_OFFSET(insn, pos)))
      |                                    ^~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:120:3: note: expanded from macro 'REG_OFFSET'
  120 |         (SHIFT_RIGHT((insn), (pos) - LOG_REGBYTES) & REG_MASK)
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:114:18: note: expanded from macro 'SHIFT_RIGHT'
  114 |         ((y) < 0 ? ((x) << -(y)) : ((x) >> (y)))
      |                         ^  ~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:764:3: warning: shift count is negative [-Wshift-count-negative]
  764 |                 SET_RD(insn, &vcpu->arch.guest_context,
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  765 |                         (ulong)data32 << shift >> shift);
      |                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:133:35: note: expanded from macro 'SET_RD'
  133 | #define SET_RD(insn, regs, val) (*REG_PTR(insn, SH_RD, regs) = (val))
      |                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:123:29: note: expanded from macro 'REG_PTR'
  123 |         ((ulong *)((ulong)(regs) + REG_OFFSET(insn, pos)))
      |                                    ^~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:120:3: note: expanded from macro 'REG_OFFSET'
  120 |         (SHIFT_RIGHT((insn), (pos) - LOG_REGBYTES) & REG_MASK)
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:114:18: note: expanded from macro 'SHIFT_RIGHT'
  114 |         ((y) < 0 ? ((x) << -(y)) : ((x) >> (y)))
      |                         ^  ~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:769:3: warning: shift count is negative [-Wshift-count-negative]
  769 |                 SET_RD(insn, &vcpu->arch.guest_context,
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  770 |                         (ulong)data64 << shift >> shift);
      |                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:133:35: note: expanded from macro 'SET_RD'
  133 | #define SET_RD(insn, regs, val) (*REG_PTR(insn, SH_RD, regs) = (val))
      |                                   ^~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:123:29: note: expanded from macro 'REG_PTR'
  123 |         ((ulong *)((ulong)(regs) + REG_OFFSET(insn, pos)))
      |                                    ^~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:120:3: note: expanded from macro 'REG_OFFSET'
  120 |         (SHIFT_RIGHT((insn), (pos) - LOG_REGBYTES) & REG_MASK)
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:114:18: note: expanded from macro 'SHIFT_RIGHT'
  114 |         ((y) < 0 ? ((x) << -(y)) : ((x) >> (y)))
      |                         ^  ~~~~
6 warnings and 2 errors generated.
make[5]: *** [/build/tmppzcv6jdf/scripts/Makefile.build:203: arch/riscv/kvm/vcpu_insn.o] Error 1
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:412:10: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  412 |                 insn = RVC_RS2S(insn) << SH_RD;
      |                        ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:124:31: note: expanded from macro 'RVC_RS2S'
  124 | #define RVC_RS2S(insn)                  (8 + RV_X(insn, SH_RS2C, 3))
      |                                              ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:420:10: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  420 |                 insn = RVC_RS2S(insn) << SH_RD;
      |                        ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:124:31: note: expanded from macro 'RVC_RS2S'
  124 | #define RVC_RS2S(insn)                  (8 + RV_X(insn, SH_RS2C, 3))
      |                                              ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:428:10: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  428 |                 insn = RVC_RS2S(insn) << SH_RD;
      |                        ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:124:31: note: expanded from macro 'RVC_RS2S'
  124 | #define RVC_RS2S(insn)                  (8 + RV_X(insn, SH_RS2C, 3))
      |                                              ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:450:3: warning: shift count is negative [-Wshift-count-negative]
  450 |                 SET_RD(insn, regs, val.data_ulong << shift >> shift);
      |                 ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:145:36: note: expanded from macro 'SET_RD'
  145 | #define SET_RD(insn, regs, val)         (*REG_PTR(insn, SH_RD, regs) = (val))
      |                                           ^~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:137:28: note: expanded from macro 'REG_PTR'
  137 |         (ulong *)((ulong)(regs) + REG_OFFSET(insn, pos))
      |                                   ^~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:134:3: note: expanded from macro 'REG_OFFSET'
  134 |         (SHIFT_RIGHT((insn), (pos) - LOG_REGBYTES) & REG_MASK)
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:128:18: note: expanded from macro 'SHIFT_RIGHT'
  128 |         ((y) < 0 ? ((x) << -(y)) : ((x) >> (y)))
      |                         ^  ~~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:510:20: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  510 |                 val.data_ulong = GET_RS2S(insn, regs);
      |                                  ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:142:41: note: expanded from macro 'GET_RS2S'
  142 | #define GET_RS2S(insn, regs)            (*REG_PTR(RVC_RS2S(insn), 0, regs))
      |                                                   ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:124:31: note: expanded from macro 'RVC_RS2S'
  124 | #define RVC_RS2S(insn)                  (8 + RV_X(insn, SH_RS2C, 3))
      |                                              ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:517:18: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  517 |                 val.data_u64 = GET_F64_RS2S(insn, regs);
      |                                ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:259:46: note: expanded from macro 'GET_F64_RS2S'
  259 | #define GET_F64_RS2S(insn, regs) (get_f64_rs(RVC_RS2S(insn), 0, regs))
      |                                              ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:124:31: note: expanded from macro 'RVC_RS2S'
  124 | #define RVC_RS2S(insn)                  (8 + RV_X(insn, SH_RS2C, 3))
      |                                              ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:526:20: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
  526 |                 val.data_ulong = GET_F32_RS2S(insn, regs);
      |                                  ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:263:46: note: expanded from macro 'GET_F32_RS2S'
  263 | #define GET_F32_RS2S(insn, regs) (get_f32_rs(RVC_RS2S(insn), 0, regs))
      |                                              ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:124:31: note: expanded from macro 'RVC_RS2S'
  124 | #define RVC_RS2S(insn)                  (8 + RV_X(insn, SH_RS2C, 3))
      |                                              ^
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:482:19: warning: shift count is negative [-Wshift-count-negative]
  482 |         val.data_ulong = GET_RS2(insn, regs);
      |                          ^~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:140:32: note: expanded from macro 'GET_RS2'
  140 | #define GET_RS2(insn, regs)             (*REG_PTR(insn, SH_RS2, regs))
      |                                           ^~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:137:28: note: expanded from macro 'REG_PTR'
  137 |         (ulong *)((ulong)(regs) + REG_OFFSET(insn, pos))
      |                                   ^~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:134:3: note: expanded from macro 'REG_OFFSET'
  134 |         (SHIFT_RIGHT((insn), (pos) - LOG_REGBYTES) & REG_MASK)
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:128:18: note: expanded from macro 'SHIFT_RIGHT'
  128 |         ((y) < 0 ? ((x) << -(y)) : ((x) >> (y)))
      |                         ^  ~~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:510:20: warning: shift count is negative [-Wshift-count-negative]
  510 |                 val.data_ulong = GET_RS2S(insn, regs);
      |                                  ^~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:142:33: note: expanded from macro 'GET_RS2S'
  142 | #define GET_RS2S(insn, regs)            (*REG_PTR(RVC_RS2S(insn), 0, regs))
      |                                           ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:137:28: note: expanded from macro 'REG_PTR'
  137 |         (ulong *)((ulong)(regs) + REG_OFFSET(insn, pos))
I: config: PASS in 0:00:08.859826
I: default: FAIL in 0:01:14.407584
I: kernel: SKIP in 0:00:00.000005
I: xipkernel: SKIP in 0:00:00.000002
I: modules: FAIL in 0:00:00.067043
I: dtbs: PASS in 0:00:01.153876
I: dtbs-legacy: SKIP in 0:00:00.004867
I: debugkernel: SKIP in 0:00:00.000006
I: headers: PASS in 0:00:00.986231
I: build output in /build/tmp.Hq76sLxdqI
      |                                   ^~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:134:3: note: expanded from macro 'REG_OFFSET'
  134 |         (SHIFT_RIGHT((insn), (pos) - LOG_REGBYTES) & REG_MASK)
      |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:128:34: note: expanded from macro 'SHIFT_RIGHT'
  128 |         ((y) < 0 ? ((x) << -(y)) : ((x) >> (y)))
      |                                         ^  ~~~
3 warnings and 6 errors generated.
make[5]: *** [/build/tmppzcv6jdf/scripts/Makefile.build:203: arch/riscv/kernel/traps_misaligned.o] Error 1
make[5]: Target 'arch/riscv/kvm/' not remade because of errors.
make[4]: *** [/build/tmppzcv6jdf/scripts/Makefile.build:461: arch/riscv/kvm] Error 2
make[5]: Target 'arch/riscv/kernel/' not remade because of errors.
make[4]: *** [/build/tmppzcv6jdf/scripts/Makefile.build:461: arch/riscv/kernel] Error 2
make[4]: Target 'arch/riscv/' not remade because of errors.
make[3]: *** [/build/tmppzcv6jdf/scripts/Makefile.build:461: arch/riscv] Error 2
make[3]: Target './' not remade because of errors.
make[2]: *** [/build/tmppzcv6jdf/Makefile:2006: .] Error 2
make[2]: Target '__all' not remade because of errors.
make[1]: *** [/build/tmppzcv6jdf/Makefile:248: __sub-make] Error 2
make[1]: Target '__all' not remade because of errors.
make: *** [Makefile:248: __sub-make] Error 2
make: Target '__all' not remade because of errors.
rm -rf /build/tmp.Hq76sLxdqI/build/modinstall
make --silent --keep-going --jobs=48 O=/build/tmp.Hq76sLxdqI/build INSTALL_MOD_STRIP=1 INSTALL_MOD_PATH=/build/tmp.Hq76sLxdqI/build/modinstall ARCH=riscv CROSS_COMPILE=riscv64-linux- LLVM=1 'CC=ccache clang' 'HOSTCC=ccache clang' modules_install
make[3]: *** No rule to make target 'modules.order', needed by '/build/tmp.Hq76sLxdqI/build/modinstall/lib/modules/6.15.0-rc1-00017-ge25c49c73413/modules.order'.
make[3]: *** No rule to make target 'modules.builtin', needed by '/build/tmp.Hq76sLxdqI/build/modinstall/lib/modules/6.15.0-rc1-00017-ge25c49c73413/modules.builtin'.
make[3]: *** No rule to make target 'modules.builtin.modinfo', needed by '/build/tmp.Hq76sLxdqI/build/modinstall/lib/modules/6.15.0-rc1-00017-ge25c49c73413/modules.builtin.modinfo'.
make[3]: Target '__modinst' not remade because of errors.
make[2]: *** [/build/tmppzcv6jdf/Makefile:1912: modules_install] Error 2
make[1]: *** [/build/tmppzcv6jdf/Makefile:248: __sub-make] Error 2
make[1]: Target 'modules_install' not remade because of errors.
make: *** [Makefile:248: __sub-make] Error 2
make: Target 'modules_install' not remade because of errors.
make --silent --keep-going --jobs=48 O=/build/tmp.Hq76sLxdqI/build INSTALL_DTBS_PATH=/build/tmp.Hq76sLxdqI/build/dtbsinstall/dtbs ARCH=riscv CROSS_COMPILE=riscv64-linux- LLVM=1 'CC=ccache clang' 'HOSTCC=ccache clang' dtbs
rm -rf /build/tmp.Hq76sLxdqI/build/dtbsinstall
mkdir -p /build/tmp.Hq76sLxdqI/build/dtbsinstall/dtbs
make --silent --keep-going --jobs=48 O=/build/tmp.Hq76sLxdqI/build INSTALL_DTBS_PATH=/build/tmp.Hq76sLxdqI/build/dtbsinstall/dtbs ARCH=riscv CROSS_COMPILE=riscv64-linux- LLVM=1 'CC=ccache clang' 'HOSTCC=ccache clang' dtbs_install
tar --sort=name --owner=tuxmake:1000 --group=tuxmake:1000 --mtime=@1745313594 --clamp-mtime -caf /build/tmp.Hq76sLxdqI/build/dtbs.tar -C /build/tmp.Hq76sLxdqI/build/dtbsinstall dtbs
rm -rf /build/tmp.Hq76sLxdqI/build/install_hdr
make --silent --keep-going --jobs=48 O=/build/tmp.Hq76sLxdqI/build INSTALL_HDR_PATH=/build/tmp.Hq76sLxdqI/build/install_hdr/ ARCH=riscv CROSS_COMPILE=riscv64-linux- LLVM=1 'CC=ccache clang' 'HOSTCC=ccache clang' headers_install
tar --sort=name --owner=tuxmake:1000 --group=tuxmake:1000 --mtime=@1745313594 --clamp-mtime -caf /build/tmp.Hq76sLxdqI/build/headers.tar -C /build/tmp.Hq76sLxdqI/build/install_hdr .
warnings/errors:
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:549:10: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:664:12: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:640:9: warning: shift count is negative [-Wshift-count-negative]
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:664:12: warning: shift count is negative [-Wshift-count-negative]
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:754:3: warning: shift count is negative [-Wshift-count-negative]
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:759:3: warning: shift count is negative [-Wshift-count-negative]
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:764:3: warning: shift count is negative [-Wshift-count-negative]
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:769:3: warning: shift count is negative [-Wshift-count-negative]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:412:10: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:420:10: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:428:10: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:450:3: warning: shift count is negative [-Wshift-count-negative]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:510:20: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:517:18: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:526:20: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:482:19: warning: shift count is negative [-Wshift-count-negative]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:510:20: warning: shift count is negative [-Wshift-count-negative]

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 2: "[2/3] riscv: Strengthen duplicate and inconsistent definition of RV_X()"
build-rv64-clang-allmodconfig
Desc: Builds riscv64 allmodconfig with Clang, and checks for errors and added warnings
Duration: 637.04 seconds
Result: ERROR
Output:

Redirect to /build/tmp.U0fp2lNOKz and /build/tmp.lMAnZNAKSw
Tree base:
b1543b05fcff ("riscv: Fix typo EXRACT -> EXTRACT")
Building the whole tree with the patch
error:
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:540:10: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:549:10: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:656:12: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:664:12: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:403:10: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:412:10: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:420:10: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:503:20: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:510:20: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:517:18: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/elf_kexec.c:426:18: error: call to undeclared function 'RV_X'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]



real	10m29.725s
user	404m58.115s
sys	77m12.574s

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 2: "[2/3] riscv: Strengthen duplicate and inconsistent definition of RV_X()"
build-rv64-gcc-allmodconfig
Desc: Builds riscv64 allmodconfig with GCC, and checks for errors and added warnings
Duration: 910.15 seconds
Result: ERROR
Output:

Redirect to /build/tmp.HZCShlIQAT and /build/tmp.ytzo3qvoZ2
Tree base:
b1543b05fcff ("riscv: Fix typo EXRACT -> EXTRACT")
Building the whole tree with the patch
error:
/build/tmppzcv6jdf/arch/riscv/kvm/vcpu_insn.c:110:38: error: implicit declaration of function 'RV_X' [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:124:46: error: implicit declaration of function 'RV_X' [-Wimplicit-function-declaration]
/build/tmppzcv6jdf/arch/riscv/kernel/elf_kexec.c:348:11: error: implicit declaration of function 'RV_X' [-Wimplicit-function-declaration]



real	15m3.916s
user	591m50.935s
sys	94m3.277s

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 2: "[2/3] riscv: Strengthen duplicate and inconsistent definition of RV_X()"
build-rv64-nommu-k210-defconfig
Desc: Builds riscv64 defconfig with NOMMU for K210
Duration: 17.85 seconds
Result: ERROR
Output:

Full log:
W: Support for running offline not available (unshare: unshare failed: Operation not permitted)
I: config: PASS in 0:00:03.475666
I: default: FAIL in 0:00:10.312426
I: kernel: SKIP in 0:00:00.000006
I: xipkernel: SKIP in 0:00:00.000002
I: modules: SKIP in 0:00:00.002563
I: dtbs: PASS in 0:00:00.739124
I: dtbs-legacy: SKIP in 0:00:00.004735
I: debugkernel: SKIP in 0:00:00.000005
I: headers: PASS in 0:00:01.072897
I: build output in /build/tmp.RboPivHM3e
tuxmake --download-all-korg-gcc-toolchains --target-arch=riscv --kconfig=nommu_k210_defconfig --toolchain=gcc --wrapper=ccache --environment=KBUILD_BUILD_TIMESTAMP=@1621270510 --environment=KBUILD_BUILD_USER=tuxmake --environment=KBUILD_BUILD_HOST=tuxmake --environment=KCFLAGS=-ffile-prefix-map=/build/tmp.RboPivHM3e/build/= --runtime=null --image=docker.io/tuxmake/riscv_gcc CROSS_COMPILE=riscv64-linux- config default kernel xipkernel modules dtbs dtbs-legacy debugkernel headers
make --silent --keep-going --jobs=48 O=/build/tmp.RboPivHM3e/build ARCH=riscv CROSS_COMPILE=riscv64-linux- 'CC=ccache riscv64-linux-gcc' 'HOSTCC=ccache gcc' nommu_k210_defconfig
make --silent --keep-going --jobs=48 O=/build/tmp.RboPivHM3e/build ARCH=riscv CROSS_COMPILE=riscv64-linux- 'CC=ccache riscv64-linux-gcc' 'HOSTCC=ccache gcc'
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c: In function 'handle_scalar_misaligned_load':
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:124:46: error: implicit declaration of function 'RV_X' [-Wimplicit-function-declaration]
  124 | #define RVC_RS2S(insn)                  (8 + RV_X(insn, SH_RS2C, 3))
      |                                              ^~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:403:24: note: in expansion of macro 'RVC_RS2S'
  403 |                 insn = RVC_RS2S(insn) << SH_RD;
      |                        ^~~~~~~~
make[5]: *** [/build/tmppzcv6jdf/scripts/Makefile.build:203: arch/riscv/kernel/traps_misaligned.o] Error 1
make[5]: Target 'arch/riscv/kernel/' not remade because of errors.
make[4]: *** [/build/tmppzcv6jdf/scripts/Makefile.build:461: arch/riscv/kernel] Error 2
make[4]: Target 'arch/riscv/' not remade because of errors.
make[3]: *** [/build/tmppzcv6jdf/scripts/Makefile.build:461: arch/riscv] Error 2
make[3]: Target './' not remade because of errors.
make[2]: *** [/build/tmppzcv6jdf/Makefile:2006: .] Error 2
make[2]: Target '__all' not remade because of errors.
make[1]: *** [/build/tmppzcv6jdf/Makefile:248: __sub-make] Error 2
make[1]: Target '__all' not remade because of errors.
make: *** [Makefile:248: __sub-make] Error 2
make: Target '__all' not remade because of errors.
make --silent --keep-going --jobs=48 O=/build/tmp.RboPivHM3e/build INSTALL_DTBS_PATH=/build/tmp.RboPivHM3e/build/dtbsinstall/dtbs ARCH=riscv CROSS_COMPILE=riscv64-linux- 'CC=ccache riscv64-linux-gcc' 'HOSTCC=ccache gcc' dtbs
rm -rf /build/tmp.RboPivHM3e/build/dtbsinstall
mkdir -p /build/tmp.RboPivHM3e/build/dtbsinstall/dtbs
make --silent --keep-going --jobs=48 O=/build/tmp.RboPivHM3e/build INSTALL_DTBS_PATH=/build/tmp.RboPivHM3e/build/dtbsinstall/dtbs ARCH=riscv CROSS_COMPILE=riscv64-linux- 'CC=ccache riscv64-linux-gcc' 'HOSTCC=ccache gcc' dtbs_install
tar --sort=name --owner=tuxmake:1000 --group=tuxmake:1000 --mtime=@1745313594 --clamp-mtime -caf /build/tmp.RboPivHM3e/build/dtbs.tar -C /build/tmp.RboPivHM3e/build/dtbsinstall dtbs
rm -rf /build/tmp.RboPivHM3e/build/install_hdr
make --silent --keep-going --jobs=48 O=/build/tmp.RboPivHM3e/build INSTALL_HDR_PATH=/build/tmp.RboPivHM3e/build/install_hdr/ ARCH=riscv CROSS_COMPILE=riscv64-linux- 'CC=ccache riscv64-linux-gcc' 'HOSTCC=ccache gcc' headers_install
tar --sort=name --owner=tuxmake:1000 --group=tuxmake:1000 --mtime=@1745313594 --clamp-mtime -caf /build/tmp.RboPivHM3e/build/headers.tar -C /build/tmp.RboPivHM3e/build/install_hdr .
warnings/errors:
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:124:46: error: implicit declaration of function 'RV_X' [-Wimplicit-function-declaration]

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 2: "[2/3] riscv: Strengthen duplicate and inconsistent definition of RV_X()"
build-rv64-nommu-k210-virt
Desc: Builds riscv64 defconfig with NOMMU for the virt platform
Duration: 19.06 seconds
Result: ERROR
Output:

Full log:
W: Support for running offline not available (unshare: unshare failed: Operation not permitted)
I: config: PASS in 0:00:03.367599
I: default: FAIL in 0:00:11.922088
I: kernel: SKIP in 0:00:00.000008
I: xipkernel: SKIP in 0:00:00.000004
I: modules: SKIP in 0:00:00.001716
I: dtbs: PASS in 0:00:00.683642
I: dtbs-legacy: SKIP in 0:00:00.004604
I: debugkernel: SKIP in 0:00:00.000004
I: headers: PASS in 0:00:01.054807
I: build output in /build/tmp.R0AJijM25g
tuxmake --download-all-korg-gcc-toolchains --target-arch=riscv --kconfig=nommu_virt_defconfig --toolchain=gcc --wrapper=ccache --environment=KBUILD_BUILD_TIMESTAMP=@1621270510 --environment=KBUILD_BUILD_USER=tuxmake --environment=KBUILD_BUILD_HOST=tuxmake --environment=KCFLAGS=-ffile-prefix-map=/build/tmp.R0AJijM25g/build/= --runtime=null --image=docker.io/tuxmake/riscv_gcc CROSS_COMPILE=riscv64-linux- config default kernel xipkernel modules dtbs dtbs-legacy debugkernel headers
make --silent --keep-going --jobs=48 O=/build/tmp.R0AJijM25g/build ARCH=riscv CROSS_COMPILE=riscv64-linux- 'CC=ccache riscv64-linux-gcc' 'HOSTCC=ccache gcc' nommu_virt_defconfig
make --silent --keep-going --jobs=48 O=/build/tmp.R0AJijM25g/build ARCH=riscv CROSS_COMPILE=riscv64-linux- 'CC=ccache riscv64-linux-gcc' 'HOSTCC=ccache gcc'
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c: In function 'handle_scalar_misaligned_load':
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:124:46: error: implicit declaration of function 'RV_X' [-Wimplicit-function-declaration]
  124 | #define RVC_RS2S(insn)                  (8 + RV_X(insn, SH_RS2C, 3))
      |                                              ^~~~
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:403:24: note: in expansion of macro 'RVC_RS2S'
  403 |                 insn = RVC_RS2S(insn) << SH_RD;
      |                        ^~~~~~~~
make[5]: *** [/build/tmppzcv6jdf/scripts/Makefile.build:203: arch/riscv/kernel/traps_misaligned.o] Error 1
make[5]: Target 'arch/riscv/kernel/' not remade because of errors.
make[4]: *** [/build/tmppzcv6jdf/scripts/Makefile.build:461: arch/riscv/kernel] Error 2
make[4]: Target 'arch/riscv/' not remade because of errors.
make[3]: *** [/build/tmppzcv6jdf/scripts/Makefile.build:461: arch/riscv] Error 2
make[3]: Target './' not remade because of errors.
make[2]: *** [/build/tmppzcv6jdf/Makefile:2006: .] Error 2
make[2]: Target '__all' not remade because of errors.
make[1]: *** [/build/tmppzcv6jdf/Makefile:248: __sub-make] Error 2
make[1]: Target '__all' not remade because of errors.
make: *** [Makefile:248: __sub-make] Error 2
make: Target '__all' not remade because of errors.
make --silent --keep-going --jobs=48 O=/build/tmp.R0AJijM25g/build INSTALL_DTBS_PATH=/build/tmp.R0AJijM25g/build/dtbsinstall/dtbs ARCH=riscv CROSS_COMPILE=riscv64-linux- 'CC=ccache riscv64-linux-gcc' 'HOSTCC=ccache gcc' dtbs
rm -rf /build/tmp.R0AJijM25g/build/dtbsinstall
mkdir -p /build/tmp.R0AJijM25g/build/dtbsinstall/dtbs
make --silent --keep-going --jobs=48 O=/build/tmp.R0AJijM25g/build INSTALL_DTBS_PATH=/build/tmp.R0AJijM25g/build/dtbsinstall/dtbs ARCH=riscv CROSS_COMPILE=riscv64-linux- 'CC=ccache riscv64-linux-gcc' 'HOSTCC=ccache gcc' dtbs_install
tar --sort=name --owner=tuxmake:1000 --group=tuxmake:1000 --mtime=@1745313594 --clamp-mtime -caf /build/tmp.R0AJijM25g/build/dtbs.tar -C /build/tmp.R0AJijM25g/build/dtbsinstall dtbs
rm -rf /build/tmp.R0AJijM25g/build/install_hdr
make --silent --keep-going --jobs=48 O=/build/tmp.R0AJijM25g/build INSTALL_HDR_PATH=/build/tmp.R0AJijM25g/build/install_hdr/ ARCH=riscv CROSS_COMPILE=riscv64-linux- 'CC=ccache riscv64-linux-gcc' 'HOSTCC=ccache gcc' headers_install
tar --sort=name --owner=tuxmake:1000 --group=tuxmake:1000 --mtime=@1745313594 --clamp-mtime -caf /build/tmp.R0AJijM25g/build/headers.tar -C /build/tmp.R0AJijM25g/build/install_hdr .
warnings/errors:
/build/tmppzcv6jdf/arch/riscv/kernel/traps_misaligned.c:124:46: error: implicit declaration of function 'RV_X' [-Wimplicit-function-declaration]

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 2: "[2/3] riscv: Strengthen duplicate and inconsistent definition of RV_X()"
checkpatch
Desc: Runs checkpatch.pl on the patch
Duration: 1.29 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 2: "[2/3] riscv: Strengthen duplicate and inconsistent definition of RV_X()"
dtb-warn-rv64
Desc: Checks for Device Tree warnings/errors
Duration: 38.01 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 2: "[2/3] riscv: Strengthen duplicate and inconsistent definition of RV_X()"
header-inline
Desc: Detects static functions without inline keyword in header files
Duration: 0.23 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 2: "[2/3] riscv: Strengthen duplicate and inconsistent definition of RV_X()"
kdoc
Desc: Detects for kdoc errors
Duration: 0.90 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 2: "[2/3] riscv: Strengthen duplicate and inconsistent definition of RV_X()"
module-param
Desc: Detect module_param changes
Duration: 0.24 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 2: "[2/3] riscv: Strengthen duplicate and inconsistent definition of RV_X()"
verify-fixes
Desc: Verifies that the Fixes: tags exist
Duration: 0.21 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 2: "[2/3] riscv: Strengthen duplicate and inconsistent definition of RV_X()"
verify-signedoff
Desc: Verifies that Signed-off-by: tags are correct
Duration: 0.27 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 3: "[3/3] riscv: Move all duplicate insn parsing macros into asm/insn.h"
build-rv32-defconfig
Desc: Builds riscv32 defconfig
Duration: 102.55 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 3: "[3/3] riscv: Move all duplicate insn parsing macros into asm/insn.h"
build-rv64-clang-allmodconfig
Desc: Builds riscv64 allmodconfig with Clang, and checks for errors and added warnings
Duration: 916.32 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 3: "[3/3] riscv: Move all duplicate insn parsing macros into asm/insn.h"
build-rv64-gcc-allmodconfig
Desc: Builds riscv64 allmodconfig with GCC, and checks for errors and added warnings
Duration: 1219.66 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 3: "[3/3] riscv: Move all duplicate insn parsing macros into asm/insn.h"
build-rv64-nommu-k210-defconfig
Desc: Builds riscv64 defconfig with NOMMU for K210
Duration: 20.22 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 3: "[3/3] riscv: Move all duplicate insn parsing macros into asm/insn.h"
build-rv64-nommu-k210-virt
Desc: Builds riscv64 defconfig with NOMMU for the virt platform
Duration: 21.24 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 3: "[3/3] riscv: Move all duplicate insn parsing macros into asm/insn.h"
checkpatch
Desc: Runs checkpatch.pl on the patch
Duration: 2.19 seconds
Result: WARNING
Output:

CHECK: Prefer using the BIT macro
#167: FILE: arch/riscv/include/asm/insn.h:428:
+#define REGBYTES			(1 << LOG_REGBYTES)

total: 0 errors, 0 warnings, 1 checks, 455 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

Commit d515ccdbc0fc ("riscv: Move all duplicate insn parsing macros into asm/insn.h") has style problems, please review.

NOTE: Ignored message types: ALLOC_SIZEOF_STRUCT CAMELCASE COMMIT_LOG_LONG_LINE GIT_COMMIT_ID MACRO_ARG_REUSE NO_AUTHOR_SIGN_OFF

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.
total: 0 errors, 0 warnings, 1 checks, 455 lines checked
CHECK: Prefer using the BIT macro

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 3: "[3/3] riscv: Move all duplicate insn parsing macros into asm/insn.h"
dtb-warn-rv64
Desc: Checks for Device Tree warnings/errors
Duration: 38.74 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 3: "[3/3] riscv: Move all duplicate insn parsing macros into asm/insn.h"
header-inline
Desc: Detects static functions without inline keyword in header files
Duration: 0.25 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 3: "[3/3] riscv: Move all duplicate insn parsing macros into asm/insn.h"
kdoc
Desc: Detects for kdoc errors
Duration: 0.91 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 3: "[3/3] riscv: Move all duplicate insn parsing macros into asm/insn.h"
module-param
Desc: Detect module_param changes
Duration: 0.25 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 3: "[3/3] riscv: Move all duplicate insn parsing macros into asm/insn.h"
verify-fixes
Desc: Verifies that the Fixes: tags exist
Duration: 0.23 seconds
Result: PASS

@linux-riscv-bot
Copy link
Copy Markdown
Author

linux-riscv-bot commented Apr 22, 2025

Patch 3: "[3/3] riscv: Move all duplicate insn parsing macros into asm/insn.h"
verify-signedoff
Desc: Verifies that Signed-off-by: tags are correct
Duration: 0.29 seconds
Result: PASS

@linux-riscv-bot linux-riscv-bot force-pushed the workflow__riscv__fixes branch from 6c2725a to c8da138 Compare April 24, 2025 20:46
@linux-riscv-bot linux-riscv-bot deleted the pw955600 branch April 30, 2025 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@linux-riscv-bot