On 11/25/2015 02:23 PM, Markus Armbruster wrote: > From: Paolo Bonzini <pbonz...@redhat.com> > > Replace the contents of the tokens GQueue with a simple struct. This cuts > the amount of memory allocated by tests/check-qjson from ~500MB to ~20MB, > and the execution time from 600ms to 80ms on my laptop. Still a lot (some > could be saved by using an intrusive list, such as QSIMPLEQ, instead of > the GQueue), but the savings are already massive and the right thing to > do would probably be to get rid of json-streamer completely. > > Signed-off-by: Paolo Bonzini <pbonz...@redhat.com> > Message-Id: <1448300659-23559-5-git-send-email-pbonz...@redhat.com> > [Straightforwardly rebased on my patches] > Signed-off-by: Markus Armbruster <arm...@redhat.com> > --- > include/qapi/qmp/json-streamer.h | 7 +++ > qobject/json-parser.c | 115 > ++++++++++++++++----------------------- > qobject/json-streamer.c | 19 +++---- > 3 files changed, 63 insertions(+), 78 deletions(-) >
Reviewed-by: Eric Blake <ebl...@redhat.com> -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature
