Bug 28742

Summary: CD_BUFFER_BLOCKS used in ide ioctl CDIOCREADAUDIO defined with different values..
Product: Base System Reporter: Peter <pb>
Component: kernAssignee: Søren Schmidt <sos>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Unspecified   
Hardware: Any   
OS: Any   

Description Peter 2001-07-06 00:10:02 UTC
CD_BUFFER_BLOCKS defined as
13 in  sys/dev/ata/atapi-cd.c
8  in  sys/i386/isa/wd_cd.c

1) Depending on compile race it may be defined differently

2) The limit is 27 (inclusive) what I remember. From the fact that the
   dma manage to transfer a maximum of 27 * 2352 bytes blocks in one 64k
   max transfer. (Might be different issue with PCI-IDE dunno)
   More frames (buffers), will allow more continous block reads per
   jitter correction. Thus higher efficiency.

(basepath /usr/src)

Fix: 

Common define for CD_BUFFER_BLOCKS. And define it as 27.
How-To-Repeat: read sources :)
Comment 1 Peter Pentchev freebsd_committer freebsd_triage 2001-07-06 07:43:43 UTC
Responsible Changed
From-To: freebsd-bugs->sos

Over to the ATA maintainer.
Comment 2 Søren Schmidt freebsd_committer freebsd_triage 2001-09-06 10:08:40 UTC
State Changed
From-To: open->closed