usbatm subdriver registration w/o .driver_info

Duncan Sands duncan.sands at math.u-psud.fr
Thu Jan 27 06:30:47 EST 2005


Hi Roman, you need to add a lock, otherwise this

> +	list_for_each_entry(driver, &usbatm_driver_list, driver_list)
> +		if (usb_match_id(intf, driver->id_table))
> +			break;
> +

might run at the same time as this

> +	list_add(&driver->driver_list, &usbatm_driver_list);

or this

> +	list_del(&driver->driver_list);

corrupting the list.  Or are you relying on the big kernel lock?

Ciao,

Duncan.



More information about the Usbatm mailing list