home *** CD-ROM | disk | FTP | other *** search
/ Meeting Pearls 3 / Meeting_Pearls_III.iso / Pearls / tcp / Networking / Sana-II / sana2-cslip.readme < prev    next >
Text File  |  1995-05-13  |  4KB  |  98 lines

  1.  
  2. This is a modified version of Commodores example SANA driver:
  3. the slip.device. As released, it contained a number of bugs and
  4. misfeatures.
  5.  
  6. Installation:
  7. ------------
  8. -   It is best to get the full, original SANA developer package. It
  9.     is available on Fish disk number 779. Newer versions may be
  10.     available by ftp.
  11.  
  12. -   Summary: Place the file (c)slip.device in DEVS:Networks/ and
  13.     (c)slip<unit>.config in ENV(ARC):SANA2/.
  14.  
  15. Additions:
  16. ---------
  17. -   Now also header-compressed slip. (See RFC-1144 by Van Jacobson) This 
  18.     is even compatible with non-comressing slip drivers, as it first 
  19.     tries to trigger the other side of the connection into using 
  20.     compression as well, but ceases these attempts if they appear 
  21.     unsuccessful. In any case, once the other side starts sending 
  22.     compressed headers, it is turned on in the local-to-remote direction 
  23.     as well.
  24.  
  25. Corrected bugs:
  26. --------------
  27. -   The AbortIO function erroneously assumed that A3 was the Unit
  28.     pointer. Even Commodore's official example in the RKM gets that
  29.     wrong (see page 567).
  30.  
  31. -   S2_GETGLOBALSTATS always returned an error, even though it worked.
  32.     Well, it got the microseconds of the last online wrong.
  33.  
  34. -   S2_TRACKTYPE had two bugs: it would always think a type that you
  35.     requested to track would not yet be tracked, even if it would
  36.     get the packet type right, which it didn't. As a result, every
  37.     call to S2_TRACKTYPE allocated memory for the statistics, yet
  38.     S2_GETTYPESTATS always claimed the type wasn't being tracked.
  39.  
  40. -   S2_GETSTATIONADDRESS should return 2 addresses, the current address,
  41.     and the ROM address. It used to give only a 0.0.0.0 "ROM" address in
  42.     the wrong location. Now, it considers the address in the config
  43.     file the "ROM" address, and the one from S2_CONFIGINTERFACE is
  44.     the current address. Initially both are the same (and both are
  45.     futher ignored since they are meaningless to slip).
  46.  
  47. -   S2_DEVICEQUERY didn't give all required information, and claimed a
  48.     weird size for it.
  49.  
  50. -   Flags set when opening the serial device should be in io->io_SerFlags,
  51.     not in the flags argument to OpenDevice().
  52.  
  53. -   Returned read requests from the serial device were sized by their
  54.     io_Length instead of io_Actual.
  55.  
  56. -   Serial input could potentially overflow the input buffer.
  57.  
  58. -   The code assumed, by using registered arguments, that the callback 
  59.     routines would have their arguments in the correct registers.  Now 
  60.     the type of the pointer explicitly declares the calling convention.
  61.  
  62. -   And the worst of all: it attempted this EXTREMELY STUPID TRICK:
  63.  
  64.     #define SLIPBase ((struct SLIPDevice *)__builtin_getreg(14))
  65.  
  66.     This is terrible!!! That people write code like this! How can you be
  67.     sure the compiler won't use A6 for a register variable somewhere and
  68.     make it unusable in the functions that it calls? In fact, SAS/C 6.2
  69.     does something like that, in ReadConfig..., when it first calls an Exec
  70.     function and then a DOS function, and in both cases fetches the
  71.     required base pointer off A6.
  72.  
  73.     If you want global register variables, use a compiler that supports
  74.     them, such as gcc.
  75.  
  76. Corrected misfeatures:
  77. ---------------------
  78.  
  79. -   The device now runs at the same priority as the opener. This helps
  80.     preventing starvation of the opener in case of high amounts of
  81.     input.
  82.  
  83. -   Uses the serial.device's SERF_EOFMODE mode. This prevents the
  84.     slip.device from hogging the CPU at high priority, and even at equal
  85.     priority improves efficiency.
  86.  
  87. Debatable issues:
  88. ----------------
  89.  
  90. -   Optionally, serial device reads are only initiated when they are
  91.     (already) requested by the caller. This is contrary to the SANA
  92.     philosophy, and therefore makes debugging protocol stacks more
  93.     difficult. On the other hand, why not use the serial device's buffer to
  94.     keep input, instead of dropping it? This also reduces CPU load in case
  95.     the device is open but unused for some reason.  (As it was, the
  96.     slip.device would continually read and decode packets, even if nobody
  97.     wanted them.)
  98.