[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] audio: fix pc speaker init
From: |
Gerd Hoffmann |
Subject: |
Re: [Qemu-devel] [PATCH] audio: fix pc speaker init |
Date: |
Thu, 24 Jan 2019 11:47:45 +0100 |
User-agent: |
NeoMutt/20180716 |
> > +static int pcspk_audio_init(ISABus *bus)
> > +{
> > + isa_create_simple (bus, TYPE_PC_SPEAKER);
> > + return 0;
>
> Previously we had soundhw_init() calling pcspk_audio_init() and ignore
> failures. Now since you use isa_create_simple() which calls
> qdev_init_nofail(), failure will directly exit(1).
> Is it the expected behavior?
The change wasn't intentional, but I think it is better to throw an
error instead of silently ignoring the problem.
In practice it probably doesn't make much of a difference, there isn't
much why isa_create_simple could fail ...
cheers,
Gerd