public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
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!" ****

  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