[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2] oslib-posix: Fix compiler warning and some d
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v2] oslib-posix: Fix compiler warning and some data types |
Date: |
Sun, 15 Oct 2017 17:32:49 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 14/10/2017 18:53, Philippe Mathieu-Daudé wrote:
>> @@ -328,7 +328,7 @@ static void *do_touch_pages(void *arg)
>> * don't need to write at all so we don't cause
>> * wear on the storage backing the region...
>> */
>> - *(volatile char *)addr = *addr;
>> + *addr = *addr;
> I personally prefer the other form which is mostly self-explicit when
> reviewing this code.
>
> Declaring addr non volatile and using volatile cast here:
> Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
>
I agree with Philippe; in genereal, volatile is more of a property of
the access rather than the variable.
Paolo