Re: [Xen-devel] [PATCH v3] xenstat: handle more than 1024 domains

[Ian Campbell <ian.campbell@xxxxxxxxxx>]

On Fri, 2016-01-15 at 17:07 +0100, Juergen Gross wrote:
> get_domain_ids() in libxenstat used by read_attributes_qdisk() is
> limited to 1024 domains. Remove that limit.
> 
> Signed-off-by: Juergen Gross <jgross@xxxxxxxx>
> ---
> V3: Obey 80 character line length limit as requested by Ian Campbell.
> ÂÂÂÂIn order to not have to wrap so many lines just carve out the loop
> ÂÂÂÂbody into a sub-function.

That's a completely reasonable way to avoid excess wrapping (and should
probably be done more often!)

>  As this changes the patch structure, drop
> ÂÂÂÂthe Reviewed-by tag of Wei.
> 
> Signed-off-by: Juergen Gross <jgross@xxxxxxxx>

I took a look at the "git show -b" version, which was a lot easier to read:

Acked-by: Ian Campbell <ian.campbell@xxxxxxxxxx>


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel