Opened 13 years ago

Closed 12 years ago

#10784 closed Patch - Feature (fixed)

match EIT to channels without looking at ONID or TSID in dtv_multiplex

Reported by: dekarl@… Owned by: Stuart Auchterlonie
Priority: minor Milestone: 0.27
Component: MythTV - EIT Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

This patch allows the EITScanner to find the right channel even if

  • the broadcaster messed up their feed (e.g. feeding DVB-C from DVB-S with mashed up multiplexes. The PMT/SDT gets properly rewritten with new ONID/TSID but the EIT is copied unchanged)
  • the channel scanner couldn't figure out ONID/TSID

Use cases to test

  • passive scan within a transport (worked very well)
  • active scan within a transport (I need to look at it again)
  • any scan across transports (I can't test it due to no access to DVB-S)

Use cases that do not work due to missing glue code

  • EIT from IPTV, etc.

Attachments (2)

0007-Handle-EIT-events-of-the-active-transport-without-lo.patch​ (10.4 KB) - added by dekarl@… 13 years ago.
first version of the patch
0007-Handle-EIT-events-of-the-active-transport-without-lo.patch-v2​ (11.4 KB) - added by dekarl@… 13 years ago.
v2: actually accept any SDT when ONID/TSID are 0

Download all attachments as: .zip

Change History (8)

Changed 13 years ago by dekarl@…

Attachment: 0007-Handle-EIT-events-of-the-active-transport-without-lo.patch​ added

first version of the patch

Changed 13 years ago by dekarl@…

Attachment: 0007-Handle-EIT-events-of-the-active-transport-without-lo.patch-v2​ added

v2: actually accept any SDT when ONID/TSID are 0

comment:1 Changed 13 years ago by dekarl@…

v2 of the patch actually works with NULL as ONID/TSID in dtv_multiplex

comment:2 Changed 13 years ago by dekarl@…

I've been running with the patch in production and it worked so well, in active scan, I forgot to put the ONID/TSID back in.

Still to test DVB with EIT(other) (common on DVB-S) but that just uses the old code path which worked before.

comment:3 Changed 13 years ago by Raymond Wagner

Type: Bug Report - General → Patch - Feature

comment:4 Changed 13 years ago by Stuart Auchterlonie <stuarta@…>

In b4110ae5f6a744b768711e4926439b83b201e105/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:5 Changed 13 years ago by Stuart Auchterlonie

Milestone: unknown → 0.27
Status: new → assigned

comment:6 Changed 12 years ago by Karl Dietz <dekarl@…>

Resolution: → fixed
Status: assigned → closed

In 3c3dd7c5372aefe9b597f2fc0b3509a61e4f2e37/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available
Note: See TracTickets for help on using tickets.