cvs commit: src/sys/geom geom_gpt.c

Poul-Henning Kamp phk at phk.freebsd.dk
Fri Mar 18 12:25:12 PST 2005


In message <20050318185753.GS591 at darkness.comp.waw.pl>, Pawel Jakub Dawidek wri
tes:

>+> >Does any of these changes fix the panic when attempting to mount an
>+> >audio CD-ROM?
>+>=20
>+> Uhh, dunno, what panics ?  Do you know which GEOM class ?
>
>panic: wrong offset 65536 for sectorsize 2352

>ffs_mountfs(c154c6a8,c107d000,c1078450,c113a0c0,0) at ffs_mountfs+0x23e
>ffs_mount(c107d000,c1078450,0,0,c154b354) at ffs_mount+0x925

FFS and other filesystems needs to find out that they cannot work with
the sectorsize proposed.

We can either do that by turning the panic into an EIO or by explicitly
checking in the filesystems.

I guess EIO makes most sense, and since I havn't seen this panic find
anything important for a long time I guess that's the way.

Anyone want to do it ?

-- 
Poul-Henning Kamp       | UNIX since Zilog Zeus 3.20
phk at FreeBSD.ORG         | TCP/IP since RFC 956
FreeBSD committer       | BSD since 4.3-tahoe    
Never attribute to malice what can adequately be explained by incompetence.


More information about the cvs-all mailing list