On 7/18/23 15:50, Luca Bonissi wrote:
On 32bit qemu-user targets, memory allocation failed after about 2GB due to incorrect signed (instead of the correct unsigned) "last" parameter in pageflags_find and pageflags_next functions (file accel/tcg/user-exec.c).

The parameter, on 32bit targets, will be signed-extent to the 64bit final uint64_t parameters, leading to incorrect comparison on the RBTree (only the first call to mmap on the upper 2GB memory will be successful).

Following the patch to fix the bug (re-submit to add "signed-off-by"):

Signed-off-by: Luca Bonissi <q...@bonslack.org>

Reviewed-by: Richard Henderson <richard.hender...@linaro.org>

Don't reply to previous patches with a new patch -- tooling doesn't handle it.
I've applied this by hand.


r~

Reply via email to