#13775 closed bug (fixed)

readcd clone option causes Haiku to KDL

Reported by: vidrep Owned by: nobody
Priority: normal Milestone: Unscheduled
Component: Drivers/Disk/ATA Version: R1/Development
Keywords: Cc:
Blocked By: Blocking:
Has a Patch: no Platform: All

Description (last modified by diver)

hrev51543 x86_64

Mount CDROM

readcd dev=10,1,0 -clone f=clone

-clone option causes Haiku to KDL 100% of the time

Attachments (4)

readcd.png (18.4 KB) - added by vidrep 20 months ago.
syslog (131.2 KB) - added by vidrep 20 months ago.
readcd_KDL.JPG (1.4 MB) - added by vidrep 20 months ago.
syslog.2 (16.6 KB) - added by vidrep 16 months ago.

Download all attachments as: .zip

Change History (15)

Changed 20 months ago by vidrep

Attachment: readcd.png added

comment:1 Changed 20 months ago by vidrep

Has a Patch: set

Changed 20 months ago by vidrep

Attachment: syslog added

Changed 20 months ago by vidrep

Attachment: readcd_KDL.JPG added

comment:2 Changed 20 months ago by waddlesplash

Component: - GeneralDrivers/Disk/ATA

comment:3 Changed 20 months ago by vidrep

Also KDL's on x86_gcc2h. Change platform to "All".

comment:4 Changed 20 months ago by vidrep

Platform: x86-64All

comment:5 Changed 20 months ago by korli

Has a Patch: unset

comment:6 Changed 20 months ago by diver

Description: modified (diff)

comment:7 Changed 16 months ago by vidrep

Still an issue. Tested with hrev51846 x86_gcc2h, using thew following command:

readcd dev=10,1,0 -clone -nocorr -retries=0 f=clone

Haiku KDL immediately. syslog attached

Changed 16 months ago by vidrep

Attachment: syslog.2 added

comment:8 Changed 16 months ago by korli

Hi, readcd seems to send a command with a zero datalength, which is unusual: https://github.com/jobermayr/cdrtools/blob/master/readcd/readcd.c#L1034

It seems though this case should be supported by avoiding DMA when no SG list is provided. https://git.haiku-os.org/haiku/tree/src/add-ons/kernel/bus_managers/ata/ATAPIDevice.cpp#n38

comment:9 Changed 16 months ago by korli

Please test with hrev51851, hopefully it helps.

comment:10 Changed 16 months ago by vidrep

Works now! Close the ticket please. Thanks korli.

comment:11 Changed 16 months ago by korli

Resolution: fixed
Status: newclosed

Thanks for the feedback!

Note: See TracTickets for help on using tickets.