From mboxrd@z Thu Jan  1 00:00:00 1970
From: Juergen Gross <jgross@suse.com>
Subject: Re: pyxc_linux_build() in
 tools/python/xen/lowlevel/xc/xc.c still needed?
Date: Tue, 6 Oct 2015 11:02:49 +0200
Message-ID: <56138E39.7020804@suse.com>
References: <56136B21.1030704@suse.com> <1444121781.5302.52.camel@citrix.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"; Format="flowed"
Content-Transfer-Encoding: 7bit
Return-path: <xen-devel-bounces@lists.xen.org>
In-Reply-To: <1444121781.5302.52.camel@citrix.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: Ian Campbell <ian.campbell@citrix.com>, "xen-devel@lists.xen.org" <xen-devel@lists.xen.org>, Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>, =?UTF-8?Q?Roger_Pau_Monn=c3=a9?= <roger.pau@citrix.com>
Cc: Wei Liu <wei.liu2@citrix.com>, Ian Jackson <ian.jackson@eu.citrix.com>, Stefano Stabellini <stefano.stabellini@eu.citrix.com>
List-Id: xen-devel@lists.xenproject.org

On 10/06/2015 10:56 AM, Ian Campbell wrote:
> On Tue, 2015-10-06 at 08:33 +0200, Juergen Gross wrote:
>> Do we have any requirements to be compatible to old releases regarding
>> the functions in tools/python/xen/lowlevel/xc/xc.c ?
>
> IMHO, no.

Okay, thanks.

I'll check for other unused functions as well and do some patches to
remove it (or them) from tools/python/xen/lowlevel/xc/xc.c

> There are also too many compatibility shims in front of the domain builder,
> i.e. all the stuff in tools/libxc/xc_dom_compat_linux.c which IMHO could be
> nuked too.

Okay, I'll look into this as well.

>> Especially pyxc_linux_build() isn't used anywhere in the tree. And this
>> is the only function setting xc_dom_image.superpages, thus removing it
>> would give us the chance to remove some code of the domain builder.
>
> Right.
>
> Removing the superpages option from the non-Python might cause concern for
> some. Adding Konrad since IIRC PV superpages is a thing Oracle implemented
> and might still care about.
>
> Also adding Roger since he is reworking the HVM builder to use the PV one,
> which for all I know might involve using the superpage option (but probably
> not the PV code to set it up).

I'm starting with the layers above the domain builder. This should give
all others interested in the features some time to raise concerns about
removing them.


Juergen