[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 00/59] Linux user for 6.1 patches
From: |
Laurent Vivier |
Subject: |
Re: [PULL 00/59] Linux user for 6.1 patches |
Date: |
Tue, 18 May 2021 07:13:11 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1 |
Hi,
I'm going to send an updated PR with fixes for problem reported by checkpatch.
Thanks,
Laurent
Le 17/05/2021 à 21:15, no-reply@patchew.org a écrit :
> Patchew URL: 20210517181424.8093-1-laurent@vivier.eu/">https://patchew.org/QEMU/20210517181424.8093-1-laurent@vivier.eu/
>
>
>
> Hi,
>
> This series seems to have some coding style problems. See output below for
> more information:
>
> Type: series
> Message-id: 20210517181424.8093-1-laurent@vivier.eu
> Subject: [PULL 00/59] Linux user for 6.1 patches
>
> === 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
> 32de74a..367196c master -> master
> - [tag update] patchew/20210517174947.1199649-1-philmd@redhat.com ->
> patchew/20210517174947.1199649-1-philmd@redhat.com
> * [new tag] patchew/20210517181424.8093-1-laurent@vivier.eu ->
> patchew/20210517181424.8093-1-laurent@vivier.eu
> Switched to a new branch 'test'
> 7db920b linux-user/elfload: add s390x core dumping support
> 5b6f11c linux-user/elfload: fix filling psinfo->pr_psargs
> 2b8cc26 linux-user: Tidy TARGET_NR_rt_sigaction
> e1e89de linux-user/alpha: Share code for TARGET_NR_sigaction
> 9872e82 linux-user/alpha: Define TARGET_ARCH_HAS_KA_RESTORER
> 49f59b4 linux-user: Honor TARGET_ARCH_HAS_SA_RESTORER in do_syscall
> e63cb74 linux-user: Pass ka_restorer to do_sigaction
> ce80cea linux-user/alpha: Rename the sigaction restorer field
> 59a7fe5 linux-user/alpha: Fix rt sigframe return
> ede5d61 linux-user: use GDateTime for formatting timestamp for core file
> d9c71e6 linux-user: Fix erroneous conversion in copy_file_range
> 00d88c8 linux-user: Add copy_file_range to strace.list
> b563b62 linux-user/s390x: Handle vector regs in signal stack
> 75cbffb linux-user/s390x: Clean up signal.c
> 9c32aac linux-user/s390x: Add build asserts for sigset sizes
> 0bec884 linux-user/s390x: Fix frame_addr corruption in setup_frame
> b197ab9 linux-user/s390x: Add stub sigframe argument for last_break
> bd5b9e8 linux-user/s390x: Set psw.mask properly for the signal handler
> fe8d7a1 linux-user/s390x: Clean up single-use gotos in signal.c
> c694fba linux-user/s390x: Tidy save_sigregs
> 81015fb linux-user/s390x: Use tswap_sigset in setup_rt_frame
> 2afaee9 linux-user/s390x: Fix sigcontext sregs value
> c335465 linux-user/s390x: Fix trace in restore_regs
> e488167 linux-user/s390x: Remove restore_sigregs return value
> ffa5802 linux-user/s390x: Remove PSW_ADDR_AMODE
> f835fe4 linux-user/s390x: Use uint16_t for signal retcode
> 32267a1 linux-user/s390x: Fix sigframe types
> 2fd8343 tests/tcg/sparc64: Re-enable linux-test
> 47a7850 linux-user/sparc: Implement sparc64 rt signals
> eb9b8de linux-user/sparc: Implement sparc32 rt signals
> 6a02eb6 linux-user/sparc: Add 64-bit support to fpu save/restore
> 7acfaa8 linux-user/sparc: Minor corrections to do_sigreturn
> 37c8475 linux-user/sparc: Clean up setup_frame
> b62d96d linux-user/sparc: Add rwin_save to signal frame
> b27662c linux-user/sparc: Save and restore fpu in signal frame
> a4a46f6 linux-user/sparc: Clean up get_sigframe
> eac9897 linux-user/sparc: Split out save_reg_win
> c75a3e5 linux-user/sparc: Use target_pt_regs
> 104931a linux-user/sparc: Fix the stackframe structure
> b6f9c25 linux-user/sparc: Remove target_rt_signal_frame as unused
> ed9e640 linux-user/sparc: Remove target_sigcontext as unused
> fb62578 linux-user/sparc: Merge sparc64/ into sparc/
> ee45b6d linux-user/sparc: Merge sparc64 termbits.h
> 5d9706c linux-user/sparc: Merge sparc64 target_structs.h
> fbdd320 linux-user/sparc: Merge sparc64 target_elf.h
> c73cc71 linux-user/sparc: Merge sparc64 target_syscall.h
> 05a3287 linux-user/sparc: Clean up init_thread
> d9551e1 linux-user/sparc: Include TARGET_STACK_BIAS in get_sp_from_cpustate
> be699c8 linux-user: Pass CPUArchState to target_restore_altstack
> 8ad510f linux-user: Pass CPUArchState to do_sigaltstack
> 5482bec linux-user: Use target_restore_altstack in all sigreturn
> ad4e4e2 linux-user: Split out target_restore_altstack
> 7ab6de0 linux-user: Add strace support for printing arguments of llseek
> 0ece12d linux-user/arm: Simplify accumulating and raising fpa11 exceptions
> 5dd2463 linux-user/arm: Do not fill in si_code for fpa11 exceptions
> fa4ab02 linux-user/arm: Do not emulate fpa11 in thumb mode
> c61b3a7 linux-user/arm: Split out emulate_arm_fpa11
> 4e217fb linux-user: strace now handles unshare syscall args correctly
> cfc582d linux-user: Remove dead code
>
> === OUTPUT BEGIN ===
> 1/59 Checking commit cfc582dd467e (linux-user: Remove dead code)
> 2/59 Checking commit 4e217fb6e89d (linux-user: strace now handles unshare
> syscall args correctly)
> 3/59 Checking commit c61b3a7d7c99 (linux-user/arm: Split out
> emulate_arm_fpa11)
> 4/59 Checking commit fa4ab02afc4d (linux-user/arm: Do not emulate fpa11 in
> thumb mode)
> 5/59 Checking commit 5dd2463d299b (linux-user/arm: Do not fill in si_code for
> fpa11 exceptions)
> 6/59 Checking commit 0ece12d2d24a (linux-user/arm: Simplify accumulating and
> raising fpa11 exceptions)
> 7/59 Checking commit 7ab6de0b0489 (linux-user: Add strace support for
> printing arguments of llseek)
> 8/59 Checking commit ad4e4e24824c (linux-user: Split out
> target_restore_altstack)
> 9/59 Checking commit 5482becd8dd6 (linux-user: Use target_restore_altstack in
> all sigreturn)
> 10/59 Checking commit 8ad510f1e2f9 (linux-user: Pass CPUArchState to
> do_sigaltstack)
> 11/59 Checking commit be699c84a441 (linux-user: Pass CPUArchState to
> target_restore_altstack)
> 12/59 Checking commit d9551e125d76 (linux-user/sparc: Include
> TARGET_STACK_BIAS in get_sp_from_cpustate)
> 13/59 Checking commit 05a3287b727c (linux-user/sparc: Clean up init_thread)
> 14/59 Checking commit c73cc71c1f45 (linux-user/sparc: Merge sparc64
> target_syscall.h)
> 15/59 Checking commit fbdd3201a581 (linux-user/sparc: Merge sparc64
> target_elf.h)
> 16/59 Checking commit 5d9706cfb373 (linux-user/sparc: Merge sparc64
> target_structs.h)
> 17/59 Checking commit ee45b6ddcf2d (linux-user/sparc: Merge sparc64
> termbits.h)
> 18/59 Checking commit fb62578dc2a3 (linux-user/sparc: Merge sparc64/ into
> sparc/)
> WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
> #57:
> deleted file mode 100644
>
> total: 0 errors, 1 warnings, 13 lines checked
>
> Patch 18/59 has style problems, please review. If any of these errors
> are false positives report them to the maintainer, see
> CHECKPATCH in MAINTAINERS.
> 19/59 Checking commit ed9e6403b7c1 (linux-user/sparc: Remove
> target_sigcontext as unused)
> 20/59 Checking commit b6f9c259dcfe (linux-user/sparc: Remove
> target_rt_signal_frame as unused)
> 21/59 Checking commit 104931a73ea9 (linux-user/sparc: Fix the stackframe
> structure)
> 22/59 Checking commit c75a3e5af1ce (linux-user/sparc: Use target_pt_regs)
> 23/59 Checking commit eac9897cd631 (linux-user/sparc: Split out save_reg_win)
> 24/59 Checking commit a4a46f628639 (linux-user/sparc: Clean up get_sigframe)
> 25/59 Checking commit b27662c4e849 (linux-user/sparc: Save and restore fpu in
> signal frame)
> 26/59 Checking commit b62d96d3871c (linux-user/sparc: Add rwin_save to signal
> frame)
> 27/59 Checking commit 37c84750ef15 (linux-user/sparc: Clean up setup_frame)
> 28/59 Checking commit 7acfaa83fd2b (linux-user/sparc: Minor corrections to
> do_sigreturn)
> 29/59 Checking commit 6a02eb64b8fa (linux-user/sparc: Add 64-bit support to
> fpu save/restore)
> 30/59 Checking commit eb9b8def30c9 (linux-user/sparc: Implement sparc32 rt
> signals)
> 31/59 Checking commit 47a7850440f8 (linux-user/sparc: Implement sparc64 rt
> signals)
> 32/59 Checking commit 2fd8343979f4 (tests/tcg/sparc64: Re-enable linux-test)
> 33/59 Checking commit 32267a1fe861 (linux-user/s390x: Fix sigframe types)
> 34/59 Checking commit f835fe46a35f (linux-user/s390x: Use uint16_t for signal
> retcode)
> 35/59 Checking commit ffa58028e721 (linux-user/s390x: Remove PSW_ADDR_AMODE)
> 36/59 Checking commit e4881679e15c (linux-user/s390x: Remove restore_sigregs
> return value)
> 37/59 Checking commit c33546509b36 (linux-user/s390x: Fix trace in
> restore_regs)
> 38/59 Checking commit 2afaee9f88b5 (linux-user/s390x: Fix sigcontext sregs
> value)
> 39/59 Checking commit 81015fb7ae34 (linux-user/s390x: Use tswap_sigset in
> setup_rt_frame)
> 40/59 Checking commit c694fba9a40b (linux-user/s390x: Tidy save_sigregs)
> 41/59 Checking commit fe8d7a179041 (linux-user/s390x: Clean up single-use
> gotos in signal.c)
> 42/59 Checking commit bd5b9e8bae3d (linux-user/s390x: Set psw.mask properly
> for the signal handler)
> 43/59 Checking commit b197ab9b7234 (linux-user/s390x: Add stub sigframe
> argument for last_break)
> 44/59 Checking commit 0bec884690d3 (linux-user/s390x: Fix frame_addr
> corruption in setup_frame)
> 45/59 Checking commit 9c32aac51d45 (linux-user/s390x: Add build asserts for
> sigset sizes)
> 46/59 Checking commit 75cbffbff7ac (linux-user/s390x: Clean up signal.c)
> 47/59 Checking commit b563b6274f41 (linux-user/s390x: Handle vector regs in
> signal stack)
> ERROR: suspect code indent for conditional statements (4, 7)
> #68: FILE: linux-user/s390x/signal.c:151:
> + for (i = 0; i < 16; ++i) {
> + __put_user(env->vregs[i][1], &ext->vxrs_low[i]);
>
> ERROR: suspect code indent for conditional statements (4, 7)
> #71: FILE: linux-user/s390x/signal.c:154:
> + for (i = 0; i < 16; ++i) {
> + __put_user(env->vregs[i + 16][0], &ext->vxrs_high[i][0]);
>
> ERROR: suspect code indent for conditional statements (4, 7)
> #128: FILE: linux-user/s390x/signal.c:321:
> + for (i = 0; i < 16; ++i) {
> + __get_user(env->vregs[i][1], &ext->vxrs_low[i]);
>
> ERROR: suspect code indent for conditional statements (4, 7)
> #131: FILE: linux-user/s390x/signal.c:324:
> + for (i = 0; i < 16; ++i) {
> + __get_user(env->vregs[i + 16][0], &ext->vxrs_high[i][0]);
>
> total: 4 errors, 0 warnings, 127 lines checked
>
> Patch 47/59 has style problems, please review. If any of these errors
> are false positives report them to the maintainer, see
> CHECKPATCH in MAINTAINERS.
>
> 48/59 Checking commit 00d88c824125 (linux-user: Add copy_file_range to
> strace.list)
> 49/59 Checking commit d9c71e62bcc0 (linux-user: Fix erroneous conversion in
> copy_file_range)
> 50/59 Checking commit ede5d61c022c (linux-user: use GDateTime for formatting
> timestamp for core file)
> 51/59 Checking commit 59a7fe51d6d0 (linux-user/alpha: Fix rt sigframe return)
> 52/59 Checking commit ce80ceaff20b (linux-user/alpha: Rename the sigaction
> restorer field)
> 53/59 Checking commit e63cb7421e0c (linux-user: Pass ka_restorer to
> do_sigaction)
> ERROR: code indent should never use tabs
> #67: FILE: linux-user/syscall.c:9019:
> +^I ret = get_errno(do_sigaction(arg1, pact, &oact, 0));$
>
> total: 1 errors, 0 warnings, 97 lines checked
>
> Patch 53/59 has style problems, please review. If any of these errors
> are false positives report them to the maintainer, see
> CHECKPATCH in MAINTAINERS.
>
> 54/59 Checking commit 49f59b42510e (linux-user: Honor
> TARGET_ARCH_HAS_SA_RESTORER in do_syscall)
> 55/59 Checking commit 9872e828af1f (linux-user/alpha: Define
> TARGET_ARCH_HAS_KA_RESTORER)
> 56/59 Checking commit e1e89de11042 (linux-user/alpha: Share code for
> TARGET_NR_sigaction)
> 57/59 Checking commit 2b8cc2689cae (linux-user: Tidy TARGET_NR_rt_sigaction)
> 58/59 Checking commit 5b6f11c55ddf (linux-user/elfload: fix filling
> psinfo->pr_psargs)
> 59/59 Checking commit 7db920b89baa (linux-user/elfload: add s390x core
> dumping support)
> === OUTPUT END ===
>
> Test command exited with code: 1
>
>
> The full log is available at
> 20210517181424.8093-1-laurent@vivier.eu/testing.checkpatch/?type=message">http://patchew.org/logs/20210517181424.8093-1-laurent@vivier.eu/testing.checkpatch/?type=message.
> ---
> Email generated automatically by Patchew [https://patchew.org/].
> Please send your feedback to patchew-devel@redhat.com
>
- [PULL 56/59] linux-user/alpha: Share code for TARGET_NR_sigaction, (continued)
- [PULL 56/59] linux-user/alpha: Share code for TARGET_NR_sigaction, Laurent Vivier, 2021/05/17
- [PULL 52/59] linux-user/alpha: Rename the sigaction restorer field, Laurent Vivier, 2021/05/17
- [PULL 54/59] linux-user: Honor TARGET_ARCH_HAS_SA_RESTORER in do_syscall, Laurent Vivier, 2021/05/17
- [PULL 59/59] linux-user/elfload: add s390x core dumping support, Laurent Vivier, 2021/05/17
- [PULL 47/59] linux-user/s390x: Handle vector regs in signal stack, Laurent Vivier, 2021/05/17
- [PULL 49/59] linux-user: Fix erroneous conversion in copy_file_range, Laurent Vivier, 2021/05/17
- [PULL 53/59] linux-user: Pass ka_restorer to do_sigaction, Laurent Vivier, 2021/05/17
- [PULL 57/59] linux-user: Tidy TARGET_NR_rt_sigaction, Laurent Vivier, 2021/05/17
- [PULL 38/59] linux-user/s390x: Fix sigcontext sregs value, Laurent Vivier, 2021/05/17
- Re: [PULL 00/59] Linux user for 6.1 patches, no-reply, 2021/05/17
- Re: [PULL 00/59] Linux user for 6.1 patches,
Laurent Vivier <=
- [PULL 00/59] Linux user for 6.1 patches, Laurent Vivier, 2021/05/18
- [PULL 05/59] linux-user/arm: Do not fill in si_code for fpa11 exceptions, Laurent Vivier, 2021/05/18
- [PULL 07/59] linux-user: Add strace support for printing arguments of llseek, Laurent Vivier, 2021/05/18
- [PULL 04/59] linux-user/arm: Do not emulate fpa11 in thumb mode, Laurent Vivier, 2021/05/18
- [PULL 03/59] linux-user/arm: Split out emulate_arm_fpa11, Laurent Vivier, 2021/05/18
- [PULL 02/59] linux-user: strace now handles unshare syscall args correctly, Laurent Vivier, 2021/05/18
- [PULL 09/59] linux-user: Use target_restore_altstack in all sigreturn, Laurent Vivier, 2021/05/18
- [PULL 06/59] linux-user/arm: Simplify accumulating and raising fpa11 exceptions, Laurent Vivier, 2021/05/18
- [PULL 10/59] linux-user: Pass CPUArchState to do_sigaltstack, Laurent Vivier, 2021/05/18
- [PULL 11/59] linux-user: Pass CPUArchState to target_restore_altstack, Laurent Vivier, 2021/05/18