Opened 17 years ago
Closed 16 years ago
#5604 closed patch (fixed)
DeviceReadBuffer: limit poll timeouts
| Reported by: | Owned by: | danielk | |
|---|---|---|---|
| Priority: | minor | Milestone: | unknown |
| Component: | mythtv | Version: | unknown |
| Severity: | medium | Keywords: | DeviceReadBuffer |
| Cc: | Ticket locked: | no |
Description
The attached patch will limit the number of consecutive poll timeouts.
Old behavior would just loop forever. New behavior is to enter an error state. This gives the "recorder" an opportunity to reset the device.
Attachments (2)
Change History (8)
by , 17 years ago
| Attachment: | DeviceReadBuffer-polltimeout.patch added |
|---|
comment:1 by , 17 years ago
I also have a patch which causes the DeviceReadBuffer to continue to read from the device even when the buffer is full, but it needs more testing. It will apply on top of DeviceReadBuffer-polltimeout.patch
comment:2 by , 17 years ago
I am going to split DeviceReadBuffer-polltimeout.patch into two pieces. A polltimeout piece, and a mutex refactoring piece. So, don't bother to commit the current patch, as is.
comment:3 by , 17 years ago
(In [18130]) Refs #5604. Finishes Qt4 port of DeviceReadBuffer. Qt4 QWaitCondition should use the DRB lock.
by , 17 years ago
| Attachment: | DeviceReadBuffer-polltimeout.2.patch added |
|---|
Updated patch for current trunk
comment:4 by , 17 years ago
Decided against splitting this into two patches, so just attached a refresh against trunk.
comment:5 by , 17 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → accepted |
comment:6 by , 16 years ago
| Resolution: | → fixed |
|---|---|
| Status: | accepted → closed |
(In [20617]) Fixes #5604. Error out in DeviceReadBuffer can't read from a device for a few seconds instead of looping forever waiting for the device to respond.
limits the number of consecutive poll timeouts.