httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ryan Bloom <...@covalent.net>
Subject Re: [PATCH] Standardize AcceptMutex config
Date Thu, 20 Sep 2001 02:53:47 GMT
On Wednesday 19 September 2001 07:40 pm, Justin Erenkrantz wrote:
> Calling the AcceptMutex proc_pthread is just silly.  1.3 got it
> right by calling it pthread.
>
> This would also be one less thing for a person who uses this new
> feature in 1.3 to change when they upgrade to 2.0.
>
> Would anyone raise a fit if I committed this?  -- justin

Why is calling it proc_pthread silly?  We are talking about a pthread based
process lock.  Personally, I think Apache 1.3 should be changed, especially
since it hasn't been released yet.  My concern is that calling it a pthread lock
makes it sound like we are just locking threads.

Ryan

> Index: server/mpm_common.c
> ===================================================================
> RCS file: /home/cvs/httpd-2.0/server/mpm_common.c,v
> retrieving revision 1.67
> diff -u -r1.67 mpm_common.c
> --- server/mpm_common.c	2001/09/18 22:13:57	1.67
> +++ server/mpm_common.c	2001/09/20 02:29:39
> @@ -604,7 +604,7 @@
>      }
>  #endif
>  #if APR_HAS_PROC_PTHREAD_SERIALIZE
> -    else if (!strcasecmp(arg, "proc_pthread")) {
> +    else if (!strcasecmp(arg, "pthread")) {
>          ap_accept_lock_mech = APR_LOCK_PROC_PTHREAD;
>      }
>  #endif
> @@ -621,7 +621,7 @@
>                             ", sysvsem"
>  #endif
>  #if APR_HAS_PROC_PTHREAD_SERIALIZE
> -                           ", proc_pthread"
> +                           ", pthread"
>  #endif
>                             , NULL);
>      }

-- 

______________________________________________________________
Ryan Bloom				rbb@apache.org
Covalent Technologies			rbb@covalent.net
--------------------------------------------------------------

Mime
View raw message