Opened 14 years ago

Closed 7 years ago

#10745 closed Bug Report - General (Fixed)

Myth player not properly resetting after resolution change

Reported by: JYA Owned by: Mark Kendall
Priority: minor Milestone: 31.0
Component: MythTV - General Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description (last modified by JYA)

Attached are 3 samples, all mpeg-ts streams with a H264 video stream and a AAC audio stream

bipbop234.ts is made of 3 x 10s. First block is at 480x320, 2nd and 3rd are at 1280x720. Between 2nd and 3rd it's just a change of video bitrate. This file plays fine with the avfd h264 reset hack. The file exhibit the same problem as #10715

nasa-8s2.ts has a resolution change occurring at 8s... About 30% of the time, myth with the avfd hack will crash and the AAC audio will stop decoding properly.

nasa-27s.ts is like nasa-8s2.ts but with the change over occurring at 27s.

The problem exhibited by those 3 files ultimately is breaking proper handling of HLS stream should the player determine that a lower bitrate stream is to be used.

Note that a recent version of mplayer will play all those files just fine.

Attachments (1)

h264reset.patch (14.6 KB ) - added by JYA 14 years ago.
This patch close the h264 decoder and re-open it if a change of resolution is detected

Download all attachments as: .zip

Change History (11)

comment:1 by JYA, 14 years ago

Description: modified (diff)

by JYA, 14 years ago

Attachment: h264reset.patch added

This patch close the h264 decoder and re-open it if a change of resolution is detected

comment:3 by Daniel Thor Kristjansson <danielk@…>, 14 years ago

Resolution: fixed
Status: newclosed

In 83e227a15b64ed217638269f8eca1037774a187f/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:4 by Daniel Thor Kristjansson <danielk@…>, 14 years ago

In 7593192d066a75ee9f55887bc8880dc35dc5b285/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:5 by JYA, 13 years ago

Resolution: fixed
Status: closednew

Re-opening ticket as this fix was really just a hack and doesn't handle all changes appropriately.

comment:6 by JYA, 13 years ago

Mark #11740 as a duplicate of this bug...

comment:7 by Raymond Wagner, 13 years ago

Milestone: 0.260.26.2

comment:8 by paulh, 12 years ago

Milestone: 0.26.2unknown

comment:9 by Mark Kendall, 7 years ago

Milestone: unknown31.0
Owner: changed from danielk to Mark Kendall
Status: newaccepted
Version: UnspecifiedMaster Head

Samples generally play fine in master with VDPAU and VAAPI but are still not handling decoder release properly (just a concurrent resource issue).

NVDEC falls over however - because it does not like concurrent decoders.

comment:10 by Mark Kendall, 7 years ago

Resolution: Fixed
Status: acceptedclosed

Stream changes are now handled *relatively* seemlessly by VAAPI, VDPAU and NVDEC for both decode only and direct rendering.

MMAL/V4L2 appear to be ok but not tested extensively.

I can't test VideoToolBox at the moment.

MediaCodec is the problem child and is still a work in progress.

Resolution changes are firmly on my radar and form part of my normal testing - so closing - given this ticket is 8 years old:)

Note: See TracTickets for help on using tickets.