On Wed, Aug 09, 2017 at 03:41:48PM +0800, Yi Sun wrote:
> This patch implements a new get hw info interface suitable for all psr 
> allocation
> features and the whole flow. It also enables MBA support in tools to get MBA
> HW info.
> 
> Signed-off-by: Yi Sun <yi.y....@linux.intel.com>

I would suggest you add to the commit message to better describe
the content of the patch. Like:

    Introduce a bunch of things in libxc ... Implement a new function on
    libxl to do XXX, the existing function libxl_psr_cat_get_info is
    changed to use the new function ...

And then you will discover certain things are better split out to
separate patches so reviewers can more easily review / ack them.

I have gone through this patch. The code looks sensible but I'm not yet
confident to say it is alright. I would appreciate you break this patch
down a bit if possible.

I will leave reviewing the xl bits to Chao -- he knows the relationship
between various features better than I do.

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

Reply via email to