Re: [PATCH] libxl: add LIBXL_HAVE_<foo> for outstanding_pages and outstanding_memkb

[Dario Faggioli <dario.faggioli@citrix.com>]

[-- Attachment #1.1: Type: text/plain, Size: 1318 bytes --]

On ven, 2013-06-07 at 06:49 -0700, Konrad Rzeszutek Wilk wrote:
> On Fri, Jun 07, 2013 at 12:50:41AM +0200, Dario Faggioli wrote:
> > Commits d0782481 and bec8f17e added these two fields in libxl_physinfo
> 
> One usually includes right after the git commit the title
> of the patches, so:
> 
> d0782481 ("xl: export 'outstanding_pages' value from xcinfo") and
> ...
>
Ok, I think I see why that could be useful. Note taken, thanks. :-)

> > and in libxl_dominfo, respectively, but did not include the needed
> > LIBXL_HAVE_<foo> runes. Adding them.
> > 
> > Signed-off-by: Dario Faggioli <dario.faggioli@citrix.com>
> 
> Besides the above mention commit description part (And one tweak below)
> please also add
> Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> 
> or Acked-by - whichever George is looking for.
> 
Ok, thanks again. George, Ian, should I respin incorporating Konrad's
feedback (both to the changelog and to the code comment, of course)? Is
this fine for being picked up in 4.3?

Regards,
Dario

-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli
Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)


[-- Attachment #1.2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 198 bytes --]

[-- Attachment #2: Type: text/plain, Size: 126 bytes --]

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