[haiku-bugs] Re: [Haiku] #3473: Haiku does not recognise some multi session CDs

  • From: "Ziusudra" <trac@xxxxxxxxxxxx>
  • Date: Sun, 13 Jun 2010 21:11:28 -0000

#3473: Haiku does not recognise some multi session CDs
-------------------------------------------------+--------------------------
 Reporter:  bbjimmy                              |       Owner:  axeld         
     Type:  bug                                  |      Status:  new           
 Priority:  normal                               |   Milestone:  R1            
Component:  File Systems/cdda                    |     Version:  R1/Development
 Keywords:  corum session mixed-mode EnhancedCD  |   Blockedby:  6130          
    Patch:  1                                    |    Platform:  All           
 Blocking:                                       |  
-------------------------------------------------+--------------------------

Comment(by Ziusudra):

 >Am I correct that mixed mode CDs are no longer detected by CDDA, and
 require manual mounting?

 No, CDDA will claim and automount the audio session for both types of
 mixed-mode CD. Other filesystem drivers will correctly claim and automount
 the data session.


 >Also, since it still would be a viable solution, reporting a lower value
 sounds like a better solution than returning an error.

 No. If CDDA claims low for a data session and no other filesystem driver
 claims it, then CDDA will automount both sessions. Both will appear the
 same, as the audio session, and access to what is really the data session
 will panic to KDL. I have an EnhancedCD where the data session is HFS, for
 which we have no driver, where this happens if CDDA claims low.


 >You should always write (... & ...) != 0

 OK, session::is_audio() in Disc.cpp will need to be fixed. Does that also
 apply to the short form that appears throughout the debug TRACEs?:
 {{{
 (entries[i].control & kControlDataTrack ? "data" : "audio")
 }}}


 >I saw that you worked around this issue at another case with extra ()
 around the term

 Actually, I have a bad habit of using too many parathesis that I need to
 break. :/

-- 
Ticket URL: <http://dev.haiku-os.org/ticket/3473#comment:13>
Haiku <http://dev.haiku-os.org>
Haiku - the operating system.

Other related posts: