Re: Can't mount a mixed mode CD from acd

Craig Rodrigues wrote:
> On Thu, Sep 22, 2005 at 10:24:03AM +0930, Daniel O'Connor wrote:
> 
>>I have burnt a mixed mode audio/data CD (as 'CD Extra!'). I used k3b to make 
>>it and it has 34 tracks in the first session that are audio, and 1 data track 
>>in the second session.
>>
>>If I mount -t cd9660 /dev/acd0 /cdrom I get a panic..
>>panic: wrong offset 427728896 for sectorsize 2352
>>g_io_request()
>>
>>If I mount with cd0 it works OK, but I'd say that's because it just assumed a 
>>2k sector size.
> 
> 
> What do you mean by "If I mount with cd0"?  Do you have a SCSI
> CD-ROM on your system as well?

I have atapicam in my kernel so the same drive appears as acd0 and cd0.

> Can you try this patch?

Not at the moment, hopefully soon :)

Although I'm not sure how it would work because the sector size *really 
is* 2048 bytes for the second session. Unfortunately it is 2352 bytes 
for the first session and this duality may be confusing Geom (if it's 
even supposed to be able to handle dual sector sizes)

> Index: cd9660_vfsops.c
> ===================================================================
> RCS file: /home/ncvs/src/sys/isofs/cd9660/cd9660_vfsops.c,v
> retrieving revision 1.141
> diff -u -u -r1.141 cd9660_vfsops.c
> --- cd9660_vfsops.c     14 Aug 2005 04:19:36 -0000      1.141
> +++ cd9660_vfsops.c     22 Sep 2005 02:20:09 -0000
> _at__at_ -247,9 +247,9 _at__at_
>  
>         /* This is the "logical sector size".  The standard says this
>          * should be 2048 or the physical sector size on the device,
> -        * whichever is greater.  For now, we'll just use a constant.
> +        * whichever is greater.
>          */
> -       iso_bsize = ISO_DEFAULT_BLOCK_SIZE;
> +       iso_bsize = cp->provider->sectorsize;
>  
>         joliet_level = 0;
>         if (1 != vfs_scanopt(mp->mnt_optnew, "ssector", "%d", &ssector))
> 
> 
> 

--
Daniel O'Connor software and network engineer
for Genesis Software - http://www.gsoft.com.au
"The nice thing about standards is that there
are so many of them to choose from."
   -- Andrew Tanenbaum
GPG Fingerprint - 5596 B766 97C0 0E94 4347 295E E593 DC20 7B3F CE8C