[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] linux-user: Fix do_store_exclusive for shared m
From: |
no-reply |
Subject: |
Re: [Qemu-devel] [PATCH] linux-user: Fix do_store_exclusive for shared memory of interprocess. |
Date: |
Sat, 15 Oct 2016 09:04:00 -0700 (PDT) |
Hi,
Your series seems to have some coding style problems. See output below for
more information:
Subject: [Qemu-devel] [PATCH] linux-user: Fix do_store_exclusive for shared
memory of interprocess.
Type: series
Message-id: address@hidden
=== TEST SCRIPT BEGIN ===
#!/bin/bash
BASE=base
n=1
total=$(git log --oneline $BASE.. | wc -l)
failed=0
# Useful git options
git config --local diff.renamelimit 0
git config --local diff.renames True
commits="$(git log --format=%H --reverse $BASE..)"
for c in $commits; do
echo "Checking PATCH $n/$total: $(git show --no-patch --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
Switched to a new branch 'test'
03ca3dc linux-user: Fix do_store_exclusive for shared memory of interprocess.
=== OUTPUT BEGIN ===
Checking PATCH 1/1: linux-user: Fix do_store_exclusive for shared memory of
interprocess....
ERROR: code indent should never use tabs
#20: FILE: linux-user/main.c:2315:
+#define cmpxchg_user(old, new, gaddr, target_type)^I^I^I\$
ERROR: code indent should never use tabs
#21: FILE: linux-user/main.c:2316:
+({^I^I^I^I^I^I^I^I^I\$
ERROR: code indent should never use tabs
#22: FILE: linux-user/main.c:2317:
+ abi_ulong __gaddr = (gaddr);^I^I^I^I^I\$
ERROR: code indent should never use tabs
#23: FILE: linux-user/main.c:2318:
+ target_type *__hptr;^I^I^I^I^I^I\$
ERROR: code indent should never use tabs
#24: FILE: linux-user/main.c:2319:
+ abi_long __ret = 0;^I^I^I^I^I^I^I\$
ERROR: do not use assignment in if condition
#25: FILE: linux-user/main.c:2320:
+ if ((__hptr = lock_user(VERIFY_WRITE, __gaddr, sizeof(target_type), 0))) {
\
ERROR: braces {} are necessary for all arms of this statement
#25: FILE: linux-user/main.c:2320:
+ if ((__hptr = lock_user(VERIFY_WRITE, __gaddr, sizeof(target_type), 0))) {
\
[...]
+ } else \
[...]
ERROR: code indent should never use tabs
#26: FILE: linux-user/main.c:2321:
+ if ((old) != atomic_cmpxchg(__hptr, (old), (new)))^I^I\$
ERROR: braces {} are necessary for all arms of this statement
#26: FILE: linux-user/main.c:2321:
+ if ((old) != atomic_cmpxchg(__hptr, (old), (new))) \
[...]
ERROR: code indent should never use tabs
#27: FILE: linux-user/main.c:2322:
+ __ret = -TARGET_EAGAIN;^I^I^I^I^I\$
ERROR: code indent should never use tabs
#28: FILE: linux-user/main.c:2323:
+ unlock_user(__hptr, __gaddr, sizeof(target_type));^I^I\$
ERROR: code indent should never use tabs
#29: FILE: linux-user/main.c:2324:
+ } else^I^I^I^I^I^I^I^I\$
ERROR: code indent should never use tabs
#30: FILE: linux-user/main.c:2325:
+ __ret = -TARGET_EFAULT;^I^I^I^I^I^I\$
ERROR: code indent should never use tabs
#31: FILE: linux-user/main.c:2326:
+ __ret;^I^I^I^I^I^I^I^I\$
WARNING: line over 80 characters
#34: FILE: linux-user/main.c:2329:
+#define cmpxchg_user_u32(old, new, gaddr) cmpxchg_user((old), (new), (gaddr),
uint32_t)
WARNING: line over 80 characters
#35: FILE: linux-user/main.c:2330:
+#define cmpxchg_user_u64(old, new, gaddr) cmpxchg_user((old), (new), (gaddr),
uint64_t)
total: 14 errors, 2 warnings, 40 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.
=== OUTPUT END ===
Test command exited with code: 1
---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to address@hidden