qemu-devel
[Top][All Lists]
Advanced

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

Re: [PATCH] vga: don't abort when adding a duplicate isa-vga device


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




reply via email to

[Prev in Thread] Current Thread [Next in Thread]