Hi, This series seems to have some coding style problems. See output below for more information:
Type: series Message-id: [email protected] Subject: [Qemu-devel] [PATCH v3 00/10] target/arm: Implement v8.1-Atomics === TEST SCRIPT BEGIN === #!/bin/bash BASE=base n=1 total=$(git log --oneline $BASE.. | wc -l) failed=0 git config --local diff.renamelimit 0 git config --local diff.renames True git config --local diff.algorithm histogram commits="$(git log --format=%H --reverse $BASE..)" for c in $commits; do echo "Checking PATCH $n/$total: $(git log -n 1 --format=%s $c)..." if ! git show $c --format=email | ./scripts/checkpatch.pl --mailback -; then failed=1 echo fi n=$((n+1)) done exit $failed === TEST SCRIPT END === Updating 3c8cf5a9c21ff8782164d1def7f44bd888713384 From https://github.com/patchew-project/qemu t [tag update] patchew/[email protected] -> patchew/[email protected] Switched to a new branch 'test' 03c4e7c45f target/arm: Enable ARM_FEATURE_V8_ATOMICS for user-only bfd47ba4bc target/arm: Implement CAS and CASP 92484463d0 target/arm: Fill in disas_ldst_atomic 13b7933509 target/arm: Introduce ARM_FEATURE_V8_ATOMICS and initial decode 97dea408aa target/riscv: Use new atomic min/max expanders 72116c58cb tcg: Use GEN_ATOMIC_HELPER_FN for opposite endian atomic add cb24fc8fed tcg: Introduce atomic helpers for integer min/max 255505b55f target/xtensa: Use new min/max expanders f82f53c47a target/arm: Use new min/max expanders 5d8965c70b tcg: Introduce helpers for integer min/max === OUTPUT BEGIN === Checking PATCH 1/10: tcg: Introduce helpers for integer min/max... Checking PATCH 2/10: target/arm: Use new min/max expanders... Checking PATCH 3/10: target/xtensa: Use new min/max expanders... Checking PATCH 4/10: tcg: Introduce atomic helpers for integer min/max... ERROR: memory barrier without comment #56: FILE: accel/tcg/atomic_template.h:137: + smp_mb(); \ ERROR: memory barrier without comment #96: FILE: accel/tcg/atomic_template.h:285: + smp_mb(); \ total: 2 errors, 0 warnings, 236 lines checked Your patch has style problems, please review. If any of these errors are false positives report them to the maintainer, see CHECKPATCH in MAINTAINERS. Checking PATCH 5/10: tcg: Use GEN_ATOMIC_HELPER_FN for opposite endian atomic add... Checking PATCH 6/10: target/riscv: Use new atomic min/max expanders... Checking PATCH 7/10: target/arm: Introduce ARM_FEATURE_V8_ATOMICS and initial decode... Checking PATCH 8/10: target/arm: Fill in disas_ldst_atomic... Checking PATCH 9/10: target/arm: Implement CAS and CASP... Checking PATCH 10/10: target/arm: Enable ARM_FEATURE_V8_ATOMICS for user-only... === OUTPUT END === Test command exited with code: 1 --- Email generated automatically by Patchew [http://patchew.org/]. Please send your feedback to [email protected]
