From patchwork Wed Oct 4 19:44:00 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tanu Kaskinen X-Patchwork-Id: 9985431 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id A9AFD60586 for ; Wed, 4 Oct 2017 19:45:31 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 9A2BB28BDC for ; Wed, 4 Oct 2017 19:45:31 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 8CEAB28BE1; Wed, 4 Oct 2017 19:45:31 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-1.6 required=2.0 tests=BAYES_00,DKIM_SIGNED, RCVD_IN_DNSWL_NONE, RCVD_IN_SBL, T_DKIM_INVALID autolearn=no version=3.3.1 Received: from alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 5CFAF28BDC for ; Wed, 4 Oct 2017 19:45:29 +0000 (UTC) Received: from alsa0.perex.cz (localhost [127.0.0.1]) by alsa0.perex.cz (Postfix) with ESMTP id B43E526741E; Wed, 4 Oct 2017 21:45:27 +0200 (CEST) X-Original-To: alsa-devel@alsa-project.org Delivered-To: alsa-devel@alsa-project.org Received: by alsa0.perex.cz (Postfix, from userid 1000) id 9CC6D26741F; Wed, 4 Oct 2017 21:45:25 +0200 (CEST) Received: from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com [66.111.4.25]) by alsa0.perex.cz (Postfix) with ESMTP id EA490267419 for ; Wed, 4 Oct 2017 21:45:20 +0200 (CEST) Received: from compute5.internal (compute5.nyi.internal [10.202.2.45]) by mailout.nyi.internal (Postfix) with ESMTP id EB3D620EB0; Wed, 4 Oct 2017 15:45:18 -0400 (EDT) Received: from frontend1 ([10.202.2.160]) by compute5.internal (MEProxy); Wed, 04 Oct 2017 15:45:18 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:date:from:message-id:subject:to :x-me-sender:x-me-sender:x-sasl-enc:x-sasl-enc; s=fm1; bh=uxoevG TcM+jDY/O3XN28ov0cWRhq5D+4op68fGd+Rjo=; b=TVz8Zjg30qFTTKHmxzALrd EeUF8jQK+1qnkCuzWmSW0cSnsdmWPRMcGn9CdOekcVb+iQiUpwrKNyXFKvHEgp/y 7UXIODQgWwpeIBmEH2iV+ro3fnHQtgA1rj2c9zF5kaejrkBjF1ll5kQaBCYYxOx5 73zXwVEKIDzvlH7EuzUZ37Zumv/eGJMNn4E6ae/3N7FElS+J5AxvnwD87p0NukSn OGT3wXuzOXn2AJxjs5LV1z8VpGhBLp73PiuMGopSqJ0wSJmLCeCRHARVOeQ/aafj azYmyGF1ywR72c5y4cYs4bcrZ1jlRFreV6lsG34wXNFbARESyXLZ0S1gRrbRAhTg == X-ME-Sender: X-Sasl-enc: e7wvZEOVSApzsmxgWiefjq2m3unHN53dy39T9zOm+3ew 1507146318 Received: from localhost.localdomain (unknown [192.40.95.39]) by mail.messagingengine.com (Postfix) with ESMTPA id 457637FA6A; Wed, 4 Oct 2017 15:45:18 -0400 (EDT) From: Tanu Kaskinen To: alsa-devel@alsa-project.org Date: Wed, 4 Oct 2017 22:44:00 +0300 Message-Id: <20171004194400.14401-1-tanuk@iki.fi> X-Mailer: git-send-email 2.14.2 Cc: Takashi Iwai Subject: [alsa-devel] [PATCH] conf: HdmiLpeAudio: remove the "front" pcm definition X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , MIME-Version: 1.0 Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org X-Virus-Scanned: ClamAV using ClamSMTP PulseAudio assumes that the "front" pcm device always refers to an analog device, not HDMI. While that assumption is not really valid, the reality is that without that assumption PulseAudio can't know whether "front" and "hdmi" refer to a different or the same device. The HDMI LPE driver doesn't allow audio streaming while the HDMI cable is unplugged, so PulseAudio has to know when it's plugged in and when it's not. If both "front" and "hdmi" devices exist, PulseAudio will notice that HDMI is unplugged, but it doesn't know that "front" refers to the same device, and PulseAudio will try to use the "front" device with bad consequences. The kernel driver's refusal to stream any audio makes PulseAudio enter an infinite loop and then the kernel kills PulseAudio, because it consumes too much CPU time in a realtime thread. While the looping in PulseAudio could probably be fixed, that wouldn't change the fact that PulseAudio thinks that there is an analog device. I believe it's best to avoid having the same device as both "front" and "hdmi" in alsa-lib. I removed also the surround configuration includes. I don't think they had any effect anyway, so I wonder why they were there in the first place. BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=100488 --- src/conf/cards/HdmiLpeAudio.conf | 24 ------------------------ 1 file changed, 24 deletions(-) diff --git a/src/conf/cards/HdmiLpeAudio.conf b/src/conf/cards/HdmiLpeAudio.conf index 9fa30da0..a1e493da 100644 --- a/src/conf/cards/HdmiLpeAudio.conf +++ b/src/conf/cards/HdmiLpeAudio.conf @@ -2,30 +2,6 @@ # Configuration for the Intel HDMI/DP LPE audio # - - -HdmiLpeAudio.pcm.front.0 { - @args [ CARD ] - @args.CARD { - type string - } - type softvol - slave.pcm { - type hw - card $CARD - } - control { - name "PCM Playback Volume" - card $CARD - } -} - - - - - - - HdmiLpeAudio.pcm.hdmi.0 {