Hi Marc-André,
On Thu, Oct 10, 2024 at 12:20 PM Marc-André Lureau
<marcandre.lur...@gmail.com> wrote:
>
> Hi Roman
>
> On Thu, Oct 10, 2024 at 1:28 PM Roman Penyaev <r.peni...@gmail.com> wrote:
>>
>> `mux_cnt` struct member never goes negative or decrements,
>> so mux chardev can be !busy only when there are no
>> frontends attached. This patch fixes the always-true
>> check.
>>
>> Fixes: a4afa548fc6d ("char: move front end handlers in CharBackend")
>> Signed-off-by: Roman Penyaev <r.peni...@gmail.com>
>> Cc: "Marc-André Lureau" <marcandre.lur...@redhat.com>
>> Cc: qemu-devel@nongnu.org
>
>
> Reviewed-by: Marc-André Lureau <marcandre.lur...@redhat.com>
>
> That would be worth some new tests for chardev removal. It seems to be
> lacking. And mux probably need extra fixing. I can take a look if you don't.
I've just sent an attempt to fix the removal of frontends, plus some test cases.
Please take a look.
Thanks.
--
Roman
