Modify
Warning Please read the Ticket HowTo before creating or commenting on a ticket. Failure to do so may cause your ticket to be rejected or result in a slower response.

Opened 6 years ago

Closed 6 years ago

Last modified 3 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@… 6 years ago.
mythtv_alsa_non_mmap_fallback.patch (3.1 KB) - added by wberrier@… 6 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@… 6 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 6 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 6 years ago by wberrier@…

Changed 6 years ago by wberrier@…

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

comment:2 Changed 6 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 6 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 6 years ago by anonymous

Was this patch merged into trunk?

comment:4 Changed 6 years ago by wberrier@…

No, this patch has not been committed.

comment:5 Changed 6 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 6 years ago by mythtv-bugs@…

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

comment:7 Changed 6 years ago by stuartm

  • Milestone changed from unknown to 0.21

comment:8 Changed 6 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 6 years ago by danielk

  • Owner changed from ijr to danielk
  • Version changed from 0.20 to head

comment:10 Changed 6 years ago by danielk

  • Resolution set to fixed
  • Status changed from new to closed

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

comment:11 Changed 6 years ago by anonymous

comment:12 Changed 6 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 6 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 6 years ago by hasuf@…

filed a new ticket, #4225 for the backport.

comment:15 Changed 5 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 5 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 3 years ago by stuartm

  • Ticket locked set

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'new'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.