All of lore.kernel.org
 help / color / mirror / Atom feed
From: Konstantin Khlebnikov <khlebnikov@openvz.org>
To: Takashi Iwai <tiwai@suse.de>,
	"alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>
Cc: "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Jaroslav Kysela <perex@perex.cz>,
	David Henningsson <david.henningsson@canonical.com>
Subject: Re: [ALSA hda 3.4-rc] suspend2ram regression in snd_hda_intel
Date: Sun, 06 May 2012 12:32:14 +0400	[thread overview]
Message-ID: <4FA6370E.9000702@openvz.org> (raw)
In-Reply-To: <4FA38371.9020205@openvz.org>

Konstantin Khlebnikov wrote:
> Mainline commit 785f857d1cb0856b612b46a0545b74aa2596e44a
> ("ALSA: hda - Set codec to D3 forcibly even if not used")
> breaks sound for my notebook (thinkpad x220),
> looks like codec cannot wake from power-save state after resume.
> After reverting looks like all works fine.
>
> how to reproduce:
> * unplug the power cord (to enable hda power safe mode)
> * play something and stop
> * wait until the background noise disappears (codec went into power-save mode)
> * suspend
> * resume
> * play something again and hear the silence
> --
> 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/

Bump

BTW, I don't see any locks around hda power-save code, for example  hda_power_work() reads
codec->power_on and codec->power_count without any locks. This seems racy.

  reply	other threads:[~2012-05-06  8:32 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-05-04  7:21 [ALSA hda 3.4-rc] suspend2ram regression in snd_hda_intel Konstantin Khlebnikov
2012-05-06  8:32 ` Konstantin Khlebnikov [this message]
2012-05-06 10:37   ` Takashi Iwai
2012-05-06 10:37     ` Takashi Iwai
2012-05-08  7:13     ` Konstantin Khlebnikov
2012-05-08  8:11       ` Takashi Iwai
2012-05-08  8:11         ` Takashi Iwai
2012-05-08  8:49         ` Takashi Iwai
2012-05-08  8:49           ` Takashi Iwai
2012-05-08 13:05           ` Konstantin Khlebnikov
2012-05-08 13:08             ` [PATCH bugfix v3.4] ALSA: hda - fix "silence after S3" regression Konstantin Khlebnikov
2012-05-08 13:25               ` Takashi Iwai
2012-05-08 13:25                 ` Takashi Iwai
2012-05-08 13:38                 ` Konstantin Khlebnikov
2012-05-08 14:29                   ` Takashi Iwai
2012-05-08 14:29                     ` Takashi Iwai

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4FA6370E.9000702@openvz.org \
    --to=khlebnikov@openvz.org \
    --cc=alsa-devel@alsa-project.org \
    --cc=david.henningsson@canonical.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=perex@perex.cz \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.