Re: [PATCH v1] softmmu/physmem: fix wrong assertion in qemu_ram_alloc_internal()

2021-08-03 Thread Peter Xu
On Mon, Aug 02, 2021 at 05:22:38PM +0200, David Hildenbrand wrote: > When adding RAM_NORESERVE, we forgot to remove the old assertion when > adding the updated one, most probably when reworking the patches or > rebasing. We can easily crash QEMU by adding > -object memory-backend-ram,id=mem0,size

Re: [PATCH v1] softmmu/physmem: fix wrong assertion in qemu_ram_alloc_internal()

2021-08-02 Thread Philippe Mathieu-Daudé
On 8/2/21 5:22 PM, David Hildenbrand wrote: > When adding RAM_NORESERVE, we forgot to remove the old assertion when > adding the updated one, most probably when reworking the patches or > rebasing. We can easily crash QEMU by adding > -object memory-backend-ram,id=mem0,size=500G,reserve=off > to

[PATCH v1] softmmu/physmem: fix wrong assertion in qemu_ram_alloc_internal()

2021-08-02 Thread David Hildenbrand
When adding RAM_NORESERVE, we forgot to remove the old assertion when adding the updated one, most probably when reworking the patches or rebasing. We can easily crash QEMU by adding -object memory-backend-ram,id=mem0,size=500G,reserve=off to the QEMU cmdline: qemu-system-x86_64: ../softmmu/phy