Re: [Xen-devel] [PATCH 08/35] libxl: Replace libxl__qmp_initializations by ev_qmp calls

[Anthony PERARD <anthony.perard@xxxxxxxxxx>]

On Tue, Sep 17, 2019 at 06:02:18PM +0100, Ian Jackson wrote:
> Anthony PERARD writes ("[PATCH 08/35] libxl: Replace 
> libxl__qmp_initializations by ev_qmp calls"):
> > Setup a timeout of 10s for all the commands. It used to be about 5s
> > per commands.
> 
> This patch is quite hard to review because it is a
> rewrite/rearrangement and I can't see where all the pieces come from.
> 
> How hard would it be to make a pre-patch to shuffle the code to the
> same place as it's going, and change the variable names etc. ?

I think that would be possible, with a few XXX comments for some
intermediary code.

I'll give it a try.

> If that's hard then I guess I can go through it line by line...
> 
> Thanks,
> Ian.

-- 
Anthony PERARD

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel