Re: [Xen-devel] [PATCH] expose shared page count through xenlight

[Adin Scannell <adin@xxxxxxxxxxxxxxx>]

Hi Ian,

On Wed, Sep 21, 2011 at 5:34 AM, Ian Campbell <Ian.Campbell@xxxxxxxxxx> wrote:
> Can we only print this info in verbose mode, or only if >=1 domain has
> shared pages or something? I don't suppose just printing the total
> number of shared pages across the system is as useful as printing the nr
> for each domain?

Sure. I'll move the shared pages to verbose output and resend.

I'll also see about throwing in the total number of shared frames to
the info command (separate patch).

> Also, was the change from \t to explicit padding in the header
> intentional? I agree it's a bit odd to use \t in the header and spaces
> in the actual info lines.

It was intentional, as the \t's happen to line up very badly after the
change (and they should both be either spaces or tabs anyways).

Cheers,
-Adin

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel