Opened 12 years ago

Closed 12 years ago

Last modified 10 years ago

#3598 closed defect (fixed)

MythFrontend doesn't work with some sound systems

Reported by: Jeremy Nickurak <mythtv-bugs@…> Owned by: danielk
Priority: minor Milestone: 0.21
Component: mythtv Version: head
Severity: medium Keywords:
Cc: Ticket locked: yes

Description

2007-06-11 10:19:20.441 AudioOutput? Error: Access type not available: Invalid argument

After some discussion on IRC in #pulseaudio :

<mezcalero> the problem is here

http://svn.mythtv.org/svn/trunk/mythtv/libs/libmyth/audiooutputalsa.cpp it assumes that SND_PCM_ACCESS_MMAP_INTERLEAVED is available that assumption is bogus only normal PCI hw has mmap it's the same issue why we have trouble to support quake with padsp people just assume that mmap is available everywhere they should stop doing that

<mezcalero> atrus: you might want to file a bug and tell them that they need to fix their code <mezcalero> otherweise they will always have trouble with software backends

or with a little bit more exotic audio hardware and i really wonder why they do that in the first place.

Attachments (3)

mythtv_pulseaudio_alsa_fix.patch (1.1 KB) - added by wberrier@… 12 years ago.
mythtv_alsa_non_mmap_fallback.patch (3.1 KB) - added by wberrier@… 12 years ago.
Fall back to non-mmap if mmap isn't available
mythtv_alsa_non_mmap_fallback.2.patch (3.2 KB) - added by wberrier@… 12 years ago.
Fall back to non-mmap if mmap isn't available (same patch, but include full path to changes files in diff)

Download all attachments as: .zip

Change History (20)

comment:1 Changed 12 years ago by wberrier@…

myth works fine for me without MMAP (although I didn't try digital out).

Looking at the history, MMAP has been used from the beginning of the alsa support.

Any reason this simple patch shouldn't be committed? Native alsa continues to work, and it allows myth to be used with the PulseAudio? alsa-plugin.

Changed 12 years ago by wberrier@…

Changed 12 years ago by wberrier@…

Fall back to non-mmap if mmap isn't available

comment:2 Changed 12 years ago by wberrier@…

Here's an updated patch, mythtv_alsa_non_mmap_fallback.patch, so that people who really want mmap will have no objections to this fix.

Changed 12 years ago by wberrier@…

Fall back to non-mmap if mmap isn't available (same patch, but include full path to changes files in diff)

comment:3 Changed 12 years ago by anonymous

Was this patch merged into trunk?

comment:4 Changed 12 years ago by wberrier@…

No, this patch has not been committed.

comment:5 Changed 12 years ago by coling

Just tried this patch and it fixed my pulseaudio/myth problems, so unless there is any specific reason not to, can someone apply it to trunk please?

comment:6 Changed 12 years ago by mythtv-bugs@…

Likewise here, it's made my usage of myth substantially happier.

comment:7 Changed 12 years ago by stuartm

Milestone: unknown0.21

comment:8 Changed 12 years ago by anonymous

I would also like to see this change commited, as it would make MythTV work with PulseAudio? without problems :)

comment:9 Changed 12 years ago by danielk

Owner: changed from Isaac Richards to danielk
Version: 0.20head

comment:10 Changed 12 years ago by danielk

Resolution: fixed
Status: newclosed

(In [14785]) Fixes #3598. Applies patch from Wade Berrier to fall back to RW_INTERLEAVED ALSA access if MMAP access fails.

comment:11 Changed 12 years ago by anonymous

comment:12 Changed 12 years ago by amessina@…

Could this be ported to 0.20.x-fixes branch for those (many) who use Axel Thimm's RPMs in Fedora 8?

comment:13 Changed 12 years ago by wberrier@…

I would also like to see this committed to the 'fixes' branch so it would be included in packman's mythtv packages.

comment:14 Changed 12 years ago by hasuf@…

filed a new ticket, #4225 for the backport.

comment:15 Changed 11 years ago by pembo13@…

What is the status of this? I have mythtv-frontend-0.21-192.fc9.i386 from ATrpms and I can't hear a thing on my F9 system with PulseAudio?. I'm trying to determine if the patch simply isn't in, if it's not in the RPM that I have, or if it's a problem local to my machine.

comment:16 Changed 11 years ago by wberrier@…

This has been committed, but you need to set myth to use alsa [1], and you need to have the pulseaudio alsa plugin installed.

  1. http://www.mythtv.org/wiki/index.php/ALSA

comment:18 Changed 10 years ago by stuartm

Ticket locked: set
Note: See TracTickets for help on using tickets.