On Saturday 16 November 2013 17:30:25 simon@xxxxxxxxxxxxx wrote: > This patch appears to work OK with my DualShock/SixAxis controller (USB > connection), but causes a machine lockup when used with my Intec wired > controller. > > The Intec controller works OK up to the point when I start rumble, > sometimes the motor runs some times it doesn't. I am using SDL2's > 'testhaptic' application. Thanks a lot for this bug report. The testhaptic was a good testcase because it is a heavy user and can reproduce the problem quite easily. I've only tested it using testrumble and own programs which didn't seem to trigger the problem here. The problem is easy to explain: * usb_control_msg/usb_interrupt_msg/usb_bulk_msg/... may sleep * sony_play_effect may gets called in an softirq context (atomic) So it is a bad choice to use hid_output_raw_report (which calls usb_control_msg) in a ff_memless control function. I will just send a revert patch in some minutes. Kind regards, Sven
Attachment:
signature.asc
Description: This is a digitally signed message part.
