All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stefan Richter <stefanr@s5r6.in-berlin.de>
To: Wakko Warner <wakko@animx.eu.org>
Cc: linux-kernel@vger.kernel.org, linux-scsi@vger.kernel.org
Subject: Re: Burning multiple DVDs at one time
Date: Sun, 18 Mar 2012 15:01:27 +0100	[thread overview]
Message-ID: <20120318150127.5f8db40c@stein> (raw)
In-Reply-To: <20120318133059.GB10172@animx.eu.org>

On Mar 18 Wakko Warner wrote:
> Stefan Richter wrote:
> > On Mar 17 Wakko Warner wrote:
> > > Thanks.  I looked at the patch.  I would just like to confirm that I can
> > > patch my 3.0.0 vanilla kernel, compile the sr module, unload the current and
> > > load the patched one without the need to reboot.
> > 
> > Yes, this should be possible.
> > 
> > Oh, I only noticed just know that you also wrote:
> > 
> > > > > The kernel is a vanilla kernel v3.0.0.  (This also happened with 2.6.35)
> > 
> > In 2.6.35, the Big Kernel Lock was still in place there.  That lock
> > behaved differently from a plain mutex --- notably it was released when a
> > thread went to sleep --- so maybe there is more to your problem than just
> > sr_mutex blocking unrelated sr accesses.
> 
> There is a good possibility that I was mistaken on the 2.6.35.
> 
> I manually applied your patch to 3.0.0, compiled, removed sr-mod module and
> inserted the newly compiled one.
> 
> With the vanilla one, I would see the buf % in wodim drop while the fifo was
> full.  With the patch (removing the mutex), buf stayed above 98% and the
> burn speed was 16x at the end (10x avg).
> 
> Seems to have fixed my problem.

Very nice; so there is an almost easy fix.  Now I or somebody else need to
work out the required cdrom_device_info protection on which Arnd and James
commented.
-- 
Stefan Richter
-=====-===-- --== =--=-
http://arcgraph.de/sr/

  reply	other threads:[~2012-03-18 14:01 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-03-15 22:53 Burning multiple DVDs at one time Wakko Warner
2012-03-17  8:56 ` Stefan Richter
2012-03-17 10:53   ` Emmanuel Florac
2012-03-17 10:53     ` Emmanuel Florac
2012-03-17 12:47     ` Stefan Richter
2012-03-17 12:47       ` Stefan Richter
2012-03-17 14:22   ` Wakko Warner
2012-03-17 15:28     ` Stefan Richter
2012-03-18 13:30       ` Wakko Warner
2012-03-18 14:01         ` Stefan Richter [this message]
2012-04-28 16:02           ` Wakko Warner
2012-04-28 16:55             ` Stefan Richter
  -- strict thread matches above, loose matches on Subject: below --
2012-03-09  3:04 Wakko Warner

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=20120318150127.5f8db40c@stein \
    --to=stefanr@s5r6.in-berlin.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=wakko@animx.eu.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.