This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


Re: [Xen-devel] [PATCH] libxl: added libxl compatibility with physical b

To: Ian Campbell <Ian.Campbell@xxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] libxl: added libxl compatibility with physical backend file for NetBSD
From: Christoph Egger <Christoph.Egger@xxxxxxx>
Date: Fri, 22 Jul 2011 15:46:04 +0200
Cc: Roger Pau Monné <roger.pau@xxxxxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 22 Jul 2011 06:47:16 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <1311341983.12772.66.camel@xxxxxxxxxxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
References: <CAPLaKK5QW_PiGcKcKoH-g4VF8zM_VFmpjGiuW5DbNN9kHeMXJg@xxxxxxxxxxxxxx> <1311341983.12772.66.camel@xxxxxxxxxxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; NetBSD amd64; en-US; rv: Gecko/20110523 Lightning/1.0b3pre Thunderbird/3.1.10
On 07/22/11 15:39, Ian Campbell wrote:
On Fri, 2011-07-22 at 13:38 +0100, Roger Pau Monné wrote:

This patch fixes the problems described in a previous thread regarding
PV machines not booting on NetBSD with libxl toolstack. The patch also
prevents libxl from adding PCI entries to xenstore if there is no PCI
device configured in the guest.

Thanks Roger.

In general we prefer to see individual fixes split up into separate
patches for review and commit. Also please can you add to your ~/.hgrc:
         showfunc = True
It prints the function name in the hunk header which makes it easier to

The change to the hotplug script seems odd, if the "xtype" paramter is
not needed then just remove it altogether, otherwise we need to teach
libxl what to write.

The PCI change looks good, although you might as well throw the for()
loop under the check too.

"#ifdef" etc should be in column 0, but the content should be indented


The checks on LIBXL_DISK_FORMAT_EMPTY and S_ISBLK are not really related
so I would not combine them even though the return is the same.
Otherwise the HAVE_PHY_BACKEND_FILE_SUPPORT change itself looks ok

What is DONT_REMOVE_VBD_FROM_STORE for? Is this because xenbackendd does
it for you or something else?

The hotplug script invoked by xenbackendd removes the vbd entry
via xenstored-rm.

Does your changed xenbackendd still work with xend? Oh, I see what the
hotplug script change was about now -- I guess just remove the commented
out line?



---to satisfy European Law for business letters:
Advanced Micro Devices GmbH
Einsteinring 24, 85689 Dornach b. Muenchen
Geschaeftsfuehrer: Alberto Bozzo, Andrew Bowd
Sitz: Dornach, Gemeinde Aschheim, Landkreis Muenchen
Registergericht Muenchen, HRB Nr. 43632

Xen-devel mailing list