openkylin
/
linux
mirror of https://gitee.com/openkylin/linux.git
9
0
Fork 0
Code Issues Projects Releases Wiki Activity

staging: usbip: userspace: migrate usbip_host_driver to libudev 

This patch modifies usbip_host_driver to use libudev.

Signed-off-by: Valentina Manea <valentina.manea.m@gmail.com>
Reviewed-by: Shuah Khan <shuah.kh@samsung.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
Valentina Manea 2014-03-08 14:53:26 +02:00 committed by Greg Kroah-Hartman
parent ecc13b7255
commit 021aed8453
6 changed files with 139 additions and 258 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

76
drivers/staging/usbip/userspace/libsrc/usbip_common.c
View File

@ -2,6 +2,7 @@
* Copyright (C) 2005-2007 Takahiro Hirofuchi * Copyright (C) 2005-2007 Takahiro Hirofuchi
*/ */
#include <libudev.h>
#include "usbip_common.h" #include "usbip_common.h"
#include "names.h" #include "names.h"
@ -12,6 +13,8 @@ int usbip_use_syslog;
int usbip_use_stderr; int usbip_use_stderr;
int usbip_use_debug; int usbip_use_debug;
extern struct udev *udev_context;
struct speed_string { struct speed_string {
int num; int num;
char *speed; char *speed;
@ -111,75 +114,48 @@ void dump_usb_device(struct usbip_usb_device *udev)
} }
int read_attr_value(struct sysfs_device *dev, const char *name, int read_attr_value(struct udev_device *dev, const char *name,
const char *format) const char *format)
{ {
char attrpath[SYSFS_PATH_MAX]; const char *attr;
struct sysfs_attribute *attr;
int num = 0; int num = 0;
int ret; int ret;
snprintf(attrpath, sizeof(attrpath), "%s/%s", dev->path, name); attr = udev_device_get_sysattr_value(dev, name);
attr = sysfs_open_attribute(attrpath);
if (!attr) { if (!attr) {
dbg("sysfs_open_attribute failed: %s", attrpath); err("udev_device_get_sysattr_value failed");
return 0;
}
ret = sysfs_read_attribute(attr);
if (ret < 0) {
dbg("sysfs_read_attribute failed");
goto err; goto err;
} }
ret = sscanf(attr->value, format, &num); ret = sscanf(attr, format, &num);
if (ret < 1) { if (ret < 1) {
dbg("sscanf failed"); err("sscanf failed");
goto err; goto err;
} }
err: err:
sysfs_close_attribute(attr);
return num; return num;
} }
int read_attr_speed(struct sysfs_device *dev) int read_attr_speed(struct udev_device *dev)
{ {
char attrpath[SYSFS_PATH_MAX]; const char *speed;
struct sysfs_attribute *attr;
char speed[100];
int ret;
snprintf(attrpath, sizeof(attrpath), "%s/%s", dev->path, "speed"); speed = udev_device_get_sysattr_value(dev, "speed");
if (!speed) {
attr = sysfs_open_attribute(attrpath); err("udev_device_get_sysattr_value failed");
if (!attr) {
dbg("sysfs_open_attribute failed: %s", attrpath);
return 0;
}
ret = sysfs_read_attribute(attr);
if (ret < 0) {
dbg("sysfs_read_attribute failed");
goto err; goto err;
} }
ret = sscanf(attr->value, "%99s\n", speed);
if (ret < 1) {
dbg("sscanf failed");
goto err;
}
err:
sysfs_close_attribute(attr);
for (int i = 0; speed_strings[i].speed != NULL; i++) { for (int i = 0; speed_strings[i].speed != NULL; i++) {
if (!strcmp(speed, speed_strings[i].speed)) if (!strcmp(speed, speed_strings[i].speed))
return speed_strings[i].num; return speed_strings[i].num;
} }
err:
return USB_SPEED_UNKNOWN; return USB_SPEED_UNKNOWN;
} }
@ -190,9 +166,10 @@ int read_attr_speed(struct sysfs_device *dev)
} while (0) } while (0)
int read_usb_device(struct sysfs_device *sdev, struct usbip_usb_device *udev) int read_usb_device(struct udev_device *sdev, struct usbip_usb_device *udev)
{ {
uint32_t busnum, devnum; uint32_t busnum, devnum;
const char *path, *name;
READ_ATTR(udev, uint8_t, sdev, bDeviceClass, "%02x\n"); READ_ATTR(udev, uint8_t, sdev, bDeviceClass, "%02x\n");
READ_ATTR(udev, uint8_t, sdev, bDeviceSubClass, "%02x\n"); READ_ATTR(udev, uint8_t, sdev, bDeviceSubClass, "%02x\n");
@ -209,10 +186,13 @@ int read_usb_device(struct sysfs_device *sdev, struct usbip_usb_device *udev)
READ_ATTR(udev, uint8_t, sdev, devnum, "%d\n"); READ_ATTR(udev, uint8_t, sdev, devnum, "%d\n");
udev->speed = read_attr_speed(sdev); udev->speed = read_attr_speed(sdev);
strncpy(udev->path, sdev->path, SYSFS_PATH_MAX); path = udev_device_get_syspath(sdev);
strncpy(udev->busid, sdev->name, SYSFS_BUS_ID_SIZE); name = udev_device_get_sysname(sdev);
sscanf(sdev->name, "%u-%u", &busnum, &devnum); strncpy(udev->path, path, SYSFS_PATH_MAX);
strncpy(udev->busid, name, SYSFS_BUS_ID_SIZE);
sscanf(name, "%u-%u", &busnum, &devnum);
udev->busnum = busnum; udev->busnum = busnum;
return 0; return 0;
@ -222,13 +202,13 @@ int read_usb_interface(struct usbip_usb_device *udev, int i,
struct usbip_usb_interface *uinf) struct usbip_usb_interface *uinf)
{ {
char busid[SYSFS_BUS_ID_SIZE]; char busid[SYSFS_BUS_ID_SIZE];
struct sysfs_device *sif; struct udev_device *sif;
sprintf(busid, "%s:%d.%d", udev->busid, udev->bConfigurationValue, i); sprintf(busid, "%s:%d.%d", udev->busid, udev->bConfigurationValue, i);
sif = sysfs_open_device("usb", busid); sif = udev_device_new_from_subsystem_sysname(udev_context, "usb", busid);
if (!sif) { if (!sif) {
dbg("sysfs_open_device(\"usb\", \"%s\") failed", busid); err("udev_device_new_from_subsystem_sysname %s failed", busid);
return -1; return -1;
} }
@ -236,8 +216,6 @@ int read_usb_interface(struct usbip_usb_device *udev, int i,
READ_ATTR(uinf, uint8_t, sif, bInterfaceSubClass, "%02x\n"); READ_ATTR(uinf, uint8_t, sif, bInterfaceSubClass, "%02x\n");
READ_ATTR(uinf, uint8_t, sif, bInterfaceProtocol, "%02x\n"); READ_ATTR(uinf, uint8_t, sif, bInterfaceProtocol, "%02x\n");
sysfs_close_device(sif);
return 0; return 0;
} }

5
drivers/staging/usbip/userspace/libsrc/usbip_common.h
View File

@ -6,6 +6,7 @@
#define __USBIP_COMMON_H #define __USBIP_COMMON_H
#include <sysfs/libsysfs.h> #include <sysfs/libsysfs.h>
#include <libudev.h>
#include <stdint.h> #include <stdint.h>
#include <stdio.h> #include <stdio.h>
@ -118,8 +119,8 @@ struct usbip_usb_device {
void dump_usb_interface(struct usbip_usb_interface *); void dump_usb_interface(struct usbip_usb_interface *);
void dump_usb_device(struct usbip_usb_device *); void dump_usb_device(struct usbip_usb_device *);
int read_usb_device(struct sysfs_device *sdev, struct usbip_usb_device *udev); int read_usb_device(struct udev_device *sdev, struct usbip_usb_device *udev);
int read_attr_value(struct sysfs_device *dev, const char *name, int read_attr_value(struct udev_device *dev, const char *name,
const char *format); const char *format);
int read_usb_interface(struct usbip_usb_device *udev, int i, int read_usb_interface(struct usbip_usb_device *udev, int i,
struct usbip_usb_interface *uinf); struct usbip_usb_interface *uinf);

265
drivers/staging/usbip/userspace/libsrc/usbip_host_driver.c
View File

@ -18,102 +18,65 @@
#include <sys/types.h> #include <sys/types.h>
#include <sys/stat.h> #include <sys/stat.h>
#include <fcntl.h>
#include <errno.h> #include <errno.h>
#include <unistd.h> #include <unistd.h>
#include <libudev.h>
#include "usbip_common.h" #include "usbip_common.h"
#include "usbip_host_driver.h" #include "usbip_host_driver.h"
#include "list.h"
#include "sysfs_utils.h"
#undef PROGNAME #undef PROGNAME
#define PROGNAME "libusbip" #define PROGNAME "libusbip"
struct usbip_host_driver *host_driver; struct usbip_host_driver *host_driver;
struct udev *udev_context;
#define SYSFS_OPEN_RETRIES 100
static int32_t read_attr_usbip_status(struct usbip_usb_device *udev) static int32_t read_attr_usbip_status(struct usbip_usb_device *udev)
{ {
char attrpath[SYSFS_PATH_MAX]; char status_attr_path[SYSFS_PATH_MAX];
struct sysfs_attribute *attr; int fd;
int length;
char status;
int value = 0; int value = 0;
int rc;
struct stat s;
int retries = SYSFS_OPEN_RETRIES;
/* This access is racy! snprintf(status_attr_path, SYSFS_PATH_MAX, "%s/usbip_status",
*
* Just after detach, our driver removes the sysfs
* files and recreates them.
*
* We may try and fail to open the usbip_status of
* an exported device in the (short) window where
* it has been removed and not yet recreated.
*
* This is a bug in the interface. Nothing we can do
* except work around it here by polling for the sysfs
* usbip_status to reappear.
*/
snprintf(attrpath, SYSFS_PATH_MAX, "%s/usbip_status",
udev->path); udev->path);
while (retries > 0) { if ((fd = open(status_attr_path, O_RDONLY)) < 0) {
if (stat(attrpath, &s) == 0) err("error opening attribute %s", status_attr_path);
break;
if (errno != ENOENT) {
dbg("stat failed: %s", attrpath);
return -1; return -1;
} }
usleep(10000); /* 10ms */ length = read(fd, &status, 1);
retries--; if (length < 0) {
} err("error reading attribute %s", status_attr_path);
close(fd);
if (retries == 0)
dbg("usbip_status not ready after %d retries",
SYSFS_OPEN_RETRIES);
else if (retries < SYSFS_OPEN_RETRIES)
dbg("warning: usbip_status ready after %d retries",
SYSFS_OPEN_RETRIES - retries);
attr = sysfs_open_attribute(attrpath);
if (!attr) {
dbg("sysfs_open_attribute failed: %s", attrpath);
return -1; return -1;
} }
rc = sysfs_read_attribute(attr); value = atoi(&status);
if (rc) {
dbg("sysfs_read_attribute failed: %s", attrpath);
sysfs_close_attribute(attr);
return -1;
}
value = atoi(attr->value);
sysfs_close_attribute(attr);
return value; return value;
} }
static struct usbip_exported_device *usbip_exported_device_new(char *sdevpath) static
struct usbip_exported_device *usbip_exported_device_new(const char *sdevpath)
{ {
struct usbip_exported_device *edev = NULL; struct usbip_exported_device *edev = NULL;
struct usbip_exported_device *edev_old; struct usbip_exported_device *edev_old;
size_t size; size_t size;
int i; int i;
edev = calloc(1, sizeof(*edev)); edev = calloc(1, sizeof(struct usbip_exported_device));
if (!edev) {
dbg("calloc failed");
return NULL;
}
edev->sudev = sysfs_open_device_path(sdevpath); edev->sudev = udev_device_new_from_syspath(udev_context, sdevpath);
if (!edev->sudev) { if (!edev->sudev) {
dbg("sysfs_open_device_path failed: %s", sdevpath); err("udev_device_new_from_syspath: %s", sdevpath);
goto err; goto err;
} }
@ -124,7 +87,7 @@ static struct usbip_exported_device *usbip_exported_device_new(char *sdevpath)
goto err; goto err;
/* reallocate buffer to include usb interface data */ /* reallocate buffer to include usb interface data */
size = sizeof(*edev) + edev->udev.bNumInterfaces * size = sizeof(struct usbip_exported_device) + edev->udev.bNumInterfaces *
sizeof(struct usbip_usb_interface); sizeof(struct usbip_usb_interface);
edev_old = edev; edev_old = edev;
@ -140,149 +103,88 @@ static struct usbip_exported_device *usbip_exported_device_new(char *sdevpath)
return edev; return edev;
err: err:
if (edev && edev->sudev) if (edev->sudev)
sysfs_close_device(edev->sudev); udev_device_unref(edev->sudev);
if (edev) if (edev)
free(edev); free(edev);
return NULL; return NULL;
} }
static int check_new(struct dlist *dlist, struct sysfs_device *target)
{
struct sysfs_device *dev;
dlist_for_each_data(dlist, dev, struct sysfs_device) {
if (!strncmp(dev->bus_id, target->bus_id, SYSFS_BUS_ID_SIZE))
/* device found and is not new */
return 0;
}
return 1;
}
static void delete_nothing(void *unused_data)
{
/*
* NOTE: Do not delete anything, but the container will be deleted.
*/
(void) unused_data;
}
static int refresh_exported_devices(void) static int refresh_exported_devices(void)
{ {
/* sysfs_device of usb_device */
struct sysfs_device *sudev;
struct dlist *sudev_list;
struct dlist *sudev_unique_list;
struct usbip_exported_device *edev; struct usbip_exported_device *edev;
struct udev_enumerate *enumerate;
struct udev_list_entry *devices, *dev_list_entry;
struct udev_device *dev;
const char *path;
sudev_unique_list = dlist_new_with_delete(sizeof(struct sysfs_device), enumerate = udev_enumerate_new(udev_context);
delete_nothing); udev_enumerate_add_match_subsystem(enumerate, "usb");
udev_enumerate_scan_devices(enumerate);
sudev_list = sysfs_get_driver_devices(host_driver->sysfs_driver); devices = udev_enumerate_get_list_entry(enumerate);
if (!sudev_list) { udev_list_entry_foreach(dev_list_entry, devices) {
/* path = udev_list_entry_get_name(dev_list_entry);
* Not an error condition. There are simply no devices bound to dev = udev_device_new_from_syspath(udev_context, path);
* the driver yet.
*/
dbg("bind " USBIP_HOST_DRV_NAME ".ko to a usb device to be "
"exportable!");
return 0;
}
dlist_for_each_data(sudev_list, sudev, struct sysfs_device)
if (check_new(sudev_unique_list, sudev))
dlist_unshift(sudev_unique_list, sudev);
dlist_for_each_data(sudev_unique_list, sudev, struct sysfs_device) {
edev = usbip_exported_device_new(sudev->path);
/* Check whether device uses usbip-host driver. */
if (!strcmp(udev_device_get_driver(dev),
USBIP_HOST_DRV_NAME)) {
edev = usbip_exported_device_new(path);
if (!edev) { if (!edev) {
dbg("usbip_exported_device_new failed"); dbg("usbip_exported_device_new failed");
continue; continue;
} }
dlist_unshift(host_driver->edev_list, edev); list_add(&edev->node, &host_driver->edev_list);
host_driver->ndevs++; host_driver->ndevs++;
} }
}
dlist_destroy(sudev_unique_list);
return 0; return 0;
} }
static struct sysfs_driver *open_sysfs_host_driver(void) static void usbip_exported_device_destroy(void)
{ {
char bus_type[] = "usb"; struct list_head *i, *tmp;
char sysfs_mntpath[SYSFS_PATH_MAX]; struct usbip_exported_device *edev;
char host_drv_path[SYSFS_PATH_MAX];
struct sysfs_driver *host_drv;
int rc;
rc = sysfs_get_mnt_path(sysfs_mntpath, SYSFS_PATH_MAX); list_for_each_safe(i, tmp, &host_driver->edev_list) {
if (rc < 0) { edev = list_entry(i, struct usbip_exported_device, node);
dbg("sysfs_get_mnt_path failed"); list_del(i);
return NULL; free(edev);
} }
snprintf(host_drv_path, SYSFS_PATH_MAX, "%s/%s/%s/%s/%s",
sysfs_mntpath, SYSFS_BUS_NAME, bus_type, SYSFS_DRIVERS_NAME,
USBIP_HOST_DRV_NAME);
host_drv = sysfs_open_driver_path(host_drv_path);
if (!host_drv) {
dbg("sysfs_open_driver_path failed");
return NULL;
}
return host_drv;
}
static void usbip_exported_device_delete(void *dev)
{
struct usbip_exported_device *edev = dev;
sysfs_close_device(edev->sudev);
free(dev);
} }
int usbip_host_driver_open(void) int usbip_host_driver_open(void)
{ {
int rc; int rc;
host_driver = calloc(1, sizeof(*host_driver)); udev_context = udev_new();
if (!host_driver) { if (!udev_context) {
dbg("calloc failed"); err("udev_new failed");
return -1; return -1;
} }
host_driver->ndevs = 0; host_driver = calloc(1, sizeof(*host_driver));
host_driver->edev_list =
dlist_new_with_delete(sizeof(struct usbip_exported_device),
usbip_exported_device_delete);
if (!host_driver->edev_list) {
dbg("dlist_new_with_delete failed");
goto err_free_host_driver;
}
host_driver->sysfs_driver = open_sysfs_host_driver(); host_driver->ndevs = 0;
if (!host_driver->sysfs_driver) INIT_LIST_HEAD(&host_driver->edev_list);
goto err_destroy_edev_list;
rc = refresh_exported_devices(); rc = refresh_exported_devices();
if (rc < 0) if (rc < 0)
goto err_close_sysfs_driver; goto err_free_host_driver;
return 0; return 0;
err_close_sysfs_driver:
sysfs_close_driver(host_driver->sysfs_driver);
err_destroy_edev_list:
dlist_destroy(host_driver->edev_list);
err_free_host_driver: err_free_host_driver:
free(host_driver); free(host_driver);
host_driver = NULL; host_driver = NULL;
udev_unref(udev_context);
return -1; return -1;
} }
@ -291,30 +193,22 @@ void usbip_host_driver_close(void)
if (!host_driver) if (!host_driver)
return; return;
if (host_driver->edev_list) usbip_exported_device_destroy();
dlist_destroy(host_driver->edev_list);
if (host_driver->sysfs_driver)
sysfs_close_driver(host_driver->sysfs_driver);
free(host_driver); free(host_driver);
host_driver = NULL; host_driver = NULL;
udev_unref(udev_context);
} }
int usbip_host_refresh_device_list(void) int usbip_host_refresh_device_list(void)
{ {
int rc; int rc;
if (host_driver->edev_list) usbip_exported_device_destroy();
dlist_destroy(host_driver->edev_list);
host_driver->ndevs = 0; host_driver->ndevs = 0;
host_driver->edev_list = INIT_LIST_HEAD(&host_driver->edev_list);
dlist_new_with_delete(sizeof(struct usbip_exported_device),
usbip_exported_device_delete);
if (!host_driver->edev_list) {
dbg("dlist_new_with_delete failed");
return -1;
}
rc = refresh_exported_devices(); rc = refresh_exported_devices();
if (rc < 0) if (rc < 0)
@ -326,8 +220,7 @@ int usbip_host_refresh_device_list(void)
int usbip_host_export_device(struct usbip_exported_device *edev, int sockfd) int usbip_host_export_device(struct usbip_exported_device *edev, int sockfd)
{ {
char attr_name[] = "usbip_sockfd"; char attr_name[] = "usbip_sockfd";
char attr_path[SYSFS_PATH_MAX]; char sockfd_attr_path[SYSFS_PATH_MAX];
struct sysfs_attribute *attr;
char sockfd_buff[30]; char sockfd_buff[30];
int ret; int ret;
@ -347,40 +240,32 @@ int usbip_host_export_device(struct usbip_exported_device *edev, int sockfd)
} }
/* only the first interface is true */ /* only the first interface is true */
snprintf(attr_path, sizeof(attr_path), "%s/%s", snprintf(sockfd_attr_path, sizeof(sockfd_attr_path), "%s/%s",
edev->udev.path, attr_name); edev->udev.path, attr_name);
attr = sysfs_open_attribute(attr_path);
if (!attr) {
dbg("sysfs_open_attribute failed: %s", attr_path);
return -1;
}
snprintf(sockfd_buff, sizeof(sockfd_buff), "%d\n", sockfd); snprintf(sockfd_buff, sizeof(sockfd_buff), "%d\n", sockfd);
dbg("write: %s", sockfd_buff);
ret = sysfs_write_attribute(attr, sockfd_buff, strlen(sockfd_buff)); ret = write_sysfs_attribute(sockfd_attr_path, sockfd_buff,
strlen(sockfd_buff));
if (ret < 0) { if (ret < 0) {
dbg("sysfs_write_attribute failed: sockfd %s to %s", err("write_sysfs_attribute failed: sockfd %s to %s",
sockfd_buff, attr_path); sockfd_buff, sockfd_attr_path);
goto err_write_sockfd; return ret;
} }
dbg("connect: %s", edev->udev.busid); info("connect: %s", edev->udev.busid);
err_write_sockfd:
sysfs_close_attribute(attr);
return ret; return ret;
} }
struct usbip_exported_device *usbip_host_get_device(int num) struct usbip_exported_device *usbip_host_get_device(int num)
{ {
struct list_head *i;
struct usbip_exported_device *edev; struct usbip_exported_device *edev;
struct dlist *dlist = host_driver->edev_list;
int cnt = 0; int cnt = 0;
dlist_for_each_data(dlist, edev, struct usbip_exported_device) { list_for_each(i, &host_driver->edev_list) {
edev = list_entry(i, struct usbip_exported_device, node);
if (num == cnt) if (num == cnt)
return edev; return edev;
else else

7
drivers/staging/usbip/userspace/libsrc/usbip_host_driver.h
View File

@ -21,18 +21,19 @@
#include <stdint.h> #include <stdint.h>
#include "usbip_common.h" #include "usbip_common.h"
#include "list.h"
struct usbip_host_driver { struct usbip_host_driver {
int ndevs; int ndevs;
struct sysfs_driver *sysfs_driver;
/* list of exported device */ /* list of exported device */
struct dlist *edev_list; struct list_head edev_list;
}; };
struct usbip_exported_device { struct usbip_exported_device {
struct sysfs_device *sudev; struct udev_device *sudev;
int32_t status; int32_t status;
struct usbip_usb_device udev; struct usbip_usb_device udev;
struct list_head node;
struct usbip_usb_interface uinf[]; struct usbip_usb_interface uinf[];
}; };

22
drivers/staging/usbip/userspace/libsrc/vhci_driver.c
View File

@ -6,24 +6,27 @@
#include "vhci_driver.h" #include "vhci_driver.h"
#include <limits.h> #include <limits.h>
#include <netdb.h> #include <netdb.h>
#include <libudev.h>
#undef PROGNAME #undef PROGNAME
#define PROGNAME "libusbip" #define PROGNAME "libusbip"
struct usbip_vhci_driver *vhci_driver; struct usbip_vhci_driver *vhci_driver;
struct udev *udev_context;
static struct usbip_imported_device * static struct usbip_imported_device *
imported_device_init(struct usbip_imported_device *idev, char *busid) imported_device_init(struct usbip_imported_device *idev, char *busid)
{ {
struct sysfs_device *sudev; struct udev_device *sudev;
sudev = sysfs_open_device("usb", busid); sudev = udev_device_new_from_subsystem_sysname(udev_context,
"usb", busid);
if (!sudev) { if (!sudev) {
dbg("sysfs_open_device failed: %s", busid); dbg("udev_device_new_from_subsystem_sysname failed: %s", busid);
goto err; goto err;
} }
read_usb_device(sudev, &idev->udev); read_usb_device(sudev, &idev->udev);
sysfs_close_device(sudev); udev_device_unref(sudev);
/* add class devices of this imported device */ /* add class devices of this imported device */
struct usbip_class_device *cdev; struct usbip_class_device *cdev;
@ -410,6 +413,12 @@ int usbip_vhci_driver_open(void)
int ret; int ret;
char hc_busid[SYSFS_BUS_ID_SIZE]; char hc_busid[SYSFS_BUS_ID_SIZE];
udev_context = udev_new();
if (!udev_context) {
err("udev_new failed");
return -1;
}
vhci_driver = (struct usbip_vhci_driver *) calloc(1, sizeof(*vhci_driver)); vhci_driver = (struct usbip_vhci_driver *) calloc(1, sizeof(*vhci_driver));
if (!vhci_driver) { if (!vhci_driver) {
dbg("calloc failed"); dbg("calloc failed");
@ -461,6 +470,9 @@ int usbip_vhci_driver_open(void)
free(vhci_driver); free(vhci_driver);
vhci_driver = NULL; vhci_driver = NULL;
udev_unref(udev_context);
return -1; return -1;
} }
@ -483,6 +495,8 @@ void usbip_vhci_driver_close()
free(vhci_driver); free(vhci_driver);
vhci_driver = NULL; vhci_driver = NULL;
udev_unref(udev_context);
} }

14
drivers/staging/usbip/userspace/src/usbipd.c
View File

@ -43,6 +43,7 @@
#include "usbip_host_driver.h" #include "usbip_host_driver.h"
#include "usbip_common.h" #include "usbip_common.h"
#include "usbip_network.h" #include "usbip_network.h"
#include "list.h"
#undef PROGNAME #undef PROGNAME
#define PROGNAME "usbipd" #define PROGNAME "usbipd"
@ -93,6 +94,7 @@ static int recv_request_import(int sockfd)
struct op_common reply; struct op_common reply;
struct usbip_exported_device *edev; struct usbip_exported_device *edev;
struct usbip_usb_device pdu_udev; struct usbip_usb_device pdu_udev;
struct list_head *i;
int found = 0; int found = 0;
int error = 0; int error = 0;
int rc; int rc;
@ -107,8 +109,8 @@ static int recv_request_import(int sockfd)
} }
PACK_OP_IMPORT_REQUEST(0, &req); PACK_OP_IMPORT_REQUEST(0, &req);
dlist_for_each_data(host_driver->edev_list, edev, list_for_each(i, &host_driver->edev_list) {
struct usbip_exported_device) { edev = list_entry(i, struct usbip_exported_device, node);
if (!strncmp(req.busid, edev->udev.busid, SYSFS_BUS_ID_SIZE)) { if (!strncmp(req.busid, edev->udev.busid, SYSFS_BUS_ID_SIZE)) {
info("found requested device: %s", req.busid); info("found requested device: %s", req.busid);
found = 1; found = 1;
@ -161,12 +163,12 @@ static int send_reply_devlist(int connfd)
struct usbip_usb_device pdu_udev; struct usbip_usb_device pdu_udev;
struct usbip_usb_interface pdu_uinf; struct usbip_usb_interface pdu_uinf;
struct op_devlist_reply reply; struct op_devlist_reply reply;
struct list_head *j;
int rc, i; int rc, i;
reply.ndev = 0; reply.ndev = 0;
/* number of exported devices */ /* number of exported devices */
dlist_for_each_data(host_driver->edev_list, edev, list_for_each(j, &host_driver->edev_list) {
struct usbip_exported_device) {
reply.ndev += 1; reply.ndev += 1;
} }
info("exportable devices: %d", reply.ndev); info("exportable devices: %d", reply.ndev);
@ -184,8 +186,8 @@ static int send_reply_devlist(int connfd)
return -1; return -1;
} }
dlist_for_each_data(host_driver->edev_list, edev, list_for_each(j, &host_driver->edev_list) {
struct usbip_exported_device) { edev = list_entry(j, struct usbip_exported_device, node);
dump_usb_device(&edev->udev); dump_usb_device(&edev->udev);
memcpy(&pdu_udev, &edev->udev, sizeof(pdu_udev)); memcpy(&pdu_udev, &edev->udev, sizeof(pdu_udev));
usbip_net_pack_usb_device(1, &pdu_udev); usbip_net_pack_usb_device(1, &pdu_udev);