Re: [PATCH] qemu: Don't ignore failure when building default memory backend

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2/22/22 17:21, Ján Tomko wrote:
> On a Tuesday in 2022, Michal Privoznik wrote:
>> When building the default memory backend (which has id='pc.ram')
>> and no guest NUMA is configured then
>> qemuBuildMemCommandLineMemoryDefaultBackend() is called. However,
>> its return value is ignored which means that on invalid
>> configuration (e.g. when non-existent hugepage size was
>> requested) an error is reported into the logs but QEMU is started
>> anyway. And while QEMU does error out its error message doesn't
>> give much clue what's going on:
>>
>>  qemu-system-x86_64: Memory backend 'pc.ram' not found
>>
>> While at it, introduce a test case. While I could chose a nice
>> looking value (e.g. 4MiB) that's exactly what I wanted to avoid,
>> because while such value might not be possible on x84_64 it may
>> be possible on other arches (e.g. ppc is notoriously known for
>> supporting wide range of HP sizes). Let's stick with obviously
>> wrong value of 5MiB.
>>
>> Reported-by: Charles Polisher <chas@xxxxxxxxxx>
>> Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
>> ---
>> src/qemu/qemu_command.c                       |  5 ++--
>> .../hugepages-default-5M.x86_64-latest.err    |  1 +
>> .../qemuxml2argvdata/hugepages-default-5M.xml | 27 +++++++++++++++++++
>> tests/qemuxml2argvtest.c                      |  1 +
>> 4 files changed, 32 insertions(+), 2 deletions(-)
>> create mode 100644
>> tests/qemuxml2argvdata/hugepages-default-5M.x86_64-latest.err
>> create mode 100644 tests/qemuxml2argvdata/hugepages-default-5M.xml
>>
> 
> Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx>
> 

Thanks, pushed now.

Michal




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux