Patchew URL: https://patchew.org/QEMU/20200411041427.14828-1-jiangyi...@huawei.com/
Hi, This series seems to have some coding style problems. See output below for more information: Subject: [PATCH RFC v2 0/9] Add riscv kvm accel support Message-id: 20200411041427.14828-1-jiangyi...@huawei.com Type: series === TEST SCRIPT BEGIN === #!/bin/bash git rev-parse base > /dev/null || exit 0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram ./scripts/checkpatch.pl --mailback base.. === TEST SCRIPT END === Switched to a new branch 'test' 3705618 target/riscv: add host cpu type efb216c target/riscv: Handler KVM_EXIT_RISCV_SBI exit 56aa404 hw/riscv: PLIC update external interrupt by KVM when kvm enabled 86280f4 target/riscv: Support start kernel directly by KVM 629e8e1 target/riscv: Implement kvm_arch_put_registers c0a354d target/riscv: Implement kvm_arch_get_registers 689729d target/riscv: Implement function kvm_arch_init_vcpu 6e0b31c target/riscv: Add target/riscv/kvm.c to place the public kvm interface e8f7f24 linux-header: Update linux/kvm.h === OUTPUT BEGIN === 1/9 Checking commit e8f7f24b4bb3 (linux-header: Update linux/kvm.h) 2/9 Checking commit 6e0b31c6be80 (target/riscv: Add target/riscv/kvm.c to place the public kvm interface) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #38: new file mode 100644 total: 0 errors, 1 warnings, 141 lines checked Patch 2/9 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 3/9 Checking commit 689729d06f73 (target/riscv: Implement function kvm_arch_init_vcpu) 4/9 Checking commit c0a354dd075c (target/riscv: Implement kvm_arch_get_registers) 5/9 Checking commit 629e8e18e44d (target/riscv: Implement kvm_arch_put_registers) 6/9 Checking commit 86280f4d8337 (target/riscv: Support start kernel directly by KVM) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #132: new file mode 100644 total: 0 errors, 1 warnings, 115 lines checked Patch 6/9 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 7/9 Checking commit 56aa404c8db0 (hw/riscv: PLIC update external interrupt by KVM when kvm enabled) 8/9 Checking commit efb216c472d6 (target/riscv: Handler KVM_EXIT_RISCV_SBI exit) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #80: new file mode 100644 WARNING: architecture specific defines should be avoided #94: FILE: target/riscv/sbi_ecall_interface.h:10: +#ifndef __SBI_ECALL_INTERFACE_H__ ERROR: code indent should never use tabs #100: FILE: target/riscv/sbi_ecall_interface.h:16: +#define^ISBI_EXT_0_1_SET_TIMER^I^I^I0x0$ ERROR: code indent should never use tabs #101: FILE: target/riscv/sbi_ecall_interface.h:17: +#define SBI_EXT_0_1_CONSOLE_PUTCHAR^I^I0x1$ ERROR: code indent should never use tabs #102: FILE: target/riscv/sbi_ecall_interface.h:18: +#define SBI_EXT_0_1_CONSOLE_GETCHAR^I^I0x2$ ERROR: code indent should never use tabs #103: FILE: target/riscv/sbi_ecall_interface.h:19: +#define SBI_EXT_0_1_CLEAR_IPI^I^I^I0x3$ ERROR: code indent should never use tabs #104: FILE: target/riscv/sbi_ecall_interface.h:20: +#define SBI_EXT_0_1_SEND_IPI^I^I^I0x4$ ERROR: code indent should never use tabs #105: FILE: target/riscv/sbi_ecall_interface.h:21: +#define SBI_EXT_0_1_REMOTE_FENCE_I^I^I0x5$ ERROR: code indent should never use tabs #106: FILE: target/riscv/sbi_ecall_interface.h:22: +#define SBI_EXT_0_1_REMOTE_SFENCE_VMA^I^I0x6$ ERROR: code indent should never use tabs #107: FILE: target/riscv/sbi_ecall_interface.h:23: +#define SBI_EXT_0_1_REMOTE_SFENCE_VMA_ASID^I0x7$ ERROR: code indent should never use tabs #108: FILE: target/riscv/sbi_ecall_interface.h:24: +#define SBI_EXT_0_1_SHUTDOWN^I^I^I0x8$ ERROR: code indent should never use tabs #109: FILE: target/riscv/sbi_ecall_interface.h:25: +#define SBI_EXT_BASE^I^I^I^I0x10$ ERROR: code indent should never use tabs #110: FILE: target/riscv/sbi_ecall_interface.h:26: +#define SBI_EXT_TIME^I^I^I^I0x54494D45$ ERROR: code indent should never use tabs #111: FILE: target/riscv/sbi_ecall_interface.h:27: +#define SBI_EXT_IPI^I^I^I^I0x735049$ ERROR: code indent should never use tabs #112: FILE: target/riscv/sbi_ecall_interface.h:28: +#define SBI_EXT_RFENCE^I^I^I^I0x52464E43$ ERROR: code indent should never use tabs #113: FILE: target/riscv/sbi_ecall_interface.h:29: +#define SBI_EXT_HSM^I^I^I^I0x48534D$ ERROR: code indent should never use tabs #116: FILE: target/riscv/sbi_ecall_interface.h:32: +#define SBI_EXT_BASE_GET_SPEC_VERSION^I^I0x0$ ERROR: code indent should never use tabs #117: FILE: target/riscv/sbi_ecall_interface.h:33: +#define SBI_EXT_BASE_GET_IMP_ID^I^I^I0x1$ ERROR: code indent should never use tabs #118: FILE: target/riscv/sbi_ecall_interface.h:34: +#define SBI_EXT_BASE_GET_IMP_VERSION^I^I0x2$ ERROR: code indent should never use tabs #119: FILE: target/riscv/sbi_ecall_interface.h:35: +#define SBI_EXT_BASE_PROBE_EXT^I^I^I0x3$ ERROR: code indent should never use tabs #120: FILE: target/riscv/sbi_ecall_interface.h:36: +#define SBI_EXT_BASE_GET_MVENDORID^I^I0x4$ ERROR: code indent should never use tabs #121: FILE: target/riscv/sbi_ecall_interface.h:37: +#define SBI_EXT_BASE_GET_MARCHID^I^I0x5$ ERROR: code indent should never use tabs #122: FILE: target/riscv/sbi_ecall_interface.h:38: +#define SBI_EXT_BASE_GET_MIMPID^I^I^I0x6$ ERROR: code indent should never use tabs #125: FILE: target/riscv/sbi_ecall_interface.h:41: +#define SBI_EXT_TIME_SET_TIMER^I^I^I0x0$ ERROR: code indent should never use tabs #128: FILE: target/riscv/sbi_ecall_interface.h:44: +#define SBI_EXT_IPI_SEND_IPI^I^I^I0x0$ ERROR: code indent should never use tabs #131: FILE: target/riscv/sbi_ecall_interface.h:47: +#define SBI_EXT_RFENCE_REMOTE_FENCE_I^I^I0x0$ ERROR: code indent should never use tabs #132: FILE: target/riscv/sbi_ecall_interface.h:48: +#define SBI_EXT_RFENCE_REMOTE_SFENCE_VMA^I0x1$ ERROR: code indent should never use tabs #133: FILE: target/riscv/sbi_ecall_interface.h:49: +#define SBI_EXT_RFENCE_REMOTE_SFENCE_VMA_ASID^I0x2$ ERROR: code indent should never use tabs #134: FILE: target/riscv/sbi_ecall_interface.h:50: +#define SBI_EXT_RFENCE_REMOTE_HFENCE_GVMA^I0x3$ ERROR: code indent should never use tabs #135: FILE: target/riscv/sbi_ecall_interface.h:51: +#define SBI_EXT_RFENCE_REMOTE_HFENCE_GVMA_VMID^I0x4$ ERROR: code indent should never use tabs #136: FILE: target/riscv/sbi_ecall_interface.h:52: +#define SBI_EXT_RFENCE_REMOTE_HFENCE_VVMA^I0x5$ ERROR: code indent should never use tabs #137: FILE: target/riscv/sbi_ecall_interface.h:53: +#define SBI_EXT_RFENCE_REMOTE_HFENCE_VVMA_ASID^I0x6$ ERROR: code indent should never use tabs #140: FILE: target/riscv/sbi_ecall_interface.h:56: +#define SBI_EXT_HSM_HART_START^I^I^I0x0$ ERROR: code indent should never use tabs #141: FILE: target/riscv/sbi_ecall_interface.h:57: +#define SBI_EXT_HSM_HART_STOP^I^I^I0x1$ ERROR: code indent should never use tabs #142: FILE: target/riscv/sbi_ecall_interface.h:58: +#define SBI_EXT_HSM_HART_GET_STATUS^I^I0x2$ ERROR: code indent should never use tabs #144: FILE: target/riscv/sbi_ecall_interface.h:60: +#define SBI_HSM_HART_STATUS_STARTED^I^I0x0$ ERROR: code indent should never use tabs #145: FILE: target/riscv/sbi_ecall_interface.h:61: +#define SBI_HSM_HART_STATUS_STOPPED^I^I0x1$ ERROR: code indent should never use tabs #146: FILE: target/riscv/sbi_ecall_interface.h:62: +#define SBI_HSM_HART_STATUS_START_PENDING^I0x2$ ERROR: code indent should never use tabs #147: FILE: target/riscv/sbi_ecall_interface.h:63: +#define SBI_HSM_HART_STATUS_STOP_PENDING^I0x3$ ERROR: code indent should never use tabs #149: FILE: target/riscv/sbi_ecall_interface.h:65: +#define SBI_SPEC_VERSION_MAJOR_OFFSET^I^I24$ ERROR: code indent should never use tabs #150: FILE: target/riscv/sbi_ecall_interface.h:66: +#define SBI_SPEC_VERSION_MAJOR_MASK^I^I0x7f$ ERROR: code indent should never use tabs #151: FILE: target/riscv/sbi_ecall_interface.h:67: +#define SBI_SPEC_VERSION_MINOR_MASK^I^I0xffffff$ ERROR: code indent should never use tabs #152: FILE: target/riscv/sbi_ecall_interface.h:68: +#define SBI_EXT_VENDOR_START^I^I^I0x09000000$ ERROR: code indent should never use tabs #153: FILE: target/riscv/sbi_ecall_interface.h:69: +#define SBI_EXT_VENDOR_END^I^I^I0x09FFFFFF$ total: 42 errors, 2 warnings, 128 lines checked Patch 8/9 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. 9/9 Checking commit 3705618846a1 (target/riscv: add host cpu type) === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20200411041427.14828-1-jiangyi...@huawei.com/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com
