|
From: | Jose Ricardo Ziviani |
Subject: | Re: [PATCH] vga: don't abort when adding a duplicate isa-vga device |
Date: | Mon, 16 Aug 2021 09:34:11 -0300 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.12.0 |
Hello Thomas and Gerd, Thank you for reviewing it. Sending a v2 soon. Thank you very much! On 16/08/2021 02:05, Gerd Hoffmann wrote:
Hi,+ if (qemu_ram_block_by_name("vga.vram")) { + error_report("vga.vram is already registered, ignoring this device"); + return; + }I think we should not ignore the error, but rather turn this into a proper error (instead of aborting).Yes. Silently fixing up things automatically tends to be worse long-term. take care, Gerd
[Prev in Thread] | Current Thread | [Next in Thread] |