Any comments on the implementation? If there is anything improper, I can update.

Thanks,
Chunyan

>>> On 6/25/2015 at 06:07 PM, in message
<1435226838-3067-4-git-send-email-cy...@suse.com>, Chunyan Liu <cy...@suse.com>
wrote: 
> Add pvusb APIs, including: 
>  - attach/detach (create/destroy) virtual usb controller. 
>  - attach/detach usb device 
>  - list usb controller and usb devices 
>  - some other helper functions 
>  
> Signed-off-by: Chunyan Liu <cy...@suse.com> 
> Signed-off-by: Simon Cao <caobosi...@gmail.com> 
> --- 
> changes: 
>   - Use macros DEFINE_DEVICE_ADD and DEFINE_DEVICES_ADD for adding 
>     usbctrl and usb, update all related codes. 
>   - Use an extend macro DEFINE_DEVICE_REMOVE_EXT for removimg usbctrl
>     update all related codes. 
>   - Remove busid from libxl_device_usb definition, keep bus.addr only, 
>     update all related codes. 
>   - Remove documentation since it's mostly about design, move to 
>     cover-letter. Some parts are moved to code comments. 
>   - Address some other comments 
>  
>  tools/libxl/Makefile                 |    2 +- 
>  tools/libxl/libxl.c                  |   53 ++ 
>  tools/libxl/libxl.h                  |   64 ++ 
>  tools/libxl/libxl_device.c           |    4 + 
>  tools/libxl/libxl_internal.h         |   20 +- 
>  tools/libxl/libxl_osdeps.h           |   13 + 
>  tools/libxl/libxl_pvusb.c            | 1305  
> ++++++++++++++++++++++++++++++++++ 
>  tools/libxl/libxl_types.idl          |   50 ++ 
>  tools/libxl/libxl_types_internal.idl |    1 + 
>  tools/libxl/libxl_utils.c            |   16 + 
>  tools/libxl/libxl_utils.h            |    5 + 
>  11 files changed, 1531 insertions(+), 2 deletions(-) 
>  create mode 100644 tools/libxl/libxl_pvusb.c 
>  
> diff --git a/tools/libxl/Makefile b/tools/libxl/Makefile 
> index cc9c152..b820105 100644 
> --- a/tools/libxl/Makefile 
> +++ b/tools/libxl/Makefile 
> @@ -95,7 +95,7 @@ LIBXL_OBJS = flexarray.o libxl.o libxl_create.o libxl_dm.o  
> libxl_pci.o \ 
>                       libxl_internal.o libxl_utils.o libxl_uuid.o \ 
>                       libxl_json.o libxl_aoutils.o libxl_numa.o libxl_vnuma.o 
> \ 
>                       libxl_save_callout.o _libxl_save_msgs_callout.o \ 
> -                     libxl_qmp.o libxl_event.o libxl_fork.o $(LIBXL_OBJS-y) 
> +                     libxl_qmp.o libxl_event.o libxl_fork.o libxl_pvusb.o 
> $(LIBXL_OBJS-y) 
>  LIBXL_OBJS += libxl_genid.o 
>  LIBXL_OBJS += _libxl_types.o libxl_flask.o _libxl_types_internal.o 
>   
> diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c 
> index 6570476..6542127 100644 
> --- a/tools/libxl/libxl.c 
> +++ b/tools/libxl/libxl.c 
> @@ -4233,11 +4233,54 @@ DEFINE_DEVICE_REMOVE(vtpm, destroy, 1) 
>   
>   
> /**************************************************************************** 
> **/ 
>   
> +/* Macro for defining device remove/destroy functions for usbctrl */ 
> +/* Following functions are defined: 
> + * libxl_device_usbctrl_remove 
> + * libxl_device_usbctrl_destroy 
> + */ 
> + 
> +#define DEFINE_DEVICE_REMOVE_EXT(type, removedestroy, f)                \ 
> +    int libxl_device_##type##_##removedestroy(libxl_ctx *ctx,           \ 
> +        uint32_t domid, libxl_device_##type *type,                      \ 
> +        const libxl_asyncop_how *ao_how)                                \ 
> +    {                                                                   \ 
> +        AO_CREATE(ctx, domid, ao_how);                                  \ 
> +        libxl__device *device;                                          \ 
> +        libxl__ao_device *aodev;                                        \ 
> +        int rc;                                                         \ 
> +                                                                        \ 
> +        GCNEW(device);                                                  \ 
> +        rc = libxl__device_from_##type(gc, domid, type, device);        \ 
> +        if (rc != 0) goto out;                                          \ 
> +                                                                        \ 
> +        GCNEW(aodev);                                                   \ 
> +        libxl__prepare_ao_device(ao, aodev);                            \ 
> +        aodev->action = LIBXL__DEVICE_ACTION_REMOVE;                    \ 
> +        aodev->dev = device;                                            \ 
> +        aodev->callback = device_addrm_aocomplete;                      \ 
> +        aodev->force = f;                                               \ 
> +        libxl__initiate_device_##type##_remove(egc, aodev);             \ 
> +                                                                        \ 
> +    out:                                                                \ 
> +        if (rc) return AO_ABORT(rc);                                    \ 
> +        return AO_INPROGRESS;                                           \ 
> +    } 
> + 
> + 
> +DEFINE_DEVICE_REMOVE_EXT(usbctrl, remove, 0) 
> +DEFINE_DEVICE_REMOVE_EXT(usbctrl, destroy, 1) 
> + 
> +#undef DEFINE_DEVICE_REMOVE_EXT 
> + 
> +/************************************************************************** 
> ****/ 
> + 
>  /* Macro for defining device addition functions in a compact way */ 
>  /* The following functions are defined: 
>   * libxl_device_disk_add 
>   * libxl_device_nic_add 
>   * libxl_device_vtpm_add 
> + * libxl_device_usbctrl_add 
> + * libxl_device_usb_add 
>   */ 
>   
>  #define DEFINE_DEVICE_ADD(type)                                         \ 
> @@ -4269,6 +4312,12 @@ DEFINE_DEVICE_ADD(nic) 
>  /* vtpm */ 
>  DEFINE_DEVICE_ADD(vtpm) 
>   
> +/* usbctrl */ 
> +DEFINE_DEVICE_ADD(usbctrl) 
> + 
> +/* usb */ 
> +DEFINE_DEVICE_ADD(usb) 
> + 
>  #undef DEFINE_DEVICE_ADD 
>   
>   
> /**************************************************************************** 
> **/ 
> @@ -6770,6 +6819,10 @@ int libxl_retrieve_domain_configuration(libxl_ctx  
> *ctx, uint32_t domid, 
>   
>      MERGE(pci, pcidevs, COMPARE_PCI, {}); 
>   
> +    MERGE(usbctrl, usbctrls, COMPARE_USBCTRL, {}); 
> + 
> +    MERGE(usb, usbs, COMPARE_USB, {}); 
> + 
>      /* Take care of removable device. We maintain invariant in the 
>       * insert / remove operation so that: 
>       * 1. if xenstore is "empty" while JSON is not, the result 
> diff --git a/tools/libxl/libxl.h b/tools/libxl/libxl.h 
> index 0a7913b..6669835 100644 
> --- a/tools/libxl/libxl.h 
> +++ b/tools/libxl/libxl.h 
> @@ -123,6 +123,23 @@ 
>  #define LIBXL_HAVE_DOMAIN_NODEAFFINITY 1 
>   
>  /* 
> + * LIBXL_HAVE_PVUSB indicates the functions for doing hot-plug of 
> + * USB devices through pvusb. 
> + * 
> + * With this functionality, one can add/remove USB controllers to/from 
> + * guest, and attach/detach USB devices to/from USB controllers. To add 
> + * USB controllers and USB devices, one can either adding USB controllers 
> + * first and then attaching USB devices to some USB controller, or adding 
> + * USB devices to guest directly, it will automatically create a USB 
> + * controller for USB devices to attach. To remove USB controllers or USB 
> + * devices, one can either remove USB devices under USB controller one by 
> + * one and then remove USB controller, or remove USB controller directly, 
> + * it will remove all USB devices under it automatically. 
> + * 
> + */ 
> +#define LIBXL_HAVE_PVUSB 1 
> + 
> +/* 
>   * LIBXL_HAVE_BUILDINFO_HVM_VENDOR_DEVICE indicates that the 
>   * libxl_vendor_device field is present in the hvm sections of 
>   * libxl_domain_build_info. This field tells libxl which 
> @@ -1269,6 +1286,53 @@ int libxl_cdrom_insert(libxl_ctx *ctx, uint32_t domid, 
>  
> libxl_device_disk *disk, 
>                         const libxl_asyncop_how *ao_how) 
>                         LIBXL_EXTERNAL_CALLERS_ONLY; 
>   
> +/* USB Controllers*/ 
> +int libxl_device_usbctrl_add(libxl_ctx *ctx, uint32_t domid, 
> +                         libxl_device_usbctrl *usbctrl, 
> +                         const libxl_asyncop_how *ao_how) 
> +                         LIBXL_EXTERNAL_CALLERS_ONLY; 
> + 
> +int libxl_device_usbctrl_remove(libxl_ctx *ctx, uint32_t domid, 
> +                         libxl_device_usbctrl *usbctrl, 
> +                         const libxl_asyncop_how *ao_how) 
> +                         LIBXL_EXTERNAL_CALLERS_ONLY; 
> + 
> +int libxl_device_usbctrl_destroy(libxl_ctx *ctx, uint32_t domid, 
> +                         libxl_device_usbctrl *usbctrl, 
> +                         const libxl_asyncop_how *ao_how) 
> +                         LIBXL_EXTERNAL_CALLERS_ONLY; 
> + 
> +libxl_device_usbctrl *libxl_device_usbctrl_list(libxl_ctx *ctx, 
> +                            uint32_t domid, int *num); 
> + 
> +void libxl_device_usbctrl_list_free(libxl_device_usbctrl* list, int nr); 
> + 
> + 
> +int libxl_device_usbctrl_getinfo(libxl_ctx *ctx, uint32_t domid, 
> +                                libxl_device_usbctrl *usbctrl, 
> +                                libxl_usbctrlinfo *usbctrlinfo); 
> + 
> +/* USB Devices */ 
> +int libxl_device_usb_add(libxl_ctx *ctx, uint32_t domid, libxl_device_usb  
> *usb, 
> +                         const libxl_asyncop_how *ao_how) 
> +                         LIBXL_EXTERNAL_CALLERS_ONLY; 
> + 
> +int libxl_device_usb_remove(libxl_ctx *ctx, uint32_t domid,  
> libxl_device_usb *usb, 
> +                            const libxl_asyncop_how *ao_how) 
> +                            LIBXL_EXTERNAL_CALLERS_ONLY; 
> + 
> +libxl_device_usb * 
> +libxl_device_usb_list(libxl_ctx *ctx, uint32_t domid, int *num); 
> + 
> +libxl_device_usb * 
> +libxl_device_usb_list_per_usbctrl(libxl_ctx *ctx, uint32_t domid, 
> +                                  libxl_devid usbctrl, int *num); 
> + 
> +void libxl_device_usb_list_free(libxl_device_usb* list, int nr); 
> + 
> +int libxl_device_usb_getinfo(libxl_ctx *ctx, libxl_device_usb *usb, 
> +                             libxl_usbinfo *usbinfo); 
> + 
>  /* Network Interfaces */ 
>  int libxl_device_nic_add(libxl_ctx *ctx, uint32_t domid, libxl_device_nic  
> *nic, 
>                           const libxl_asyncop_how *ao_how) 
> diff --git a/tools/libxl/libxl_device.c b/tools/libxl/libxl_device.c 
> index 93bb41e..9869a21 100644 
> --- a/tools/libxl/libxl_device.c 
> +++ b/tools/libxl/libxl_device.c 
> @@ -676,6 +676,10 @@ void libxl__devices_destroy(libxl__egc *egc,  
> libxl__devices_remove_state *drs) 
>                  aodev->action = LIBXL__DEVICE_ACTION_REMOVE; 
>                  aodev->dev = dev; 
>                  aodev->force = drs->force; 
> +                if (dev->backend_kind == LIBXL__DEVICE_KIND_VUSB) { 
> +                    libxl__initiate_device_usbctrl_remove(egc, aodev); 
> +                    continue; 
> +                } 
>                  libxl__initiate_device_remove(egc, aodev); 
>              } 
>          } 
> diff --git a/tools/libxl/libxl_internal.h b/tools/libxl/libxl_internal.h 
> index 0965e08..6a8d4ee 100644 
> --- a/tools/libxl/libxl_internal.h 
> +++ b/tools/libxl/libxl_internal.h 
> @@ -2438,6 +2438,14 @@ _hidden void libxl__device_vtpm_add(libxl__egc *egc,  
> uint32_t domid, 
>                                     libxl_device_vtpm *vtpm, 
>                                     libxl__ao_device *aodev); 
>   
> +_hidden void libxl__device_usbctrl_add(libxl__egc *egc, uint32_t domid, 
> +                               libxl_device_usbctrl *usbctrl, 
> +                               libxl__ao_device *aodev); 
> + 
> +_hidden void libxl__device_usb_add(libxl__egc *egc, uint32_t domid, 
> +                                   libxl_device_usb *usb, 
> +                                   libxl__ao_device *aodev); 
> + 
>  /* Internal function to connect a vkb device */ 
>  _hidden int libxl__device_vkb_add(libxl__gc *gc, uint32_t domid, 
>                                    libxl_device_vkb *vkb); 
> @@ -2470,6 +2478,13 @@ _hidden void  
> libxl__wait_device_connection(libxl__egc*, 
>  _hidden void libxl__initiate_device_remove(libxl__egc *egc, 
>                                             libxl__ao_device *aodev); 
>   
> +_hidden int libxl__device_from_usbctrl(libxl__gc *gc, uint32_t domid, 
> +                               libxl_device_usbctrl *usbctrl, 
> +                               libxl__device *device); 
> + 
> +_hidden void libxl__initiate_device_usbctrl_remove(libxl__egc *egc, 
> +                                                   libxl__ao_device  
> *aodev); 
> + 
>  /* 
>   * libxl__get_hotplug_script_info returns the args and env that should 
>   * be passed to the hotplug script for the requested device. 
> @@ -3645,7 +3660,10 @@ static inline void libxl__update_config_vtpm(libxl__gc 
>  
> *gc, 
>  #define COMPARE_PCI(a, b) ((a)->func == (b)->func &&    \ 
>                             (a)->bus == (b)->bus &&      \ 
>                             (a)->dev == (b)->dev) 
> - 
> +#define COMPARE_USB(a, b) ((a)->hostbus == (b)->hostbus &&  \ 
> +                           (a)->hostaddr == (b)->hostaddr) 
> +#define COMPARE_USBCTRL(a, b) ((a)->devid == (b)->devid) 
> +  
>  /* DEVICE_ADD 
>   * 
>   * Add a device in libxl_domain_config structure 
> diff --git a/tools/libxl/libxl_osdeps.h b/tools/libxl/libxl_osdeps.h 
> index 08eaf0c..55caf71 100644 
> --- a/tools/libxl/libxl_osdeps.h 
> +++ b/tools/libxl/libxl_osdeps.h 
> @@ -24,6 +24,8 @@ 
>  #define _GNU_SOURCE 
>   
>  #if defined(__NetBSD__) 
> +#define SYSFS_USB_DEV          "/sys/bus/usb/devices" 
> +#define SYSFS_USBBACK_DRIVER   "/kern/xen/usb" 
>  #define SYSFS_PCI_DEV          "/sys/bus/pci/devices" 
>  #define SYSFS_PCIBACK_DRIVER   "/kern/xen/pci" 
>  #define NETBACK_NIC_NAME       "xvif%ui%d" 
> @@ -31,6 +33,8 @@ 
>  #elif defined(__OpenBSD__) 
>  #include <util.h> 
>  #elif defined(__linux__) 
> +#define SYSFS_USB_DEV          "/sys/bus/usb/devices" 
> +#define SYSFS_USBBACK_DRIVER   "/sys/bus/usb/drivers/usbback" 
>  #define SYSFS_PCI_DEV          "/sys/bus/pci/devices" 
>  #define SYSFS_PCIBACK_DRIVER   "/sys/bus/pci/drivers/pciback" 
>  #define NETBACK_NIC_NAME       "vif%u.%d" 
> @@ -38,12 +42,21 @@ 
>  #elif defined(__sun__) 
>  #include <stropts.h> 
>  #elif defined(__FreeBSD__) 
> +#define SYSFS_USB_DEV          "/dev/null" 
> +#define SYSFS_USBBACK_DRIVER   "/dev/null" 
>  #define SYSFS_PCI_DEV          "/dev/null" 
>  #define SYSFS_PCIBACK_DRIVER   "/dev/null" 
>  #define NETBACK_NIC_NAME       "xnb%u.%d" 
>  #include <libutil.h> 
>  #endif 
>   
> +#ifndef SYSFS_USBBACK_DRIVER 
> +#error define SYSFS_USBBACK_DRIVER for your platform 
> +#endif 
> +#ifndef SYSFS_USB_DEV 
> +#error define SYSFS_USB_DEV for your platform 
> +#endif 
> + 
>  #ifndef SYSFS_PCIBACK_DRIVER 
>  #error define SYSFS_PCIBACK_DRIVER for your platform 
>  #endif 
> diff --git a/tools/libxl/libxl_pvusb.c b/tools/libxl/libxl_pvusb.c 
> new file mode 100644 
> index 0000000..d8cacd4 
> --- /dev/null 
> +++ b/tools/libxl/libxl_pvusb.c 
> @@ -0,0 +1,1305 @@ 
> +/* 
> + * 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. with the special 
> + * exception on linking described in file LICENSE. 
> + * 
> + * 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_osdeps.h" /* must come before any other headers */ 
> + 
> +#include "libxl_internal.h" 
> + 
> +#define USBBACK_INFO_PATH "/libxl/usbback" 
> + 
> +#define USBHUB_CLASS_CODE 0x09 
> + 
> +/* Utility to read backend xenstore keys */ 
> +#define READ_BACKEND(tgc, subpath)                                    \ 
> +            libxl__xs_read(tgc, XBT_NULL, GCSPRINTF("%s/" subpath,  
> be_path)) 
> + 
> +/* Utility to read frontend xenstore keys */ 
> +#define READ_FRONTEND(tgc, subpath)                                   \ 
> +            libxl__xs_read(tgc, XBT_NULL, GCSPRINTF("%s/" subpath,  
> fe_path)) 
> + 
> +static int libxl__device_usbctrl_setdefault(libxl__gc *gc, uint32_t domid, 
> +                                            libxl_device_usbctrl *usbctrl) 
> +{ 
> +    int rc; 
> + 
> +    if (!usbctrl->version) 
> +        usbctrl->version = 2; 
> + 
> +    if (!usbctrl->ports) 
> +        usbctrl->ports = 8; 
> + 
> +    if (usbctrl->protocol == LIBXL_USB_PROTOCOL_AUTO) 
> +        usbctrl->protocol = LIBXL_USB_PROTOCOL_PV; 
> + 
> +    rc = libxl__resolve_domid(gc, usbctrl->backend_domname, 
> +                              &usbctrl->backend_domid); 
> +    return rc; 
> +} 
> + 
> +int libxl__device_from_usbctrl(libxl__gc *gc, uint32_t domid, 
> +                               libxl_device_usbctrl *usbctrl, 
> +                               libxl__device *device) 
> +{ 
> +    device->backend_devid   = usbctrl->devid; 
> +    device->backend_domid   = usbctrl->backend_domid; 
> +    device->backend_kind    = LIBXL__DEVICE_KIND_VUSB; 
> +    device->devid           = usbctrl->devid; 
> +    device->domid           = domid; 
> +    device->kind            = LIBXL__DEVICE_KIND_VUSB; 
> + 
> +    return 0; 
> +} 
> + 
> +/* Add usbctrl information to xenstore. 
> + * 
> + * Adding a usb controller will add a new 'vusb' device in xenstore, and 
> + * add corresponding frontend, backend information to it. According to 
> + * "update_json", decide wether to update json config file. 
> + */ 
> +static int libxl__device_usbctrl_add_xenstore(libxl__gc *gc, uint32_t  
> domid, 
> +                                              libxl_device_usbctrl  
> *usbctrl, 
> +                                              bool update_json) 
> +{ 
> +    libxl__device *device; 
> +    flexarray_t *front; 
> +    flexarray_t *back; 
> +    xs_transaction_t t = XBT_NULL; 
> +    int i, rc; 
> +    libxl_domain_config d_config; 
> +    libxl_device_usbctrl usbctrl_saved; 
> +    libxl__domain_userdata_lock *lock = NULL; 
> + 
> +    libxl_domain_config_init(&d_config); 
> +    libxl_device_usbctrl_init(&usbctrl_saved); 
> +    libxl_device_usbctrl_copy(CTX, &usbctrl_saved, usbctrl); 
> + 
> +    GCNEW(device); 
> +    rc = libxl__device_from_usbctrl(gc, domid, usbctrl, device); 
> +    if (rc) goto out; 
> + 
> +    front = flexarray_make(gc, 4, 1); 
> +    back = flexarray_make(gc, 12, 1); 
> + 
> +    flexarray_append_pair(back, "frontend-id", GCSPRINTF("%d", domid)); 
> +    flexarray_append_pair(back, "online", "1"); 
> +    flexarray_append_pair(back, "state", "1"); 
> +    flexarray_append_pair(back, "type", 
> +                    (char 
> *)libxl_usb_protocol_to_string(usbctrl->protocol)); 
> +    flexarray_append_pair(back, "usb-ver", GCSPRINTF("%d", 
> usbctrl->version)); 
> +    flexarray_append_pair(back, "num-ports", GCSPRINTF("%d", 
> usbctrl->ports)); 
> +    flexarray_append_pair(back, "port", ""); 
> +    for (i = 0; i < usbctrl->ports; i++) 
> +        flexarray_append_pair(back, GCSPRINTF("port/%d", i + 1), ""); 
> + 
> +    flexarray_append_pair(front, "backend-id", 
> +                          GCSPRINTF("%d", usbctrl->backend_domid)); 
> +    flexarray_append_pair(front, "state", "1"); 
> + 
> +    if (update_json) { 
> +        lock = libxl__lock_domain_userdata(gc, domid); 
> +        if (!lock) { 
> +            rc = ERROR_LOCK_FAIL; 
> +            goto out; 
> +        } 
> + 
> +        rc = libxl__get_domain_configuration(gc, domid, &d_config); 
> +        if (rc) goto out; 
> + 
> +        DEVICE_ADD(usbctrl, usbctrls, domid, &usbctrl_saved, 
> +                   COMPARE_USBCTRL, &d_config); 
> +    } 
> + 
> +    for (;;) { 
> +        rc = libxl__xs_transaction_start(gc, &t); 
> +        if (rc) goto out; 
> + 
> +        rc = libxl__device_exists(gc, t, device); 
> +        if (rc < 0) goto out; 
> +        if (rc == 1) { 
> +            /* already exists in xenstore */ 
> +            LOG(ERROR, "device already exists in xenstore"); 
> +            rc = ERROR_DEVICE_EXISTS; 
> +            goto out; 
> +        } 
> + 
> +        if (update_json) { 
> +            rc = libxl__set_domain_configuration(gc, domid, &d_config); 
> +            if (rc) goto out; 
> +        } 
> + 
> +        libxl__device_generic_add(gc, t, device, 
> +                          libxl__xs_kvs_of_flexarray(gc, back, back->count), 
> +                          libxl__xs_kvs_of_flexarray(gc, front,  
> front->count), 
> +                          NULL); 
> + 
> +        rc = libxl__xs_transaction_commit(gc, &t); 
> +        if (!rc) break; 
> +        if (rc < 0) goto out; 
> +    } 
> + 
> +out: 
> +    libxl__xs_transaction_abort(gc, &t); 
> +    if (lock) libxl__unlock_domain_userdata(lock); 
> +    libxl_device_usbctrl_dispose(&usbctrl_saved); 
> +    libxl_domain_config_dispose(&d_config); 
> +    return rc; 
> +} 
> + 
> +/* AO operation to add a usb controller. 
> + * 
> + * Generally, it does: 
> + * 1) fill in necessary usb controler information with default value 
> + * 2) write usb controller frontend/backend info to xenstore, update json 
> + *    config file if necessary. 
> + * 3) wait for device connection. PVUSB frontend and backend driver will 
> + *    probe xenstore paths and build connection between frontend and  
> backend. 
> + */ 
> +void libxl__device_usbctrl_add(libxl__egc *egc, uint32_t domid, 
> +                               libxl_device_usbctrl *usbctrl, 
> +                               libxl__ao_device *aodev) 
> +{ 
> +    STATE_AO_GC(aodev->ao); 
> +    libxl__device *device; 
> +    int rc; 
> + 
> +    rc = libxl__device_usbctrl_setdefault(gc, domid, usbctrl); 
> +    if (rc < 0) goto out; 
> + 
> +    if (usbctrl->devid == -1) { 
> +        usbctrl->devid = libxl__device_nextid(gc, domid, "vusb"); 
> +        if (usbctrl->devid < 0) { 
> +            rc = ERROR_FAIL; 
> +            goto out; 
> +        } 
> +    } 
> + 
> +    rc = libxl__device_usbctrl_add_xenstore(gc, domid, usbctrl, 
> +                                            aodev->update_json); 
> +    if (rc) goto out; 
> + 
> +    GCNEW(device); 
> +    rc = libxl__device_from_usbctrl(gc, domid, usbctrl, device); 
> +    if (rc) goto out; 
> + 
> +    aodev->dev = device; 
> +    aodev->action = LIBXL__DEVICE_ACTION_ADD; 
> +    libxl__wait_device_connection(egc, aodev); 
> + 
> +    rc = 0; 
> + 
> +out: 
> +    aodev->rc = rc; 
> +    if (rc) aodev->callback(egc, aodev); 
> +    return; 
> +} 
> + 
> +static int 
> +libxl__device_usb_list_per_usbctrl(libxl__gc *gc, uint32_t domid, 
> +                                   libxl_devid usbctrl, 
> +                                   libxl_device_usb **usbs, int *num); 
> + 
> +static int 
> +libxl__device_usb_remove(libxl__gc *gc, uint32_t domid, libxl_device_usb  
> *usb); 
> + 
> +int libxl_devid_to_device_usbctrl(libxl_ctx *ctx, 
> +                                  uint32_t domid, 
> +                                  int devid, 
> +                                  libxl_device_usbctrl *usbctrl) 
> +{ 
> +    GC_INIT(ctx); 
> +    libxl_device_usbctrl *usbctrls; 
> +    int nb = 0; 
> +    int i, rc = -1; 
> + 
> +    usbctrls = libxl_device_usbctrl_list(ctx, domid, &nb); 
> +    if (!nb) goto out; 
> + 
> +    libxl_device_usbctrl_init(usbctrl); 
> +    for (i = 0; i < nb; i++) { 
> +        if (devid == usbctrls[i].devid) { 
> +            *usbctrl = usbctrls[i]; 
> +            rc = 0; 
> +            break; 
> +        } 
> +    } 
> + 
> +    libxl_device_usbctrl_list_free(usbctrls, nb); 
> + 
> +out: 
> +    GC_FREE; 
> +    return rc; 
> +} 
> + 
> +/* AO function to remove a usb controller. 
> + * 
> + * Generally, it does: 
> + * 1) check if the usb controller exists or not 
> + * 2) remove all usb devices under controller 
> + * 3) remove usb controller information from xenstore 
> + */ 
> +void libxl__initiate_device_usbctrl_remove(libxl__egc *egc, 
> +                                           libxl__ao_device *aodev) 
> +{ 
> +    STATE_AO_GC(aodev->ao); 
> +    libxl_device_usb *usbs = NULL; 
> +    int numusb = 0; 
> +    int i, rc; 
> +    uint32_t domid = ao->domid; 
> +    int usbctrl_devid = aodev->dev->devid; 
> + 
> +    /* Remove usb devices first */ 
> +    rc  = libxl__device_usb_list_per_usbctrl(gc, domid, usbctrl_devid, 
> +                                             &usbs, &numusb); 
> +    if (rc) goto out; 
> + 
> +    for (i = 0; i < numusb; i++) { 
> +        if (libxl__device_usb_remove(gc, domid, &usbs[i])) { 
> +            LOG(ERROR, "libxl__device_usb_remove failed"); 
> +            rc = ERROR_FAIL; 
> +            goto out; 
> +        } 
> +    } 
> + 
> +    libxl_device_usb_list_free(usbs, numusb); 
> + 
> +    /* Remove usbctrl */ 
> +    return libxl__initiate_device_remove(egc, aodev); 
> + 
> +out: 
> +    libxl_device_usb_list_free(usbs, numusb); 
> +    aodev->rc = rc; 
> +    if (rc) aodev->callback(egc, aodev); 
> +    return; 
> +} 
> + 
> +libxl_device_usbctrl * 
> +libxl_device_usbctrl_list(libxl_ctx *ctx, uint32_t domid, int *num) 
> +{ 
> +    GC_INIT(ctx); 
> +    libxl_device_usbctrl *usbctrls = NULL; 
> +    char *path = NULL; 
> +    char **dir = NULL; 
> +    unsigned int ndirs = 0; 
> + 
> +    *num = 0; 
> + 
> +    path = GCSPRINTF("%s/device/vusb", 
> +                     libxl__xs_get_dompath(gc, domid)); 
> +    dir = libxl__xs_directory(gc, XBT_NULL, path, &ndirs); 
> + 
> +    if (dir && ndirs) { 
> +        usbctrls = libxl__zalloc(NOGC, sizeof(*usbctrls) * ndirs); 
> +        libxl_device_usbctrl* usbctrl; 
> +        libxl_device_usbctrl* end = usbctrls + ndirs; 
> +        for (usbctrl = usbctrls; usbctrl < end; usbctrl++, dir++, (*num)++)  
> { 
> +            const char *tmp, *be_path; 
> +            const char *fe_path = GCSPRINTF("%s/%s", path, *dir); 
> + 
> +            libxl_device_usbctrl_init(usbctrl); 
> +            usbctrl->devid = atoi(*dir); 
> + 
> +            be_path = READ_FRONTEND(gc, "backend"); 
> +            if (!be_path) goto outerr; 
> + 
> +            tmp = READ_FRONTEND(gc, "backend-id"); 
> +            if (!tmp) goto outerr; 
> +            usbctrl->backend_domid = atoi(tmp); 
> + 
> +            tmp = READ_BACKEND(gc, "usb-ver"); 
> +            if (!tmp) goto outerr; 
> +            usbctrl->version = atoi(tmp); 
> + 
> +            tmp = READ_BACKEND(gc, "num-ports"); 
> +            if (!tmp) goto outerr; 
> +            usbctrl->ports = atoi(tmp); 
> + 
> +            tmp = READ_BACKEND(gc, "type"); 
> +            if (!tmp) goto outerr; 
> +            libxl_usb_protocol_from_string(tmp, &usbctrl->protocol); 
> +        } 
> +    } 
> + 
> +    goto out; 
> + 
> +outerr: 
> +    LOG(ERROR, "Unable to list USB Controllers"); 
> +    libxl_device_usbctrl_list_free(usbctrls, *num); 
> +    *num = 0; 
> +    usbctrls = NULL; 
> + 
> +out: 
> +    GC_FREE; 
> +    return usbctrls; 
> +} 
> + 
> +int libxl_device_usbctrl_getinfo(libxl_ctx *ctx, uint32_t domid, 
> +                                libxl_device_usbctrl *usbctrl, 
> +                                libxl_usbctrlinfo *usbctrlinfo) 
> +{ 
> +    GC_INIT(ctx); 
> +    char *dompath; 
> +    const char *fe_path, *be_path, *tmp; 
> +    int rc = 0; 
> + 
> +    usbctrlinfo->devid = usbctrl->devid; 
> + 
> +    dompath = libxl__xs_get_dompath(gc, domid); 
> +    fe_path = GCSPRINTF("%s/device/vusb/%d", dompath, usbctrlinfo->devid); 
> +    be_path = READ_FRONTEND(gc, "backend"); 
> +    if (!be_path) { 
> +        rc = ERROR_FAIL; 
> +        goto out; 
> +    } 
> +    usbctrlinfo->backend = libxl__strdup(NOGC, be_path); 
> + 
> +    tmp = READ_FRONTEND(gc, "backend-id"); 
> +    usbctrlinfo->backend_id = tmp ? strtoul(tmp, NULL, 10) : -1; 
> + 
> +    tmp = READ_FRONTEND(gc, "state"); 
> +    usbctrlinfo->state = tmp ? strtoul(tmp, NULL, 10) : -1; 
> + 
> +    tmp = READ_FRONTEND(gc, "event-channel"); 
> +    usbctrlinfo->evtch = tmp ? strtoul(tmp, NULL, 10) : -1; 
> + 
> +    tmp = READ_FRONTEND(gc, "urb-ring-ref"); 
> +    usbctrlinfo->ref_urb = tmp ? strtoul(tmp, NULL, 10) : -1; 
> + 
> +    tmp = READ_FRONTEND(gc, "conn-ring-ref"); 
> +    usbctrlinfo->ref_conn= tmp ? strtoul(tmp, NULL, 10) : -1; 
> + 
> +    tmp = READ_BACKEND(gc, "frontend"); 
> +    usbctrlinfo->frontend = libxl__strdup(NOGC, tmp); 
> + 
> +    tmp = READ_BACKEND(gc, "frontend-id"); 
> +    usbctrlinfo->frontend_id = tmp ? strtoul(tmp, NULL, 10) : -1; 
> + 
> +    tmp = READ_BACKEND(gc, "num-ports"); 
> +    usbctrlinfo->ports = tmp ? strtoul(tmp, NULL, 10) : -1; 
> + 
> +    tmp = READ_BACKEND(gc, "usb-ver"); 
> +    usbctrlinfo->version = tmp ? strtoul(tmp, NULL, 10) : -1; 
> + 
> +    tmp = READ_BACKEND(gc, "type"); 
> +    libxl_usb_protocol_from_string(tmp, &usbctrlinfo->protocol); 
> + 
> +out: 
> +    GC_FREE; 
> +    return rc; 
> +} 
> + 
> +static char *usb_busaddr_to_busid(libxl__gc *gc, int bus, int addr) 
> +{ 
> +    libxl_ctx *ctx = CTX; 
> +    struct dirent *de; 
> +    DIR *dir; 
> +    char *busid = NULL; 
> + 
> +    assert(bus > 0 && addr > 0); 
> + 
> +    if (!(dir = opendir(SYSFS_USB_DEV))) 
> +        return NULL; 
> + 
> +    while((de = readdir(dir))) { 
> +        char *filename; 
> +        void *buf; 
> +        int busnum = -1; 
> +        int devnum = -1; 
> + 
> +        if (!de->d_name) 
> +            continue; 
> + 
> +        filename = GCSPRINTF(SYSFS_USB_DEV"/%s/devnum", de->d_name); 
> +        if (!libxl_read_sysfs_file_contents(ctx, filename, &buf, NULL)) 
> +            sscanf(buf, "%x", &devnum); 
> + 
> +        filename = GCSPRINTF(SYSFS_USB_DEV"/%s/busnum", de->d_name); 
> +        if (!libxl_read_sysfs_file_contents(ctx, filename, &buf, NULL)) 
> +            sscanf(buf, "%x", &busnum); 
> + 
> +        if (bus == busnum && addr == devnum) { 
> +            busid = libxl__strdup(NOGC, de->d_name); 
> +            break; 
> +        } 
> +    } 
> + 
> +    closedir(dir); 
> +    return busid; 
> +} 
> + 
> +static void usb_busaddr_from_busid(libxl__gc *gc, char *busid, 
> +                                   int *bus, int *addr) 
> +{ 
> +    libxl_ctx *ctx = CTX; 
> +    char *filename; 
> +    void *buf; 
> + 
> +    assert(busid); 
> + 
> +    filename = GCSPRINTF(SYSFS_USB_DEV"/%s/busnum", busid); 
> +    if (!libxl_read_sysfs_file_contents(ctx, filename, &buf, NULL)) 
> +        sscanf(buf, "%x", bus); 
> + 
> +    filename = GCSPRINTF(SYSFS_USB_DEV"/%s/devnum", busid); 
> +    if (!libxl_read_sysfs_file_contents(ctx, filename, &buf, NULL)) 
> +        sscanf(buf, "%x", addr); 
> +} 
> + 
> +static int 
> +libxl__device_usb_assigned_list(libxl__gc *gc, 
> +                                libxl_device_usb **list, int *num) 
> +{ 
> +    char **domlist; 
> +    unsigned int nd = 0, i, j; 
> +    char *be_path; 
> +    libxl_device_usb *usb; 
> + 
> +    *list = NULL; 
> +    *num = 0; 
> + 
> +    domlist = libxl__xs_directory(gc, XBT_NULL, "/local/domain", &nd); 
> +    be_path = GCSPRINTF("/local/domain/%d/backend/vusb",  
> LIBXL_TOOLSTACK_DOMID); 
> +    for (i = 0; i < nd; i++) { 
> +        char *path, *num_ports, **ctrl_list; 
> +        unsigned int nc = 0; 
> +        path = GCSPRINTF("%s/%s", be_path, domlist[i]); 
> +        ctrl_list = libxl__xs_directory(gc, XBT_NULL, path , &nc); 
> + 
> +        for (j = 0; j < nc; j++) { 
> +            path = GCSPRINTF("%s/%s/%s/num-ports", be_path, 
> +                             domlist[i], ctrl_list[j]); 
> +            num_ports = libxl__xs_read(gc, XBT_NULL, path); 
> +            if (num_ports) { 
> +                int nport = atoi(num_ports), k; 
> +                char *devpath, *busid; 
> + 
> +                for (k = 0; k < nport; k++) { 
> +                    devpath = GCSPRINTF("%s/%s/%s/port/%u", be_path, 
> +                                        domlist[i], ctrl_list[j], k + 1); 
> +                    busid = libxl__xs_read(gc, XBT_NULL, devpath); 
> +                    /* If there are USB device attached, add it to list */ 
> +                    if (busid && strcmp(busid, "")) { 
> +                        GCREALLOC_ARRAY(*list, *num + 1); 
> +                        usb = *list + *num; 
> +                        usb->ctrl = atoi(ctrl_list[j]); 
> +                        usb->port = k + 1; 
> +                        usb_busaddr_from_busid(gc, busid, 
> +                                               &usb->hostbus, 
> &usb->hostaddr); 
> +                        (*num)++; 
> +                    } 
> +                } 
> +            } 
> +        } 
> +    } 
> + 
> +    return 0; 
> +} 
> + 
> +static bool is_usb_in_array(libxl_device_usb *usbs, int num, 
> +                            libxl_device_usb *usb) 
> +{ 
> +    int i; 
> + 
> +    for (i = 0; i < num; i++) { 
> +        if (COMPARE_USB(&usbs[i], usb)) 
> +            return true; 
> +    } 
> + 
> +    return false; 
> +} 
> + 
> +/* check if USB device is already assigned to a domain */ 
> +static bool is_usb_assigned(libxl__gc *gc, libxl_device_usb *usb) 
> +{ 
> +    libxl_device_usb *usbs; 
> +    int rc, num; 
> + 
> +    rc = libxl__device_usb_assigned_list(gc, &usbs, &num); 
> +    if (rc) { 
> +        LOG(ERROR, "Fail to get assigned usb list"); 
> +        return true; 
> +    } 
> + 
> +    return is_usb_in_array(usbs, num, usb); 
> +} 
> + 
> +/* check if USB device type is assignable */ 
> +static bool is_usb_assignable(libxl__gc *gc, libxl_device_usb *usb) 
> +{ 
> +    libxl_ctx *ctx = CTX; 
> +    int classcode; 
> +    char *filename; 
> +    void *buf = NULL; 
> +    char *busid = NULL; 
> + 
> +    assert(usb->hostbus > 0 && usb->hostaddr > 0); 
> +    busid = usb_busaddr_to_busid(gc, usb->hostbus, usb->hostaddr); 
> + 
> +    filename = GCSPRINTF(SYSFS_USB_DEV"/%s/bDeviceClass", busid); 
> +    if (libxl_read_sysfs_file_contents(ctx, filename, &buf, NULL)) 
> +        return false; 
> + 
> +    sscanf(buf, "%x", &classcode); 
> +    return classcode != USBHUB_CLASS_CODE; 
> +} 
> + 
> +/* get usb devices under certain usb controller */ 
> +static int 
> +libxl__device_usb_list_per_usbctrl(libxl__gc *gc, uint32_t domid, 
> +                                   libxl_devid usbctrl, 
> +                                   libxl_device_usb **usbs, int *num) 
> +{ 
> +    char *be_path, *num_devs; 
> +    int n, i; 
> + 
> +    *usbs = NULL; 
> +    *num = 0; 
> + 
> +    be_path = GCSPRINTF("%s/backend/vusb/%d/%d", 
> +                        libxl__xs_get_dompath(gc, LIBXL_TOOLSTACK_DOMID), 
> +                        domid, usbctrl); 
> +    num_devs = libxl__xs_read(gc, XBT_NULL, 
> +                              GCSPRINTF("%s/num-ports", be_path)); 
> +    if (!num_devs) 
> +        return 0; 
> + 
> +    n = atoi(num_devs); 
> +    *usbs = libxl__calloc(NOGC, n, sizeof(libxl_device_usb)); 
> + 
> +    for (i = 0; i < n; i++) { 
> +        char *busid; 
> +        libxl_device_usb *usb = NULL; 
> + 
> +        busid = libxl__xs_read(gc, XBT_NULL, 
> +                               GCSPRINTF("%s/port/%d", be_path, i + 1)); 
> +        if (busid && strcmp(busid, "")) { 
> +            usb = *usbs + *num; 
> +            usb->ctrl = usbctrl; 
> +            usb->port = i + 1; 
> +            usb_busaddr_from_busid(gc, busid, 
> +                                   &usb->hostbus, &usb->hostaddr); 
> +            (*num)++; 
> +        } 
> +    } 
> + 
> +    return 0; 
> +} 
> + 
> +/* get all usb devices of the domain */ 
> +static libxl_device_usb * 
> +libxl__device_usb_list(libxl__gc *gc, uint32_t domid, int *num) 
> +{ 
> +    char **usbctrls; 
> +    unsigned int nd = 0, i, j; 
> +    char *be_path; 
> +    int rc; 
> +    libxl_device_usb *usbs = NULL; 
> + 
> +    *num = 0; 
> + 
> +    be_path = GCSPRINTF("/local/domain/%d/backend/vusb/%d", 
> +                        LIBXL_TOOLSTACK_DOMID, domid); 
> +    usbctrls = libxl__xs_directory(gc, XBT_NULL, be_path, &nd); 
> + 
> +    for (i = 0; i < nd; i++) { 
> +        int nc = 0; 
> +        libxl_device_usb *tmp = NULL; 
> +        rc = libxl__device_usb_list_per_usbctrl(gc, domid, 
> +                                                atoi(usbctrls[i]), &tmp,  
> &nc); 
> +        if (!nc) continue; 
> + 
> +        usbs = libxl__realloc(NOGC, usbs, sizeof(*usbs) * (*num + nc)); 
> +        for (j = 0; j < nc; j++) { 
> +            usbs[*num] = tmp[j]; 
> +            (*num)++; 
> +        } 
> +        libxl_device_usb_list_free(tmp, nc); 
> +    } 
> +    return usbs; 
> +} 
> + 
> +libxl_device_usb * 
> +libxl_device_usb_list(libxl_ctx *ctx, uint32_t domid, int *num) 
> +{ 
> +    GC_INIT(ctx); 
> +    libxl_device_usb *usbs = NULL; 
> + 
> +    usbs = libxl__device_usb_list(gc, domid, num); 
> + 
> +    GC_FREE; 
> +    return usbs; 
> +} 
> + 
> +libxl_device_usb * 
> +libxl_device_usb_list_per_usbctrl(libxl_ctx *ctx, uint32_t domid, 
> +                                  libxl_devid usbctrl, int *num) 
> +{ 
> +    GC_INIT(ctx); 
> +    libxl_device_usb *usbs = NULL; 
> + 
> +    libxl__device_usb_list_per_usbctrl(gc, domid, usbctrl, &usbs, num); 
> + 
> +    GC_FREE; 
> +    return usbs; 
> +} 
> + 
> +/* find first unused controller:port and give that to usb device */ 
> +static int 
> +libxl__device_usb_set_default_usbctrl(libxl__gc *gc, uint32_t domid, 
> +                                      libxl_device_usb *usb) 
> +{ 
> +    libxl_ctx *ctx = CTX; 
> +    libxl_device_usbctrl *usbctrls = NULL; 
> +    int numctrl = 0; 
> +    int i, j, rc = -1; 
> +    char *be_path, *tmp; 
> + 
> +    usbctrls = libxl_device_usbctrl_list(ctx, domid, &numctrl); 
> +    if (!numctrl) 
> +        goto out; 
> + 
> +    for (i = 0; i < numctrl; i++) { 
> +        for (j = 0; j < usbctrls[i].ports; j++) { 
> +            be_path = GCSPRINTF("%s/backend/vusb/%d/%d/port/%d", 
> +                                libxl__xs_get_dompath(gc,  
> LIBXL_TOOLSTACK_DOMID), 
> +                                domid, usbctrls[i].devid, j + 1); 
> +            tmp = libxl__xs_read(gc, XBT_NULL, be_path); 
> +            if (tmp && !strcmp(tmp, "")) { 
> +                usb->ctrl = usbctrls[i].devid; 
> +                usb->port = j + 1; 
> +                rc = 0; 
> +                break; 
> +            } 
> +        } 
> +    } 
> + 
> +out: 
> +    libxl_device_usbctrl_list_free(usbctrls, numctrl); 
> +    return rc; 
> +} 
> + 
> +/* Fill in usb information with default value. 
> + * 
> + * Generally, it does: 
> + * 1) if "controller" is not specified: 
> + *    - if "port" is not specified, try to find an available  
> controller:port, 
> + *      if found, use that; otherwise, create a new controller, use this 
> + *      controller and its first port 
> + *    - if "port" is specified, report error. 
> + * 2) if "controller" is specified, but port is not specified: 
> + *    try to find an available port under this controller, if found, use 
> + *    that, otherwise, report error. 
> + * 3) if both "controller" and "port" are specified: 
> + *    check the controller:port is available, if not, report error. 
> + */ 
> +static int libxl__device_usb_setdefault(libxl__gc *gc, uint32_t domid, 
> +                                        libxl_device_usb *usb, 
> +                                        bool update_json) 
> +{ 
> +    int rc = -1; 
> +    char *be_path, *tmp; 
> + 
> +    if (usb->ctrl == -1) { 
> +        if (usb->port) { 
> +            LOG(ERROR, "USB controller must be specified if you specify  
> port ID"); 
> +            return ERROR_INVAL; 
> +        } 
> + 
> +        rc = libxl__device_usb_set_default_usbctrl(gc, domid, usb); 
> +        /* If no existing controller to host this usb device, add a new one  
> */ 
> +        if (rc) { 
> +            libxl_device_usbctrl *usbctrl; 
> +            GCNEW(usbctrl); 
> +            libxl_device_usbctrl_init(usbctrl); 
> + 
> +            rc = libxl__device_usbctrl_setdefault(gc, domid, usbctrl); 
> +            if (rc < 0) goto out; 
> + 
> +            if (usbctrl->devid == -1) { 
> +                usbctrl->devid = libxl__device_nextid(gc, domid, "vusb"); 
> +                if (usbctrl->devid < 0) { 
> +                    goto out; 
> +                } 
> +            } 
> + 
> +            rc = libxl__device_usbctrl_add_xenstore(gc, domid, usbctrl, 
> +                                                    update_json); 
> +            if (rc) goto out; 
> + 
> +            usb->ctrl = usbctrl->devid; 
> +            usb->port = 1; 
> +        } 
> +    } else if (!usb->port) { 
> +        /* Valid port starts from 1. Choose port for us. */ 
> +        int i, ports; 
> + 
> +        be_path = GCSPRINTF("%s/backend/vusb/%d/%d", 
> +                            libxl__xs_get_dompath(gc,  
> LIBXL_TOOLSTACK_DOMID), 
> +                            domid, usb->ctrl); 
> +        tmp = READ_BACKEND(gc, "num-ports"); 
> +        ports = tmp ? atoi(tmp) : 0; 
> +        for (i = 0; i < ports; i++) { 
> +            tmp = libxl__xs_read(gc, XBT_NULL, 
> +                                 GCSPRINTF("%s/port/%d", be_path, i + 1)); 
> +            if (tmp && !strcmp(tmp, "")) { 
> +                usb->port = i + 1; 
> +                break; 
> +            } 
> +        } 
> + 
> +        if (!usb->port) { 
> +            LOG(ERROR, "No available port under specified controller"); 
> +            goto out; 
> +        } 
> +    } else { 
> +        be_path = GCSPRINTF("%s/backend/vusb/%d/%d/port/%d", 
> +                            libxl__xs_get_dompath(gc,  
> LIBXL_TOOLSTACK_DOMID), 
> +                            domid, usb->ctrl, usb->port); 
> +        tmp = libxl__xs_read(gc, XBT_NULL, be_path); 
> +        if (!tmp || strcmp(tmp, "")) { 
> +            LOG(ERROR, "The controller port isn't available"); 
> +            goto out; 
> +        } 
> +    } 
> + 
> +    rc = 0; 
> + 
> +out: 
> +    return rc; 
> +} 
> + 
> +/* Add usb information to xenstore 
> + * 
> + * Adding a usb device won't create new 'vusb' device, but only write 
> + * the device busid to the controller:port in xenstore. 
> + */ 
> +static int libxl__device_usb_add_xenstore(libxl__gc *gc, uint32_t domid, 
> +                                          libxl_device_usb *usb, 
> +                                          bool update_json) 
> +{ 
> +    char *be_path; 
> +    char *busid; 
> +    int rc; 
> +    xs_transaction_t t = XBT_NULL; 
> +    libxl_domain_config d_config; 
> +    libxl_device_usb usb_saved; 
> +    libxl__domain_userdata_lock *lock = NULL; 
> + 
> +    libxl_domain_config_init(&d_config); 
> +    libxl_device_usb_init(&usb_saved); 
> +    libxl_device_usb_copy(CTX, &usb_saved, usb); 
> + 
> +    busid = usb_busaddr_to_busid(gc, usb->hostbus, usb->hostaddr); 
> +    if (!busid) { 
> +        LOG(DEBUG, "Fail to get busid of usb device"); 
> +        goto out; 
> +    } 
> + 
> +    if (update_json) { 
> +        lock = libxl__lock_domain_userdata(gc, domid); 
> +        if (!lock) { 
> +            rc = ERROR_LOCK_FAIL; 
> +            goto out; 
> +        } 
> + 
> +        rc = libxl__get_domain_configuration(gc, domid, &d_config); 
> +        if (rc) goto out; 
> + 
> +        DEVICE_ADD(usb, usbs, domid, &usb_saved, COMPARE_USB, &d_config); 
> +    } 
> + 
> +    for (;;) { 
> +        rc = libxl__xs_transaction_start(gc, &t); 
> +        if (rc) goto out; 
> + 
> +        if (update_json) { 
> +            rc = libxl__set_domain_configuration(gc, domid, &d_config); 
> +            if (rc) goto out; 
> +        } 
> + 
> +        be_path = GCSPRINTF("%s/backend/vusb/%d/%d/port/%d", 
> +                            libxl__xs_get_dompath(gc,  
> LIBXL_TOOLSTACK_DOMID), 
> +                            domid, usb->ctrl, usb->port); 
> + 
> +        LOG(DEBUG, "Adding new usb device to xenstore"); 
> +        if (libxl__xs_write_checked(gc, t, be_path, busid)) 
> +            goto out; 
> + 
> +        rc = libxl__xs_transaction_commit(gc, &t); 
> +        if (!rc) break; 
> +        if (rc < 0) goto out; 
> +    } 
> + 
> +    rc = 0; 
> + 
> +out: 
> +    if (lock) libxl__unlock_domain_userdata(lock); 
> +    libxl_device_usb_dispose(&usb_saved); 
> +    libxl_domain_config_dispose(&d_config); 
> +    return rc; 
> +} 
> + 
> +static int libxl__device_usb_remove_xenstore(libxl__gc *gc, uint32_t domid, 
> +                                             libxl_device_usb *usb) 
> +{ 
> +    char *be_path; 
> + 
> +    be_path = GCSPRINTF("%s/backend/vusb/%d/%d/port/%d", 
> +                        libxl__xs_get_dompath(gc, LIBXL_TOOLSTACK_DOMID), 
> +                        domid, usb->ctrl, usb->port); 
> +    LOG(DEBUG, "Removing USB device from xenstore"); 
> +    if (libxl__xs_write_checked(gc,XBT_NULL, be_path, "")) 
> +        return ERROR_FAIL; 
> + 
> +    return 0; 
> +} 
> + 
> +/* bind/unbind usb device interface */ 
> +static int unbind_usb_intf(libxl__gc *gc, char *intf, char **drvpath) 
> +{ 
> +    char *path, *spath, *dp = NULL; 
> +    int fd = -1; 
> +    int rc = 0; 
> +    struct stat st; 
> + 
> +    spath = GCSPRINTF(SYSFS_USB_DEV"/%s/driver", intf); 
> +    if (!lstat(spath, &st)) { 
> +        /* Find the canonical path to the driver. */ 
> +        dp = libxl__zalloc(gc, PATH_MAX); 
> +        dp = realpath(spath, dp); 
> + 
> +        path = GCSPRINTF("%s/unbind", spath); 
> +        fd = open(path, O_WRONLY); 
> +        if (fd < 0) 
> +            return ERROR_FAIL; 
> +        rc = write(fd, intf, strlen(intf)); 
> +        close(fd); 
> +        if (rc < 0) 
> +            return ERROR_FAIL; 
> +    } 
> + 
> +    if (drvpath) 
> +        *drvpath = dp; 
> + 
> +    return 0; 
> +} 
> + 
> +static int bind_usb_intf(libxl__gc *gc, char *intf, char *drvpath) 
> +{ 
> +    char *path; 
> +    struct stat st; 
> +    int fd, rc = 0; 
> + 
> +    path = GCSPRINTF("%s/%s", drvpath, intf); 
> +    rc = lstat(path, &st); 
> +    /* already bind, return */ 
> +    if (rc == 0) 
> +        return 0; 
> + 
> +    path = GCSPRINTF("%s/bind", drvpath); 
> +    fd = open(path, O_WRONLY); 
> +    if (fd < 0) 
> +        return ERROR_FAIL; 
> + 
> +    rc = write(fd, intf, strlen(intf)); 
> +    close(fd); 
> +    if (rc < 0) 
> +        return ERROR_FAIL; 
> + 
> +    return 0; 
> +} 
> + 
> +/* Is usb interface bound to usbback? */ 
> +static int usb_intf_is_assigned(libxl__gc *gc, char *intf) 
> +{ 
> +    char *spath; 
> +    int rc; 
> +    struct stat st; 
> + 
> +    spath = GCSPRINTF(SYSFS_USBBACK_DRIVER"/%s", intf); 
> +    rc = lstat(spath, &st); 
> + 
> +    if (rc == 0) 
> +        return 1; 
> +    if (rc < 0 && errno == ENOENT) 
> +        return 0; 
> +    LOGE(ERROR, "Accessing %s", spath); 
> +    return -1; 
> +} 
> + 
> +static int usb_get_all_interfaces(libxl__gc *gc, libxl_device_usb *usb, 
> +                                  char ***intfs, int *num) 
> +{ 
> +    DIR *dir; 
> +    struct dirent *entry; 
> +    char *buf; 
> +    char *busid; 
> +    int rc = 0; 
> + 
> +    *intfs = NULL; 
> +    *num = 0; 
> + 
> +    busid = usb_busaddr_to_busid(gc, usb->hostbus, usb->hostaddr); 
> +    if (!busid) { 
> +        rc = ERROR_FAIL; 
> +        goto out; 
> +    } 
> + 
> +    buf = GCSPRINTF("%s:", busid); 
> + 
> +    if (!(dir = opendir(SYSFS_USB_DEV))) { 
> +        rc = ERROR_FAIL; 
> +        goto out; 
> +    } 
> + 
> +    while ((entry = readdir(dir)) != NULL) { 
> +        if (!strncmp(entry->d_name, buf, strlen(buf))) { 
> +            GCREALLOC_ARRAY(*intfs, *num + 1); 
> +            if (*intfs == NULL) { 
> +                rc = ERROR_FAIL; 
> +                goto out; 
> +            } 
> +            (*intfs)[*num] = libxl__strdup(gc, entry->d_name); 
> +            (*num)++; 
> +        } 
> +    } 
> + 
> +    closedir(dir); 
> + 
> +out: 
> +    return rc; 
> +} 
> + 
> +/* Encode usb interface so that it could be written to xenstore as a key. 
> + * 
> + * Since xenstore key cannot include '.' or ':', we'll change '.' to '_', 
> + * change ':' to '-'. For example, 3-1:2.1 will be encoded to 3-1-2_1. 
> + * This will be used to save original driver of USB device to xenstore. 
> + */ 
> +static char *usb_interface_xenstore_encode(char *busid) 
> +{ 
> +    char *str = strdup(busid); 
> +    int i, len = strlen(str); 
> + 
> +    for (i = 0; i < len; i++) { 
> +        if (str[i] == '.') 
> +            str[i] = '_'; 
> +         if (str[i] == ':') 
> +            str[i] = '-'; 
> +    } 
> +    return str; 
> +} 
> + 
> +/* Unbind USB device from "usbback" driver. 
> + * 
> + * If there are many interfaces under USB device, check each interface, 
> + * unbind from "usbback" driver and rebind to its original driver. 
> + */ 
> +static int usbback_dev_unassign(libxl__gc *gc, libxl_device_usb *usb) 
> +{ 
> +    char **intfs = NULL; 
> +    char *path; 
> +    int num = 0, i; 
> +    int rc = 0; 
> +    char *busid; 
> +    char *usb_encode = NULL; 
> + 
> +    if (usb_get_all_interfaces(gc, usb, &intfs, &num) < 0) 
> +        return ERROR_FAIL; 
> + 
> +    busid = usb_busaddr_to_busid(gc, usb->hostbus, usb->hostaddr); 
> +    usb_encode = usb_interface_xenstore_encode(busid); 
> + 
> +    for (i = 0; i < num; i++) { 
> +        char *intf = intfs[i]; 
> +        char *drvpath = NULL; 
> + 
> +        /* check if the USB interface is already bound to "usbbcak" */ 
> +        if (usb_intf_is_assigned(gc, intf) > 0) { 
> +            /* unbind interface from usbback driver */ 
> +            if (unbind_usb_intf(gc, intf, NULL) < 0) { 
> +                rc = ERROR_FAIL; 
> +                goto out; 
> +            } 
> +        } 
> + 
> +        /* bind interface to its originial driver */ 
> +        drvpath = libxl__xs_read(gc, XBT_NULL, 
> +                  GCSPRINTF(USBBACK_INFO_PATH"/%s/%s/driver_path", 
> +                  usb_encode, usb_interface_xenstore_encode(intf))); 
> +        if (drvpath && bind_usb_intf(gc, intf, drvpath)) 
> +            LOGE(WARN, "Couldn't bind %s to %s", intf, drvpath); 
> +    } 
> + 
> +    /* finally, remove xs driver path */ 
> +    path = GCSPRINTF(USBBACK_INFO_PATH"/%s", usb_encode); 
> +    libxl__xs_rm_checked(gc, XBT_NULL, path); 
> + 
> +out: 
> +    free(usb_encode); 
> +    return rc; 
> +} 
> + 
> +/* Bind USB device to "usbback" driver. 
> + * 
> + * If there are many interfaces under USB device, check each interface, 
> + * unbind from original driver and bind to "usbback" driver. 
> + */ 
> +static int usbback_dev_assign(libxl__gc *gc, libxl_device_usb *usb) 
> +{ 
> +    char **intfs = NULL; 
> +    int num = 0, i; 
> +    int rc = 0; 
> +    char *busid; 
> +    char *usb_encode = NULL; 
> + 
> +    if (usb_get_all_interfaces(gc, usb, &intfs, &num) < 0) 
> +        return ERROR_FAIL; 
> + 
> +    busid = usb_busaddr_to_busid(gc, usb->hostbus, usb->hostaddr); 
> +    usb_encode = usb_interface_xenstore_encode(busid); 
> + 
> +    for (i = 0; i < num; i++) { 
> +        char *intf = intfs[i]; 
> +        char *path = NULL; 
> +        char *drvpath = NULL; 
> + 
> +        /* already assigned to usbback */ 
> +        if (usb_intf_is_assigned(gc, intf) > 0) 
> +            continue; 
> + 
> +        /* unbind interface from original driver */ 
> +        if (unbind_usb_intf(gc, intf, &drvpath) < 0) { 
> +            rc = ERROR_FAIL; 
> +            goto out_rebind; 
> +        } 
> + 
> +        if (drvpath) { 
> +            /* write driver path to xenstore for later rebinding */ 
> +            path = GCSPRINTF(USBBACK_INFO_PATH"/%s/%s/driver_path", 
> +                             usb_encode,  
> usb_interface_xenstore_encode(intf)); 
> +            if (libxl__xs_write_checked(gc, XBT_NULL, path, drvpath) < 0) { 
> +                LOG(WARN, "Write of %s to node %s failed", drvpath, path); 
> +            } 
> +        } 
> + 
> +        /* bind interface to usbback */ 
> +        if (bind_usb_intf(gc, intf, SYSFS_USBBACK_DRIVER) < 0) { 
> +            LOGE(ERROR, "Couldn't bind %s to %s", intf,  
> SYSFS_USBBACK_DRIVER); 
> +            rc = ERROR_FAIL; 
> +            goto out_rebind; 
> +        } 
> +    } 
> + 
> +    goto out; 
> + 
> +out_rebind: 
> +    /* some interfaces might be bound to usbback, unbind it then and 
> +     * rebind to its original driver 
> +     */ 
> +    usbback_dev_unassign(gc, usb); 
> +out: 
> +    free(usb_encode); 
> +    return rc; 
> +} 
> + 
> +/* AO operation to add a usb device. 
> + * 
> + * Generally, it does: 
> + * 1) check if the usb device type is assignable 
> + * 2) check if the usb device is already assigned to a domain 
> + * 3) add 'busid' of the usb device to xenstore contoller/port/. 
> + *    (PVUSB driver watches the xenstore changes and will detect that.) 
> + * 4) unbind usb device from original driver and bind to usbback. 
> + *    If usb device has many interfaces, then: 
> + *    - unbind each interface from its original driver and bind to usbback. 
> + *    - store the original driver to xenstore for later rebinding when 
> + *      detaching the device. 
> + */ 
> +void libxl__device_usb_add(libxl__egc *egc, uint32_t domid, 
> +                           libxl_device_usb *usb, 
> +                           libxl__ao_device *aodev) 
> +{ 
> +    STATE_AO_GC(aodev->ao); 
> +    int rc = -1; 
> +    char *busid = NULL; 
> + 
> +    assert(usb->hostbus > 0 && usb->hostaddr > 0); 
> + 
> +    busid = usb_busaddr_to_busid(gc, usb->hostbus, usb->hostaddr); 
> +    if (!busid) { 
> +        LOG(ERROR, "USB device doesn't exist in sysfs"); 
> +        goto out; 
> +    } 
> + 
> +    if (!is_usb_assignable(gc, usb)) { 
> +        LOG(ERROR, "USB device is not assignable."); 
> +        goto out; 
> +    } 
> + 
> +    /* check usb device is already assigned */ 
> +    if (is_usb_assigned(gc, usb)) { 
> +        LOG(ERROR, "USB device is already attached to a domain."); 
> +        goto out; 
> +    } 
> + 
> +    rc = libxl__device_usb_setdefault(gc, domid, usb, aodev->update_json); 
> +    if (rc) goto out; 
> + 
> +    rc = libxl__device_usb_add_xenstore(gc, domid, usb, aodev->update_json); 
> +    if (rc) goto out; 
> + 
> +    rc = usbback_dev_assign(gc, usb); 
> +    if (rc) { 
> +        libxl__device_usb_remove_xenstore(gc, domid, usb); 
> +        goto out; 
> +    } 
> + 
> +    libxl__ao_complete(egc, ao, 0); 
> +    rc = 0; 
> + 
> +out: 
> +    aodev->rc = rc; 
> +    if (rc) aodev->callback(egc, aodev); 
> +    return; 
> +} 
> + 
> +int libxl_ctrlport_to_device_usb(libxl_ctx *ctx, 
> +                                 uint32_t domid, 
> +                                 int ctrl, 
> +                                 int port, 
> +                                 libxl_device_usb *usb) 
> +{ 
> +    GC_INIT(ctx); 
> +    char *dompath, *be_path, *busid; 
> +    int rc = ERROR_FAIL; 
> + 
> +    dompath = libxl__xs_get_dompath(gc, domid); 
> +    if (!dompath) 
> +        goto out; 
> + 
> +    be_path = libxl__xs_read(gc, XBT_NULL, 
> +                  GCSPRINTF("%s/device/vusb/%d/backend", dompath, ctrl)); 
> +    if (!be_path) 
> +        goto out; 
> + 
> +    busid = libxl__xs_read(gc, XBT_NULL, 
> +                           GCSPRINTF("%s/port/%d", be_path, port)); 
> +    if (busid && strcmp(busid, "")) { 
> +        usb->ctrl = ctrl; 
> +        usb->port = port; 
> +        usb_busaddr_from_busid(gc, busid, &usb->hostbus, &usb->hostaddr); 
> +        rc = 0; 
> +    } 
> + 
> +out: 
> +    GC_FREE; 
> +    return rc; 
> +} 
> + 
> +/* Operation to remove usb device. 
> + * 
> + * Generally, it does: 
> + * 1) check if the usb device is assigned to the domain 
> + * 2) remove the usb device from xenstore controller/port. 
> + * 3) unbind usb device from usbback and rebind to its original driver. 
> + *    If usb device has many interfaces, do it to each interface. 
> + */ 
> +static int libxl__device_usb_remove(libxl__gc *gc, uint32_t domid, 
> +                                    libxl_device_usb *usb) 
> +{ 
> +    if (libxl__device_usb_remove_xenstore(gc, domid, usb)) 
> +        return -1; 
> + 
> +    usbback_dev_unassign(gc, usb); 
> + 
> +    return 0; 
> +} 
> + 
> +int libxl_device_usb_remove(libxl_ctx *ctx, uint32_t domid, 
> +                            libxl_device_usb *usb, 
> +                            const libxl_asyncop_how *ao_how) 
> + 
> +{ 
> +    AO_CREATE(ctx, domid, ao_how); 
> +    int rc; 
> + 
> +    rc = libxl__device_usb_remove(gc, domid, usb); 
> + 
> +    libxl__ao_complete(egc, ao, rc); 
> +    return AO_INPROGRESS; 
> +} 
> + 
> +int libxl_device_usb_getinfo(libxl_ctx *ctx, libxl_device_usb *usb, 
> +                             libxl_usbinfo *usbinfo) 
> +{ 
> +    GC_INIT(ctx); 
> +    char *filename; 
> +    char *busid; 
> +    void *buf = NULL; 
> +    int buflen, rc; 
> + 
> +    usbinfo->ctrl = usb->ctrl; 
> +    usbinfo->port = usb->port; 
> + 
> +    busid = usb_busaddr_to_busid(gc, usb->hostbus, usb->hostaddr); 
> +    if (!busid) { 
> +        rc = ERROR_FAIL; 
> +        goto out; 
> +    } 
> + 
> +    filename = GCSPRINTF(SYSFS_USB_DEV"/%s/devnum", busid); 
> +    if (!libxl_read_sysfs_file_contents(ctx, filename, &buf, NULL)) 
> +        sscanf(buf, "%x", &usbinfo->devnum); 
> + 
> +    filename = GCSPRINTF(SYSFS_USB_DEV"/%s/busnum", busid); 
> +    if (!libxl_read_sysfs_file_contents(ctx, filename, &buf, NULL)) 
> +        sscanf(buf, "%x", &usbinfo->busnum); 
> + 
> +    filename = GCSPRINTF(SYSFS_USB_DEV"/%s/idVendor", busid); 
> +    if (!libxl_read_sysfs_file_contents(ctx, filename, &buf, NULL)) 
> +        sscanf(buf, "%x", &usbinfo->idVendor); 
> + 
> +    filename = GCSPRINTF(SYSFS_USB_DEV"/%s/idProduct", busid); 
> +    if (!libxl_read_sysfs_file_contents(ctx, filename, &buf, NULL)) 
> +        sscanf(buf, "%x", &usbinfo->idProduct); 
> + 
> +    filename = GCSPRINTF(SYSFS_USB_DEV"/%s/manufacturer", busid); 
> +    if (!libxl_read_sysfs_file_contents(ctx, filename, &buf, &buflen) && 
> +        buflen > 0) { 
> +        /* replace \n to \0 */ 
> +        if (((char *)buf)[buflen - 1] == '\n') 
> +            ((char *)buf)[buflen - 1] = '\0'; 
> +        usbinfo->manuf = libxl__strdup(NOGC, buf); 
> +   } 
> + 
> +    filename = GCSPRINTF(SYSFS_USB_DEV"/%s/product", busid); 
> +    if (!libxl_read_sysfs_file_contents(ctx, filename, &buf, &buflen) && 
> +        buflen > 0) { 
> +        /* replace \n to \0 */ 
> +        if (((char *)buf)[buflen - 1] == '\n') 
> +            ((char *)buf)[buflen - 1] = '\0'; 
> +        usbinfo->prod = libxl__strdup(NOGC, buf); 
> +    } 
> + 
> +    rc = 0; 
> + 
> +out: 
> +    GC_FREE; 
> +    return rc; 
> +} 
> +/* 
> + * Local variables: 
> + * mode: C 
> + * c-basic-offset: 4 
> + * indent-tabs-mode: nil 
> + * End: 
> + */ 
> diff --git a/tools/libxl/libxl_types.idl b/tools/libxl/libxl_types.idl 
> index 23f27d4..5278d03 100644 
> --- a/tools/libxl/libxl_types.idl 
> +++ b/tools/libxl/libxl_types.idl 
> @@ -541,6 +541,28 @@ libxl_device_pci = Struct("device_pci", [ 
>      ("seize", bool), 
>      ]) 
>   
> +libxl_usb_protocol = Enumeration("usb_protocol", [ 
> +    (0, "AUTO"), 
> +    (1, "PV"), 
> +    (2, "QEMU"), 
> +    ]) 
> + 
> +libxl_device_usbctrl = Struct("device_usbctrl", [ 
> +    ("protocol", libxl_usb_protocol), 
> +    ("devid", libxl_devid), 
> +    ("version", integer), 
> +    ("ports", integer), 
> +    ("backend_domid", libxl_domid), 
> +    ("backend_domname", string), 
> +   ]) 
> + 
> +libxl_device_usb = Struct("device_usb", [ 
> +    ("ctrl", libxl_devid), 
> +    ("port", integer), 
> +    ("hostbus",   integer), 
> +    ("hostaddr",  integer), 
> +    ]) 
> + 
>  libxl_device_dtdev = Struct("device_dtdev", [ 
>      ("path", string), 
>      ]) 
> @@ -572,6 +594,8 @@ libxl_domain_config = Struct("domain_config", [ 
>      ("nics", Array(libxl_device_nic, "num_nics")), 
>      ("pcidevs", Array(libxl_device_pci, "num_pcidevs")), 
>      ("dtdevs", Array(libxl_device_dtdev, "num_dtdevs")), 
> +    ("usbctrls", Array(libxl_device_usbctrl, "num_usbctrls")), 
> +    ("usbs", Array(libxl_device_usb, "num_usbs")), 
>      ("vfbs", Array(libxl_device_vfb, "num_vfbs")), 
>      ("vkbs", Array(libxl_device_vkb, "num_vkbs")), 
>      ("vtpms", Array(libxl_device_vtpm, "num_vtpms")), 
> @@ -620,6 +644,32 @@ libxl_vtpminfo = Struct("vtpminfo", [ 
>      ("uuid", libxl_uuid), 
>      ], dir=DIR_OUT) 
>   
> +libxl_usbctrlinfo = Struct("usbctrlinfo", [ 
> +    ("protocol", libxl_usb_protocol), 
> +    ("devid", libxl_devid), 
> +    ("version", integer), 
> +    ("ports", integer), 
> +    ("backend", string), 
> +    ("backend_id", uint32), 
> +    ("frontend", string), 
> +    ("frontend_id", uint32), 
> +    ("state", integer), 
> +    ("evtch", integer), 
> +    ("ref_urb", integer), 
> +    ("ref_conn", integer), 
> +    ], dir=DIR_OUT) 
> + 
> +libxl_usbinfo = Struct("usbinfo", [ 
> +    ("ctrl", libxl_devid), 
> +    ("port", integer), 
> +    ("busnum", integer), 
> +    ("devnum", integer), 
> +    ("idVendor", integer), 
> +    ("idProduct", integer), 
> +    ("prod", string), 
> +    ("manuf", string), 
> +    ], dir=DIR_OUT) 
> + 
>  libxl_vcpuinfo = Struct("vcpuinfo", [ 
>      ("vcpuid", uint32), 
>      ("cpu", uint32), 
> diff --git a/tools/libxl/libxl_types_internal.idl  
> b/tools/libxl/libxl_types_internal.idl 
> index 5e55685..696f5f8 100644 
> --- a/tools/libxl/libxl_types_internal.idl 
> +++ b/tools/libxl/libxl_types_internal.idl 
> @@ -22,6 +22,7 @@ libxl__device_kind = Enumeration("device_kind", [ 
>      (6, "VKBD"), 
>      (7, "CONSOLE"), 
>      (8, "VTPM"), 
> +    (9, "VUSB"), 
>      ]) 
>   
>  libxl__console_backend = Enumeration("console_backend", [ 
> diff --git a/tools/libxl/libxl_utils.c b/tools/libxl/libxl_utils.c 
> index dc5465e..a627662 100644 
> --- a/tools/libxl/libxl_utils.c 
> +++ b/tools/libxl/libxl_utils.c 
> @@ -1187,6 +1187,22 @@ int libxl__random_bytes(libxl__gc *gc, uint8_t *buf,  
> size_t len) 
>      return ret; 
>  } 
>   
> +void libxl_device_usbctrl_list_free(libxl_device_usbctrl* list, int nr) 
> +{ 
> +   int i; 
> +   for (i = 0; i < nr; i++) 
> +      libxl_device_usbctrl_dispose(&list[i]); 
> +   free(list); 
> +} 
> + 
> +void libxl_device_usb_list_free(libxl_device_usb* list, int nr) 
> +{ 
> +   int i; 
> +   for (i = 0; i < nr; i++) 
> +      libxl_device_usb_dispose(&list[i]); 
> +   free(list); 
> +} 
> + 
>  /* 
>   * Local variables: 
>   * mode: C 
> diff --git a/tools/libxl/libxl_utils.h b/tools/libxl/libxl_utils.h 
> index 1c1761d..321df9d 100644 
> --- a/tools/libxl/libxl_utils.h 
> +++ b/tools/libxl/libxl_utils.h 
> @@ -76,6 +76,11 @@ int libxl_uuid_to_device_vtpm(libxl_ctx *ctx, uint32_t  
> domid, 
>                                 libxl_uuid *uuid, libxl_device_vtpm *vtpm); 
>  int libxl_devid_to_device_vtpm(libxl_ctx *ctx, uint32_t domid, 
>                                 int devid, libxl_device_vtpm *vtpm); 
> +int libxl_devid_to_device_usbctrl(libxl_ctx *ctx, uint32_t domid, 
> +                                  int devid, libxl_device_usbctrl  
> *usbctrl); 
> +int libxl_ctrlport_to_device_usb(libxl_ctx *ctx, uint32_t domid, 
> +                                 int ctrl, int port, 
> +                                 libxl_device_usb *usb); 
>   
>  int libxl_bitmap_alloc(libxl_ctx *ctx, libxl_bitmap *bitmap, int n_bits); 
>      /* Allocated bimap is from malloc, libxl_bitmap_dispose() to be 
 


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to