diff -r 2760576b0d7c -r 9996d4f06e70 tools/Rules.mk --- a/tools/Rules.mk +++ b/tools/Rules.mk @@ -26,8 +26,19 @@ LDLIBS_libxenguest = -L$(XEN_LIBXC) -lxe CFLAGS_libxenstore = -I$(XEN_XENSTORE) $(CFLAGS_include) LDLIBS_libxenstore = -L$(XEN_XENSTORE) -lxenstore +ifeq ($(CONFIG_Linux),y) +LIBXL_BLKTAP = y +else +LIBXL_BLKTAP = n +endif + +ifeq ($(LIBXL_BLKTAP),y) CFLAGS_libblktapctl = -I$(XEN_BLKTAP2)/control -I$(XEN_BLKTAP2)/include $(CFLAGS_include) LDLIBS_libblktapctl = -L$(XEN_BLKTAP2)/control -lblktapctl +else +CFLAGS_libblktapctl = +LDLIBS_libblktapctl = +endif X11_LDPATH = -L/usr/X11R6/$(LIBLEAFDIR) diff -r 2760576b0d7c -r 9996d4f06e70 tools/libxl/Makefile --- a/tools/libxl/Makefile +++ b/tools/libxl/Makefile @@ -18,6 +18,12 @@ CFLAGS += $(CFLAGS_libxenctrl) $(CFLAGS_ LIBS = $(LDLIBS_libxenctrl) $(LDLIBS_libxenguest) $(LDLIBS_libxenstore) $(LDLIBS_libblktapctl) $(UTIL_LIBS) LIBXL_OBJS-y = osdeps.o libxl_paths.o libxl_bootloader.o +ifeq ($(LIBXL_BLKTAP),y) +LIBXL_OBJS-y += libxl_blktap2.o +else +LIBXL_OBJS-y += libxl_noblktap2.o +endif + LIBXL_OBJS = flexarray.o libxl.o libxl_pci.o libxl_dom.o libxl_exec.o libxl_xshelp.o libxl_device.o libxl_internal.o xenguest.o libxl_utils.o $(LIBXL_OBJS-y) AUTOINCS= libxlu_cfg_y.h libxlu_cfg_l.h diff -r 2760576b0d7c -r 9996d4f06e70 tools/libxl/libxl.c --- a/tools/libxl/libxl.c +++ b/tools/libxl/libxl.c @@ -36,7 +36,6 @@ #include "libxl_utils.h" #include "libxl_internal.h" #include "flexarray.h" -#include "tap-ctl.h" #define PAGE_TO_MEMKB(pages) ((pages) * 4) @@ -1697,27 +1696,6 @@ int libxl_confirm_device_model_startup(l /******************************************************************************/ -static char *get_blktap2_device(libxl_gc *gc, - const char *name, const char *type) -{ - int minor = tap_ctl_find_minor(type, name); - if (minor < 0) - return NULL; - return libxl_sprintf(gc, "/dev/xen/blktap-2/tapdev%d", minor); -} - -static char *make_blktap2_device(libxl_gc *gc, - const char *name, const char *type) -{ - char *params, *devname = NULL; - int err; - params = libxl_sprintf(gc, "%s:%s", type, name); - err = tap_ctl_create(params, &devname); - if (!err) - libxl_ptr_add(gc, devname); - return err ? NULL : devname; -} - int libxl_device_disk_add(libxl_ctx *ctx, uint32_t domid, libxl_device_disk *disk) { libxl_gc gc = LIBXL_INIT_GC(ctx); @@ -1772,14 +1750,13 @@ int libxl_device_disk_add(libxl_ctx *ctx case PHYSTYPE_FILE: /* let's pretend is tap:aio for the moment */ disk->phystype = PHYSTYPE_AIO; - case PHYSTYPE_AIO: case PHYSTYPE_QCOW: case PHYSTYPE_QCOW2: case PHYSTYPE_VHD: { - const char *msg; - if (!tap_ctl_check(&msg)) { - const char *type = device_disk_string_of_phystype(disk->phystype); - char *dev; - dev = get_blktap2_device(&gc, disk->physpath, type); - if (!dev) - dev = make_blktap2_device(&gc, disk->physpath, type); + case PHYSTYPE_AIO: + case PHYSTYPE_QCOW: + case PHYSTYPE_QCOW2: + case PHYSTYPE_VHD: + if (libxl_blktap_enabled(ctx)) { + const char *dev = libxl_blktap_devpath(ctx, + disk->physpath, disk->phystype); if (!dev) { rc = ERROR_FAIL; goto out_free; @@ -1802,7 +1779,7 @@ int libxl_device_disk_add(libxl_ctx *ctx device.backend_kind = DEVICE_TAP; break; - } + default: XL_LOG(ctx, XL_LOG_ERROR, "unrecognized disk physical type: %d\n", disk->phystype); rc = ERROR_INVAL; @@ -1874,7 +1851,8 @@ int libxl_device_disk_del(libxl_ctx *ctx char * libxl_device_disk_local_attach(libxl_ctx *ctx, libxl_device_disk *disk) { libxl_gc gc = LIBXL_INIT_GC(ctx); - char *dev = NULL, *ret; + const char *dev = NULL; + char *ret; int phystype = disk->phystype; switch (phystype) { case PHYSTYPE_PHY: { @@ -1885,16 +1863,14 @@ char * libxl_device_disk_local_attach(li case PHYSTYPE_FILE: /* let's pretend is tap:aio for the moment */ phystype = PHYSTYPE_AIO; - case PHYSTYPE_AIO: case PHYSTYPE_QCOW: case PHYSTYPE_QCOW2: case PHYSTYPE_VHD: { - const char *msg; - if (!tap_ctl_check(&msg)) { - const char *type = device_disk_string_of_phystype(phystype); - dev = get_blktap2_device(&gc, disk->physpath, type); - if (!dev) - dev = make_blktap2_device(&gc, disk->physpath, type); - } + case PHYSTYPE_AIO: + case PHYSTYPE_QCOW: + case PHYSTYPE_QCOW2: + case PHYSTYPE_VHD: + if (libxl_blktap_enabled(ctx)) + dev = libxl_blktap_devpath(ctx, disk->physpath, phystype); break; - } + default: XL_LOG(ctx, XL_LOG_ERROR, "unrecognized disk physical type: %d\n", phystype); break; diff -r 2760576b0d7c -r 9996d4f06e70 tools/libxl/libxl_blktap2.c --- /dev/null +++ b/tools/libxl/libxl_blktap2.c @@ -0,0 +1,51 @@ +/* + * Copyright (C) 2010 Advanced Micro Devices + * Author Christoph Egger + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU Lesser General Public License as published + * by the Free Software Foundation; version 2.1 only. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Lesser General Public License for more details. + */ + +#include "libxl.h" +#include "libxl_osdeps.h" +#include "libxl_internal.h" + +#include "tap-ctl.h" + +int libxl_blktap_enabled(libxl_ctx *ctx) +{ + const char *msg; + return !tap_ctl_check(&msg); +} + +const char *libxl_blktap_devpath(libxl_ctx *ctx, + const char *disk, + libxl_disk_phystype phystype) +{ + const char *type; + char *params, *devname; + int minor, err; + + type = device_disk_string_of_phystype(phystype); + minor = tap_ctl_find_minor(type, disk); + if (minor >= 0) { + devname = libxl_sprintf(ctx, "/dev/xen/blktap-2/tapdev%d", minor); + if (devname) + return devname; + } + + params = libxl_sprintf(ctx, "%s:%s", type, disk); + err = tap_ctl_create(params, &devname); + if (!err) { + libxl_ptr_add(ctx, devname); + return devname; + } + + return NULL; +} diff -r 2760576b0d7c -r 9996d4f06e70 tools/libxl/libxl_internal.h --- a/tools/libxl/libxl_internal.h +++ b/tools/libxl/libxl_internal.h @@ -249,4 +249,23 @@ _hidden char *libxl_abs_path(libxl_gc *g _hidden char *_libxl_domid_to_name(libxl_gc *gc, uint32_t domid); _hidden char *_libxl_poolid_to_name(libxl_gc *gc, uint32_t poolid); +/* + * blktap2 support + */ + +/* libxl_blktap_enabled: + * return true if blktap/blktap2 support is available. + */ +int libxl_blktap_enabled(libxl_ctx *ctx); + +/* libxl_blktap_devpath: + * Argument: path and disk image as specified in config file. + * The type specifies whether this is aio, qcow, qcow2, etc. + * returns device path xenstore wants to have. returns NULL + * if no device corresponds to the disk. + */ +const char *libxl_blktap_devpath(libxl_ctx *ctx, + const char *disk, + libxl_disk_phystype phystype); + #endif diff -r 2760576b0d7c -r 9996d4f06e70 tools/libxl/libxl_noblktap2.c --- /dev/null +++ b/tools/libxl/libxl_noblktap2.c @@ -0,0 +1,29 @@ +/* + * Copyright (C) 2010 Advanced Micro Devices + * Author Christoph Egger + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU Lesser General Public License as published + * by the Free Software Foundation; version 2.1 only. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Lesser General Public License for more details. + */ + +#include "libxl.h" +#include "libxl_osdeps.h" +#include "libxl_internal.h" + +int libxl_blktap_enabled(libxl_ctx *ctx) +{ + return 0; +} + +const char *libxl_blktap_devpath(libxl_ctx *ctx, + const char *disk, + libxl_disk_phystype phystype) +{ + return NULL; +}