On Mon, May 01, 2017 at 07:47:25PM +0200, David Härdeman wrote: >On Mon, May 01, 2017 at 05:49:53PM +0100, Sean Young wrote: >>On Thu, Apr 27, 2017 at 10:34:03PM +0200, David Härdeman wrote: >>> The device core infrastructure is based on the presumption that >>> once a driver calls device_add(), it must be ready to accept >>> userspace interaction. >>> >>> This requires splitting rc_setup_rx_device() into two functions >>> and reorganizing rc_register_device() so that as much work >>> as possible is performed before calling device_add(). >>> >> >>With this patch applied, I'm no longer getting any scancodes from >>my rc devices. >> >>David, please can you test your patches before submitting. I have to go >>over them meticulously because I cannot assume you've tested them. > >I did test this patch and I just redid the tests, both with rc-loopback >and with a mceusb receiver. I'm seeing scancodes on the input device as >well as pulse-space readings on the lirc device in both tests. > >I did the tests with only this patch applied and the lirc-use-after-free >(v3). What hardware did you test with? > >Meanwhile, I'll try rebasing my patches to the latest version of the >media-master git tree and test again. I rebased the patches onto media-master (commit 3622d3e77ecef090b5111e3c5423313f11711dfa) and tested again. I still can't reproduce the problems you're having :/ -- David Härdeman
