From: James Bottomley <James.Bottomley@HansenPartnership.com>
To: Mikael Pettersson <mikpe@it.uu.se>
Cc: Stefan Richter <stefanr@s5r6.in-berlin.de>,
FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>,
crmafra2@gmail.com, linux-kernel@vger.kernel.org,
htejun@gmail.com, jens.axboe@oracle.com,
linux-ide@vger.kernel.org, linux-scsi@vger.kernel.org
Subject: Re: [BUG] 2.6.25-rc2-git3 broke cdrecord
Date: Mon, 03 Mar 2008 08:51:53 -0600 [thread overview]
Message-ID: <1204555913.3043.1.camel@localhost.localdomain> (raw)
In-Reply-To: <18379.53492.171481.255371@harpo.it.uu.se>
On Mon, 2008-03-03 at 11:20 +0100, Mikael Pettersson wrote:
> Mikael Pettersson writes:
> > Stefan Richter writes:
> > > FUJITA Tomonori wrote:
> > > > On Mon, 3 Mar 2008 02:48:23 +0100 (CET)
> > > > Stefan Richter <stefanr@s5r6.in-berlin.de> wrote:
> > > >> CD burning still fails for me via ide-cd, although in a
> > > >> different way than without the patch. (Without the patch, ide-cd also
> > > >> fails like described by you for SATA.)
> > > >>
> > > >> With the patch, the failure is as follows. I don't know if this is
> > > >> related to the data length arithmetic or something entirely different;
> > > >> all I learned so far is that 2.6.24 works. Application log from
> > > >> 2.6.25-rc3 + Tejun's patch + Gentoo's cdrkit-1.1.6 + k3b:
> > > >
> > > > Can you try the following patch?
> > > >
> > > > http://lkml.org/lkml/2008/3/2/218
> > >
> > > Fails just like with Tejun's patch, alas.
> >
> > At this point I must confess I feel very uneasy about this situation.
> >
> > Dare I suggest the perhaps hereditary solution of reverting these
> > changes and doing them right after 2.6.25 final?
>
> Doh! s/hereditary/heretical/
Unfortunately not. This patch set was first introduced to fix a panic
in the SAS drivers that support SATAPI because they couldn't cope with
the way libata mangled the sg list.
We know what the problem is, we just have to find an acceptable way of
fixing it.
James
next prev parent reply other threads:[~2008-03-03 14:52 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-02 20:53 [BUG] 2.6.25-rc2-git3 broke cdrecord Mikael Pettersson
2008-03-02 23:34 ` Carlos R. Mafra
2008-03-03 0:34 ` Mikael Pettersson
2008-03-03 1:48 ` Stefan Richter
2008-03-03 8:26 ` FUJITA Tomonori
2008-03-03 9:08 ` Stefan Richter
2008-03-03 9:45 ` Mikael Pettersson
2008-03-03 10:20 ` Mikael Pettersson
2008-03-03 10:29 ` Tejun Heo
2008-03-03 14:51 ` James Bottomley [this message]
2008-03-03 17:39 ` Stefan Richter
2008-03-03 18:00 ` Boris Petkov
2008-03-03 21:41 ` Stefan Richter
2008-03-03 0:04 ` James Bottomley
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=1204555913.3043.1.camel@localhost.localdomain \
--to=james.bottomley@hansenpartnership.com \
--cc=crmafra2@gmail.com \
--cc=fujita.tomonori@lab.ntt.co.jp \
--cc=htejun@gmail.com \
--cc=jens.axboe@oracle.com \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=mikpe@it.uu.se \
--cc=stefanr@s5r6.in-berlin.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox