Re: patch-2.1.117 adds bad ide_xlate_1024()

Andries.Brouwer@cwi.nl
Tue, 25 Aug 1998 23:05:15 +0200 (MET DST)


> From torvalds@transmeta.com Tue Aug 25 19:28:49 1998
>
> Ok. Can you actually send me a diff

The diff below does: 1. Removes the ide_xlate_1024() that we don't like,
2. adds one more case to the ide_xlate_1024("[PTBL]")

Andries

--- genhd.c~ Sun Aug 16 20:48:40 1998
+++ genhd.c Tue Aug 25 22:48:32 1998
@@ -411,28 +411,22 @@
} else {
/*
* Examine the partition table for common translations.
- * This is necessary for drives for situations where
- * the translated geometry is unavailable from the BIOS.
+ * This is useful for drives in situations where the
+ * translated geometry is unavailable from the BIOS.
*/
- int xlate_done = 0;
- for (i = 0; i < 4 && !xlate_done; i++) {
+ for (i = 0; i < 4; i++) {
struct partition *q = &p[i];
if (NR_SECTS(q)
&& (q->sector & 63) == 1
&& (q->end_sector & 63) == 63) {
unsigned int heads = q->end_head + 1;
- if (heads == 32 || heads == 64 || heads == 128 || heads == 255) {
-
+ if (heads == 32 || heads == 64 ||
+ heads == 128 || heads == 255 ||
+ heads == 240) {
(void) ide_xlate_1024(dev, heads, " [PTBL]");
- xlate_done = 1;
+ break;
}
}
- }
- if (!xlate_done) {
- /*
- * Default translation is equivalent of "BIOS LBA":
- */
- ide_xlate_1024(dev, -2, " [LBA]");
}
}
}

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.altern.org/andrebalsa/doc/lkml-faq.html