Re: [PATCH] monitor/qmp: resume monitor when clearing its queue

2019-10-10 Thread Markus Armbruster
Wolfgang Bumiller writes: > On Wed, Oct 09, 2019 at 09:18:04PM +0200, Markus Armbruster wrote: >> Wolfgang Bumiller writes: >> >> > On Wed, Oct 09, 2019 at 10:39:44AM +0200, Markus Armbruster wrote: >> >> Cc: Marc-André for additional monitor and chardev expertise. >> >> >> >> Wolfgang Bumille

Re: [PATCH] monitor/qmp: resume monitor when clearing its queue

2019-10-10 Thread Wolfgang Bumiller
On Wed, Oct 09, 2019 at 09:18:04PM +0200, Markus Armbruster wrote: > Wolfgang Bumiller writes: > > > On Wed, Oct 09, 2019 at 10:39:44AM +0200, Markus Armbruster wrote: > >> Cc: Marc-André for additional monitor and chardev expertise. > >> > >> Wolfgang Bumiller writes: > >> > >> > When a monit

Re: [PATCH] monitor/qmp: resume monitor when clearing its queue

2019-10-09 Thread Markus Armbruster
Wolfgang Bumiller writes: > On Wed, Oct 09, 2019 at 10:39:44AM +0200, Markus Armbruster wrote: >> Cc: Marc-André for additional monitor and chardev expertise. >> >> Wolfgang Bumiller writes: >> >> > When a monitor's queue is filled up in handle_qmp_command() >> > it gets suspended. It's the di

Re: [PATCH] monitor/qmp: resume monitor when clearing its queue

2019-10-09 Thread Wolfgang Bumiller
On Wed, Oct 09, 2019 at 10:39:44AM +0200, Markus Armbruster wrote: > Cc: Marc-André for additional monitor and chardev expertise. > > Wolfgang Bumiller writes: > > > When a monitor's queue is filled up in handle_qmp_command() > > it gets suspended. It's the dispatcher bh's job currently to > > r

Re: [PATCH] monitor/qmp: resume monitor when clearing its queue

2019-10-09 Thread Markus Armbruster
Cc: Marc-André for additional monitor and chardev expertise. Wolfgang Bumiller writes: > When a monitor's queue is filled up in handle_qmp_command() > it gets suspended. It's the dispatcher bh's job currently to > resume the monitor, which it does after processing an event > from the queue. Howe

[PATCH] monitor/qmp: resume monitor when clearing its queue

2019-10-02 Thread Wolfgang Bumiller
When a monitor's queue is filled up in handle_qmp_command() it gets suspended. It's the dispatcher bh's job currently to resume the monitor, which it does after processing an event from the queue. However, it is possible for a CHR_EVENT_CLOSED event to be processed before before the bh is scheduled