xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
From: Mukesh Rathor <mukesh.rathor@oracle.com>
To: Ian Campbell <Ian.Campbell@citrix.com>
Cc: xen-devel@lists.xenproject.org, Ian.Jackson@eu.citrix.com
Subject: Re: [V2 PATCH 3/4] PVH xen tools: libxl changes to create a PVH guest.
Date: Mon, 9 Sep 2013 18:40:47 -0700	[thread overview]
Message-ID: <20130909184047.74250ae6@mantra.us.oracle.com> (raw)
In-Reply-To: <1378716579.19967.10.camel@kazak.uk.xensource.com>

On Mon, 9 Sep 2013 09:49:39 +0100
Ian Campbell <Ian.Campbell@citrix.com> wrote:

> On Fri, 2013-09-06 at 17:28 -0700, Mukesh Rathor wrote:
> > V2: set c_info->hap to default value of pvh, and use LOG(ERROR...).
> >     Move PVH check to xc_dom_gnttab_init.
> >     Move xl doc change here.
> > 
> > Signed-off-by: Mukesh Rathor <mukesh.rathor@oracle.com>
> > ---
> >  docs/man/xl.cfg.pod.5        |    3 +++
> >  tools/libxl/libxl.h          |    6 ++++++
> >  tools/libxl/libxl_create.c   |   12 ++++++++++++
> >  tools/libxl/libxl_dom.c      |    2 ++
> >  tools/libxl/libxl_internal.h |    1 +
> >  tools/libxl/libxl_types.idl  |    1 +
> >  tools/libxl/libxl_x86.c      |    4 +++-
> >  tools/libxl/xl_cmdimpl.c     |    1 +
> >  8 files changed, 29 insertions(+), 1 deletions(-)
> > 
> > diff --git a/docs/man/xl.cfg.pod.5 b/docs/man/xl.cfg.pod.5
> > index 08d6cc4..4c925c9 100644
> > --- a/docs/man/xl.cfg.pod.5
> > +++ b/docs/man/xl.cfg.pod.5
> > @@ -634,6 +634,9 @@ if your particular guest kernel does not
> > require this behaviour then it is safe to allow this to be enabled
> > but you may wish to disable it anyway.
> >  
> > +=item B<pvh=BOOLEAN>
> > +Selects whether to run this guest in an HVM container. Default is
> > 0.
> 
>                               ^ PV
> 
> Will help stop people becoming confused and thinking this turns on HVM
> mode.

The change is under PV guest. HVM section os separate. But I can add
it anyways.

thanks
Mukesh

  reply	other threads:[~2013-09-10  1:40 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-09-07  0:28 [V2 PATCH 0/5]: PVH xen tools: tools changes to create PVH guest Mukesh Rathor
2013-09-07  0:28 ` [V2 PATCH 1/4] PVH xen tools: In Elf code, add hvm callback vector Mukesh Rathor
2013-09-09  8:33   ` Ian Campbell
2013-09-07  0:28 ` [V2 PATCH 2/4] PVH xen tools: libxc changes to build a PVH guest Mukesh Rathor
2013-09-09  8:42   ` Ian Campbell
2013-09-10  1:40     ` Mukesh Rathor
2013-09-10  8:27       ` Ian Campbell
2013-09-07  0:28 ` [V2 PATCH 3/4] PVH xen tools: libxl changes to create " Mukesh Rathor
2013-09-09  8:49   ` Ian Campbell
2013-09-10  1:40     ` Mukesh Rathor [this message]
2013-09-10  8:28       ` Ian Campbell
2013-09-07  0:28 ` [V2 PATCH 4/4] PVH xen tools: xenstored changes Mukesh Rathor

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20130909184047.74250ae6@mantra.us.oracle.com \
    --to=mukesh.rathor@oracle.com \
    --cc=Ian.Campbell@citrix.com \
    --cc=Ian.Jackson@eu.citrix.com \
    --cc=xen-devel@lists.xenproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).