Re: [Xen-devel] [PATCH v4 4/4] docs: add document to introduce CDP command

2015-09-25 Thread Ian Campbell
On Fri, 2015-09-25 at 18:16 +0800, He Chen wrote: > [...] > > And if cdp is not enabled: > > [...] > Right above. > > > xl psr-cat-cbm-set -c -d 0xd00dfeed > > > > *ERRROR* > > [now: cbm=] > > > > In current code, it is valid since -c & -d have the same behaviour as > neither of them. > So,

Re: [Xen-devel] [PATCH v4 4/4] docs: add document to introduce CDP command

2015-09-25 Thread He Chen
On Fri, Sep 25, 2015 at 10:58:58AM +0100, Ian Campbell wrote: > On Fri, 2015-09-25 at 17:29 +0800, He Chen wrote: > > On Thu, Sep 24, 2015 at 12:22:02PM +0100, Ian Campbell wrote: > > > On Thu, 2015-09-17 at 17:35 +0800, He Chen wrote: > > > > Add new CDP options with CAT commands in xl interface m

Re: [Xen-devel] [PATCH v4 4/4] docs: add document to introduce CDP command

2015-09-25 Thread Ian Campbell
On Fri, 2015-09-25 at 17:29 +0800, He Chen wrote: > On Thu, Sep 24, 2015 at 12:22:02PM +0100, Ian Campbell wrote: > > On Thu, 2015-09-17 at 17:35 +0800, He Chen wrote: > > > Add new CDP options with CAT commands in xl interface man page. > > > Add description of CDP in xl-psr.markdown. > > > > It

Re: [Xen-devel] [PATCH v4 4/4] docs: add document to introduce CDP command

2015-09-25 Thread He Chen
On Thu, Sep 24, 2015 at 12:22:02PM +0100, Ian Campbell wrote: > On Thu, 2015-09-17 at 17:35 +0800, He Chen wrote: > > Add new CDP options with CAT commands in xl interface man page. > > Add description of CDP in xl-psr.markdown. > > It would have been fine to include this in the previous patch by

Re: [Xen-devel] [PATCH v4 4/4] docs: add document to introduce CDP command

2015-09-24 Thread Jan Beulich
>>> On 24.09.15 at 13:22, wrote: > On Thu, 2015-09-17 at 17:35 +0800, He Chen wrote: >> Add new CDP options with CAT commands in xl interface man page. >> --- a/docs/man/xl.pod.1 >> +++ b/docs/man/xl.pod.1 >> @@ -1530,6 +1530,12 @@ applications. In the Xen implementation, CAT is >> used to control

Re: [Xen-devel] [PATCH v4 4/4] docs: add document to introduce CDP command

2015-09-24 Thread Ian Campbell
On Thu, 2015-09-17 at 17:35 +0800, He Chen wrote: > Add new CDP options with CAT commands in xl interface man page. > Add description of CDP in xl-psr.markdown. It would have been fine to include this in the previous patch by the way. > > Signed-off-by: He Chen > --- > docs/man/xl.pod.1