lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Wed, 07 Mar 2007 23:44:52 +0100
From:	Takashi Iwai <tiwai@...e.de>
To:	Michal Piotrowski <michal.k.k.piotrowski@...il.com>
Cc:	Andrew Morton <akpm@...ux-foundation.org>,
	Randy Cushman <rcushman_linux@...thlink.net>,
	Jaroslav Kysela <perex@...e.cz>, alsa-devel@...a-project.org,
	linux-kernel@...r.kernel.org,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	Adrian Bunk <bunk@...sta.de>
Subject: Re: mm snapshot broken-out-2007-03-05-02-22.tar.gz uploaded

At Wed, 07 Mar 2007 23:22:36 +0100,
Michal Piotrowski wrote:
> 
> Hi Takashi,
> 
> Takashi Iwai napisaƂ(a):
> > At Wed, 7 Mar 2007 11:50:24 -0800,
> > Andrew Morton wrote:
> >> On Wed, 7 Mar 2007 20:41:30 +0100 "Michal Piotrowski" <michal.k.k.piotrowski@...il.com> wrote:
> >>
> >>>>> 831466f4ad2b5fe23dff77edbe6a7c244435e973 is first bad commit
> >>>>> commit 831466f4ad2b5fe23dff77edbe6a7c244435e973
> >>>>> Author: Randy Cushman <rcushman_linux@...thlink.net>
> >>>>> Date:   Tue Dec 19 18:42:16 2006 +0100
> >>>>>
> >>>>>     [ALSA] ac97 - fix microphone and line_in selection logic
> >>>>>
> >>>>>     This patch fixes the Microphone and LINE_IN select logic for
> >>>>>     Analog Devices surround codecs with shared jacks.  The existing
> >>>>>     code can never utilize the shared jacks for Microphone and LINE_IN
> >>>>>     due to the reversed jack selection logic.  The patched code
> >>>>>     correctly selects the shared jack for input if the 'Channel Mode'
> >>>>>     selector does not specify that the jack is to be used for output.
> >>>>>     Specifically, in '2ch' mode the Center/LFE jack is used for
> >>>>>     microphone input and the Surround jack is used for LINE_IN,
> >>>>>     in '4ch' mode the Center/LFE jack is used for microphone input
> >>>>>     and the Surround jack is used for output, and in '6ch' mode
> >>>>>     both jacks are used for output.
> >>>>>
> >>>>>     Signed-off-by: Randy Cushman <rcushman_linux@...thlink.net>
> >>>>>     Signed-off-by: Takashi Iwai <tiwai@...e.de>
> >>>>>     Signed-off-by: Jaroslav Kysela <perex@...e.cz>
> >>>>>
> >>>> Is 2.6.21-rc3 similarly broken?
> >>> Yes.
> >>>
> >> OK.  Can you please confirm that applying the below fixes 2.6.21-rc3?
> >>
> >> If so, I'll queue it up for application in a week or two if there is no
> >> progress on this.
> > 
> > [sorry for the late interrupt - I'm just back from a vacation]
> > 
> > Could you postpone reverting for a while?
> > I guess this possible regression is limited only to a certain type of
> > ac97 codec chip.  The patch applied there was a fix for many codec
> > chips.
> > 
> > Which codec chip are you using specifically?  Please check
> > /proc/asound/card0/codec97#0/* file(s).
> 
> 0-0/0: Analog Devices AD1985

OK, that's good to know.

> 	control.43 {
> 		comment.access 'read write'
> 		comment.type BOOLEAN
> 		comment.count 1
> 		iface MIXER
> 		name 'Spread Front to Surround and Center/LFE'
> 		value true
> 	}

So, you're playing two-channel samples and get no output from surround
and CLFE?  Or, does it happen if you play a real 5.1 channel file?
(for example, try speaker-test -c6)

Also, it'd be helpful if you compare
/proc/asound/card0/codec97#0/ac97#0-0+regs file before and after the
patch.  This reveals which register bits differ actually.


thanks,

Takashi
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ