Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755814AbZLYQFV (ORCPT ); Fri, 25 Dec 2009 11:05:21 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1754193AbZLYQFU (ORCPT ); Fri, 25 Dec 2009 11:05:20 -0500 Received: from cantor2.suse.de ([195.135.220.15]:49299 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753150AbZLYQFU (ORCPT ); Fri, 25 Dec 2009 11:05:20 -0500 Date: Fri, 25 Dec 2009 17:05:16 +0100 Message-ID: From: Takashi Iwai To: Maciej Rutecki Cc: Sergei Trofimovich , linux-kernel@vger.kernel.org, =?UTF-8?B?w4lyaWM=?= Piel Subject: Re: intel hda regression: 2.6.33-rc{1,2}: hd-audio0 eats 25% CPU (according to top) In-Reply-To: <8db1092f0912250731u46637a55lccb77e6cdb376a57@mail.gmail.com> References: <20091225122107.357d7102@mosly> <20091225162528.5dbbbea0@mosly> <8db1092f0912250731u46637a55lccb77e6cdb376a57@mail.gmail.com> User-Agent: Wanderlust/2.15.6 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.7 Emacs/23.1 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3255 Lines: 75 At Fri, 25 Dec 2009 16:31:58 +0100, Maciej Rutecki wrote: > > 2009/12/25 Sergei Trofimovich : > > > Done: > > > > d56757abc11a21996d9839c0d4e3b2c3666cd318 is the first bad commit > > commit d56757abc11a21996d9839c0d4e3b2c3666cd318 > > Author: Takashi Iwai > > Date:   Wed Nov 18 08:00:14 2009 +0100 > > > >    ALSA: hda - Replace the rest of jack-detections with snd_hda_jack_detect() > > > >    Signed-off-by: Takashi Iwai > > > > :040000 040000 46040881f7bc4b662a784ac66a6482c2a5b3dca5 6a76aa47a3d0cccad0c8a3ae62cb63ecbdd9b44d M      sound > > > > Bisect log: > > git bisect start '--' 'sound/pci/hda' > > # bad: [55639353a0035052d9ea6cfe4dde0ac7fcbb2c9f] Linux 2.6.33-rc1 > > git bisect bad 55639353a0035052d9ea6cfe4dde0ac7fcbb2c9f > > # good: [22763c5cf3690a681551162c15d34d935308c8d7] Linux 2.6.32 > > git bisect good 22763c5cf3690a681551162c15d34d935308c8d7 > > # good: [123c07aeddd71fbb295842a8c19866e780b9a100] ALSA: hda_intel: Digital PC Beep - change behaviour for input layer > > git bisect good 123c07aeddd71fbb295842a8c19866e780b9a100 > > # good: [123c07aeddd71fbb295842a8c19866e780b9a100] ALSA: hda_intel: Digital PC Beep - change behaviour for input layer > > git bisect good 123c07aeddd71fbb295842a8c19866e780b9a100 > > # bad: [0b587fc4d35afb1bc0fc3d890084bb14c78372dc] ALSA: hda: Fix max PCM level to 0 dB for Fujitsu-Siemens laptops using CX2054 > > 9 (Venice) > > git bisect bad 0b587fc4d35afb1bc0fc3d890084bb14c78372dc > > # good: [23ccc2bd246a5bdb1ac03dc9040a0585c1890ef3] ALSA: intelhdmi - export monitor-presence and ELD-valid status > > git bisect good 23ccc2bd246a5bdb1ac03dc9040a0585c1890ef3 > > # bad: [d56757abc11a21996d9839c0d4e3b2c3666cd318] ALSA: hda - Replace the rest of jack-detections with snd_hda_jack_detect() > > git bisect bad d56757abc11a21996d9839c0d4e3b2c3666cd318 > > # good: [848de598eef9603d6f2c174f90fded4e63ac5e23] ALSA: intelhdmi - sticky infoframe > > git bisect good 848de598eef9603d6f2c174f90fded4e63ac5e23 > > # good: [81bf31e2d0a6a9f5d83da0a757f8ca03db908162] ALSA: intelhdmi - sticky channel count > > git bisect good 81bf31e2d0a6a9f5d83da0a757f8ca03db908162 > > # good: [83d605fd63e704419ccb92d48b735c6890ce3d6a] ALSA: hda - show EPSS capability in proc > > git bisect good 83d605fd63e704419ccb92d48b735c6890ce3d6a > > > > Sorry, was unable to revert that commit on top of current 2.6.33-rc2 and verify it's guilty. > > > >> Takashi > > > > -- > > > >  Sergei > > > > Good work. I already find some time to test, and I have the same > bisection's result, so I think that this commit cause problem. OK, thanks for quick tests. It's a bit surprising that this resulted in the high CPU usage instead of erroneous behavior. This should be rather a bug of codec chip... Anyway, Segei, which codec is on your machine? Is it Analog Device one? IIRC, the one's on Maciej and Eric machines are AD codecs. If so, we can just revert the changes to patch_analog.c in that commit. thanks, Takashi -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/