Re: [PATCH v22] tools: CMDs and APIs for Cache Monitoring Technology

[Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>]

On Fri, Oct 10, 2014 at 09:56:17AM +0100, Wei Liu wrote:
> On Fri, Oct 10, 2014 at 09:37:24AM +0100, Ian Campbell wrote:
> > On Fri, 2014-10-10 at 09:31 +0100, Jan Beulich wrote:
> > > >>> On 10.10.14 at 10:26, <Ian.Campbell@citrix.com> wrote:
> > > > On Fri, 2014-10-10 at 14:29 +0800, Chao Peng wrote:
> > > >> On Thu, Oct 09, 2014 at 10:09:08AM +0100, Wei Liu wrote:
> > > >> > On Thu, Oct 09, 2014 at 12:02:04PM +0800, Chao Peng wrote:
> > > >> > > Introduced some new xl commands to enable/disable Cache Monitoring
> > > >> > > Technology(CMT) feature.
> > > >> > > 
> > > >> > > The following two commands is to attach/detach the CMT feature
> > > >> > > to/from a certain domain.
> > > >> > > 
> > > >> > > $ xl psr-cmt-attach domid
> > > >> > > $ xl psr-cmt-detach domid
> > > >> > > 
> > > >> > > This command is to display the CMT information, such as L3 cache
> > > >> > > occupancy.
> > > >> > > 
> > > >> > > $ xl psr-cmt-show cache_occupancy <domid>
> > > >> > > 
> > > >> > > Signed-off-by: Dongxiao Xu <dongxiao.xu@intel.com>
> > > >> > > Signed-off-by: Chao Peng <chao.p.peng@linux.intel.com>
> > > >> > 
> > > >> > Acked-by: Wei Liu <wei.liu2@citrix.com>
> > > >> > 
> > > >> > In your later contribution please don't forget to carry over any tags
> > > >> > you got during review.
> > > >> > 
> > > >> > Wei.
> > > >> 
> > > >> So konrad, is it the time to allow this patch in?
> > > > 
> > > > Has Konrad not already acked this series? Given that the hypervisor side
> > > > went in earlier in the week I was going to assume so (i.e. I even tried
> > > > to apply v21) and just go ahead.
> > > 
> > > Iirc he explicitly excluded this patch from his ack, since he wants
> > > maintainer acks to be in place before giving his.
> > 
> > Ah, ok. I'll wait then
> > 
> 
> See <20141008124917.GD13391@laptop.dumpdata.com>
> 
> I think he implicitly acked this patch.

I was waiting for Wei or Ian's Ack before giving it an Ack.

And since that has been given (see above):

Release-Acked-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>

Thank you!
> 
> Wei.
> 
> > 
> > Ian.