permissions: 0.989
peripherals: 0.969
hypervisor: 0.969
register: 0.957
user-level: 0.956
operating system: 0.956
risc-v: 0.954
debug: 0.951
device: 0.951
graphic: 0.950
PID: 0.949
vnc: 0.948
assembly: 0.944
VMM: 0.938
semantic: 0.937
performance: 0.936
x86: 0.936
arm: 0.935
alpha: 0.931
virtual: 0.931
ppc: 0.929
architecture: 0.928
files: 0.918
KVM: 0.916
socket: 0.913
network: 0.913
kernel: 0.909
TCG: 0.906
boot: 0.902
mistranslation: 0.897
i386: 0.895
--------------------
TCG: 0.893
boot: 0.799
files: 0.766
register: 0.658
device: 0.619
PID: 0.516
VMM: 0.497
risc-v: 0.481
vnc: 0.477
debug: 0.235
peripherals: 0.180
operating system: 0.170
hypervisor: 0.162
kernel: 0.124
alpha: 0.121
virtual: 0.104
architecture: 0.073
assembly: 0.048
semantic: 0.039
user-level: 0.019
socket: 0.019
KVM: 0.014
performance: 0.012
permissions: 0.011
network: 0.008
graphic: 0.004
arm: 0.002
x86: 0.002
mistranslation: 0.002
ppc: 0.002
i386: 0.002

[Qemu-devel] [BUG] checkpatch.pl hangs on target/mips/msa_helper.c

If  checkpatch.pl is applied (using switch "-f") on file 
target/mips/msa_helper.c, it will hang.

There is a workaround for this particular file:

These lines in msa_helper.c:

        uint## BITS ##_t S = _S, T = _T;                            \
        uint## BITS ##_t as, at, xs, xt, xd;                        \

should be replaced with:

        uint## BITS ## _t S = _S, T = _T;                           \
        uint## BITS ## _t as, at, xs, xt, xd;                       \

(a space is added after the second "##" in each line)

The workaround is found by partial deleting and undeleting of the code in 
msa_helper.c in binary search fashion.

This workaround will soon be submitted by me as a patch within a series on misc 
MIPS issues.

I took a look at checkpatch.pl code, and it looks it is fairly complicated to 
fix the issue, since it happens in the code segment involving intricate logic 
conditions.

Regards,
Aleksandar

On Wed, Jul 04, 2018 at 03:35:18PM +0000, Aleksandar Markovic wrote:
>
If  checkpatch.pl is applied (using switch "-f") on file
>
target/mips/msa_helper.c, it will hang.
>
>
There is a workaround for this particular file:
>
>
These lines in msa_helper.c:
>
>
uint## BITS ##_t S = _S, T = _T;                            \
>
uint## BITS ##_t as, at, xs, xt, xd;                        \
>
>
should be replaced with:
>
>
uint## BITS ## _t S = _S, T = _T;                           \
>
uint## BITS ## _t as, at, xs, xt, xd;                       \
>
>
(a space is added after the second "##" in each line)
>
>
The workaround is found by partial deleting and undeleting of the code in
>
msa_helper.c in binary search fashion.
>
>
This workaround will soon be submitted by me as a patch within a series on
>
misc MIPS issues.
>
>
I took a look at checkpatch.pl code, and it looks it is fairly complicated to
>
fix the issue, since it happens in the code segment involving intricate logic
>
conditions.
Thanks for figuring this out, Aleksandar.  Not sure if anyone else has
the apetite to fix checkpatch.pl.

Stefan
signature.asc
Description:
PGP signature

On 07/11/2018 09:36 AM, Stefan Hajnoczi wrote:
>
On Wed, Jul 04, 2018 at 03:35:18PM +0000, Aleksandar Markovic wrote:
>
> If  checkpatch.pl is applied (using switch "-f") on file
>
> target/mips/msa_helper.c, it will hang.
>
>
>
> There is a workaround for this particular file:
>
>
>
> These lines in msa_helper.c:
>
>
>
>         uint## BITS ##_t S = _S, T = _T;                            \
>
>         uint## BITS ##_t as, at, xs, xt, xd;                        \
>
>
>
> should be replaced with:
>
>
>
>         uint## BITS ## _t S = _S, T = _T;                           \
>
>         uint## BITS ## _t as, at, xs, xt, xd;                       \
>
>
>
> (a space is added after the second "##" in each line)
>
>
>
> The workaround is found by partial deleting and undeleting of the code in
>
> msa_helper.c in binary search fashion.
>
>
>
> This workaround will soon be submitted by me as a patch within a series on
>
> misc MIPS issues.
>
>
>
> I took a look at checkpatch.pl code, and it looks it is fairly complicated
>
> to fix the issue, since it happens in the code segment involving intricate
>
> logic conditions.
>
>
Thanks for figuring this out, Aleksandar.  Not sure if anyone else has
>
the apetite to fix checkpatch.pl.
Anyone else but Paolo ;P
http://lists.nongnu.org/archive/html/qemu-devel/2018-07/msg01250.html
signature.asc
Description:
OpenPGP digital signature