home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #18 / NN_1992_18.iso / spool / comp / unix / bsd / 4364 < prev    next >
Encoding:
Internet Message Format  |  1992-08-18  |  8.7 KB
  1. Path: sparky!uunet!sun-barr!cs.utexas.edu!zaphod.mps.ohio-state.edu!uwm.edu!bionet!ucselx!crash!fpm
  2. From: fpm@crash.cts.com (Frank Maclachlan)
  3. Newsgroups: comp.unix.bsd
  4. Subject: Bugs in DMA code (w/ patch)
  5. Summary: Bugs in DMA code fixed
  6. Keywords: DMA, patch
  7. Message-ID: <1992Aug18.142737.16819@crash>
  8. Date: 18 Aug 92 21:27:36 GMT
  9. Followup-To: comp.unix.bsd
  10. Organization: CTS Network Services (crash, ctsnet), El Cajon, CA
  11. Lines: 298
  12.  
  13. I found a few problems in '/usr/src/sys.386bsd/i386/isa/isa.c'.
  14.  
  15. At line 389 in isa_dmarangecheck(), the automatic variable priorpage
  16. is used without being initially set to 0.  This causes the function
  17. to flag special handling for virtually all DMA transfer requests.
  18. Also, no check is made for DMA requests crossing DMA page boundaries
  19. (64k for DMA chans 0..3, 128k for DMA chans 4..7).  This problem is
  20. masked by priorpage not being initialized - almost all DMA is done
  21. to/from safe 'bounce' buffers which don't cross DMA page boundaries
  22. and the data are block moved from/to the user's buffer.
  23.  
  24. In line 359, nbytes is doubled by shifting it left 1 bit; it
  25. should instead be divided by 2 by shifting right 1 bit position.
  26. This occurs when any of the word mode DMA channels (4..7) is used.
  27.  
  28. The following patch fixes the above problems and makes some cosmetic
  29. changes to isa_dmacascade() and isa_dmastart():
  30.  
  31.  
  32. *** isa.c.ORIG    Tue Jul 14 12:25:24 1992
  33. --- isa.c    Tue Aug 18 14:12:49 1992
  34. ***************
  35. *** 39,44 ****
  36. --- 39,50 ----
  37.   
  38.   /*
  39.    * code to manage AT bus
  40. +  *
  41. +  * 92/08/18  Frank P. MacLachlan (fpm@crash.cts.com):
  42. +  * Fixed uninitialized variable problem and added code to deal
  43. +  * with DMA page boundaries in isa_dmarangecheck().  Fixed word
  44. +  * mode DMA count compution and reorganized DMA setup code in
  45. +  * isa_dmastart()
  46.    */
  47.   
  48.   #include "param.h"
  49. ***************
  50. *** 58,63 ****
  51. --- 64,85 ----
  52.   #include "i386/isa/ic/i8237.h"
  53.   #include "i386/isa/ic/i8042.h"
  54.   
  55. + /*
  56. + **  Register definitions for DMA controller 1 (channels 0..3):
  57. + */
  58. + #define    DMA1_CHN(c)    (IO_DMA1 + 1*(2*(c)))    /* addr reg for channel c */
  59. + #define    DMA1_SMSK    (IO_DMA1 + 1*10)    /* single mask register */
  60. + #define    DMA1_MODE    (IO_DMA1 + 1*11)    /* mode register */
  61. + #define    DMA1_FFC    (IO_DMA1 + 1*12)    /* clear first/last FF */
  62. + 
  63. + /*
  64. + **  Register definitions for DMA controller 2 (channels 4..7):
  65. + */
  66. + #define    DMA2_CHN(c)    (IO_DMA1 + 2*(2*(c)))    /* addr reg for channel c */
  67. + #define    DMA2_SMSK    (IO_DMA2 + 2*10)    /* single mask register */
  68. + #define    DMA2_MODE    (IO_DMA2 + 2*11)    /* mode register */
  69. + #define    DMA2_FFC    (IO_DMA2 + 2*12)    /* clear first/last FF */
  70. + 
  71.   int config_isadev(struct isa_device *, u_short *);
  72.   #ifdef notyet
  73.   struct rlist *isa_iomem;
  74. ***************
  75. *** 278,298 ****
  76.    * external dma control by a board.
  77.    */
  78.   void isa_dmacascade(unsigned chan)
  79. ! {    int modeport;
  80. ! 
  81.       if (chan > 7)
  82.           panic("isa_dmacascade: impossible request"); 
  83.   
  84.       /* set dma channel mode, and set dma channel mode */
  85. !     if ((chan & 4) == 0)
  86. !         modeport = IO_DMA1 + 0xb;
  87. !     else
  88. !         modeport = IO_DMA2 + 0x16;
  89. !     outb(modeport, DMA37MD_CASCADE | (chan & 3));
  90. !     if ((chan & 4) == 0)
  91. !         outb(modeport - 1, chan & 3);
  92. !     else
  93. !         outb(modeport - 2, chan & 3);
  94.   }
  95.   
  96.   /*
  97. --- 300,317 ----
  98.    * external dma control by a board.
  99.    */
  100.   void isa_dmacascade(unsigned chan)
  101. ! {
  102.       if (chan > 7)
  103.           panic("isa_dmacascade: impossible request"); 
  104.   
  105.       /* set dma channel mode, and set dma channel mode */
  106. !     if ((chan & 4) == 0) {
  107. !         outb(DMA1_MODE, DMA37MD_CASCADE | chan);
  108. !         outb(DMA1_SMSK, chan);
  109. !     } else {
  110. !         outb(DMA2_MODE, DMA37MD_CASCADE | (chan & 3));
  111. !         outb(DMA2_SMSK, chan & 3);
  112. !     }
  113.   }
  114.   
  115.   /*
  116. ***************
  117. *** 301,313 ****
  118.    */
  119.   void isa_dmastart(int flags, caddr_t addr, unsigned nbytes, unsigned chan)
  120.   {    vm_offset_t phys;
  121. !     int modeport, waport, mskport;
  122.       caddr_t newaddr;
  123.   
  124. !     if (chan > 7 || nbytes > (1<<16))
  125.           panic("isa_dmastart: impossible request"); 
  126.   
  127. !     if (isa_dmarangecheck(addr, nbytes)) {
  128.           if (dma_bounce[chan] == 0)
  129.               dma_bounce[chan] =
  130.                   /*(caddr_t)malloc(MAXDMASZ, M_TEMP, M_WAITOK);*/
  131. --- 320,334 ----
  132.    */
  133.   void isa_dmastart(int flags, caddr_t addr, unsigned nbytes, unsigned chan)
  134.   {    vm_offset_t phys;
  135. !     int waport;
  136.       caddr_t newaddr;
  137.   
  138. !     if (    chan > 7
  139. !         || (chan < 4 && nbytes > (1<<16))
  140. !         || (chan >= 4 && (nbytes > (1<<17) || (u_int)addr & 1)))
  141.           panic("isa_dmastart: impossible request"); 
  142.   
  143. !     if (isa_dmarangecheck(addr, nbytes, chan)) {
  144.           if (dma_bounce[chan] == 0)
  145.               dma_bounce[chan] =
  146.                   /*(caddr_t)malloc(MAXDMASZ, M_TEMP, M_WAITOK);*/
  147. ***************
  148. *** 325,372 ****
  149.       /* translate to physical */
  150.       phys = pmap_extract(pmap_kernel(), (vm_offset_t)addr);
  151.   
  152. -     /* set dma channel mode, and reset address ff */
  153. -     if ((chan & 4) == 0)
  154. -         modeport = IO_DMA1 + 0xb;
  155. -     else
  156. -         modeport = IO_DMA2 + 0x16;
  157. -     if (flags & B_READ)
  158. -         outb(modeport, DMA37MD_SINGLE|DMA37MD_WRITE|(chan&3));
  159. -     else
  160. -         outb(modeport, DMA37MD_SINGLE|DMA37MD_READ|(chan&3));
  161. -     if ((chan & 4) == 0)
  162. -         outb(modeport + 1, 0);
  163. -     else
  164. -         outb(modeport + 2, 0);
  165. - 
  166. -     /* send start address */
  167.       if ((chan & 4) == 0) {
  168. !         waport =  IO_DMA1 + (chan<<1);
  169.           outb(waport, phys);
  170.           outb(waport, phys>>8);
  171. !     } else {
  172. !         waport =  IO_DMA2 + ((chan - 4)<<2);
  173. !         outb(waport, phys>>1);
  174. !         outb(waport, phys>>9);
  175. !     }
  176. !     outb(dmapageport[chan], phys>>16);
  177.   
  178. !     /* send count */
  179. !     if ((chan & 4) == 0) {
  180.           outb(waport + 1, --nbytes);
  181.           outb(waport + 1, nbytes>>8);
  182.       } else {
  183. !         nbytes <<= 1;
  184.           outb(waport + 2, --nbytes);
  185.           outb(waport + 2, nbytes>>8);
  186. -     }
  187.   
  188. !     /* unmask channel */
  189. !     if ((chan & 4) == 0)
  190. !         mskport =  IO_DMA1 + 0x0a;
  191. !     else
  192. !         mskport =  IO_DMA2 + 0x14;
  193. !     outb(mskport, chan & 3);
  194.   }
  195.   
  196.   void isa_dmadone(int flags, caddr_t addr, int nbytes, int chan)
  197. --- 346,401 ----
  198.       /* translate to physical */
  199.       phys = pmap_extract(pmap_kernel(), (vm_offset_t)addr);
  200.   
  201.       if ((chan & 4) == 0) {
  202. !         /*
  203. !          * Program one of DMA channels 0..3.  These are
  204. !          * byte mode channels.
  205. !          */
  206. !         /* set dma channel mode, and reset address ff */
  207. !         if (flags & B_READ)
  208. !             outb(DMA1_MODE, DMA37MD_SINGLE|DMA37MD_WRITE|chan);
  209. !         else
  210. !             outb(DMA1_MODE, DMA37MD_SINGLE|DMA37MD_READ|chan);
  211. !         outb(DMA1_FFC, 0);
  212. ! 
  213. !         /* send start address */
  214. !         waport =  DMA1_CHN(chan);
  215.           outb(waport, phys);
  216.           outb(waport, phys>>8);
  217. !         outb(dmapageport[chan], phys>>16);
  218.   
  219. !         /* send count */
  220.           outb(waport + 1, --nbytes);
  221.           outb(waport + 1, nbytes>>8);
  222. + 
  223. +         /* unmask channel */
  224. +         outb(DMA1_SMSK, chan);
  225.       } else {
  226. !         /*
  227. !          * Program one of DMA channels 4..7.  These are
  228. !          * word mode channels.
  229. !          */
  230. !         /* set dma channel mode, and reset address ff */
  231. !         if (flags & B_READ)
  232. !             outb(DMA2_MODE, DMA37MD_SINGLE|DMA37MD_WRITE|(chan&3));
  233. !         else
  234. !             outb(DMA2_MODE, DMA37MD_SINGLE|DMA37MD_READ|(chan&3));
  235. !         outb(DMA2_FFC, 0);
  236. ! 
  237. !         /* send start address */
  238. !         waport = DMA2_CHN(chan - 4);
  239. !         outb(waport, phys>>1);
  240. !         outb(waport, phys>>9);
  241. !         outb(dmapageport[chan], phys>>16);
  242. ! 
  243. !         /* send count */
  244. !         nbytes >>= 1;
  245.           outb(waport + 2, --nbytes);
  246.           outb(waport + 2, nbytes>>8);
  247.   
  248. !         /* unmask channel */
  249. !         outb(DMA2_SMSK, chan & 3);
  250. !     }
  251.   }
  252.   
  253.   void isa_dmadone(int flags, caddr_t addr, int nbytes, int chan)
  254. ***************
  255. *** 382,393 ****
  256.   
  257.   /*
  258.    * Check for problems with the address range of a DMA transfer
  259. !  * (non-contiguous physical pages, outside of bus address space).
  260.    * Return true if special handling needed.
  261.    */
  262.   
  263. ! isa_dmarangecheck(caddr_t va, unsigned length) {
  264. !     vm_offset_t phys, priorpage, endva;
  265.   
  266.       endva = (vm_offset_t)round_page(va + length);
  267.       for (; va < (caddr_t) endva ; va += NBPG) {
  268. --- 411,424 ----
  269.   
  270.   /*
  271.    * Check for problems with the address range of a DMA transfer
  272. !  * (non-contiguous physical pages, outside of bus address space,
  273. !  * crossing DMA page boundaries).
  274.    * Return true if special handling needed.
  275.    */
  276.   
  277. ! isa_dmarangecheck(caddr_t va, unsigned length, unsigned chan) {
  278. !     vm_offset_t phys, priorpage = 0, endva;
  279. !     u_int dma_pgmsk = (chan & 4) ?  ~(128*1024-1) : ~(64*1024-1);
  280.   
  281.       endva = (vm_offset_t)round_page(va + length);
  282.       for (; va < (caddr_t) endva ; va += NBPG) {
  283. ***************
  284. *** 397,404 ****
  285.               panic("isa_dmacheck: no physical page present");
  286.           if (phys > ISARAM_END) 
  287.               return (1);
  288. !         if (priorpage && priorpage + NBPG != phys)
  289. !             return (1);
  290.           priorpage = phys;
  291.       }
  292.       return (0);
  293. --- 428,440 ----
  294.               panic("isa_dmacheck: no physical page present");
  295.           if (phys > ISARAM_END) 
  296.               return (1);
  297. !         if (priorpage) {
  298. !             if (priorpage + NBPG != phys)
  299. !                 return (1);
  300. !             /* check if crossing a DMA page boundary */
  301. !             if (((u_int)priorpage ^ (u_int)phys) & dma_pgmsk)
  302. !                 return (1);
  303. !         }
  304.           priorpage = phys;
  305.       }
  306.       return (0);
  307. --
  308. UUCP: {hplabs!hp-sdd ucsd nosc}!crash!fpm
  309. ARPA: crash!fpm@nosc.mil
  310. INET: fpm@crash.cts.com
  311.