[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

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

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



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.