Re: [PATCH for 6.2? v2] Revert "vga: don't abort when adding a duplicate isa-vga device"

["Philippe Mathieu-Daudé" <f4bug@amsat.org>]

On Mon, Dec 6, 2021 at 11:50 AM Philippe Mathieu-Daudé <f4bug@amsat.org> wrote:
> On 12/6/21 11:43, Philippe Mathieu-Daudé wrote:
> > On 12/6/21 10:52, Alex Bennée wrote:
> >> This reverts commit 7852a77f598635a67a222b6c1463c8b46098aed2.
> >>
> >> The check is bogus as it ends up finding itself and falling over.
> >>
> >> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> >> Resolves: https://gitlab.com/qemu-project/qemu/-/issues/733
> >> Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>
> >> Message-Id: <20211202164929.1119036-1-alex.bennee@linaro.org>
> >>
> >> ---
> >> v2
> >>   - fixed bug link
> >
> > IIUC Cornelia & Mark said the correct fix is simply:
> >
> > -- >8 --
> > -    if (object_resolve_path_type("", TYPE_ISA_VGA, NULL)) {
> > +    if (!object_resolve_path_type("", TYPE_ISA_VGA, NULL)) {
> > ---
>
> Bah I just noticed your update on v1:
> https://www.mail-archive.com/qemu-devel@nongnu.org/msg854596.html
>
> So:
>
> Acked-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

I meant:
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

> and queued to mips-fixes, thanks.

(still valid).