All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Benny Sjostrand <gorm@cucumelo.org>
Cc: alsa-devel@lists.sourceforge.net
Subject: Re: [PATCH] cs46xx update SMP fixes
Date: Thu, 22 Aug 2002 15:40:43 +0200	[thread overview]
Message-ID: <s5h65y3t3j8.wl@alsa2.suse.de> (raw)
In-Reply-To: <3D64E811.30005@cucumelo.org>

At Thu, 22 Aug 2002 15:33:05 +0200,
Benny Sjostrand wrote:
> 
> >
> >
> >could you rewrite mutex with spinlock if not too long held?
> >
> >you cannot use mutex on trigger since this could be called from
> >interrupt handlers, too (in order to stop the stream when xrun is
> >detected).
> >
> I believe that the critical sections are too long and will lose to much 
> performance
> on a SMP machines that's why i've choose the mutex instead of spin_locks.
> However, that the trigger functions could be called from interrupt 
> context is new to me,
> thanks to pointing it out.
> Mutexes are used in _trigger function in current CVS too, not directly 
> in trigger function,
> but in _link and _unlink function called by _trigger, and SMP is broken, 
> so in  some
> way we need to fix it.
> 
> I'll try to come with another solution, if you have any suggestion how 
> to solve it, then great.

how about to do pcm_link/unlink in open/close or prepare callback?
they are definitely not called from interrupts.


Takashi


-------------------------------------------------------
This sf.net email is sponsored by: OSDN - Tired of that same old
cell phone?  Get a new here for FREE!
https://www.inphonic.com/r.asp?r=sourceforge1&refcode1=vs3390

  reply	other threads:[~2002-08-22 13:40 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-08-22  9:59 [PATCH] cs46xx update SMP fixes Benny Sjostrand
2002-08-22 12:44 ` Takashi Iwai
2002-08-22 13:33   ` Benny Sjostrand
2002-08-22 13:40     ` Takashi Iwai [this message]
2002-08-22 14:13       ` Benny Sjostrand
2002-08-22 14:27         ` Takashi Iwai
     [not found]           ` <3D652BA1.8010600@cucumelo.org>
2002-08-23  8:21             ` 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=s5h65y3t3j8.wl@alsa2.suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@lists.sourceforge.net \
    --cc=gorm@cucumelo.org \
    /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.