home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #31 / NN_1992_31.iso / spool / comp / unix / bsd / 10632 < prev    next >
Encoding:
Text File  |  1992-12-25  |  3.4 KB  |  127 lines
  1. Path: sparky!uunet!pipex!bnr.co.uk!uknet!mcsun!fuug!kiae!demos!newsserv
  2. From: ache@astral.msk.su (Andrew A. Chernov, Black Mage)
  3. Newsgroups: comp.unix.bsd
  4. Subject: [386bsd] patch for cgd's multiport com driver, select() bug fix
  5. Message-ID: <lFjp1EhiA0@astral.msk.su>
  6. Date: 23 Dec 92 08:01:49 GMT
  7. Sender: news-service@newcom.kiae.su
  8. Reply-To: ache@astral.msk.su
  9. Organization: Ha-oh-lahm Yetzirah
  10. Lines: 115
  11.  
  12. I just found this bug, maybe, it present in standard com driver too...
  13. I don't use it and can't check.
  14.  
  15. BUG DESCRIPTION:
  16. select() call don't work at all.
  17.  
  18. WHY:
  19. ttselect() kernel routine used for coms too, its code say:
  20.     register struct tty *tp = &cdevsw[major(dev)].d_ttys[minor(dev)];
  21. but for cgd's multiport com driver it must be:
  22.     register struct tty *tp = &com_tty[UNIT(dev)];
  23.  
  24. FIX:
  25. Simple, duplicate ttselect() code into comselect() code
  26. with replacing above. Modify conf.c ttselect()-->comselect()
  27.  
  28.  
  29. *** /sys/i386/isa/com.c.cgd.was       Thu Dec 10 01:07:29 1992
  30. --- /sys/i386/isa/com.c       Wed Dec 23 10:11:01 1992
  31. ***************
  32. *** 932,934 ****
  33. --- 932,973 ----
  34.       return 0;
  35.   }
  36.   #endif
  37. + 
  38. + int
  39. + comselect(dev, rw, p)
  40. +     dev_t dev;
  41. +     int rw;
  42. +     struct proc *p;
  43. + {
  44. +     register struct tty *tp = &com_tty[UNIT(dev)];
  45. +     int nread;
  46. +     int s = spltty();
  47. + 
  48. +     switch (rw) {
  49. + 
  50. +     case FREAD:
  51. +         nread = ttnread(tp);
  52. +         if (nread > 0 || 
  53. +            ((tp->t_cflag&CLOCAL) == 0 && (tp->t_state&TS_CARR_ON) == 0))
  54. +             goto win;
  55. +         if (tp->t_rsel && tp->t_rsel->p_wchan == (caddr_t)&selwait)
  56. +             tp->t_state |= TS_RCOLL;
  57. +         else
  58. +             tp->t_rsel = p;
  59. +         break;
  60. + 
  61. +     case FWRITE:
  62. +         if (RB_LEN(&tp->t_out) <= tp->t_lowat)
  63. +             goto win;
  64. +         if (tp->t_wsel && tp->t_wsel->p_wchan == (caddr_t)&selwait)
  65. +             tp->t_state |= TS_WCOLL;
  66. +         else
  67. +             tp->t_wsel = p;
  68. +         break;
  69. +     }
  70. +     splx(s);
  71. +     return (0);
  72. + win:
  73. +     splx(s);
  74. +     return (1);
  75. + }
  76. *** /sys/i386/i386/conf.c.cgd.was      Mon Nov 23 08:00:07 1992
  77. --- /sys/i386/i386/conf.c      Wed Dec 23 10:13:12 1992
  78. ***************
  79. *** 152,158 ****
  80.   
  81.   #include "com.h"
  82.   #if NCOM > 0
  83. ! int    comopen(),comclose(),comread(),comwrite(),comioctl();
  84.   #define comreset    enxio
  85.   extern    struct tty com_tty[];
  86.   #else
  87. --- 152,158 ----
  88.   
  89.   #include "com.h"
  90.   #if NCOM > 0
  91. ! int     comopen(),comclose(),comread(),comwrite(),comioctl(),comselect();
  92.   #define comreset    enxio
  93.   extern    struct tty com_tty[];
  94.   #else
  95. ***************
  96. *** 162,167 ****
  97. --- 162,168 ----
  98.   #define comwrite    enxio
  99.   #define comioctl    enxio
  100.   #define comreset    enxio
  101. + #define comselect       enxio
  102.   #define    com_tty        NULL
  103.   #endif
  104.   
  105. ***************
  106. *** 220,226 ****
  107.         logselect,    enodev,        NULL },
  108.       { comopen,    comclose,    comread,    comwrite,    /*8*/
  109.         comioctl,    enodev,        comreset,    com_tty,
  110. !       ttselect,    enodev,        NULL },
  111.       { Fdopen,    fdclose,    rawread,    rawwrite,    /*9*/
  112.         fdioctl,    enodev,        nullop,        NULL,
  113.         seltrue,    enodev,        fdstrategy },
  114. --- 221,227 ----
  115.         logselect,    enodev,        NULL },
  116.       { comopen,    comclose,    comread,    comwrite,    /*8*/
  117.         comioctl,    enodev,        comreset,    com_tty,
  118. !       comselect,    enodev,         NULL },
  119.       { Fdopen,    fdclose,    rawread,    rawwrite,    /*9*/
  120.         fdioctl,    enodev,        nullop,        NULL,
  121.         seltrue,    enodev,        fdstrategy },
  122. -- 
  123. In-This-Life:  Andrew A. Chernov    |  "Hay mas dicha, mas contento
  124. Internet:      ache@astral.msk.su   |  "Que adorar una hermosura
  125. Organization:  The RELCOM Corp.,    |  "Brujuleada entre los lejos
  126.                Moscow, Russia       |  "De lo imposible?!"  (Calderon)
  127.