Bug 116766 - IDE CD DMA + 4GB ia32e box == instant panic
Summary: IDE CD DMA + 4GB ia32e box == instant panic
Alias: None
Product: Fedora
Classification: Fedora
Component: kernel
Version: rawhide
Hardware: x86_64
OS: Linux
Target Milestone: ---
Assignee: Dave Jones
QA Contact: Brian Brock
Depends On:
Blocks: FC3Target
TreeView+ depends on / blocked
Reported: 2004-02-25 00:02 UTC by Bill Nottingham
Modified: 2015-01-04 22:04 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2004-11-03 03:58:13 UTC

Attachments (Terms of Use)
Patch fixing the issue (426 bytes, patch)
2004-07-17 01:57 UTC, Suresh Siddha
no flags Details | Diff

Description Bill Nottingham 2004-02-25 00:02:20 UTC
This is with kernel-2.6.3-1.104.

Booting the kernel and then accessing the CD (mount, checkisomd5) with
DMA gives an immediate panic, with a backtrace of:
 mark_clean + 172, pci_unmap_sg + 68, ide_destroy_dmatable + 42,
__ide_dma_end + 101, ide_cd:cdrom_read_intr:74

Booting with mem=4096M works OK.
DMA HD access works OK.

hdparm -d0 /dev/hdc causes it to then work OK.

Comment 1 Bill Nottingham 2004-03-04 15:54:49 UTC
Works in 2.4, FWIW.

Comment 2 Dave Jones 2004-06-14 23:50:28 UTC
did this get fixed ?

Comment 3 Bill Nottingham 2004-06-28 20:56:08 UTC
Appears to have been.

Comment 4 Bill Nottingham 2004-07-09 22:37:02 UTC
It's baaaaaaaaaaack....


Mounting a CD fails, says it can't read the filesystem.
chkisomd5sum on the CD device panics.

Comment 5 Suresh Siddha 2004-07-17 01:57:53 UTC
Created attachment 101992 [details]
Patch fixing the issue

With swiotlb, pci_unmap_sg() was doing pci_unmap_single() instead of 
swiotlb_unmap_sg(). This will fail because, swiotlb_map_sg() is
setting up virtual address in sg->dma_address and pci_unmap_single()
expects physical address.

Attached patch will fix the issue.

Comment 6 Suresh Siddha 2004-11-02 22:05:40 UTC
This issue is fixed in the base sometime back. Sorry forgot to update 
this bugzilla.

Comment 7 Bill Nottingham 2004-11-03 03:58:13 UTC
Yeah, haven't seen this recently.

Note You need to log in before you can comment on or make changes to this bug.