Bug#810726: linux-image-4.3.0-1-amd64: Kernel 4.3 breaks Hauppauge HD-PVR recording

2016-01-26 Thread Javier Martinez Canillas

Hello David,

Sorry for the late response.

On 01/20/2016 04:41 PM, David Engel wrote:

Javier and Mauro, this is in regards to Debian bug #810726, which I
filed last week.  I also informed Hans about it at that time.

Using git bisect, I finally identified the attached commit as the
offending one.  Sure enough, after reverting that change, my hdpvr
behaves reliably with the 4.3.3 kernel.  Further testing has revealed
that I can achieve the same result with the standard, Debian kernel by
blacklisting the ir_kbd_i2c module.



The attached commit is only fixing module autoloading for the I2C driver
so is just exposing a existent bug. If you built the driver in the image
instead of as a module, the bug should be present even with that commit
reverted.


It seems something in the IR initialization is causing the hdpvr to
become unreliable.  This is even when the IR support is not used nor
wanted.  Additionally, blacklisting the lirc_zilog module is not
sufficient to work around the problem, nor is keeping it but using the
tx_only=1 parameter.



I believe the root cause has to be found since as I said, it was only
working because the module didn't have the module alias information so
udev/kmod were not able to load the module when the kernel report the
MODALIAS uevent when the I2C device was registered.


David



Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America



Bug#810726: linux-image-4.3.0-1-amd64: Kernel 4.3 breaks Hauppauge HD-PVR recording

2016-01-27 Thread Javier Martinez Canillas

Hello David,

On 01/27/2016 12:26 AM, David Engel wrote:

On Tue, Jan 26, 2016 at 11:06:53PM -0300, Javier Martinez Canillas wrote:


[snip]





The attached commit is only fixing module autoloading for the I2C driver
so is just exposing a existent bug. If you built the driver in the image
instead of as a module, the bug should be present even with that commit
reverted.


Yes, I realize that.  The thing is the LIRC receive support on the
hdpvr has been notoriously flaky, at least in MythTV circles, and
often causes recording instability.  Previously, with the autoloading
bug, most hdpvr users weren't affected by it unless they manually
loaded the appropriate LIRC modules.  Now, with the autoloading bug
fixed, many unsuspecting hdpvr users will experience the instability
when upgrading kernels.



Fair enough, I've posted a patch [0] to partially revert the module auto
loading for the ir-kbd-i2c driver. That should "fix" the regression by
masking the real issue as it was before the mentioned commit.

Unfortunately I can't do more since I don't have access to the HD PVR hw.

[0]: https://patchwork.linuxtv.org/patch/32750/

Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America