Opened 10 years ago

Closed 10 years ago

Last modified 10 years ago

#7212 closed defect (fixed)

Running mtd as a daemon prevents keeping a DVD tray open

Reported by: superm1@… Owned by: danielk
Priority: minor Milestone: 0.22
Component: Plugin - MythVideo Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

When mtd is invoked like "mtd -d" , this is preventing the DVD drive from staying open. Running the eject command, pressing the button, or any similar actions get the tray to eject and then reject back in. Killing mtd prevents the problem.

Last verified on trunk 22101.

Attachments (1)

7212-v1.patch (2.8 KB) - added by danielk 10 years ago.
Possible fix

Download all attachments as: .zip

Change History (10)

comment:1 Changed 10 years ago by robertm

Resolution: fixed
Status: newclosed

(In [22126]) Closes #7212. Reorder the status probe in MTD to try to open the device *after* checking the tray status. Trying to open the device when the tray was open automatically closes the tray. This means that every time the tray was opened, it was immediately closed as part of the probe. Here at MythTV, we recognize that your DVR shearing off your wife's fingers is the kind of thing that will get you in trouble.

comment:2 Changed 10 years ago by robertm

(In [22128]) Refs #7212. Restore the QFile::Open, we need that to get the handle to check the tray status. Instead, split the difference and up the probe frequency from 1 second to 14. mtd will still close the tray, but at least it will do it in a more sane interval.

comment:4 Changed 10 years ago by stuartm

Ticket locked: set

comment:5 Changed 10 years ago by danielk

Owner: changed from Anduin Withers to danielk

comment:6 Changed 10 years ago by danielk

Resolution: fixed
Status: closednew

comment:7 Changed 10 years ago by danielk

Ticket locked: unset

Changed 10 years ago by danielk

Attachment: 7212-v1.patch added

Possible fix

comment:8 Changed 10 years ago by danielk

The patch is as of yet untested. But it just reverts some changes and adds some ifdef's to the original code so. If this resolves the digit removal bug, please reply in the ticket comments.

comment:8 Changed 10 years ago by danielk

Resolution: fixed
Status: newclosed

(In [22956]) Fixes #7212. Open dvd file handle with O_NONBLOCK so that it does not force the tray closed.

comment:9 Changed 10 years ago by danielk

(In [22957]) Refs #7212. Open dvd file handle with O_NONBLOCK so that it does not force the tray closed. Backport of [22956].

Note: See TracTickets for help on using tickets.