[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] softmmu: Tidy dirtylimit_dirty_ring_full_time
From: |
Juan Quintela |
Subject: |
Re: [PATCH] softmmu: Tidy dirtylimit_dirty_ring_full_time |
Date: |
Fri, 28 Apr 2023 20:38:12 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/28.2 (gnu/linux) |
Thomas Huth <thuth@redhat.com> wrote:
> On 28/04/2023 12.34, Richard Henderson wrote:
>> Drop inline marker: let compiler decide.
>> Change return type to uint64_t: this matches the computation in the
>> return statement and the local variable assignment in the caller.
>> Rename local to dirty_ring_size_MB to fix typo.
>> Simplify conversion to MiB via qemu_target_page_bits and right shift.
>> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Not that I expect the amount of bytes in one machine to not fit in
64bit, but good change.
About the signesdness of variables, I really hate it, specially that
compiler not help here where arguments, operands and assignment place is
unsigned. Sniff.