[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] hostmem-file: fix pmem file size check
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH] hostmem-file: fix pmem file size check |
Date: |
Fri, 23 Aug 2019 15:14:38 -0300 |
On Fri, Aug 23, 2019 at 02:56:32PM +0100, Stefan Hajnoczi wrote:
> Commit 314aec4a6e06844937f1677f6cba21981005f389 ("hostmem-file: reject
> invalid pmem file sizes") added a file size check that verifies the
> hostmem object's size parameter against the actual devdax pmem file.
> This is useful because getting the size wrong results in confusing
> errors inside the guest.
>
> However, the code doesn't work properly for files where struct
> stat::st_size is zero. Hostmem-file's ->alloc() function returns early
> without setting an Error, causing the following assertion failure:
>
> qemu/memory.c:2215: memory_region_get_ram_ptr: Assertion `mr->ram_block'
> failed.
>
> This patch handles the case where qemu_get_pmem_size() returns 0 but
> there is no error.
>
> Fixes: 314aec4a6e06844937f1677f6cba21981005f389
> Signed-off-by: Stefan Hajnoczi <address@hidden>
Queued, thanks.
--
Eduardo