Re: [PATCH v5 0/6] Add strace support for printing arguments of selected syscalls

[no-reply]

Patchew URL: 
20200619123331.17387-1-filip.bozuta@syrmia.com/">https://patchew.org/QEMU/20200619123331.17387-1-filip.bozuta@syrmia.com/



Hi,

This series seems to have some coding style problems. See output below for
more information:

Subject: [PATCH v5 0/6] Add strace support for printing arguments of selected 
syscalls
Type: series
Message-id: 20200619123331.17387-1-filip.bozuta@syrmia.com

=== 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/159257395689.52908.4409314503988289481.stgit@naples-babu.amd.com -> 
patchew/159257395689.52908.4409314503988289481.stgit@naples-babu.amd.com
 * [new tag]         patchew/20200619135844.23307-1-alex.bennee@linaro.org -> 
patchew/20200619135844.23307-1-alex.bennee@linaro.org
Switched to a new branch 'test'
fc51e74 linux-user: Add strace support for printing arguments of fallocate()
0a4c5ce linux-user: Add strace support for printing arguments of 
chown()/lchown()
1f88017 linux-user: Add strace support for printing arguments of lseek()
0e4e7f6 linux-user: Add strace support for printing argument of syscalls used 
for extended attributes
60bbb8d linux-user: Add strace support for a group of syscalls
61a78c4 linux-user: Extend strace support to enable argument printing after 
syscall execution

=== OUTPUT BEGIN ===
1/6 Checking commit 61a78c4db925 (linux-user: Extend strace support to enable 
argument printing after syscall execution)
2/6 Checking commit 60bbb8d99c2d (linux-user: Add strace support for a group of 
syscalls)
3/6 Checking commit 0e4e7f6c61a5 (linux-user: Add strace support for printing 
argument of syscalls used for extended attributes)
4/6 Checking commit 1f8801785b1f (linux-user: Add strace support for printing 
arguments of lseek())
5/6 Checking commit 0a4c5ce6db8c (linux-user: Add strace support for printing 
arguments of chown()/lchown())
6/6 Checking commit fc51e742a543 (linux-user: Add strace support for printing 
arguments of fallocate())
ERROR: storage class should be at the beginning of the declaration
#70: FILE: linux-user/strace.c:1138:
+UNUSED static struct flags falloc_flags[] = {

total: 1 errors, 0 warnings, 104 lines checked

Patch 6/6 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
20200619123331.17387-1-filip.bozuta@syrmia.com/testing.checkpatch/?type=message">http://patchew.org/logs/20200619123331.17387-1-filip.bozuta@syrmia.com/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [https://patchew.org/].
Please send your feedback to patchew-devel@redhat.com