You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by dr...@apache.org on 2001/12/14 14:42:17 UTC

cvs commit: httpd-2.0/server/mpm/beos beos.c

dreid       01/12/14 05:42:17

  Modified:    server/mpm/beos beos.c
  Log:
  Change the beos mpm to use the new pools creation call to try and
  improve performance a little.  Tidy up a comment.
  
  Revision  Changes    Path
  1.73      +5 -4      httpd-2.0/server/mpm/beos/beos.c
  
  Index: beos.c
  ===================================================================
  RCS file: /home/cvs/httpd-2.0/server/mpm/beos/beos.c,v
  retrieving revision 1.72
  retrieving revision 1.73
  diff -u -r1.72 -r1.73
  --- beos.c	2001/11/17 14:02:25	1.72
  +++ beos.c	2001/12/14 13:42:17	1.73
  @@ -345,7 +345,8 @@
       sigfillset(&sig_mask);
       sigprocmask(SIG_BLOCK, &sig_mask, NULL);
   
  -    apr_pool_create(&ptrans, tpool);
  +    apr_pool_create_ex(&ptrans, tpool, NULL, APR_POOL_FNEW_ALLOCATOR);
  +    apr_pool_tag(ptrans, "transaction");
   
       apr_lock_acquire(worker_thread_count_mutex);
       worker_thread_count++;
  @@ -359,9 +360,9 @@
           apr_poll_socket_add(pollset, listening_sockets[n], APR_POLLIN);
   
       while (1) {
  -        /* If we're here, then chances are (unless we're the first thread created) we're going
  -           to be held up on the accept_muetx, so doing this here shouldn't be a peformance hit.
  -           If it is, you probably need more threads...
  +        /* If we're here, then chances are (unless we're the first thread created) 
  +         * we're going to be held up in the accept mutex, so doing this here
  +         * shouldn't hurt performance.
            */
   
           this_worker_should_exit |= (ap_max_requests_per_thread != 0) && (requests_this_child <= 0);