*) Add multithread support to the fcgiapp lib and an example multithreaded
roberts [Mon, 26 Jul 1999 04:28:07 +0000 (04:28 +0000)]
    application, threaded.c.  Based on work by Dennis Payne
    <dpayne@softscape.com> and Gene Sokolov <hook@aktrad.ru>.

Modified Files: README examples/Makefile.in libfcgi/fcgiapp.c include/fcgiapp.h
Added Files: examples/threaded.c

README
examples/Makefile.in
examples/threaded.c [new file with mode: 0755]
include/fcgiapp.h
libfcgi/fcgiapp.c

diff --git a/README b/README
index b565def..cacc9cd 100755 (executable)
--- a/README
+++ b/README
@@ -2,7 +2,7 @@ FastCGI Developer's Kit README
 ------------------------------
 
     Version 2.0b2, 04 April 1997
-    $Id: README,v 1.4 1999/06/10 21:20:33 roberts Exp $
+    $Id: README,v 1.5 1999/07/26 04:28:07 roberts Exp $
     Copyright (c) 1996 Open Market, Inc.
     See the file "LICENSE.TERMS" for information on usage and redistribution
     of this file, and for a DISCLAIMER OF ALL WARRANTIES.
@@ -38,6 +38,10 @@ release of the FDK.
 Changes with devkit 2.1.1
 -------------------------
 
+ *) Add multithread support to the fcgiapp lib and an example multithreaded
+    application, threaded.c.  Based on work by Dennis Payne 
+    <dpayne@softscape.com> and Gene Sokolov <hook@aktrad.ru>.
+
  *) Remove the printf() and #include of stdio.h from examples/echo2.c.
 
  *) Remove the static initialization of _fcgi_sF[] because on glibc 2.x based
index 153288f..da6a0be 100644 (file)
@@ -3,7 +3,7 @@
 #
 #  Open Market, Inc.
 #
-#  $Id: Makefile.in,v 1.1 1997/09/16 15:36:28 stanleyg Exp $
+#  $Id: Makefile.in,v 1.2 1999/07/26 04:28:08 roberts Exp $
 #
 
 SHELL = @SHELL@
@@ -26,10 +26,13 @@ LIBDIR      = ../libfcgi
 LIBFCGI            = $(LIBDIR)/libfcgi.${L}
 TARGETS            = tiny-cgi.cgi tiny-fcgi tiny-fcgi2 tiny-authorizer \
              echo echo2 sample-store sockets echo.fcg perl tclsh \
-              SampleStore.state.0 SampleStore.state.1 log-dump
+              SampleStore.state.0 SampleStore.state.1 log-dump threaded.fcg
 
 all: $(TARGETS)
 
+threaded.fcg: threaded.c $(INCLUDES)
+       $(CC) $(CFLAGS) -D_REENTRANT threaded.c -o threaded.fcg $(LIBFCGI) -lpthread
+
 tiny-cgi.cgi: tiny-cgi.${O}
        $(CC) $(CFLAGS) tiny-cgi.${O} -o tiny-cgi.cgi
 
diff --git a/examples/threaded.c b/examples/threaded.c
new file mode 100755 (executable)
index 0000000..8618187
--- /dev/null
@@ -0,0 +1,71 @@
+/* 
+ * threaded.c -- A simple multi-threaded FastCGI application.
+ */
+
+#ifndef lint
+static const char rcsid[] = "$Id: threaded.c,v 1.1 1999/07/26 04:28:07 roberts Exp $";
+#endif /* not lint */
+
+#if defined HAVE_UNISTD_H || defined __linux__
+#include <unistd.h>
+#endif
+
+#include "fcgiapp.h"
+
+#ifdef _WIN32
+#include <process.h>
+#endif
+
+#include <pthread.h>
+
+#define THREAD_COUNT 20
+
+int count[THREAD_COUNT];
+
+static void *doit(void *a)
+{
+    int k = (int)a;
+    FCGX_Request request;
+    FCGX_Stream *in, *out, *err;
+    FCGX_ParamArray envp;
+    int i;
+
+    FCGX_InitRequest(&request);
+
+    while (FCGX_Accept_r(&in, &out, &err, &envp, &request) >= 0)
+    {
+        FCGX_FPrintF(out,
+           "Content-type: text/html\r\n"
+           "\r\n"
+           "<title>FastCGI Hello! (multi-threaded C, fcgiapp library)</title>"
+           "<h1>FastCGI Hello! (multi-threaded C, fcgiapp library)</h1>"
+           "Request counts for %d threads running on host <i>%s</i><P><CODE>",
+              THREAD_COUNT, FCGX_GetParam("SERVER_NAME", envp));
+
+        count[k]++;
+
+        for (i = 0; i < THREAD_COUNT; i++)
+            FCGX_FPrintF(out, "%5d " , count[i]);
+    }
+
+    return NULL;
+}
+
+int main(void)
+{
+    int i;
+    pthread_t id[THREAD_COUNT];
+
+    FCGX_Init();
+
+    for (i = 0; i < THREAD_COUNT; i++)
+        count[i] = 0;
+
+    for (i = 1; i < THREAD_COUNT; i++)
+        pthread_create(&id[i], NULL, doit, (void*)i);
+   
+    doit(0);
+
+    exit(0);
+}
+
index ebde17a..f05e36d 100644 (file)
@@ -9,7 +9,7 @@
  * See the file "LICENSE.TERMS" for information on usage and redistribution
  * of this file, and for a DISCLAIMER OF ALL WARRANTIES.
  *
- * $Id: fcgiapp.h,v 1.1 1997/09/16 15:36:32 stanleyg Exp $
+ * $Id: fcgiapp.h,v 1.2 1999/07/26 04:28:10 roberts Exp $
  */
 
 #ifndef _FCGIAPP_H
@@ -77,6 +77,25 @@ typedef struct FCGX_Stream {
  */
 typedef char **FCGX_ParamArray;
 
+/*
+ * State associated with a request.  
+ *
+ * Its exposed for API simplicity, DON'T use it - it WILL change!
+ */
+typedef struct FCGX_Request {
+    int ipcFd;               /* < 0 means no connection */
+    int isBeginProcessed;     /* FCGI_BEGIN_REQUEST seen */
+    int requestId;            /* valid if isBeginProcessed */
+    int keepConnection;       /* don't close ipcFd at end of request */
+    int role;
+    int appStatus;
+    int nWriters;             /* number of open writers (0..2) */
+    FCGX_Stream *inStream;
+    FCGX_Stream *outStream;
+    FCGX_Stream *errStream;
+    struct Params *paramsPtr;
+} FCGX_Request;
+
 \f
 /*
  *======================================================================
@@ -99,9 +118,92 @@ DLLAPI int FCGX_IsCGI(void);
 /*
  *----------------------------------------------------------------------
  *
+ * FCGX_Init --
+ *
+ *      Initialize the FCGX library.  Call in multi-threaded apps
+ *      before calling FCGX_Accept_r().
+ *
+ *      Returns 0 upon success.
+ *
+ *----------------------------------------------------------------------
+ */
+DLLAPI int FCGX_Init(void);
+
+/*
+ *----------------------------------------------------------------------
+ *
+ * FCGX_InitRequest --
+ *
+ *      Initialize a FCGX_Request for use with FCGX_Accept_r().
+ *
+ *----------------------------------------------------------------------
+ */
+DLLAPI void FCGX_InitRequest(FCGX_Request *request);
+
+/*
+ *----------------------------------------------------------------------
+ *
+ * FCGX_Accept_r --
+ *
+ *      Accept a new request (multi-thread safe).  Be sure to call 
+ *     FCGX_Init() first.
+ *
+ * Results:
+ *     0 for successful call, -1 for error.
+ *
+ * Side effects:
+ *
+ *      Finishes the request accepted by (and frees any
+ *      storage allocated by) the previous call to FCGX_Accept.
+ *      Creates input, output, and error streams and
+ *      assigns them to *in, *out, and *err respectively.
+ *      Creates a parameters data structure to be accessed
+ *      via getenv(3) (if assigned to environ) or by FCGX_GetParam
+ *      and assigns it to *envp.
+ *
+ *      DO NOT retain pointers to the envp array or any strings
+ *      contained in it (e.g. to the result of calling FCGX_GetParam),
+ *      since these will be freed by the next call to FCGX_Finish
+ *      or FCGX_Accept.
+ *
+ *     DON'T use the FCGX_Request, its structure WILL change.
+ *
+ *----------------------------------------------------------------------
+ */
+DLLAPI int FCGX_Accept_r(
+        FCGX_Stream **in,
+        FCGX_Stream **out,
+        FCGX_Stream **err,
+        FCGX_ParamArray *envp,
+        FCGX_Request *request);
+
+/*
+ *----------------------------------------------------------------------
+ *
+ * FCGX_Finish_r --
+ *
+ *      Finish the request (multi-thread safe).
+ *
+ * Side effects:
+ *
+ *      Finishes the request accepted by (and frees any
+ *      storage allocated by) the previous call to FCGX_Accept.
+ *
+ *      DO NOT retain pointers to the envp array or any strings
+ *      contained in it (e.g. to the result of calling FCGX_GetParam),
+ *      since these will be freed by the next call to FCGX_Finish
+ *      or FCGX_Accept.
+ *
+ *----------------------------------------------------------------------
+ */
+DLLAPI void FCGX_Finish_r(FCGX_Request *request);
+
+/*
+ *----------------------------------------------------------------------
+ *
  * FCGX_Accept --
  *
- *      Accepts a new request from the HTTP server.
+ *      Accept a new request (NOT multi-thread safe).
  *
  * Results:
  *     0 for successful call, -1 for error.
@@ -134,7 +236,7 @@ DLLAPI int FCGX_Accept(
  *
  * FCGX_Finish --
  *
- *      Finishes the current request from the HTTP server.
+ *      Finish the current request (NOT multi-thread safe).
  *
  * Side effects:
  *
index 9b65b15..cb86324 100644 (file)
@@ -12,7 +12,7 @@
  */
 
 #ifndef lint
-static const char rcsid[] = "$Id: fcgiapp.c,v 1.3 1999/02/12 00:46:41 roberts Exp $";
+static const char rcsid[] = "$Id: fcgiapp.c,v 1.4 1999/07/26 04:28:08 roberts Exp $";
 #endif /* not lint */
 
 #ifdef _WIN32
@@ -59,7 +59,15 @@ static const char rcsid[] = "$Id: fcgiapp.c,v 1.3 1999/02/12 00:46:41 roberts Ex
 #define LONG_DOUBLE long double
 #endif
 
-static int osLibInitialized = 0;
+
+/*
+ * Globals 
+ */
+static int libInitialized = 0;
+static char *webServerAddressList = NULL;
+static FCGX_Request reqData;
+static FCGX_Request *reqDataPtr = &reqData;
+
 
 static void *Malloc(size_t size)
 {
@@ -1249,23 +1257,6 @@ static unsigned char *AlignPtr8(unsigned char *p) {
     return p + u;
 }
 \f
-/*
- * State associated with a request
- */
-typedef struct ReqData {
-    int ipcFd;               /* < 0 means no connection */
-    int isBeginProcessed;     /* FCGI_BEGIN_REQUEST seen */
-    int requestId;            /* valid if isBeginProcessed */
-    int keepConnection;       /* don't close ipcFd at end of request */
-    int role;
-    int appStatus;
-    int nWriters;             /* number of open writers (0..2) */
-    FCGX_Stream *inStream;
-    FCGX_Stream *outStream;
-    FCGX_Stream *errStream;
-    ParamsPtr paramsPtr;
-} ReqData;
-
 
 /*
  * State associated with a stream
@@ -1286,7 +1277,7 @@ typedef struct FCGX_Stream_Data {
     int paddingLen;           /* reader: bytes of unread padding */
     int isAnythingWritten;    /* writer: data has been written to ipcFd */
     int rawWrite;             /* writer: write data without stream headers */
-    ReqData *reqDataPtr;      /* request data not specific to one stream */
+    FCGX_Request *reqDataPtr; /* request data not specific to one stream */
 } FCGX_Stream_Data;
 \f
 /*
@@ -1729,7 +1720,7 @@ static void FillBuffProc(FCGX_Stream *stream)
  *----------------------------------------------------------------------
  */
 static FCGX_Stream *NewStream(
-        ReqData *reqDataPtr, int bufflen, int isReader, int streamType)
+        FCGX_Request *reqDataPtr, int bufflen, int isReader, int streamType)
 {
     /*
      * XXX: It would be a lot cleaner to have a NewStream that only
@@ -1843,7 +1834,7 @@ static FCGX_Stream *SetReaderType(FCGX_Stream *stream, int streamType)
  *
  *----------------------------------------------------------------------
  */
-static FCGX_Stream *NewReader(ReqData *reqDataPtr, int bufflen, int streamType)
+static FCGX_Stream *NewReader(FCGX_Request *reqDataPtr, int bufflen, int streamType)
 {
     return NewStream(reqDataPtr, bufflen, TRUE, streamType);
 }
@@ -1860,7 +1851,7 @@ static FCGX_Stream *NewReader(ReqData *reqDataPtr, int bufflen, int streamType)
  *
  *----------------------------------------------------------------------
  */
-static FCGX_Stream *NewWriter(ReqData *reqDataPtr, int bufflen, int streamType)
+static FCGX_Stream *NewWriter(FCGX_Request *reqDataPtr, int bufflen, int streamType)
 {
     return NewStream(reqDataPtr, bufflen, FALSE, streamType);
 }
@@ -1884,7 +1875,7 @@ FCGX_Stream *CreateWriter(
         int bufflen,
         int streamType)
 {
-    ReqData *reqDataPtr = Malloc(sizeof(ReqData));
+    FCGX_Request *reqDataPtr = Malloc(sizeof(FCGX_Request));
     reqDataPtr->ipcFd = ipcFd;
     reqDataPtr->requestId = requestId;
     /*
@@ -1900,9 +1891,6 @@ FCGX_Stream *CreateWriter(
  *======================================================================
  */
 
-static int isCGI = -1;
-static int isFastCGI = -1;
-
 /*
  *----------------------------------------------------------------------
  *
@@ -1925,23 +1913,23 @@ static int isFastCGI = -1;
  */
 int FCGX_IsCGI(void)
 {
-    /*
-     * Already been here, no need to test again.
-     */
-    if(isCGI != -1) {
-        return isCGI;
-    }
+    static int isFastCGI = -1;
     
-    if(!osLibInitialized) {
-        if(OS_LibInit(NULL) == -1) {
-           exit(OS_Errno);
-       }
-       osLibInitialized = 1;
+    if (isFastCGI != -1) {
+        return !isFastCGI;
     }
+    
+    if (!libInitialized) {
+        int rc = FCGX_Init();
+        if (rc) {
+            /* exit() isn't great, but hey */
+            exit((rc < 0) ? rc : -rc);
+        }
+    }                 
 
     isFastCGI = OS_IsFcgi();
-    isCGI = !isFastCGI;
-    return isCGI;
+
+    return !isFastCGI;
 }
 \f
 /*
@@ -1963,31 +1951,105 @@ int FCGX_IsCGI(void)
  *
  *----------------------------------------------------------------------
  */
-static ReqData *reqDataPtr = NULL;
 
 void FCGX_Finish(void)
 {
-    if(reqDataPtr != NULL && reqDataPtr->inStream != NULL) {
-        /*
-         * Complete the previous request.
-         */
+    FCGX_Finish_r(reqDataPtr);
+}
+
+/*
+ *----------------------------------------------------------------------
+ *
+ * FCGX_Finish_r --
+ *
+ *      Finishes the current request from the HTTP server.
+ *
+ * Side effects:
+ *
+ *      Finishes the request accepted by (and frees any
+ *      storage allocated by) the previous call to FCGX_Accept.
+ *
+ *      DO NOT retain pointers to the envp array or any strings
+ *      contained in it (e.g. to the result of calling FCGX_GetParam),
+ *      since these will be freed by the next call to FCGX_Finish
+ *      or FCGX_Accept.
+ *
+ *----------------------------------------------------------------------
+ */
+void FCGX_Finish_r(FCGX_Request *reqDataPtr)
+{
+    if (reqDataPtr == NULL) {
+        return;
+    }
+
+    if (reqDataPtr->inStream) {
         int errStatus = FCGX_FClose(reqDataPtr->errStream);
         int outStatus = FCGX_FClose(reqDataPtr->outStream);
-        int prevRequestFailed = (errStatus != 0)
-                || (outStatus != 0)
-                || (FCGX_GetError(reqDataPtr->inStream) != 0);
+
+        if (errStatus  || outStatus 
+            || FCGX_GetError(reqDataPtr->inStream) 
+            || !reqDataPtr->keepConnection) 
+        {
+            OS_IpcClose(reqDataPtr->ipcFd);
+        }
+
         ASSERT(reqDataPtr->nWriters == 0);
+
         FreeStream(&reqDataPtr->inStream);
         FreeStream(&reqDataPtr->outStream);
         FreeStream(&reqDataPtr->errStream);
+
         FreeParams(&reqDataPtr->paramsPtr);
-        if(prevRequestFailed || !reqDataPtr->keepConnection) {
-            OS_IpcClose(reqDataPtr->ipcFd);
-            reqDataPtr->ipcFd = -1;
-        }
+    }
+
+    if (!reqDataPtr->keepConnection) {
+        reqDataPtr->ipcFd = -1;
     }
 }
 \f
+
+void FCGX_InitRequest(FCGX_Request *request)
+{
+    memset(request, 0, sizeof(FCGX_Request));
+}
+
+/*
+ *----------------------------------------------------------------------
+ *
+ * FCGX_Init --
+ *
+ *      Initilize the FCGX library.  This is called by FCGX_Accept()
+ *      but must be called by the user when using FCGX_Accept_r().
+ *
+ * Results:
+ *         0 for successful call.
+ *
+ *----------------------------------------------------------------------
+ */
+int FCGX_Init(void)
+{
+    char *p;
+    
+    if (libInitialized) {
+        return 0;
+    }
+
+    /* If our compiler doesn't play by the ISO rules for struct layout, halt. */
+    ASSERT(sizeof(FCGI_Header) == FCGI_HEADER_LEN);
+
+    memset(&reqData, 0, sizeof(FCGX_Request));
+
+    if (OS_LibInit(NULL) == -1) {
+        return OS_Errno ? OS_Errno : -9997;
+    }
+
+    p = getenv("FCGI_WEB_SERVER_ADDRS");
+    webServerAddressList = p ? StringCopy(p) : "";
+    
+    libInitialized = 1;
+    return 0;
+}
+
 /*
  *----------------------------------------------------------------------
  *
@@ -2015,8 +2077,6 @@ void FCGX_Finish(void)
  *
  *----------------------------------------------------------------------
  */
-static ReqData reqData;
-static char *webServerAddressList = NULL;
 
 int FCGX_Accept(
         FCGX_Stream **in,
@@ -2024,59 +2084,69 @@ int FCGX_Accept(
         FCGX_Stream **err,
         FCGX_ParamArray *envp)
 {
-    /*
-     * If our compiler doesn't play by the ISO rules for struct
-     * layout, halt.
-     */
-    ASSERT(sizeof(FCGI_Header) == FCGI_HEADER_LEN);
+    if (!libInitialized) {
+        int rc = FCGX_Init();
+        if (rc) {
+            return (rc < 0) ? rc : -rc;
+        }
+    }                 
 
-    if(!osLibInitialized) {
-        if(OS_LibInit(NULL) == -1) {
-           exit(OS_Errno);
-       }
-       osLibInitialized = 1;
-    }
+    return FCGX_Accept_r(in, out, err, envp, &reqData);
+}
 
-    /*
-     * If our compiler doesn't play by the ISO rules for struct
-     * layout, halt.
-     */
-    ASSERT(sizeof(FCGI_Header) == FCGI_HEADER_LEN);
-    
-    if(reqDataPtr == NULL) {
-       /*
-        * Very first call, so capture FCGI_WEB_SERVER_ADDRS from
-         * the initial environment, and initialize reqDataPtr
-         * and parts of reqData.
-        */
-        char *p = getenv("FCGI_WEB_SERVER_ADDRS");
-       if (p != NULL) {
-            webServerAddressList = StringCopy(p);
-       }
-        reqDataPtr = &reqData;
-        reqDataPtr->ipcFd = -1;
-        reqDataPtr->inStream = NULL;
-        reqDataPtr->outStream = NULL;
-        reqDataPtr->errStream = NULL;
-    } else {
-        /*
-         * Not the first call.  Finish the current request, if any.
-         */
-        FCGX_Finish();
+/*
+ *----------------------------------------------------------------------
+ *
+ * FCGX_Accept_r --
+ *
+ *      Accepts a new request from the HTTP server.
+ *
+ * Results:
+ *     0 for successful call, -1 for error.
+ *
+ * Side effects:
+ *
+ *      Finishes the request accepted by (and frees any
+ *      storage allocated by) the previous call to FCGX_Accept.
+ *      Creates input, output, and error streams and
+ *      assigns them to *in, *out, and *err respectively.
+ *      Creates a parameters data structure to be accessed
+ *      via getenv(3) (if assigned to environ) or by FCGX_GetParam
+ *      and assigns it to *envp.
+ *
+ *      DO NOT retain pointers to the envp array or any strings
+ *      contained in it (e.g. to the result of calling FCGX_GetParam),
+ *      since these will be freed by the next call to FCGX_Finish
+ *      or FCGX_Accept.
+ *
+ *----------------------------------------------------------------------
+ */
+int FCGX_Accept_r(
+        FCGX_Stream **in,
+        FCGX_Stream **out,
+        FCGX_Stream **err,
+        FCGX_ParamArray *envp,
+        FCGX_Request *reqDataPtr)
+{
+    if (!libInitialized) {
+        return -9998;
     }
-    for(;;) {
+
+    /* Finish the current request, if any. */
+    FCGX_Finish_r(reqDataPtr);
+
+    for (;;) {
         /*
          * If a connection isn't open, accept a new connection (blocking).
          * If an OS error occurs in accepting the connection,
          * return -1 to the caller, who should exit.
          */
-        if(reqDataPtr->ipcFd < 0) {
-           reqDataPtr->ipcFd = OS_FcgiIpcAccept(webServerAddressList);
-           if(reqDataPtr->ipcFd < 0) {
-                reqDataPtr = NULL;
-                   return (errno > 0) ? (0 - errno) : -9999;
-           }
-       }
+        if (reqDataPtr->ipcFd < 0) {
+            reqDataPtr->ipcFd = OS_FcgiIpcAccept(webServerAddressList);
+            if (reqDataPtr->ipcFd < 0) {
+                return (errno > 0) ? (0 - errno) : -9999;
+            }
+        }
         /*
          * A connection is open.  Read from the connection in order to
          * get the request's role and environment.  If protocol or other
@@ -2189,3 +2259,4 @@ void FCGX_SetExitStatus(int status, FCGX_Stream *stream)
     FCGX_Stream_Data *data = stream->data;
     data->reqDataPtr->appStatus = status;
 }
+