On Thu, Jan 12, 2023 at 10:28 AM Maksim Davydov <davydov-...@yandex-team.ru> wrote: > > Current 256KB is not enough for some real cases. As a possible solution > limit can be chosen to be the same as libvirt (10MB) > > Signed-off-by: Maksim Davydov <davydov-...@yandex-team.ru> > Reviewed-by: John Snow <js...@redhat.com> > --- > python/qemu/qmp/qmp_client.py | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/python/qemu/qmp/qmp_client.py b/python/qemu/qmp/qmp_client.py > index 5dcda04a75..b5772e7f32 100644 > --- a/python/qemu/qmp/qmp_client.py > +++ b/python/qemu/qmp/qmp_client.py > @@ -197,8 +197,8 @@ async def run(self, address='/tmp/qemu.socket'): > #: Logger object used for debugging messages. > logger = logging.getLogger(__name__) > > - # Read buffer limit; large enough to accept query-qmp-schema > - _limit = (256 * 1024) > + # Read buffer limit; 10MB like libvirt default > + _limit = (10 * 1024 * 1024) > > # Type alias for pending execute() result items > _PendingT = Union[Message, ExecInterruptedError] > -- > 2.25.1 >
I've staged this one patch to send with my next Python update. If by chance this series should be picked up in entirety by another maintainer before I get this one patch in: Go for it, you've got my blessing. Acked-by: John Snow <js...@redhat.com>
