From: Rogier Wolff <R.E.Wolff@BitWizard.nl>
To: Andries.Brouwer@cwi.nl
Cc: aebr@win.tue.nl, konsti@ludenkalle.de,
linux-kernel@vger.kernel.org, torvalds@osdl.org
Subject: Re: Weird partititon recocnising problem in 2.6.0-testX
Date: Fri, 14 Nov 2003 09:18:42 +0100 [thread overview]
Message-ID: <20031114081840.GA5485@bitwizard.nl> (raw)
In-Reply-To: <UTC200311101602.hAAG2an12276.aeb@smtp.cwi.nl>
On Mon, Nov 10, 2003 at 05:02:36PM +0100, Andries.Brouwer@cwi.nl wrote:
>
> Boot a kernel that remaps, say vanilla 2.4.
[...]
> Now boot a kernel that does not remap. Since you won't see
[...]
All this rebooting into kernels that DO or DON'T remap would have been
unneccesary if the remapping would not have cloned block 1 to appear
in block 0, but would just have swapped them.
In remapped mode, you now read block 1 wether you seek to block 0 or 1.
If the remapping would have swapped them, just copying over the block
from block 1 to block 0 would have allowed the removal of the stupid
remapping.
ide.c:1381
/* Yecch - this will shift the entire interval,
possibly killing some innocent following sector */
if (block == 0 && drive->remap_0_to_1 == 1)
block = 1; /* redirect MBR access to EZ-Drive partn table */
should have been:
if ((block <= 1) && drive->remap_0_to_1)
block ^= 1;
(IMHO, it's not worth going in and fixing this for say 2.4 or 2.6
kernels(*), but it IS worth noting and not making the same mistake again in
the future.....)
Roger.
(*) For example because that would change established behaviour which is
bad and can lead to surprises. If we'd change it, Andries' "how to fix"
emails would become even longer: He'd have to start: IF you have a
modern kernel, you can just ... but otherwise you'll have to .... and
then comes the whole mess I didn't fully quote.....
--
** R.E.Wolff@BitWizard.nl ** http://www.BitWizard.nl/ ** +31-15-2600998 **
*-- BitWizard writes Linux device drivers for any device you may have! --*
**** "Linux is like a wigwam - no windows, no gates, apache inside!" ****
next prev parent reply other threads:[~2003-11-14 8:18 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-11-10 16:02 Weird partititon recocnising problem in 2.6.0-testX Andries.Brouwer
2003-11-14 8:18 ` Rogier Wolff [this message]
-- strict thread matches above, loose matches on Subject: below --
2003-11-10 10:24 Konstantin Kletschke
2003-11-10 10:56 ` Andries Brouwer
2003-11-10 11:32 ` Konstantin Kletschke
2003-11-10 11:57 ` Konstantin Kletschke
2003-11-09 1:12 Konstantin Kletschke
2003-11-09 2:36 ` Andries Brouwer
2003-11-09 3:49 ` Konstantin Kletschke
2003-11-09 11:58 ` Andries Brouwer
2003-11-09 12:10 ` Stefan Smietanowski
2003-11-09 22:15 ` Konstantin Kletschke
2003-11-09 23:09 ` Konstantin Kletschke
2003-11-10 0:00 ` Andries Brouwer
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=20031114081840.GA5485@bitwizard.nl \
--to=r.e.wolff@bitwizard.nl \
--cc=Andries.Brouwer@cwi.nl \
--cc=aebr@win.tue.nl \
--cc=konsti@ludenkalle.de \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@osdl.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