Unplugging an az6007 based tuner (here a TechniSat CableStar Combo HD
CI with a CAM inserted) while it is in use can leave parts of the DVB
stack stuck on the disconnected device.
In particular:
- drxk keeps accessing the device after unplug and hides the resulting
errors from userspace.
- The CA thread can get stuck polling a removed CAM slot and prevent
the disconnect from completing.
- Applications that wait on or poll the DVB devices are neither woken
up nor told that the device is gone, so the disconnect can wait
indefinitely for them to close the devices.
This series fixes these issues by:
- treating an unreadable CAM slot status as no CAM,
- propagating -ENODEV from az6007 to drxk and stopping further device
access,
- waking up the users of the demux, dvr and CA devices and returning
-ENODEV to them, so they can handle the device removal.
Tested on Linux 6.18.44 on a Turris 1.x (PowerPC P2020, two CPUs,
non-preemptible):
- tvheadend with CAM enabled, idle, scanning and streaming,
- unplugging the tuner or the whole USB hub,
- unbinding the driver through sysfs,
- blocking reads on dvr0 and demux0.
In all tested cases the disconnect completed and blocked userspace
operations returned -ENODEV.
Some pre-existing cases where the release waits for users that are
not woken up remain. The unlocked user count check in
dvb_ca_en50221_release() and dvb_dmxdev_release() also still races
with a concurrent close(), and patches 5 and 6 make that race easier
to hit.
Josef Schlehofer (6):
media: az6007: fix CAM status polling after disconnect
media: az6007: propagate USB errors from I2C transfers
media: drxk: stop retrying after disconnect
media: drxk: stop accessing a disconnected device
media: dvb-core: dmxdev: wake up readers on release
media: dvb-core: wake up CA users on release
drivers/media/dvb-core/dmxdev.c | 31 ++++++++++++++--
drivers/media/dvb-core/dvb_ca_en50221.c | 24 ++++++++++++-
drivers/media/dvb-frontends/drxk_hard.c | 48 +++++++++++++++++++------
drivers/media/dvb-frontends/drxk_hard.h | 2 +-
drivers/media/usb/dvb-usb-v2/az6007.c | 23 ++++++------
5 files changed, 104 insertions(+), 24 deletions(-)
base-commit: df2908090cda368b01ff43709f51890076c56157
--
2.54.0 (Apple Git-157)