Commit 501950d8 authored by Alan Stern's avatar Alan Stern Committed by Greg Kroah-Hartman

USB: fix char-device disconnect handling

This patch (as1198) fixes a conceptual bug: Somewhere along the line
we managed to confuse USB class devices with USB char devices.  As a
result, the code to send a disconnect signal to userspace would not be
built if both CONFIG_USB_DEVICE_CLASS and CONFIG_USB_DEVICEFS were
disabled.

The usb_fs_classdev_common_remove() routine has been renamed to
usbdev_remove() and it is now called whenever any USB device is
removed, not just when a class device is unregistered.  The notifier
registration and unregistration calls are no longer conditionally
compiled.  And since the common removal code will always be called as
part of the char device interface, there's no need to call it again as
part of the usbfs interface; thus the invocation of
usb_fs_classdev_common_remove() has been taken out of
usbfs_remove_device().
Signed-off-by: default avatarAlan Stern <stern@rowland.harvard.edu>
Reported-by: default avatarAlon Bar-Lev <alon.barlev@gmail.com>
Tested-by: default avatarAlon Bar-Lev <alon.barlev@gmail.com>
Cc: stable <stable@kernel.org>
parent a15d95a0
...@@ -1700,7 +1700,7 @@ const struct file_operations usbdev_file_operations = { ...@@ -1700,7 +1700,7 @@ const struct file_operations usbdev_file_operations = {
.release = usbdev_release, .release = usbdev_release,
}; };
void usb_fs_classdev_common_remove(struct usb_device *udev) static void usbdev_remove(struct usb_device *udev)
{ {
struct dev_state *ps; struct dev_state *ps;
struct siginfo sinfo; struct siginfo sinfo;
...@@ -1742,10 +1742,15 @@ static void usb_classdev_remove(struct usb_device *dev) ...@@ -1742,10 +1742,15 @@ static void usb_classdev_remove(struct usb_device *dev)
{ {
if (dev->usb_classdev) if (dev->usb_classdev)
device_unregister(dev->usb_classdev); device_unregister(dev->usb_classdev);
usb_fs_classdev_common_remove(dev);
} }
static int usb_classdev_notify(struct notifier_block *self, #else
#define usb_classdev_add(dev) 0
#define usb_classdev_remove(dev) do {} while (0)
#endif
static int usbdev_notify(struct notifier_block *self,
unsigned long action, void *dev) unsigned long action, void *dev)
{ {
switch (action) { switch (action) {
...@@ -1755,15 +1760,15 @@ static int usb_classdev_notify(struct notifier_block *self, ...@@ -1755,15 +1760,15 @@ static int usb_classdev_notify(struct notifier_block *self,
break; break;
case USB_DEVICE_REMOVE: case USB_DEVICE_REMOVE:
usb_classdev_remove(dev); usb_classdev_remove(dev);
usbdev_remove(dev);
break; break;
} }
return NOTIFY_OK; return NOTIFY_OK;
} }
static struct notifier_block usbdev_nb = { static struct notifier_block usbdev_nb = {
.notifier_call = usb_classdev_notify, .notifier_call = usbdev_notify,
}; };
#endif
static struct cdev usb_device_cdev; static struct cdev usb_device_cdev;
...@@ -1798,9 +1803,8 @@ int __init usb_devio_init(void) ...@@ -1798,9 +1803,8 @@ int __init usb_devio_init(void)
* to /sys/dev * to /sys/dev
*/ */
usb_classdev_class->dev_kobj = NULL; usb_classdev_class->dev_kobj = NULL;
usb_register_notify(&usbdev_nb);
#endif #endif
usb_register_notify(&usbdev_nb);
out: out:
return retval; return retval;
...@@ -1811,8 +1815,8 @@ int __init usb_devio_init(void) ...@@ -1811,8 +1815,8 @@ int __init usb_devio_init(void)
void usb_devio_cleanup(void) void usb_devio_cleanup(void)
{ {
#ifdef CONFIG_USB_DEVICE_CLASS
usb_unregister_notify(&usbdev_nb); usb_unregister_notify(&usbdev_nb);
#ifdef CONFIG_USB_DEVICE_CLASS
class_destroy(usb_classdev_class); class_destroy(usb_classdev_class);
#endif #endif
cdev_del(&usb_device_cdev); cdev_del(&usb_device_cdev);
......
...@@ -717,7 +717,6 @@ static void usbfs_remove_device(struct usb_device *dev) ...@@ -717,7 +717,6 @@ static void usbfs_remove_device(struct usb_device *dev)
fs_remove_file (dev->usbfs_dentry); fs_remove_file (dev->usbfs_dentry);
dev->usbfs_dentry = NULL; dev->usbfs_dentry = NULL;
} }
usb_fs_classdev_common_remove(dev);
} }
static int usbfs_notify(struct notifier_block *self, unsigned long action, void *dev) static int usbfs_notify(struct notifier_block *self, unsigned long action, void *dev)
......
...@@ -152,7 +152,6 @@ extern struct usb_driver usbfs_driver; ...@@ -152,7 +152,6 @@ extern struct usb_driver usbfs_driver;
extern const struct file_operations usbfs_devices_fops; extern const struct file_operations usbfs_devices_fops;
extern const struct file_operations usbdev_file_operations; extern const struct file_operations usbdev_file_operations;
extern void usbfs_conn_disc_event(void); extern void usbfs_conn_disc_event(void);
extern void usb_fs_classdev_common_remove(struct usb_device *udev);
extern int usb_devio_init(void); extern int usb_devio_init(void);
extern void usb_devio_cleanup(void); extern void usb_devio_cleanup(void);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment