Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

USB: Fix device driver race

When a new device with a specialised device driver is plugged in, the
new driver will be modprobe()'d but the driver core will attach the
"generic" driver to the device.

After that, nothing will trigger a reprobe when the modprobe()'d device
driver has finished initialising, as the device has the "generic"
driver attached to it.

Trigger a reprobe ourselves when new specialised drivers get registered.

Fixes: 88b7381a939d ("USB: Select better matching USB drivers when available")
Signed-off-by: Bastien Nocera <hadess@hadess.net>
Cc: stable <stable@vger.kernel.org>
Acked-by: Alan Stern <stern@rowland.harvard.edu>
Link: https://lore.kernel.org/r/20200818110445.509668-3-hadess@hadess.net
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

authored by

Bastien Nocera and committed by
Greg Kroah-Hartman
d5643d22 adb6e6ac

+38 -2
+38 -2
drivers/usb/core/driver.c
··· 905 905 return 0; 906 906 } 907 907 908 + static bool is_dev_usb_generic_driver(struct device *dev) 909 + { 910 + struct usb_device_driver *udd = dev->driver ? 911 + to_usb_device_driver(dev->driver) : NULL; 912 + 913 + return udd == &usb_generic_driver; 914 + } 915 + 916 + static int __usb_bus_reprobe_drivers(struct device *dev, void *data) 917 + { 918 + struct usb_device_driver *new_udriver = data; 919 + struct usb_device *udev; 920 + int ret; 921 + 922 + if (!is_dev_usb_generic_driver(dev)) 923 + return 0; 924 + 925 + udev = to_usb_device(dev); 926 + if (usb_device_match_id(udev, new_udriver->id_table) == NULL && 927 + (!new_udriver->match || new_udriver->match(udev) != 0)) 928 + return 0; 929 + 930 + ret = device_reprobe(dev); 931 + if (ret && ret != -EPROBE_DEFER) 932 + dev_err(dev, "Failed to reprobe device (error %d)\n", ret); 933 + 934 + return 0; 935 + } 936 + 908 937 /** 909 938 * usb_register_device_driver - register a USB device (not interface) driver 910 939 * @new_udriver: USB operations for the device driver ··· 963 934 964 935 retval = driver_register(&new_udriver->drvwrap.driver); 965 936 966 - if (!retval) 937 + if (!retval) { 967 938 pr_info("%s: registered new device driver %s\n", 968 939 usbcore_name, new_udriver->name); 969 - else 940 + /* 941 + * Check whether any device could be better served with 942 + * this new driver 943 + */ 944 + bus_for_each_dev(&usb_bus_type, NULL, new_udriver, 945 + __usb_bus_reprobe_drivers); 946 + } else { 970 947 printk(KERN_ERR "%s: error %d registering device " 971 948 " driver %s\n", 972 949 usbcore_name, retval, new_udriver->name); 950 + } 973 951 974 952 return retval; 975 953 }