[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] qemu/atomic.h: prefix qemu_ to solve <stdatomic.h> collision
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH] qemu/atomic.h: prefix qemu_ to solve <stdatomic.h> collisions |
Date: |
Tue, 22 Sep 2020 10:33:44 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
On 9/22/20 10:17 AM, Stefan Hajnoczi wrote:
> On Mon, Sep 21, 2020 at 01:56:08PM -0700, no-reply@patchew.org wrote:
>> ERROR: Macros with multiple statements should be enclosed in a do - while
>> loop
>> #2968: FILE: include/qemu/atomic.h:152:
>> +#define qemu_atomic_rcu_read__nocheck(ptr, valptr) \
>> __atomic_load(ptr, valptr, __ATOMIC_RELAXED); \
>> smp_read_barrier_depends();
>>
...
>> WARNING: Block comments use a leading /* on a separate line
>> #7456: FILE: util/rcu.c:154:
>> + /* In either case, the qemu_atomic_mb_set below blocks stores that
>> free
>>
>> total: 7 errors, 4 warnings, 6507 lines checked
>
> These are pre-existing coding style issues. This is a big patch that
> tries to make as few actual changes as possible so I would rather not
> try to fix them.
What I do with automated patches triggering checkpatch errors:
- run automated conversion
- fix errors until checkpatch is happy
- run automated conversion inversed
- result is the checkpatch changes, commit them
- run automated conversion again, commit