home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1993 #3 / NN_1993_3.iso / spool / comp / os / linux / 25577 < prev    next >
Encoding:
Text File  |  1993-01-27  |  7.2 KB  |  247 lines
  1. Newsgroups: comp.os.linux
  2. Path: sparky!uunet!boulder!hazelrah!drew
  3. From: drew@hazelrah.cs.colorado.edu (Drew Eckhardt)
  4. Subject: Seagate scatter / gather patches
  5. Message-ID: <1993Jan28.000142.19523@colorado.edu>
  6. Sender: news@colorado.edu (The Daily Planet)
  7. Nntp-Posting-Host: hazelrah.cs.colorado.edu
  8. Organization: University of Colorado at Boulder
  9. References: <vera.728038000@fanaraaken.Stanford.EDU> <1993Jan26.143815.2112@unislc.uucp>
  10. Date: Thu, 28 Jan 1993 00:01:42 GMT
  11. Lines: 234
  12.  
  13. Ok, here's the first installment of the Seagate patches,   This one 
  14. implements scatter-gather (I saw a 4X speed improvement).  As far 
  15. as CPU usage and "hanging" - things should improve if your filesystems
  16. aren't too fragmented.
  17.  
  18. Note that the current Seagate routines will not transfer >
  19. ~500-600K/sec. If your disk is faster than this, you'll have to 
  20. format at something greater than a 1:1 interleave if you want to 
  21. see the same four-fold performance increase.
  22.  
  23. I have another patch on the way that will allow the use of the 
  24. Seagate's hardware handshaking, it will be posted "RSN" (the 
  25. definition of RSN being somewhere between a few hours and six months).
  26. It should rectify the maximum transfer rate restriction.
  27.  
  28. Here are some figures for your perusal : 
  29.  
  30.  
  31. IOZONE results (using recommended file sizes, minix filesystems).
  32. To reduce buffer cache size (and therefore required filesize under 
  33. IOZONE), I loaded the memory by running X11R5 + 3 xterms + xman + xload + 
  34. xsysinfo + twm.
  35.  
  36. System : i386-33, 8M main memory, 64K cache
  37. SCSI host : Seagate ST02
  38. Software : Linux .99.3 
  39. Disk : /dev/sda3 Seagate 296N, 3:1 interleave
  40.  
  41. Writing the 3 Megabyte file, 'iozone.tmp'...50.610000 seconds
  42. Reading the file...62.110000 seconds
  43.  
  44. IOZONE performance measurements:
  45.         62156 bytes/second for writing the file
  46.         50647 bytes/second for reading the file
  47.  
  48. Software : Linux .99.4 + Seagate Scatter / Gather patches
  49. Disk : /dev/sda3 Seagate 296N, 3:1 interleave
  50.  
  51. Writing the 3 Megabyte file, 'iozone.tmp'...12.840000 seconds
  52. Reading the file...16.120000 seconds
  53.  
  54. IOZONE performance measurements:
  55.         244994 bytes/second for writing the file
  56.         195144 bytes/second for reading the file
  57.  
  58. And, as a baseline : 
  59.  
  60. Software : Linux .99.4 + Seagate Scatter / Gather patches
  61. Disk : /dev/hda2 Seagate 1144A 1:1 interleave
  62.  
  63. Writing the 3 Megabyte file, 'iozone.tmp'...7.500000 seconds
  64. Reading the file...15.520000 seconds
  65.  
  66. IOZONE performance measurements:
  67.         419430 bytes/second for writing the file
  68.         202688 bytes/second for reading the file
  69.  
  70. Software : Linux .99.3 
  71.  
  72. The patch : 
  73.  
  74.  
  75. *** 1.1    1993/01/27 20:33:13
  76. --- linux/kernel/blk_drv/scsi/seagate.c    1993/01/27 22:16:53
  77. ***************
  78. *** 225,230 ****
  79. --- 225,233 ----
  80.   
  81.   static unsigned char current_target, current_lun;
  82.   static unsigned char *current_cmnd, *current_data;
  83. + static int current_nobuffs;
  84. + static struct scatterlist *current_buffer;
  85. + 
  86.   static int current_bufflen;
  87.   static void (*done_fn)(Scsi_Cmnd *) = NULL;
  88.   static Scsi_Cmnd * SCint = NULL;
  89. ***************
  90. *** 353,358 ****
  91. --- 356,363 ----
  92.       {
  93.       int len;            
  94.       unsigned char *data;    
  95. +     struct scatterlist *buffer;
  96. +     int nobuffs;
  97.       int clock;            
  98.       int temp;
  99.       Scsi_Cmnd * SCtmp;
  100. ***************
  101. *** 371,379 ****
  102.       unsigned char message = 0;
  103.       register unsigned char status_read;
  104.   
  105. -     len=bufflen;
  106. -     data=(unsigned char *) buff;
  107. - 
  108.       incommand = 0;
  109.       st0x_aborted = 0;
  110.   
  111. --- 376,381 ----
  112. ***************
  113. *** 455,463 ****
  114.                   hostno, temp);
  115.               return (DID_BAD_INTR << 16);
  116.               }
  117. !                 data=current_data;      /* WDE add */
  118.                   cmnd=current_cmnd;      /* WDE add */
  119.                   len=current_bufflen;    /* WDE add */
  120.   
  121.   /*
  122.    *     We have determined that we have been selected.  At this point, 
  123. --- 457,468 ----
  124.                   hostno, temp);
  125.               return (DID_BAD_INTR << 16);
  126.               }
  127. ! 
  128. !         buffer=current_buffer;    
  129.                   cmnd=current_cmnd;      /* WDE add */
  130. +                 data=current_data;      /* WDE add */
  131.                   len=current_bufflen;    /* WDE add */
  132. +         nobuffs=current_nobuffs;
  133.   
  134.   /*
  135.    *     We have determined that we have been selected.  At this point, 
  136. ***************
  137. *** 606,611 ****
  138. --- 611,629 ----
  139.                   }
  140.               return retcode(st0x_aborted);
  141.               }    
  142. + 
  143. + /* Establish current pointers.  Take into account scatter / gather */
  144. + 
  145. +         if ((nobuffs = SCint->use_sg)) {
  146. +                 buffer = (struct scatterlist *) SCint->buffer;
  147. +                 len = buffer->length;
  148. +                 data = (unsigned char *) buffer->address;
  149. +         } else {
  150. +                 buffer = NULL;
  151. +                 len = SCint->bufflen;
  152. +                 data = (unsigned char *) SCint->buffer;
  153. +         }
  154. + 
  155.           }
  156.   
  157.       CONTROL = BASE_CMD | CMD_DRVR_ENABLE | 
  158. ***************
  159. *** 748,753 ****
  160. --- 766,779 ----
  161.   /* clobbered */
  162.   "ebx", "ecx", "edi", "esi"); 
  163.   
  164. +                         if (!len && nobuffs) {
  165. +                                 --nobuffs;
  166. +                                 ++buffer;
  167. +                                 len = buffer->length;
  168. +                                 data = (unsigned char *) buffer->address;
  169. +                         }
  170. + 
  171. + 
  172.               break;
  173.   
  174.           case REQ_DATAIN : 
  175. ***************
  176. *** 811,816 ****
  177. --- 837,850 ----
  178.   "0" (data), "1" (len) :
  179.   /* clobbered */
  180.   "ebx", "ecx", "edi", "esi"); 
  181. + 
  182. +                         if (!len && nobuffs) {
  183. +                                 --nobuffs;
  184. +                                 ++buffer;
  185. +                                 len = buffer->length;
  186. +                                 data = (unsigned char *) buffer->address;
  187. +                         }
  188. + 
  189.               break;
  190.   
  191.           case REQ_CMDOUT : 
  192. ***************
  193. *** 858,864 ****
  194. --- 892,901 ----
  195.               case DISCONNECT :
  196.                   should_reconnect = 1;
  197.                                   current_data = data;    /* WDE add */
  198. +                 current_buffer = buffer;
  199.                                   current_bufflen = len;  /* WDE add */
  200. +                 current_nobuffs = nobuffs;
  201. + 
  202.   #if (DEBUG & (PHASE_RESELECT | PHASE_MSGIN))
  203.                   printk("scsi%d : disconnected.\n", hostno);
  204.                   done=1;
  205. ***************
  206. *** 877,891 ****
  207.                   done=1;
  208.                   break;
  209.               case SAVE_POINTERS :
  210. !                 current_data = data;    /* WDE mod */
  211.                                   current_bufflen = len;  /* WDE add */
  212.   #if (DEBUG & PHASE_MSGIN)
  213.                   printk("scsi%d : pointers saved.\n", hostno);
  214.   #endif 
  215.                   break;
  216.               case RESTORE_POINTERS:
  217. !                 data=current_data;    /* WDE mod */
  218.                   cmnd=current_cmnd;
  219.   #if (DEBUG & PHASE_MSGIN)
  220.                   printk("scsi%d : pointers restored.\n", hostno);
  221.   #endif
  222. --- 914,933 ----
  223.                   done=1;
  224.                   break;
  225.               case SAVE_POINTERS :
  226. !                 current_buffer = buffer;
  227.                                   current_bufflen = len;  /* WDE add */
  228. +                 current_data = data;    /* WDE mod */
  229. +                 current_nobuffs = nobuffs;
  230.   #if (DEBUG & PHASE_MSGIN)
  231.                   printk("scsi%d : pointers saved.\n", hostno);
  232.   #endif 
  233.                   break;
  234.               case RESTORE_POINTERS:
  235. !                 buffer=current_buffer;
  236.                   cmnd=current_cmnd;
  237. +                 data=current_data;    /* WDE mod */
  238. +                 len=current_bufflen;
  239. +                 nobuffs=current_nobuffs;
  240.   #if (DEBUG & PHASE_MSGIN)
  241.                   printk("scsi%d : pointers restored.\n", hostno);
  242.   #endif
  243. -- 
  244. Boycott AT&T for their absurd anti-BSDI lawsuit. | Drew Eckhardt
  245. Condemn Colorado for Amendment Two.         | drew@cs.colorado.edu
  246. Use Linux, the fast, flexible, and free 386 unix |  
  247.