Re: [PATCH v2 07/10] softmmu/physmem: Don't use atomic operations in ram_block_discard_(disable|require)

[David Hildenbrand]

>>  bool ram_block_discard_is_disabled(void)
>>  {
>> -    return qatomic_read(&ram_block_discard_disabled) > 0;
>> +    return qatomic_read(&ram_block_discard_disablers);
>>  }
> return value won't be bool?

The compiler does type conversion.

!= 0 -> true
== 0 -> false

[...]

> Apart from query above, looks good.
> Reviewed-by: Pankaj Gupta <pankaj.gupta@cloud.ionos.com>

Thanks!


-- 
Thanks,

David / dhildenb