qemu-s390x
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [qemu-s390x] [Qemu-devel] [PATCH v3 00/32] s390x/tcg: Vector Instruc


From: no-reply
Subject: Re: [qemu-s390x] [Qemu-devel] [PATCH v3 00/32] s390x/tcg: Vector Instruction Support Part 1
Date: Thu, 7 Mar 2019 07:49:47 -0800 (PST)

Patchew URL: https://patchew.org/QEMU/address@hidden/



Hi,

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

Type: series
Message-id: address@hidden
Subject: [Qemu-devel] [PATCH v3 00/32] s390x/tcg: Vector Instruction Support 
Part 1

=== 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
 t [tag update]            patchew/address@hidden -> patchew/address@hidden
Switched to a new branch 'test'
5848dfc88a s390x/tcg: Implement VECTOR UNPACK *
7e368477c5 s390x/tcg: Implement VECTOR STORE WITH LENGTH
609977f736 s390x/tcg: Implement VECTOR STORE MULTIPLE
db01dbf76f s390x/tcg: Implement VECTOR STORE ELEMENT
84c37b8f82 s390x/tcg: Implement VECTOR STORE
6ffbe981b0 s390x/tcg: Provide probe_write_access helper
5d1e9782e4 s390x/tcg: Implement VECTOR SIGN EXTEND TO DOUBLEWORD
d477bf8a42 s390x/tcg: Implement VECTOR SELECT
4ada4aa4e8 s390x/tcg: Implement VECTOR SCATTER ELEMENT
54495356f0 s390x/tcg: Implement VECTOR REPLICATE IMMEDIATE
53fd66335c s390x/tcg: Implement VECTOR REPLICATE
2f7d5675bf s390x/tcg: Implement VECTOR PERMUTE DOUBLEWORD IMMEDIATE
1b0ea7576b s390x/tcg: Implement VECTOR PERMUTE
00d9df92c1 s390x/tcg: Implement VECTOR PACK *
3701ea61fb s390x/tcg: Implement VECTOR MERGE (HIGH|LOW)
29f6183c82 s390x/tcg: Implement VECTOR LOAD WITH LENGTH
c7914c98ce s390x/tcg: Implement VECTOR LOAD VR FROM GRS DISJOINT
a02ffced91 s390x/tcg: Implement VECTOR LOAD VR ELEMENT FROM GR
0749abdbd4 s390x/tcg: Implement VECTOR LOAD TO BLOCK BOUNDARY
9bec1d6b0b s390x/tcg: Implement VECTOR LOAD MULTIPLE
4b839be686 s390x/tcg: Implement VECTOR LOAD LOGICAL ELEMENT AND ZERO
6198c749ec s390x/tcg: Implement VECTOR LOAD GR FROM VR ELEMENT
23f6ed0387 s390x/tcg: Implement VECTOR LOAD ELEMENT IMMEDIATE
2168f3ed0e s390x/tcg: Implement VECTOR LOAD ELEMENT
961ce46497 s390x/tcg: Implement VECTOR LOAD AND REPLICATE
24e44fb771 s390x/tcg: Implement VECTOR LOAD
128d7ccfec s390x/tcg: Implement VECTOR GENERATE MASK
4b7349dc7e s390x/tcg: Implement VECTOR GENERATE BYTE MASK
54557b2dda s390x/tcg: Implement VECTOR GATHER ELEMENT
82ba050cbd s390x/tcg: Utilities for vector instruction helpers
1066457e34 s390x/tcg: Check vector register instructions at central point
bb70511084 s390x/tcg: Define vector instruction formats

=== OUTPUT BEGIN ===
1/32 Checking commit bb7051108455 (s390x/tcg: Define vector instruction formats)
2/32 Checking commit 1066457e346b (s390x/tcg: Check vector register 
instructions at central point)
3/32 Checking commit 82ba050cbd09 (s390x/tcg: Utilities for vector instruction 
helpers)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#17: 
new file mode 100644

total: 0 errors, 1 warnings, 101 lines checked

Patch 3/32 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
4/32 Checking commit 54557b2dda03 (s390x/tcg: Implement VECTOR GATHER ELEMENT)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#56: 
new file mode 100644

total: 0 errors, 1 warnings, 155 lines checked

Patch 4/32 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
5/32 Checking commit 4b7349dc7eee (s390x/tcg: Implement VECTOR GENERATE BYTE 
MASK)
6/32 Checking commit 128d7ccfec4e (s390x/tcg: Implement VECTOR GENERATE MASK)
7/32 Checking commit 24e44fb7712f (s390x/tcg: Implement VECTOR LOAD)
8/32 Checking commit 961ce46497cf (s390x/tcg: Implement VECTOR LOAD AND 
REPLICATE)
9/32 Checking commit 2168f3ed0e0c (s390x/tcg: Implement VECTOR LOAD ELEMENT)
10/32 Checking commit 23f6ed038729 (s390x/tcg: Implement VECTOR LOAD ELEMENT 
IMMEDIATE)
11/32 Checking commit 6198c749ec94 (s390x/tcg: Implement VECTOR LOAD GR FROM VR 
ELEMENT)
12/32 Checking commit 4b839be6865f (s390x/tcg: Implement VECTOR LOAD LOGICAL 
ELEMENT AND ZERO)
13/32 Checking commit 9bec1d6b0b63 (s390x/tcg: Implement VECTOR LOAD MULTIPLE)
14/32 Checking commit 0749abdbd438 (s390x/tcg: Implement VECTOR LOAD TO BLOCK 
BOUNDARY)
WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
#97: 
new file mode 100644

total: 0 errors, 1 warnings, 96 lines checked

Patch 14/32 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
15/32 Checking commit a02ffced918f (s390x/tcg: Implement VECTOR LOAD VR ELEMENT 
FROM GR)
16/32 Checking commit c7914c98ce0e (s390x/tcg: Implement VECTOR LOAD VR FROM 
GRS DISJOINT)
17/32 Checking commit 29f6183c820a (s390x/tcg: Implement VECTOR LOAD WITH 
LENGTH)
18/32 Checking commit 3701ea61fbbc (s390x/tcg: Implement VECTOR MERGE 
(HIGH|LOW))
19/32 Checking commit 00d9df92c1a8 (s390x/tcg: Implement VECTOR PACK *)
ERROR: space prohibited between function name and open parenthesis '('
#183: FILE: target/s390x/vec_helper.c:48:
+typedef uint##TBITS##_t (*vpk##BITS##_fn)(uint##BITS##_t, int *);              
\

total: 1 errors, 0 warnings, 245 lines checked

Patch 19/32 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.

20/32 Checking commit 1b0ea7576bec (s390x/tcg: Implement VECTOR PERMUTE)
WARNING: line over 80 characters
#21: FILE: target/s390x/helper.h:144:
+DEF_HELPER_FLAGS_5(gvec_vperm, TCG_CALL_NO_RWG, void, ptr, cptr, cptr, cptr, 
i32)

total: 0 errors, 1 warnings, 58 lines checked

Patch 20/32 has style problems, please review.  If any of these errors
are false positives report them to the maintainer, see
CHECKPATCH in MAINTAINERS.
21/32 Checking commit 2f7d5675bf89 (s390x/tcg: Implement VECTOR PERMUTE 
DOUBLEWORD IMMEDIATE)
22/32 Checking commit 53fd66335c53 (s390x/tcg: Implement VECTOR REPLICATE)
23/32 Checking commit 54495356f06c (s390x/tcg: Implement VECTOR REPLICATE 
IMMEDIATE)
24/32 Checking commit 4ada4aa4e80b (s390x/tcg: Implement VECTOR SCATTER ELEMENT)
25/32 Checking commit d477bf8a42d9 (s390x/tcg: Implement VECTOR SELECT)
26/32 Checking commit 5d1e9782e43a (s390x/tcg: Implement VECTOR SIGN EXTEND TO 
DOUBLEWORD)
27/32 Checking commit 6ffbe981b0c1 (s390x/tcg: Provide probe_write_access 
helper)
28/32 Checking commit 84c37b8f821a (s390x/tcg: Implement VECTOR STORE)
29/32 Checking commit db01dbf76f04 (s390x/tcg: Implement VECTOR STORE ELEMENT)
30/32 Checking commit 609977f73603 (s390x/tcg: Implement VECTOR STORE MULTIPLE)
31/32 Checking commit 7e368477c537 (s390x/tcg: Implement VECTOR STORE WITH 
LENGTH)
32/32 Checking commit 5848dfc88ab1 (s390x/tcg: Implement VECTOR UNPACK *)
=== OUTPUT END ===

Test command exited with code: 1


The full log is available at
http://patchew.org/logs/address@hidden/testing.checkpatch/?type=message.
---
Email generated automatically by Patchew [http://patchew.org/].
Please send your feedback to address@hidden

reply via email to

[Prev in Thread] Current Thread [Next in Thread]