From mboxrd@z Thu Jan  1 00:00:00 1970
From: Wei Liu <wei.liu2@citrix.com>
Subject: Re: [PATCH 1/2] altp2m: Merge p2m_set_altp2m_mem_access
 and p2m_set_mem_access
Date: Thu, 28 Jan 2016 14:36:03 +0000
Message-ID: <20160128143603.GN25660@citrix.com>
References: <1453925201-15926-1-git-send-email-tlengyel@novetta.com>
	<20160128105010.GJ25660@citrix.com>
	<1453978536.26591.68.camel@citrix.com>
	<20160128110012.GM25660@citrix.com>
	<CAD33N+6y4yeoRN=8LnX=UthgUJi7YhbNNLCFj9=62vnoqKgvrg@mail.gmail.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
Received: from mail6.bemta5.messagelabs.com ([195.245.231.135])
	by lists.xen.org with esmtp (Exim 4.72)
	(envelope-from <prvs=828e4d37b=wei.liu2@citrix.com>)
	id 1aOng8-0001K5-18
	for xen-devel@lists.xenproject.org; Thu, 28 Jan 2016 14:36:08 +0000
Content-Disposition: inline
In-Reply-To: <CAD33N+6y4yeoRN=8LnX=UthgUJi7YhbNNLCFj9=62vnoqKgvrg@mail.gmail.com>
List-Unsubscribe: <http://lists.xen.org/cgi-bin/mailman/options/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=unsubscribe>
List-Post: <mailto:xen-devel@lists.xen.org>
List-Help: <mailto:xen-devel-request@lists.xen.org?subject=help>
List-Subscribe: <http://lists.xen.org/cgi-bin/mailman/listinfo/xen-devel>,
	<mailto:xen-devel-request@lists.xen.org?subject=subscribe>
Sender: xen-devel-bounces@lists.xen.org
Errors-To: xen-devel-bounces@lists.xen.org
To: "Lengyel, Tamas" <tlengyel@novetta.com>
Cc: Wei Liu <wei.liu2@citrix.com>, Ian Campbell <ian.campbell@citrix.com>, Razvan Cojocaru <rcojocaru@bitdefender.com>, Stefano Stabellini <stefano.stabellini@eu.citrix.com>, George Dunlap <george.dunlap@eu.citrix.com>, Andrew Cooper <andrew.cooper3@citrix.com>, Ian Jackson <ian.jackson@eu.citrix.com>, Stefano Stabellini <stefano.stabellini@citrix.com>, Jan Beulich <jbeulich@suse.com>, xen-devel@lists.xenproject.org, Keir Fraser <keir@xen.org>
List-Id: xen-devel@lists.xenproject.org

On Thu, Jan 28, 2016 at 07:30:31AM -0700, Lengyel, Tamas wrote:
> On Jan 28, 2016 4:00 AM, "Wei Liu" <wei.liu2@citrix.com> wrote:
> >
> > On Thu, Jan 28, 2016 at 10:55:36AM +0000, Ian Campbell wrote:
> > > --- a/tools/libxc/include/xenctrl.h
> > > > > +++ b/tools/libxc/include/xenctrl.h
> > > > > @@ -2027,9 +2027,6 @@ int xc_altp2m_destroy_view(xc_interface
> *handle,
> > > > > domid_t domid,
> > > > >  /* Switch all vCPUs of the domain to the specified altp2m view */
> > > > >  int xc_altp2m_switch_to_view(xc_interface *handle, domid_t domid,
> > > > >                               uint16_t view_id);
> > > > > -int xc_altp2m_set_mem_access(xc_interface *handle, domid_t domid,
> > > > > -                             uint16_t view_id, xen_pfn_t gfn,
> > > > > -                             xenmem_access_t access);
> > > >
> > > > What is the support status of these APIs in libxc? Are they supposed
> to
> > > > be stable now? Do you have opinion on making them stable interfaces?
> > >
> > > Nothing in libxc is ever a stable interface. It is always completely
> fine
> > > to change them if that is what is needed.
> > >
> >
> > Right. Ignore the first question and this patch is ready to go in when
> > those nits are fixed.
> >
> > It would still be nice if we can get a libmemaccess or some sort so that
> > people can build on top of it. Just curious if that's feasible.
> >
> > Wei.
> 
> There is LibVMI and Bitdefender also released its library so there are
> choices if folks find libxc too volatile.

Thanks, this makes sense.

Wei.

> 
> Tamas