Patchew URL: https://patchew.org/QEMU/20191011175536.GB25464@xps13.dannf/
Hi, This series seems to have some coding style problems. See output below for more information: Subject: [Bug 1805256] Re: [Qemu-devel] qemu_futex_wait() lockups in ARM64: 2 possible issues Type: series Message-id: 20191011175536.GB25464@xps13.dannf === 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' d3707c3 qemu_futex_wait() lockups in ARM64: 2 possible issues === OUTPUT BEGIN === WARNING: Block comments use a leading /* on a separate line #66: FILE: util/async.c:345: + /* Using atomic_mb_read ensures that e.g. bh->scheduled is written before ERROR: code indent should never use tabs #74: FILE: util/async.c:350: +^Ievent_notifier_set(&ctx->notifier);$ ERROR: Missing Signed-off-by: line(s) total: 2 errors, 1 warnings, 33 lines checked Commit d3707c31b5da (qemu_futex_wait() lockups in ARM64: 2 possible issues) 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/20191011175536.GB25464@xps13.dannf/testing.checkpatch/?type=message. --- Email generated automatically by Patchew [https://patchew.org/]. Please send your feedback to patchew-de...@redhat.com