Re: [PATCH 1/1] chardev/char: fix qemu_chr_is_busy() check

2024-10-14 Thread Roman Penyaev
Hi Marc-André, On Thu, Oct 10, 2024 at 12:20 PM Marc-André Lureau wrote: > > Hi Roman > > On Thu, Oct 10, 2024 at 1:28 PM Roman Penyaev 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

Re: [PATCH 1/1] chardev/char: fix qemu_chr_is_busy() check

2024-10-10 Thread Roman Penyaev
On Thu, Oct 10, 2024, 12:20 Marc-André Lureau wrote: > Hi Roman > > On Thu, Oct 10, 2024 at 1:28 PM Roman Penyaev 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 >

Re: [PATCH 1/1] chardev/char: fix qemu_chr_is_busy() check

2024-10-10 Thread Marc-André Lureau
Hi Roman On Thu, Oct 10, 2024 at 1:28 PM Roman Penyaev 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

[PATCH 1/1] chardev/char: fix qemu_chr_is_busy() check

2024-10-10 Thread Roman Penyaev
`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 Cc: "Marc-André Lureau" Cc: