Re: [PATCH] monitor: Fix slow reading

2019-12-03 Thread Yury Kotov
02.12.2019, 23:50, "Markus Armbruster" : > Yury Kotov writes: > >>  Hi! >> >>  29.11.2019, 11:22, "Markus Armbruster" : >>>  Yury Kotov writes: >>>   The monitor_can_read (as a callback of qemu_chr_fe_set_handlers)   should return size of buffer which monitor_qmp_read or monitor_read >>

Re: [PATCH] monitor: Fix slow reading

2019-12-02 Thread Denis V. Lunev
On 12/2/19 11:49 PM, Markus Armbruster wrote: > Yury Kotov writes: > >> Hi! >> >> 29.11.2019, 11:22, "Markus Armbruster" : >>> Yury Kotov writes: >>>  The monitor_can_read (as a callback of qemu_chr_fe_set_handlers)  should return size of buffer which monitor_qmp_read or monitor_read >>

Re: [PATCH] monitor: Fix slow reading

2019-12-02 Thread Markus Armbruster
Yury Kotov writes: > Hi! > > 29.11.2019, 11:22, "Markus Armbruster" : >> Yury Kotov writes: >> >>>  The monitor_can_read (as a callback of qemu_chr_fe_set_handlers) >>>  should return size of buffer which monitor_qmp_read or monitor_read >>>  can process. >>>  Currently, monitor_can_read returns

Re: [PATCH] monitor: Fix slow reading

2019-12-02 Thread Yury Kotov
Hi! 29.11.2019, 11:22, "Markus Armbruster" : > Yury Kotov writes: > >>  The monitor_can_read (as a callback of qemu_chr_fe_set_handlers) >>  should return size of buffer which monitor_qmp_read or monitor_read >>  can process. >>  Currently, monitor_can_read returns 1 as a result of logical not. >

Re: [PATCH] monitor: Fix slow reading

2019-11-29 Thread Markus Armbruster
Yury Kotov writes: > The monitor_can_read (as a callback of qemu_chr_fe_set_handlers) > should return size of buffer which monitor_qmp_read or monitor_read > can process. > Currently, monitor_can_read returns 1 as a result of logical not. > Thus, for each QMP command, len(QMD) iterations of the m

Re: [PATCH] monitor: Fix slow reading

2019-11-22 Thread Philippe Mathieu-Daudé
Cc'ing the chardev maintainers. On 11/22/19 10:23 AM, Yury Kotov wrote: The monitor_can_read (as a callback of qemu_chr_fe_set_handlers) should return size of buffer which monitor_qmp_read or monitor_read can process. Currently, monitor_can_read returns 1 as a result of logical not. Thus, for ea

[PATCH] monitor: Fix slow reading

2019-11-22 Thread Yury Kotov
The monitor_can_read (as a callback of qemu_chr_fe_set_handlers) should return size of buffer which monitor_qmp_read or monitor_read can process. Currently, monitor_can_read returns 1 as a result of logical not. Thus, for each QMP command, len(QMD) iterations of the main loop are required to handle