All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nikolaus Schulz <schulz@macnetix.de>
To: Mauro Carvalho Chehab <mchehab@infradead.org>
Cc: Jonathan Nieder <jrnieder@gmail.com>,
	Oliver Endriss <o.endriss@gmx.de>,
	Andreas Oberritter <obi@linuxtv.org>,
	<linux-media@vger.kernel.org>
Subject: Re: [PATCH] dvb: push down ioctl lock in dvb_usercopy
Date: Tue, 17 Jul 2012 22:15:57 +0200	[thread overview]
Message-ID: <20120717201557.GA18369@pcewns01.macnetix.de> (raw)
In-Reply-To: <1340300655-17696-1-git-send-email-schulz@macnetix.de>

Hi!

I don't want to press or annoy anybody, but may I ask what's the status
of this patch[1]?

On Thu, Jun 21, 2012 at 07:44:15PM +0200, schulz@macnetix.de wrote:
> Since most dvb ioctls wrap their real work with dvb_usercopy, the static mutex
> used in dvb_usercopy effectively is a global lock for dvb ioctls.
> Unfortunately, frontend ioctls can be blocked by the frontend thread for
> several seconds; this leads to unacceptable lock contention.  Mitigate that by
> pushing the mutex from dvb_usercopy down to the individual, device specific
> ioctls.

On our systems, this patch has a great effect in terms of scalability
when operating multiple DVB tuners.

Is it going to be considered?

Thanks,
Nikolaus

[1] http://patchwork.linuxtv.org/patch/12989/

      reply	other threads:[~2012-07-17 20:16 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-21 17:44 [PATCH] dvb: push down ioctl lock in dvb_usercopy schulz
2012-07-17 20:15 ` Nikolaus Schulz [this message]

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=20120717201557.GA18369@pcewns01.macnetix.de \
    --to=schulz@macnetix.de \
    --cc=jrnieder@gmail.com \
    --cc=linux-media@vger.kernel.org \
    --cc=mchehab@infradead.org \
    --cc=o.endriss@gmx.de \
    --cc=obi@linuxtv.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.