[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [BUG] checkpatch.pl hangs on target/mips/msa_helper.c
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [BUG] checkpatch.pl hangs on target/mips/msa_helper.c |
Date: |
Wed, 11 Jul 2018 10:34:54 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.0 |
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