home *** CD-ROM | disk | FTP | other *** search
/ NetNews Usenet Archive 1992 #30 / NN_1992_30.iso / spool / comp / mail / mh / 1426 < prev    next >
Encoding:
Text File  |  1992-12-21  |  6.4 KB  |  219 lines
  1. Xref: sparky comp.mail.mh:1426 comp.mail.sendmail:3045
  2. Path: sparky!uunet!haven.umd.edu!decuac!pa.dec.com!vixie
  3. From: vixie@pa.dec.com (Paul A Vixie)
  4. Newsgroups: comp.mail.mh,comp.mail.sendmail
  5. Subject: Re: MH 6.8 sendmail/smtp uses "ONEX" (w. fix for IDA sendmail)
  6. Date: 19 Dec 92 15:29:11
  7. Organization: DEC Network Software Lab
  8. Lines: 204
  9. Distribution: world
  10. Message-ID: <VIXIE.92Dec19152911@cognition.pa.dec.com>
  11. References: <19322.724536025@dbc.mtview.ca.us> <BzFu4v.MxL@news.cso.uiuc.edu>
  12. NNTP-Posting-Host: cognition.pa.dec.com
  13. In-reply-to: paul@uxc.cso.uiuc.edu's message of Fri, 18 Dec 1992 04:36:30 GMT
  14.  
  15. In the October 1 release of King James Sendmail, we find in README.decwrl...
  16.  
  17. >option O (OO in .cf or -oO on command line) sets SynchRcv which is also set
  18. >by SMTP's ONEX command; if set, we will not fork on MAIL FROM commands.  we
  19. >(lovstrand and vixie) believe that we can recover the previous state without
  20. >having to isolate ourselves to a subprocess. this has performance implications
  21. >and also helps avoid the shared-input bug for brain-dead PC mailers that do
  22. >not wait for result codes and instead fill up the TCP pipe with their entire
  23. >conversation before reading back any result codes.
  24.  
  25. ...and if you turn this on, sendmail never forks -- i.e., it does what ONEX
  26. causes now, and has the side-effect that ONEX followed by multiple MAIL's is
  27. not an error.  lennart and i chose "O" as the option to remind us of "ONEX";
  28. this actually shipped with NeXT-Step 3.0 as far as I know.
  29.  
  30. Note that this has the effect of creating all the [qdtx]f[A-Z][A-Z]\d\d\d\d\d
  31. files in the mail queue directory with the same \d\d\d\d\d (PID) field, which
  32. is not a huge problem until you start using BSMTP or otherwise sending a lot
  33. of transactions in the same session.  Finding out that you need to be creating
  34. qfAL12345 requires 11 IO-intensive system calls.  This is not the average case
  35. for us, so this is still a big performance win.  However, it's clear that more
  36. thought (a lot more!) is needed.
  37.  
  38. Diffs follow.  Before you yell about the apparently-permanent effects of ONEX
  39. on the global variable SynchRcv, think about which forks have _already_
  40. happened by the time this code gets run.  Note, also, that these diffs are
  41. very similar to Paul Pomes', and there's a reason for that :-)..
  42.  
  43. *** 1.8    1992/05/04 18:03:24
  44. --- 1.9    1992/10/01 18:06:53
  45. ***************
  46. *** 20,26 ****
  47.   
  48.   #ifndef lint
  49.   static char sccsid[] = "@(#)readcf.c    5.21 (Berkeley) 6/1/90";
  50. ! static char rcsid[] = "$Id: readcf.c,v 1.8 1992/05/04 18:03:24 vixie Exp $";
  51.   #endif /* not lint */
  52.   
  53.   # include "sendmail.h"
  54. --- 20,26 ----
  55.   
  56.   #ifndef lint
  57.   static char sccsid[] = "@(#)readcf.c    5.21 (Berkeley) 6/1/90";
  58. ! static char rcsid[] = "$Id: readcf.c,v 1.9 1992/10/01 18:06:53 vixie Exp $";
  59.   #endif /* not lint */
  60.   
  61.   # include "sendmail.h"
  62. ***************
  63. *** 701,706 ****
  64. --- 701,707 ----
  65.       extern time_t convtime();
  66.       extern int QueueLA;
  67.       extern int RefuseLA;
  68. +     extern bool SynchRcv;
  69.       extern bool trusteduser();
  70.       extern char *username();
  71.   
  72. ***************
  73. *** 901,906 ****
  74. --- 902,911 ----
  75.               CurEnv->e_flags |= EF_OLDSTYLE;
  76.           else
  77.               CurEnv->e_flags &= ~EF_OLDSTYLE;
  78. +         break;
  79. + 
  80. +       case 'O':        /* Do synchronous receive in srvrsmtp */
  81. +         SynchRcv = atobool(val);
  82.           break;
  83.   
  84.         case 'P':        /* postmaster copy address for returned mail */
  85. *** 1.8    1992/05/04 18:05:40
  86. --- srvrsmtp.c    1992/10/01 18:07:49
  87. ***************
  88. *** 25,36 ****
  89.   static char sccsid[] =
  90.     "@(#)srvrsmtp.c    5.28 (Berkeley) 6/1/90 (with SMTP)";
  91.   static char rcsid[] =
  92. !   "$Id: srvrsmtp.c,v 1.8 1992/05/04 18:05:40 vixie Exp $ (with SMTP)";
  93.   #else
  94.   static char sccsid[] =
  95.     "@(#)srvrsmtp.c    5.28 (Berkeley) 6/1/90 (without SMTP)";
  96.   static char rcsid[] =
  97. !   "$Id: srvrsmtp.c,v 1.8 1992/05/04 18:05:40 vixie Exp $ (without SMTP)";
  98.   #endif
  99.   #endif /* not lint */
  100.   
  101. --- 25,36 ----
  102.   static char sccsid[] =
  103.     "@(#)srvrsmtp.c    5.28 (Berkeley) 6/1/90 (with SMTP)";
  104.   static char rcsid[] =
  105. !   "$Id: srvrsmtp.c,v 1.9 1992/10/01 18:07:49 vixie Exp $ (with SMTP)";
  106.   #else
  107.   static char sccsid[] =
  108.     "@(#)srvrsmtp.c    5.28 (Berkeley) 6/1/90 (without SMTP)";
  109.   static char rcsid[] =
  110. !   "$Id: srvrsmtp.c,v 1.9 1992/10/01 18:07:49 vixie Exp $ (without SMTP)";
  111.   #endif
  112.   #endif /* not lint */
  113.   
  114. ***************
  115. *** 100,106 ****
  116.   };
  117.   
  118.   bool    InChild = FALSE;        /* true if running in a subprocess */
  119. ! bool    OneXact = FALSE;        /* one xaction only this run */
  120.   
  121.   #define EX_QUIT        22        /* special code for QUIT command */
  122.   
  123. --- 100,106 ----
  124.   };
  125.   
  126.   bool    InChild = FALSE;        /* true if running in a subprocess */
  127. ! bool    SynchRcv = FALSE;        /* one xaction only this run */
  128.   
  129.   #define EX_QUIT        22        /* special code for QUIT command */
  130.   
  131. ***************
  132. *** 398,403 ****
  133. --- 398,411 ----
  134.               else
  135.                   CurEnv->e_flags &= ~EF_FATALERRS;
  136.   
  137. +             /*FALLTHROUGH*/
  138. + 
  139. +           case CMDRSET:
  140. +             if (c->cmdcode == CMDRSET) {
  141. +                 /* not a fallthrough from DATA */
  142. +                 message("250", "Reset state");
  143. +             }
  144. + 
  145.               /* if in a child, pop back to our parent */
  146.               if (InChild) {
  147.   #ifdef DECWRL
  148. ***************
  149. *** 407,413 ****
  150.               }
  151.   
  152.               /* clean up a bit */
  153. !             hasmail = 0;
  154.               dropenvelope(CurEnv);
  155.               CurEnv = newenvelope(CurEnv);
  156.               CurEnv->e_flags = BlankEnvelope.e_flags;
  157. --- 415,421 ----
  158.               }
  159.   
  160.               /* clean up a bit */
  161. !             hasmail = FALSE;
  162.               dropenvelope(CurEnv);
  163.               CurEnv = newenvelope(CurEnv);
  164.               CurEnv->e_flags = BlankEnvelope.e_flags;
  165. ***************
  166. *** 416,427 ****
  167.   #endif
  168.               break;
  169.   
  170. -           case CMDRSET:        /* rset -- reset state */
  171. -             message("250", "Reset state");
  172. -             if (InChild)
  173. -                 finis();
  174. -             break;
  175. - 
  176.             case CMDVRFY:        /* vrfy -- verify address */
  177.   #ifndef DECWRL
  178.               if (!batched && !runinchild("SMTP-VRFY") > 0)
  179. --- 424,429 ----
  180. ***************
  181. *** 491,497 ****
  182.               break;
  183.   
  184.             case CMDONEX:        /* doing one transaction only */
  185. !             OneXact = TRUE;
  186.               message("200", "Only one transaction");
  187.               break;
  188.   
  189. --- 493,499 ----
  190.               break;
  191.   
  192.             case CMDONEX:        /* doing one transaction only */
  193. !             SynchRcv = TRUE;
  194.               message("200", "Only one transaction");
  195.               break;
  196.   
  197. ***************
  198. *** 656,662 ****
  199.   {
  200.       int childpid;
  201.   
  202. !     if (!OneXact)
  203.       {
  204.           childpid = dofork();
  205.           if (childpid < 0)
  206. --- 658,664 ----
  207.   {
  208.       int childpid;
  209.   
  210. !     if (!SynchRcv)
  211.       {
  212.           childpid = dofork();
  213.           if (childpid < 0)
  214. --
  215. Paul Vixie, DEC Network Systems Lab    
  216. Palo Alto, California, USA             "Don't be a rebel, or a conformist;
  217. <vixie@pa.dec.com> decwrl!vixie        they're the same thing, anyway.  Find
  218. <paul@vix.com>     vixie!paul        your own path, and stay on it."  -me
  219.