Patchew URL: https://patchew.org/QEMU/20201016184207.786698-1-richard.hender...@linaro.org/
Hi, This series seems to have some coding style problems. See output below for more information: Type: series Message-id: 20201016184207.786698-1-richard.hender...@linaro.org Subject: [PATCH v11 00/12] linux-user: User support for AArch64 BTI === 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 === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu * [new tag] patchew/20201016184207.786698-1-richard.hender...@linaro.org -> patchew/20201016184207.786698-1-richard.hender...@linaro.org Switched to a new branch 'test' adddebd tests/tcg/aarch64: Add bti smoke tests 6de9e12 linux-user/elfload: Parse GNU_PROPERTY_AARCH64_FEATURE_1_AND 9b05077 linux-user/elfload: Parse NT_GNU_PROPERTY_TYPE_0 notes e063fde linux-user/elfload: Use Error for load_elf_interp 5fa6305 linux-user/elfload: Use Error for load_elf_image 0bdf3b9 linux-user/elfload: Move PT_INTERP detection to first loop 3192943 linux-user/elfload: Adjust iteration over phdr d35ac0e linux-user/elfload: Fix coding style in load_elf_image f1019c0 linux-user/elfload: Avoid leaking interp_name using GLib memory API 46b8e04 include/elf: Add defines related to GNU property notes for AArch64 15162b4 linux-user: Set PAGE_TARGET_1 for TARGET_PROT_BTI 49ed74b linux-user/aarch64: Reset btype for signals === OUTPUT BEGIN === 1/12 Checking commit 49ed74bc1aee (linux-user/aarch64: Reset btype for signals) 2/12 Checking commit 15162b4dcbec (linux-user: Set PAGE_TARGET_1 for TARGET_PROT_BTI) 3/12 Checking commit 46b8e04781a1 (include/elf: Add defines related to GNU property notes for AArch64) 4/12 Checking commit f1019c0416c3 (linux-user/elfload: Avoid leaking interp_name using GLib memory API) 5/12 Checking commit d35ac0e1930b (linux-user/elfload: Fix coding style in load_elf_image) 6/12 Checking commit 3192943b8d75 (linux-user/elfload: Adjust iteration over phdr) 7/12 Checking commit 0bdf3b9c07f6 (linux-user/elfload: Move PT_INTERP detection to first loop) 8/12 Checking commit 5fa63056e084 (linux-user/elfload: Use Error for load_elf_image) 9/12 Checking commit e063fde7d4b1 (linux-user/elfload: Use Error for load_elf_interp) 10/12 Checking commit 9b05077fa2ed (linux-user/elfload: Parse NT_GNU_PROPERTY_TYPE_0 notes) 11/12 Checking commit 6de9e12b2adf (linux-user/elfload: Parse GNU_PROPERTY_AARCH64_FEATURE_1_AND) 12/12 Checking commit adddebd8a702 (tests/tcg/aarch64: Add bti smoke tests) WARNING: added, moved or deleted file(s), does MAINTAINERS need updating? #37: new file mode 100644 ERROR: externs should be avoided in .c files #165: FILE: tests/tcg/aarch64/bti-2.c:56: +extern char test_begin[], test_end[]; ERROR: use qemu_real_host_page_size instead of getpagesize() #199: FILE: tests/tcg/aarch64/bti-2.c:90: + void *p = mmap(0, getpagesize(), ERROR: externs should be avoided in .c files #236: FILE: tests/tcg/aarch64/bti-crt.inc.c:13: +int main(void); total: 3 errors, 1 warnings, 247 lines checked Patch 12/12 has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. === OUTPUT END === Test command exited with code: 1 The full log is available at http://patchew.org/logs/20201016184207.786698-1-richard.hender...@linaro.org/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com