Attachment 69661 Details for Bug 56735 – patch 3/3: Write to the wakeup pipe unconditionally

[patch] patch 3/3: Write to the wakeup pipe unconditionally

3.mbox (text/plain), 5.29 KB, created by Tanu Kaskinen on 2012-11-07 12:42:50 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Tanu Kaskinen

Created: 2012-11-07 12:42:50 UTC

Size: 5.29 KB

patch

obsolete

>From pulseaudio-discuss-bounces+tanuk=iki.fi@lists.freedesktop.org Mon Apr  2 08:03:41 2012
>Return-Path: <pulseaudio-discuss-bounces+tanuk=iki.fi@lists.freedesktop.org>
>Received: from compute1.internal (compute1.nyi.mail.srv.osa [10.202.2.41])
> by sloti30d2p3 (Cyrus git2.5+0-git-fastmail-7802) with LMTPA; Mon, 02 Apr
> 2012 08:03:41 -0400
>X-Sieve: CMU Sieve 2.4
>X-Spam-score: 0.0
>X-Spam-hits: BAYES_00 -1.9, RCVD_IN_DNSWL_MED -2.3, T_RP_MATCHES_RCVD
> -0.01, BAYES_USED global, SA_VERSION 3.3.1
>X-Spam-source: IP='131.252.210.177', Host='gabe.freedesktop.org',
> Country='US', FromHeader='com', MailFrom='org', XOriginatingCountry='UNK'
>X-Spam-charsets: plain='us-ascii'
>X-Resolved-to: tanuk@fastmail.fm
>X-Delivered-to: tanuk@fastmail.fm
>X-Mail-from: pulseaudio-discuss-bounces+tanuk=iki.fi@lists.freedesktop.org
>Received: from mx2.nyi.mail.srv.osa ([10.202.2.201]) by compute1.internal
> (LMTPProxy); Mon, 02 Apr 2012 08:03:41 -0400
>Received: from ikiaikainen.iki.fi (ikiaikainen.iki.fi [212.16.98.54])
> (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client
> certificate requested) by mx2.nyi.mail.srv.osa (Postfix) with ESMTPS id
> 2594776021F for <tanuk@fastmail.fm>; Mon,  2 Apr 2012 08:03:41 -0400 (EDT)
>Received: from gabe.freedesktop.org (gabe.freedesktop.org
> [131.252.210.177]) by ikiaikainen.iki.fi (8.14.5/8.14.4) with ESMTP id
> q32C3bVp021532 for <tanuk@iki.fi>; Mon, 2 Apr 2012 15:03:38 +0300 (EEST)
>Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by
> gabe.freedesktop.org (Postfix) with ESMTP id D5F2A9E710 for <tanuk@iki.fi>;
> Mon,  2 Apr 2012 05:03:36 -0700 (PDT)
>X-Original-To: pulseaudio-discuss@lists.freedesktop.org
>X-Remote-Delivered-To: pulseaudio-discuss@lists.freedesktop.org
>Received: from smtp1.digia.com (smtp1.digia.com [82.118.214.156]) by
> gabe.freedesktop.org (Postfix) with ESMTP id 75678A02AC for
> <pulseaudio-discuss@lists.freedesktop.org>; Mon,  2 Apr 2012 05:01:28 -0700
> (PDT)
>Received: from smtp1.digia.com (unknown [127.0.0.1]) by IMSVA80 (Postfix)
> with ESMTP id 7F299370044 for <pulseaudio-discuss@lists.freedesktop.org>;
> Mon,  2 Apr 2012 15:01:27 +0300 (EEST)
>Received: from IT-EXCAS01-HKI.it.local (unknown [10.9.52.70]) by
> smtp1.digia.com (Postfix) with ESMTP id 72A3837003D for
> <pulseaudio-discuss@lists.freedesktop.org>; Mon,  2 Apr 2012 15:01:27 +0300
> (EEST)
>Received: from IT-W2076.it.local (10.120.150.85) by IT-EXCAS01-HKI.it.local
> (10.9.52.75) with Microsoft SMTP Server id 14.1.355.2; Mon, 2 Apr 2012
> 15:01:26 +0300
>From: Tanu Kaskinen <tanu.kaskinen@digia.com>
>To: <pulseaudio-discuss@lists.freedesktop.org>
>Date: Mon, 2 Apr 2012 15:01:05 +0300
>Message-ID: <1333368066-32343-5-git-send-email-tanu.kaskinen@digia.com>
>X-Mailer: git-send-email 1.7.8
>In-Reply-To: <1333368066-32343-1-git-send-email-tanu.kaskinen@digia.com>
>References: <1333368066-32343-1-git-send-email-tanu.kaskinen@digia.com>
>MIME-Version: 1.0
>X-Originating-IP: [10.120.150.85]
>Subject: [pulseaudio-discuss] [PATCH 4/5] mainloop: Write to the wakeup
> pipe unconditionally when waking up the mainloop.
>X-BeenThere: pulseaudio-discuss@lists.freedesktop.org
>X-Mailman-Version: 2.1.11
>Precedence: list
>Reply-To: General PulseAudio Discussion <pulseaudio-discuss@lists.freedesktop.org>
>List-Id: General PulseAudio Discussion
> <pulseaudio-discuss.lists.freedesktop.org>
>List-Unsubscribe:
> <http://lists.freedesktop.org/mailman/options/pulseaudio-discuss>,
> <mailto:pulseaudio-discuss-request@lists.freedesktop.org?subject=unsubscribe>
>List-Archive: <http://lists.freedesktop.org/archives/pulseaudio-discuss>
>List-Post: <mailto:pulseaudio-discuss@lists.freedesktop.org>
>List-Help:
> <mailto:pulseaudio-discuss-request@lists.freedesktop.org?subject=help>
>List-Subscribe:
> <http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss>,
> <mailto:pulseaudio-discuss-request@lists.freedesktop.org?subject=subscribe>
>Content-Type: text/plain; charset="us-ascii"
>Sender: pulseaudio-discuss-bounces+tanuk=iki.fi@lists.freedesktop.org
>Errors-To: pulseaudio-discuss-bounces+tanuk=iki.fi@lists.freedesktop.org
>X-Truedomain-Domain: digia.com
>X-Truedomain-SPF: No Record
>X-Truedomain-DKIM: No Signature
>X-Truedomain-ID: 3DF5999212D00620718F791D7DDE80CF
>X-Truedomain: Neutral
>X-Evolution-Source: 1322389393.6514.21@laptop
>Content-Transfer-Encoding: 8bit
>
>If the mainloop is just about to enter polling, but m->state
>is not POLLING yet when some other thread calls
>pa_mainloop_wakeup(), the mainloop will not be woken up.
>It's safe to write to the wakeup pipe at any time, so let's
>just remove the check.
>---
> src/pulse/mainloop.c |    6 ++----
> 1 files changed, 2 insertions(+), 4 deletions(-)
>
>diff --git a/src/pulse/mainloop.c b/src/pulse/mainloop.c
>index 160ba9c..8ac8f06 100644
>--- a/src/pulse/mainloop.c
>+++ b/src/pulse/mainloop.c
>@@ -774,10 +774,8 @@ void pa_mainloop_wakeup(pa_mainloop *m) {
>     char c = 'W';
>     pa_assert(m);
> 
>-    if (m->state == STATE_POLLING) {
>-        pa_write(m->wakeup_pipe[1], &c, sizeof(c), &m->wakeup_pipe_type);
>-        pa_atomic_store(&m->wakeup_requested, TRUE);
>-    }
>+    pa_write(m->wakeup_pipe[1], &c, sizeof(c), &m->wakeup_pipe_type);
>+    pa_atomic_store(&m->wakeup_requested, TRUE);
> }
> 
> static void clear_wakeup(pa_mainloop *m) {
>-- 
>1.7.8
>
>_______________________________________________
>pulseaudio-discuss mailing list
>pulseaudio-discuss@lists.freedesktop.org
>http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss
>

Actions: View | Splinter Review

Attachments on bug 56735: 69659 | 69660 | 69661

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.