AW: [PATCH] sun disk label: fix signed int usage for sector count

From: Jurzitza, Dieter
Date: Tue Aug 29 2006 - 00:55:37 EST


Dear listmembers,
referring to my own question (and to Jeff Mahoneys email), would it be sufficient to simply change

- --- linux-2.6.17/fs/partitions/sun.c 2006-01-02 22:21:10.000000000 -0500
+++ linux-2.6.17.fix/fs/partitions/sun.c 2006-08-26 11:54:54.000000000 -0400
@@ -74,10 +74,10 @@ int sun_partition(struct parsed_partitio
spc = be16_to_cpu(label->ntrks) * be16_to_cpu(label->nsect);
for (i = 0; i < 8; i++, p++) {
unsigned long st_sector;
- - int num_sectors;
+ unsigned int num_sectors;

st_sector = be32_to_cpu(p->start_cylinder) * spc;
num_sectors = be32_to_cpu(p->num_sectors);

in order to get rid of the meaningless negative numbers? If possible this would be highly preferrable over a solution that renders this partition useless. If Sun Os cannot handle this, maybe an error message when configuring such a disk would be helpful.

Once again, thank you for your feedback,
best regards




Dieter Jurzitza


--
________________________________________________

HARMAN BECKER AUTOMOTIVE SYSTEMS

Dr.-Ing. Dieter Jurzitza
Manager Hardware Systems
System Development

Industriegebiet Ittersbach
Becker-Göring Str. 16
D-76307 Karlsbad / Germany

Phone: +49 (0)7248 71-1577
Fax: +49 (0)7248 71-1216
eMail: DJurzitza@xxxxxxxxxxxxxxxx
Internet: http://www.becker.de


> -----Ursprüngliche Nachricht-----
> Von: linux-kernel-owner@xxxxxxxxxxxxxxx
> [mailto:linux-kernel-owner@xxxxxxxxxxxxxxx] Im Auftrag von
> Jurzitza, Dieter
> Gesendet: Dienstag, 29. August 2006 06:52
> An: linux-kernel@xxxxxxxxxxxxxxx
> Betreff: [PATCH] sun disk label: fix signed int usage for sector count
>
>
> Dear listmembers,
> citing Jeff Mahoney:
>
> > I'm not sure how many people out there are using Sun disk
> labels with
> > sizes > 1TB. It seems like a pretty rare corner case, but
> there's no
> > reason any data stored in those partitions wouldn't be
> invalid, and it
> > will suddenly cut them off. Is this a rare enough
> occurrence that we
> > don't care?
>
> I am one of those guys using Sun disklabels with sizes > 1TB.
> And I would highly appreciate if nobody out there would make
> my disk drive useless by simply discarding anything larger
> than 1 TB. This would render our department server useless,
> what is everything but desirable :-)
>
> Fdisk says:
>
> Kommando (m für Hilfe):
> Disk /dev/sda (Sun disk label): 1024 heads, 63 sectors, 36187
> cylinders Units = Zylinder of 64512 * 512 bytes
>
> Gerät Flag Start End Blocks Id System
> /dev/sda1 0 1 32256 83 Linux native
> /dev/sda2 u 1 64 2032128 82 Linux Swap
> /dev/sda3 0 36187 -980235776 5 Whole disk
> /dev/sda4 64 320 8257536 1 Boot
> /dev/sda5 320 4864 146571264 83 Linux native
> /dev/sda6 4864 16384 371589120 83 Linux native
> /dev/sda7 16384 36187 638765568 83 Linux native
>
> Kommando (m für Hilfe):
>
> cat /proc/partitions says:
> /home/fred> cat /proc/partitions
> major minor #blocks name
>
> 8 0 1171874880 sda
> 8 1 32256 sda1
> 8 2 2032128 sda2
> 8 3 -980235776 sda3
> 8 4 8257536 sda4
> 8 5 146571264 sda5
> 8 6 371589120 sda6
> 8 7 638765568 sda7
>
> but do not leave sda3 useless by changing the code. If there
> would be a chance to make fdisk and /proc/partitions not to
> display negative numbers (since, though obvious, they are
> nonsense) it would be preferrable.
>
> Many thanks for your help,
> take care
>
>
> Dieter Jurzitza
>
> --
> ________________________________________________
>
> HARMAN BECKER AUTOMOTIVE SYSTEMS
>
> Dr.-Ing. Dieter Jurzitza
> Manager Hardware Systems
> System Development
>
> Industriegebiet Ittersbach
> Becker-Göring Str. 16
> D-76307 Karlsbad / Germany
>
> Phone: +49 (0)7248 71-1577
> Fax: +49 (0)7248 71-1216
> eMail: DJurzitza@xxxxxxxxxxxxxxxx
> Internet: http://www.becker.de
>
>
>
> *******************************************
> Diese E-Mail enthaelt vertrauliche und/oder rechtlich
> geschuetzte Informationen. Wenn Sie nicht der richtige
> Adressat sind oder diese E-Mail irrtuemlich erhalten haben,
> informieren Sie bitte sofort den Absender und loeschen Sie
> diese Mail. Das unerlaubte Kopieren sowie die unbefugte
> Weitergabe dieser Mail ist nicht gestattet.
>
> This e-mail may contain confidential and/or privileged
> information. If you are not the intended recipient (or have
> received this e-mail in error) please notify the sender
> immediately and delete this e-mail. Any unauthorized copying,
> disclosure or distribution of the contents in this e-mail is
> strictly forbidden.
> *******************************************
>
> -
> To unsubscribe from this list: send the line "unsubscribe
> linux-kernel" in the body of a message to
> majordomo@xxxxxxxxxxxxxxx More majordomo info at
> http://vger.kernel.org/majordomo-> info.html
> Please read the
> FAQ at http://www.tux.org/lkml/
>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/