cvs commit: src/sys/geom geom_pc98.c src/sys/dev/ata ata-all.h ata-cbus.c ata-disk.c

[Søren Schmidt]

Poul-Henning Kamp wrote:
> In message <20041009.023958.41719880.nyan at jp.FreeBSD.org>, Takahashi Yoshihiro 
> writes:
> 
>>In article <200410071737.i97Hb9TU038301 at repoman.freebsd.org>
>>Søren Schmidt <sos at FreeBSD.org> writes:
>>
>>
>>>sos         2004-10-07 17:37:09 UTC
>>>
>>>  FreeBSD src repository
>>>
>>>  Modified files:
>>>    sys/geom             geom_pc98.c 
>>>    sys/dev/ata          ata-all.h ata-cbus.c ata-disk.c 
>>>  Log:
>>>  Move the PC98 specific geometry "gunk" to geom_pc98.c where it belongs.
>>>  This also adds support for bigger disks on the controller I have access to,
>>>  and maybe others if I understood the adhoc methods used on those.
>>>  
>>>  Those with more PC98 bigdrive controllers it is hereby invited to add/fix
>>>  support for those in geom_pc98.c and not using #ifdef PC98 all over the place.
>>
>>This broke non-ATA disk supports.  We need to use different way for
>>geometry translation between ATA and non-ATA disks.  And, your
>>'BigDrive' support has some problems, it cannot use 'BigDrive' as root
>>filesystem and it is incompatibility with the other OSs.
> 
> 
> Then conditionalize the code in geom_pc98 on the provider name starting
> with "ad".

I allready did so :)

> All PC98 disk layout relevant code should be in geom_pc98 and not
> in diskdrivers.

Exactly!

However there is fripe with fdisk_pc98 and sysinstall it seems, dunno 
about those as they are way out of my territory...



-- 

-Søren