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: Roger Pau Monné <roger.pau@xxxxxxxxxxxxx>
Subject: Re: [Xen-devel] [PATCH] libxl: added libxl compatibility with physical backend file for NetBSD
From: Ian Campbell <Ian.Campbell@xxxxxxxxxx>
Date: Fri, 22 Jul 2011 14:39:43 +0100
Cc: "xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date: Fri, 22 Jul 2011 06:40:21 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <CAPLaKK5QW_PiGcKcKoH-g4VF8zM_VFmpjGiuW5DbNN9kHeMXJg@xxxxxxxxxxxxxx>
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>
Organization: Citrix Systems, Inc.
References: <CAPLaKK5QW_PiGcKcKoH-g4VF8zM_VFmpjGiuW5DbNN9kHeMXJg@xxxxxxxxxxxxxx>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
On Fri, 2011-07-22 at 13:38 +0100, Roger Pau Monné wrote:
> Hello,
> 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?

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?


Xen-devel mailing list