From: Bartlomiej Zolnierkiewicz <B.Zolnierkiewicz@elka.pw.edu.pl>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>, Andries Brouwer <aebr@win.tue.nl>
Cc: Jan Niehusmann <jan@gondor.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: IDE bug - was: Re: uncorrectable ext2 errors
Date: Tue, 12 Aug 2003 18:32:48 +0200 [thread overview]
Message-ID: <200308121832.48212.bzolnier@elka.pw.edu.pl> (raw)
In-Reply-To: <1060702567.21160.30.camel@dhcp22.swansea.linux.org.uk>
On Tuesday 12 of August 2003 17:36, Alan Cox wrote:
> On Sul, 2003-08-10 at 23:33, Andries Brouwer wrote:
> > if (drive->addressing == 1) /* 48-bit LBA */
> > return lba_48_rw_disk(drive, rq, (unsigned long long)
> > block); if (drive->select.b.lba) /* 28-bit LBA */ return
> > lba_28_rw_disk(drive, rq, (unsigned long) block); return
> > chs_rw_disk(drive, rq, (unsigned long) block);
> >
> > with checking the size of block.
> > And init_idedisk_capacity() does not check addressing.
>
> It should also issue LBA28 if the size of th range and the end block
> fall under the LBA28 limit because thst saves you valuable I/O time.
Yep, but its other issue.
> Jens had patches for that but I don't know where they went in 2.6
Jens and me agreed that they should be included after taskfile IO
is integrated as the only IO path. Otherwise there are too many
corner cases to deal with.
--bartlomiej
next prev parent reply other threads:[~2003-08-12 16:32 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-08-06 15:03 Jan Niehusmann
2003-08-06 17:06 ` Andreas Dilger
2003-08-06 18:38 ` Jan Niehusmann
2003-08-06 18:52 ` Mike Fedyk
2003-08-06 19:50 ` Jan Niehusmann
2003-08-07 11:06 ` Jan Niehusmann
2003-08-07 12:27 ` LBA48 on Promise 20265 Jan Niehusmann
2003-08-07 21:12 ` uncorrectable ext2 errors Andries Brouwer
2003-08-10 20:55 ` Jan Niehusmann
2003-08-10 21:19 ` Andries Brouwer
2003-08-10 21:34 ` Jan Niehusmann
2003-08-10 21:58 ` Andries Brouwer
2003-08-10 22:10 ` Jan Niehusmann
2003-08-10 22:33 ` IDE bug - was: " Andries Brouwer
2003-08-12 15:36 ` Alan Cox
2003-08-12 16:32 ` Bartlomiej Zolnierkiewicz [this message]
2003-08-12 22:50 ` Andries Brouwer
2003-08-13 0:21 ` Bartlomiej Zolnierkiewicz
2003-08-13 8:03 ` Jan Niehusmann
2003-08-13 9:25 ` Jan Niehusmann
2003-08-13 11:16 ` Alan Cox
2003-08-13 11:20 ` Jan Niehusmann
2003-08-13 12:12 ` Jan Niehusmann
2003-08-13 16:31 ` Jan Niehusmann
2003-08-13 18:38 ` Alan Cox
2003-08-13 12:21 ` Bartlomiej Zolnierkiewicz
2003-08-13 13:27 ` Andries Brouwer
2003-08-13 22:05 ` Jan Niehusmann
2003-08-13 22:32 ` Bartlomiej Zolnierkiewicz
2003-08-13 22:36 ` Jan Niehusmann
2003-08-13 23:14 ` Bartlomiej Zolnierkiewicz
2003-08-14 6:37 ` Jan Niehusmann
2003-08-10 23:01 ` Bartlomiej Zolnierkiewicz
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=200308121832.48212.bzolnier@elka.pw.edu.pl \
--to=b.zolnierkiewicz@elka.pw.edu.pl \
--cc=aebr@win.tue.nl \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=jan@gondor.com \
--cc=linux-kernel@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®