Opened 16 years ago
Closed 15 years ago
#8134 closed defect (Unverified)
internal dvd player: pcm_s16be improperly detected as pcm_dvd audio codec.
| Reported by: | skamithi | Owned by: | Janne Grunau |
|---|---|---|---|
| Priority: | minor | Milestone: | unknown |
| Component: | MythTV - DVD Playback | Version: | head |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description (last modified by )
dvds with audio using a mix of the PCM_DVD and PCM_S16BE codecs play really fast because AvFormatDecoder is using the wrong sample_fmt.
Skipping back or forwards makes no difference. If you play a single VOB file, it plays just fine.
Not sure when this problem started, but the bug is in trunk. may be related to the latest ffmpeg sync
Working on a fix.
Attachments (5)
Change History (19)
comment:1 by , 16 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
by , 16 years ago
| Attachment: | ticket_8134_fix.patch added |
|---|
comment:4 by , 16 years ago
Replying to skamithi:
ffmpeg first detects PCM_DVD audio codec in some dvd streams, and soon after in the same stream detects PCM_S16BE codec. this patch generates the correct avcodec when this happens to prevent either fast dvd playback or no sound. #21891 therefore becomes unnecessary. patch also revert #21891
comment:5 by , 16 years ago
submitted patch to ffmpeg group. Waiting for feedback before applying whatever amended patch the ffmpeg folks suggest.
by , 16 years ago
| Attachment: | afd_based_solution_to_8134.diff added |
|---|
ffmpeg alias suggested not changing the audio codec on the fly. this approach uses AvformatDecoder calls to avoid generating an audio codec for the stream when channels == 0, and if an audio stream with no codec is found, call scanstreams to generate the correct stream. made it dvd specific and pcm_dvd/pcm_s16be specific so it doesn't affect detection of other audio codecs.
by , 16 years ago
| Attachment: | afd_based_solution_to_8134_v2.diff added |
|---|
make scanstream check a little more broad, so expand to catch any audio codec with 1 channel or more.
by , 16 years ago
| Attachment: | afd_based_solution_to_8134_v3.diff added |
|---|
add qmutexlocker around call to scanstreams to prevent ffmpeg related crashes.
comment:6 by , 16 years ago
| Component: | MythTV - General → MythTV - DVD Playback |
|---|
by , 16 years ago
| Attachment: | ffmpeg_based_fix_for_8134.diff added |
|---|
ffmpeg based fix. change the way streams changed is called. this ensure that dvd audio streams are properly detected.
comment:7 by , 16 years ago
| Summary: | internal dvd player: fast dvd playback (not related to #7067) → internal dvd player: pcm_s16be improperly detected as pcm_dvd audio codec. |
|---|
comment:8 by , 16 years ago
comment:9 by , 15 years ago
janneg: go to http://38.114.169.249/JM_VIDEO.tar.bz2 to get the stripped iso that shows the problem. audio transitions from AC3 to PCM_S16BE in the dvd menu intros.
If you find a better way to fix this, that would be awesome. thx.
comment:10 by , 15 years ago
comment:11 by , 15 years ago
| Owner: | changed from to |
|---|
Stanley taking a break, reassigning to Janne for lack of anyone better to look at it.
comment:12 by , 15 years ago
| Status: | assigned → infoneeded |
|---|
Stanley, are you available to check whether this is now fixed? Your sample is no longer available and Mark's recent changes to the DVD audio stream change code may have fixed this.
comment:13 by , 15 years ago
| Milestone: | 0.24 → unknown |
|---|
comment:14 by , 15 years ago
| Resolution: | → Unverified |
|---|---|
| Status: | infoneeded → closed |
ffmpeg first detects PCM_DVD audio codec in some dvd streams, and soon after in the same stream detects PCM_S16BE codec. this patch generates the correct avcodec when this happens to prevent either fast dvd playback or no sound. #21891 therefore becomes unnecessary. patch also revert #21891