Re: mod_webapp.so socketpool changes..

2002-04-30 Thread simonkeary


Hi,

I'm a bit confused about the problem with the use of the APR atomic types.  Is the 
problem that it is not supported on all platforms?  If this is the case then isn't 
this a problem with the apr code?  Even if a given OS doesn't specifically offer 
atomic types then (as menioned earlier) a simple mutex and int in a struct offers the 
same functionality.

As far as I can see, all this wrapping should be part of the apr, shouldn't it?  I 
think puting in #ifdef APR_USE_ATOMICS is a bit of band-aid solution; Whether or not 
atomics are OS supported the variables in question should really be only operated on 
atomically (in a multi-threaded environment) or else the code is not thread-safe.

Also, I don't know if this has got anything to do with the  compilation problems found 
but when I first put in the atomics I found that the implementation in the (fairly 
recent) version of the apr I had was broken (on NT) and caused compilation to fail.  
Getting the latest version fixed this problem.

BTW Pier, sorry for not responding earlier - I managed to get the mod_webapp module 
compiling for Apache 2 after tweaking the 1.3 makefile to get a 2.0 makefile for NT.  
I used the apr directly from the Apache 2.0 distribution.  At the moment I am using 
Apache 2.0 on an NT intranet server with the mod_webapp at it seems to behave 
identically to the 1.3 version.

PS. If you want me to make any changes modifications (either changing the #ifdef or 
anything else) to the warp connector just let me know

PPS. I noticed a comment about mod_webapp being deprecated.  Is this the case?


Simon.

[EMAIL PROTECTED] wrote:

 
 Pier Fumagalli wrote:
  Cavan Morris [EMAIL PROTECTED] wrote:
  
  
 Attached is an ASCII file with my build process.  It may be important to not
 that I'm using the APR libraries from the webapp-module-1.0.2-tc402 and apxs
 from apache 2.0.32 not 2.0.35.
  
  
  You _need_ to use the APR libraries shipped with your Apache 2.0 version,
  and the same APXS, anyway it's bloated on my system as well. I have some
  spare time on my hands tonight, and tomorrow... I'll try to revamp the
  autoconf stuff and make it build again...
  
  NOTE
I might break something in Win32 builds. Can someone with MSVC give it a
go once they see my commits, please?
  /NOTE
  
 Question:
 when you say Yes, that's right...  do you mean yes that's right, the
 socketpool work may solve bug 8433 or yes that's right, it doesn't compile?
 
 The #if APR_HAS_THREADS is wrong that should have been #if APR_HAS_ATOMIC.
 But APR_HAS_ATOMIC is not existing and it is not possible to force APR to use a 
 generic C code for atomics (apr_force_atomic_generic cannot be set to one via 
 some configure options).
 
 Therefore when apr_atomic_t is not defined mod_webapp compilation failed miserably.
 
 I will try to change the #if APR_HAS_THREADS into #ifdef USE_ATOMICS like in the 
 mod_mem_cache.c of httpd-2.0. (As a temporary fix).
 
  
  
  Both... Yeah, the socketpool solves that problem as well (at least it
  should, what's your worker configuration in Apache 2.0? - output of httpd -l
  please :) and yes, it doesn't compile...
  
  On APR they're talking about removing atomics, since those bits actually
  depend (especially on sun hardware) on some opcodes present only on V9
  architectures (JF might explain it better than me, the only 2 RISCs I've
  coded for are PPCs and MIPSes), thus making binaries unportable from one
  architecture to another...
 
 The atomic for SPARC is using cas that is an 8+ opcode. This causes problems 
 on SPARC machines with old processors.
 
 There is a C code for machines that does not have a as atomic support.
 
  Therefore my thought is to rely on inter-process
  mutex locking, instead of atomics)...
  
  
 Thanks for your time.
  
  
  I feel lucky to have some spare time on my hands :) :) :)
  
  Pier
  
  
 -Cavan Morris
 
 - Original Message -
 From: Pier Fumagalli [EMAIL PROTECTED]
 To: Tomcat Developers List [EMAIL PROTECTED]
 Sent: Saturday, April 27, 2002 8:58 AM
 Subject: Re: mod_webapp.so socketpool changes..
 
 
 
 Cavan Morris [EMAIL PROTECTED] wrote:
 
 
 Hey Guys,
 I reported bug 8433 and am looking for a way to solve it.  I thought that
 the
 socketpool work you're doing might solve the problem but wasn't able to
 compile the latest from cvs.
 My question is do you think that what you're working on could fix the bug?
 This is a very severe problem on my system.  I have to restart both apache
 and
 tomcat if I get 2 concurrent requests.  If you've got any ideas on the java
 side I can look into it, but have no idea what to do with the C.
 
 Yes, that's right... Can you send out why it doesn't compile? That code uses
 ATOMIC, but given that atomics are going to go away in a short time from
 APR, we'll need to change it to use intra-process mutexes...
 
 It doesn't compile on OS/X as well... :(
 
 Pier
 
 --
 I think that it's extremely foolish to name a server after

Re: mod_webapp.so socketpool changes..

2002-04-30 Thread jean-frederic clere

[EMAIL PROTECTED] wrote:
  Hi,
 
  I'm a bit confused about the problem with the use of the APR atomic types.
  Is the problem that it is not supported on all platforms?

No it is not yet supported on all platforms.

   If this is the
  case then isn't this a problem with the apr code?  Even if a given OS
  doesn't specifically offer atomic types then (as menioned earlier) a simple
  mutex and int in a struct offers the same functionality.]

Yep, there is an apr_atomic.c file that could be used as generic atomic code.

 
  As far as I can see, all this wrapping should be part of the apr, shouldn't
  it?  I think puting in #ifdef APR_USE_ATOMICS is a bit of band-aid solution;
  Whether or not atomics are OS supported the variables in question should
  really be only operated on atomically (in a multi-threaded environment) or
  else the code is not thread-safe.

The status of atomic is not 100% clear in APR.
Someone from RedHat tells it will not work in user space (at least on Linux)(see 
http://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=63643).
For mod_webapp probably the best might be to use mutex instead of atomic.

 
  Also, I don't know if this has got anything to do with the  compilation
  problems found but when I first put in the atomics I found that the
  implementation in the (fairly recent) version of the apr I had was broken
  (on NT) and caused compilation to fail.  Getting the latest version fixed
  this problem.
 
  BTW Pier, sorry for not responding earlier - I managed to get the mod_webapp
  module compiling for Apache 2 after tweaking the 1.3 makefile to get a 2.0
  makefile for NT.  I used the apr directly from the Apache 2.0 distribution.
  At the moment I am using Apache 2.0 on an NT intranet server with the
  mod_webapp at it seems to behave identically to the 1.3 version.
 
  PS. If you want me to make any changes modifications (either changing the
  #ifdef or anything else) to the warp connector just let me know

I committed your changes. For the moment I think we should wait to see what 
happends in APR...

 
  PPS. I noticed a comment about mod_webapp being deprecated.  Is this the
  case?
 

No. It is just commented out in the server.xml that is delivered with the TC 
binaries.

 
  Simon.
 
  [EMAIL PROTECTED] wrote:
 
 
  Pier Fumagalli wrote:
 
  Cavan Morris [EMAIL PROTECTED] wrote:
 
 
 
  Attached is an ASCII file with my build process.  It may be important
  to not that I'm using the APR libraries from the
  webapp-module-1.0.2-tc402 and apxs
 
  from apache 2.0.32 not 2.0.35.
 
 
  You _need_ to use the APR libraries shipped with your Apache 2.0
  version, and the same APXS, anyway it's bloated on my system as well. I
  have some spare time on my hands tonight, and tomorrow... I'll try to
  revamp the autoconf stuff and make it build again...
 
  NOTE I might break something in Win32 builds. Can someone with MSVC
  give it a go once they see my commits, please? /NOTE
 
  Question: when you say Yes, that's right...  do you mean yes that's
  right, the socketpool work may solve bug 8433 or yes that's right, it
  doesn't compile?
 
  The #if APR_HAS_THREADS is wrong that should have been #if APR_HAS_ATOMIC.
  But APR_HAS_ATOMIC is not existing and it is not possible to force APR to
  use a generic C code for atomics (apr_force_atomic_generic cannot be set
  to one via some configure options).
 
  Therefore when apr_atomic_t is not defined mod_webapp compilation failed
  miserably.
 
  I will try to change the #if APR_HAS_THREADS into #ifdef USE_ATOMICS like
  in the mod_mem_cache.c of httpd-2.0. (As a temporary fix).
 
 
 
  Both... Yeah, the socketpool solves that problem as well (at least it should,
  what's your worker configuration in Apache 2.0? - output of httpd -l please
  :) and yes, it doesn't compile...
 
  On APR they're talking about removing atomics, since those bits actually
  depend (especially on sun hardware) on some opcodes present only on V9
  architectures (JF might explain it better than me, the only 2 RISCs I've
  coded for are PPCs and MIPSes), thus making binaries unportable from one
  architecture to another...
 
  The atomic for SPARC is using cas that is an 8+ opcode. This causes
  problems on SPARC machines with old processors.
 
  There is a C code for machines that does not have a as atomic support.
 
 
  Therefore my thought is to rely on inter-process mutex locking, instead
  of atomics)...
 
 
 
  Thanks for your time.
 
 
  I feel lucky to have some spare time on my hands :) :) :)
 
  Pier
 
 
 
  -Cavan Morris
 
  - Original Message - From: Pier Fumagalli
  [EMAIL PROTECTED] To: Tomcat Developers List
  [EMAIL PROTECTED] Sent: Saturday, April 27, 2002 8:58 AM
  Subject: Re: mod_webapp.so socketpool changes..
 
 
 
 
  Cavan Morris [EMAIL PROTECTED] wrote:
 
 
 
  Hey Guys, I reported bug 8433 and am looking for a way to solve
  it.  I thought that the socketpool work you're doing might solve
  the problem but wasn't able

Re: mod_webapp.so socketpool changes..

2002-04-29 Thread jean-frederic clere

Pier Fumagalli wrote:
 Cavan Morris [EMAIL PROTECTED] wrote:
 
 
Attached is an ASCII file with my build process.  It may be important to not
that I'm using the APR libraries from the webapp-module-1.0.2-tc402 and apxs
from apache 2.0.32 not 2.0.35.
 
 
 You _need_ to use the APR libraries shipped with your Apache 2.0 version,
 and the same APXS, anyway it's bloated on my system as well. I have some
 spare time on my hands tonight, and tomorrow... I'll try to revamp the
 autoconf stuff and make it build again...
 
 NOTE
   I might break something in Win32 builds. Can someone with MSVC give it a
   go once they see my commits, please?
 /NOTE
 
Question:
when you say Yes, that's right...  do you mean yes that's right, the
socketpool work may solve bug 8433 or yes that's right, it doesn't compile?

The #if APR_HAS_THREADS is wrong that should have been #if APR_HAS_ATOMIC.
But APR_HAS_ATOMIC is not existing and it is not possible to force APR to use a 
generic C code for atomics (apr_force_atomic_generic cannot be set to one via 
some configure options).

Therefore when apr_atomic_t is not defined mod_webapp compilation failed miserably.

I will try to change the #if APR_HAS_THREADS into #ifdef USE_ATOMICS like in the 
mod_mem_cache.c of httpd-2.0. (As a temporary fix).

 
 
 Both... Yeah, the socketpool solves that problem as well (at least it
 should, what's your worker configuration in Apache 2.0? - output of httpd -l
 please :) and yes, it doesn't compile...
 
 On APR they're talking about removing atomics, since those bits actually
 depend (especially on sun hardware) on some opcodes present only on V9
 architectures (JF might explain it better than me, the only 2 RISCs I've
 coded for are PPCs and MIPSes), thus making binaries unportable from one
 architecture to another...

The atomic for SPARC is using cas that is an 8+ opcode. This causes problems 
on SPARC machines with old processors.

There is a C code for machines that does not have a as atomic support.

 Therefore my thought is to rely on inter-process
 mutex locking, instead of atomics)...
 
 
Thanks for your time.
 
 
 I feel lucky to have some spare time on my hands :) :) :)
 
 Pier
 
 
-Cavan Morris

- Original Message -
From: Pier Fumagalli [EMAIL PROTECTED]
To: Tomcat Developers List [EMAIL PROTECTED]
Sent: Saturday, April 27, 2002 8:58 AM
Subject: Re: mod_webapp.so socketpool changes..



Cavan Morris [EMAIL PROTECTED] wrote:


Hey Guys,
I reported bug 8433 and am looking for a way to solve it.  I thought that
the
socketpool work you're doing might solve the problem but wasn't able to
compile the latest from cvs.
My question is do you think that what you're working on could fix the bug?
This is a very severe problem on my system.  I have to restart both apache
and
tomcat if I get 2 concurrent requests.  If you've got any ideas on the java
side I can look into it, but have no idea what to do with the C.

Yes, that's right... Can you send out why it doesn't compile? That code uses
ATOMIC, but given that atomics are going to go away in a short time from
APR, we'll need to change it to use intra-process mutexes...

It doesn't compile on OS/X as well... :(

Pier

--
I think that it's extremely foolish to name a server after the current U.S.
President. B.W. Fitzpatrick



--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]


--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]
 
 
 --
 I think that it's extremely foolish to name a server after the current U.S.
 President. B.W. Fitzpatrick
 
 
 
 --
 To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
 For additional commands, e-mail: mailto:[EMAIL PROTECTED]
 
 
 




--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]




Re: mod_webapp.so socketpool changes..

2002-04-27 Thread Pier Fumagalli

Cavan Morris [EMAIL PROTECTED] wrote:

 Hey Guys,
 I reported bug 8433 and am looking for a way to solve it.  I thought that the
 socketpool work you're doing might solve the problem but wasn't able to
 compile the latest from cvs.
 My question is do you think that what you're working on could fix the bug?
 This is a very severe problem on my system.  I have to restart both apache and
 tomcat if I get 2 concurrent requests.  If you've got any ideas on the java
 side I can look into it, but have no idea what to do with the C.

Yes, that's right... Can you send out why it doesn't compile? That code uses
ATOMIC, but given that atomics are going to go away in a short time from
APR, we'll need to change it to use intra-process mutexes...

It doesn't compile on OS/X as well... :(

Pier

--
I think that it's extremely foolish to name a server after the current U.S.
President. B.W. Fitzpatrick



--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]




Re: mod_webapp.so socketpool changes..

2002-04-27 Thread Cavan Morris

Attached is an ASCII file with my build process.  It may be important to not that I'm 
using the APR libraries from the webapp-module-1.0.2-tc402 and apxs from apache 2.0.32 
not 2.0.35.
Question:
when you say Yes, that's right...  do you mean yes that's right, the socketpool work 
may solve bug 8433 or yes that's right, it doesn't compile?

Thanks for your time.
-Cavan Morris

- Original Message - 
From: Pier Fumagalli [EMAIL PROTECTED]
To: Tomcat Developers List [EMAIL PROTECTED]
Sent: Saturday, April 27, 2002 8:58 AM
Subject: Re: mod_webapp.so socketpool changes..


 Cavan Morris [EMAIL PROTECTED] wrote:
 
  Hey Guys,
  I reported bug 8433 and am looking for a way to solve it.  I thought that the
  socketpool work you're doing might solve the problem but wasn't able to
  compile the latest from cvs.
  My question is do you think that what you're working on could fix the bug?
  This is a very severe problem on my system.  I have to restart both apache and
  tomcat if I get 2 concurrent requests.  If you've got any ideas on the java
  side I can look into it, but have no idea what to do with the C.
 
 Yes, that's right... Can you send out why it doesn't compile? That code uses
 ATOMIC, but given that atomics are going to go away in a short time from
 APR, we'll need to change it to use intra-process mutexes...
 
 It doesn't compile on OS/X as well... :(
 
 Pier
 
 --
 I think that it's extremely foolish to name a server after the current U.S.
 President. B.W. Fitzpatrick
 
 
 
 --
 To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
 For additional commands, e-mail: mailto:[EMAIL PROTECTED]
 
 


[root@lin1 webapp]# ./support/buildconf.sh 
--- Cannot run APR buildconf script
Don't a forget to download a copy of the APR sources, and to run
the buildconf script for it:
  # cd [path to APR sources]
  # ./buildconf
  # cd [path to WebApp sources]
Then remember to run ./configure script including the command line
option --with-apr=[path to APR sources]
--- Creating WebApp configure script
Creating configure ...
--- All done
[root@lin1 webapp]# ./configure --with-apr=~/webapp-module-1.0.2-tc402/apr 
--with-apxs=/usr/local/apache2/bin/apxs
creating cache ./config.cache
checking for test... /usr/bin/test
checking for true... /bin/true
checking for echo... /bin/echo
checking for grep... /bin/grep
checking for cat... /bin/cat
checking for sed... /bin/sed
checking for ln... /bin/ln
checking for rm... /bin/rm
checking host system type... i686-pc-linux-gnu
checking for sources directory... /root/jakarta-tomcat-connectors/webapp
checking for targets directory... /root/jakarta-tomcat-connectors/webapp

C-Language compilation tools
checking for ar... /usr/bin/ar
checking build system type... i686-pc-linux-gnu
checking for ranlib... ranlib
checking for gcc... gcc
checking whether the C compiler (gcc  ) works... yes
checking whether the C compiler (gcc  ) is a cross-compiler... no
checking whether we are using GNU C... yes
checking whether gcc accepts -g... yes
checking for gcc... /usr/bin/gcc
checking whether /usr/bin/gcc and cc understand -c and -o together... yes
checking how to run the C preprocessor... /usr/bin/gcc -E
checking for debug build... no

API documentation generation
checking for C API documentation... no

APR location
checking for APR sources... ~/webapp-module-1.0.2-tc402/apr
checking for APR libraries... no
checking for APR includes... no

Target web-server
checking for Apache apxs... yes
checking target module... apache-2.0

Finishing up
checking for APR compilation flags... ok
checking for Makefile targets... lib apache-2.0
updating cache ./config.cache
creating ./config.status
creating ./Makefile
creating ./Makedefs
creating ./lib/Makefile
creating ./apache-2.0/Makefile

All done. Now you can issue make. Good luck.
[root@lin1 webapp]# make
make[1]: Entering directory `/root/jakarta-tomcat-connectors/webapp'

make[1]: Entering directory lib
make[1]: Invoking make  build
make[2]: Entering directory `/root/jakarta-tomcat-connectors/webapp/lib'
make[2]: Generating pr_warp_defs.h from 
/root/jakarta-tomcat-connectors/webapp/java/org/apache/catalina/connector/warp/Constants.java
/usr/local/apache2/build/libtool /usr/bin/gcc -static -g -O2  -g -O2 -pthread 
-DLINUX=2 -D_REENTRANT -D_XOPEN_SOURCE=500 -D_BSD_SOURCE -D_SVID_SOURCE  
-I/usr/local/apache2/include -I/root/jakarta-tomcat-connectors/webapp/include -c 
wa_main.c
mkdir .libs
/usr/bin/gcc -g -O2 -g -O2 -pthread -DLINUX=2 -D_REENTRANT -D_XOPEN_SOURCE=500 
-D_BSD_SOURCE -D_SVID_SOURCE -I/usr/local/apache2/include 
-I/root/jakarta-tomcat-connectors/webapp/include -c wa_main.c   -fPIC -DPIC -o 
.libs/wa_main.lo
/usr/bin/gcc -g -O2 -g -O2 -pthread -DLINUX=2 -D_REENTRANT -D_XOPEN_SOURCE=500 
-D_BSD_SOURCE -D_SVID_SOURCE -I/usr/local/apache2/include 
-I/root/jakarta-tomcat-connectors/webapp/include -c wa_main.c -o wa_main.o /dev/null 
21
mv -f .libs/wa_main.lo wa_main.lo
/usr/local/apache2/build

Re: mod_webapp.so socketpool changes..

2002-04-27 Thread Pier Fumagalli

Cavan Morris [EMAIL PROTECTED] wrote:

 Attached is an ASCII file with my build process.  It may be important to not
 that I'm using the APR libraries from the webapp-module-1.0.2-tc402 and apxs
 from apache 2.0.32 not 2.0.35.

You _need_ to use the APR libraries shipped with your Apache 2.0 version,
and the same APXS, anyway it's bloated on my system as well. I have some
spare time on my hands tonight, and tomorrow... I'll try to revamp the
autoconf stuff and make it build again...

NOTE
  I might break something in Win32 builds. Can someone with MSVC give it a
  go once they see my commits, please?
/NOTE

 Question:
 when you say Yes, that's right...  do you mean yes that's right, the
 socketpool work may solve bug 8433 or yes that's right, it doesn't compile?

Both... Yeah, the socketpool solves that problem as well (at least it
should, what's your worker configuration in Apache 2.0? - output of httpd -l
please :) and yes, it doesn't compile...

On APR they're talking about removing atomics, since those bits actually
depend (especially on sun hardware) on some opcodes present only on V9
architectures (JF might explain it better than me, the only 2 RISCs I've
coded for are PPCs and MIPSes), thus making binaries unportable from one
architecture to another... Therefore my thought is to rely on inter-process
mutex locking, instead of atomics)...

 Thanks for your time.

I feel lucky to have some spare time on my hands :) :) :)

Pier

 -Cavan Morris
 
 - Original Message -
 From: Pier Fumagalli [EMAIL PROTECTED]
 To: Tomcat Developers List [EMAIL PROTECTED]
 Sent: Saturday, April 27, 2002 8:58 AM
 Subject: Re: mod_webapp.so socketpool changes..
 
 
 Cavan Morris [EMAIL PROTECTED] wrote:
 
 Hey Guys,
 I reported bug 8433 and am looking for a way to solve it.  I thought that
 the
 socketpool work you're doing might solve the problem but wasn't able to
 compile the latest from cvs.
 My question is do you think that what you're working on could fix the bug?
 This is a very severe problem on my system.  I have to restart both apache
 and
 tomcat if I get 2 concurrent requests.  If you've got any ideas on the java
 side I can look into it, but have no idea what to do with the C.
 
 Yes, that's right... Can you send out why it doesn't compile? That code uses
 ATOMIC, but given that atomics are going to go away in a short time from
 APR, we'll need to change it to use intra-process mutexes...
 
 It doesn't compile on OS/X as well... :(
 
 Pier
 
 --
 I think that it's extremely foolish to name a server after the current U.S.
 President. B.W. Fitzpatrick
 
 
 
 --
 To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
 For additional commands, e-mail: mailto:[EMAIL PROTECTED]
 
 
 
 --
 To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
 For additional commands, e-mail: mailto:[EMAIL PROTECTED]

--
I think that it's extremely foolish to name a server after the current U.S.
President. B.W. Fitzpatrick



--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]




Re: mod_webapp.so socketpool changes..

2002-04-27 Thread Cavan Morris

Well I changed the apr libs I was using and had the same errors.  I'm using the Worker 
MPM.

Side Note:  I would also like to know why the webapp connector is being deprecated.  
It may need a little work, but it's ease of configuration is a great feature.  I am 
concerned with it's performance (not knowing anything about it I think it could be a 
lot better) but the solution to that is to fix it not abandon a beautifully simple 
interface.

BTW: Is there someplace I can get a look at the Warp protocol?

-Cavan Morris

- Original Message - 
From: Pier Fumagalli [EMAIL PROTECTED]
To: Tomcat Developers List [EMAIL PROTECTED]
Sent: Saturday, April 27, 2002 2:03 PM
Subject: Re: mod_webapp.so socketpool changes..


 Cavan Morris [EMAIL PROTECTED] wrote:
 
  Attached is an ASCII file with my build process.  It may be important to not
  that I'm using the APR libraries from the webapp-module-1.0.2-tc402 and apxs
  from apache 2.0.32 not 2.0.35.
 
 You _need_ to use the APR libraries shipped with your Apache 2.0 version,
 and the same APXS, anyway it's bloated on my system as well. I have some
 spare time on my hands tonight, and tomorrow... I'll try to revamp the
 autoconf stuff and make it build again...
 
 NOTE
   I might break something in Win32 builds. Can someone with MSVC give it a
   go once they see my commits, please?
 /NOTE
 
  Question:
  when you say Yes, that's right...  do you mean yes that's right, the
  socketpool work may solve bug 8433 or yes that's right, it doesn't compile?
 
 Both... Yeah, the socketpool solves that problem as well (at least it
 should, what's your worker configuration in Apache 2.0? - output of httpd -l
 please :) and yes, it doesn't compile...
 
 On APR they're talking about removing atomics, since those bits actually
 depend (especially on sun hardware) on some opcodes present only on V9
 architectures (JF might explain it better than me, the only 2 RISCs I've
 coded for are PPCs and MIPSes), thus making binaries unportable from one
 architecture to another... Therefore my thought is to rely on inter-process
 mutex locking, instead of atomics)...
 
  Thanks for your time.
 
 I feel lucky to have some spare time on my hands :) :) :)
 
 Pier
 
  -Cavan Morris
  
  - Original Message -
  From: Pier Fumagalli [EMAIL PROTECTED]
  To: Tomcat Developers List [EMAIL PROTECTED]
  Sent: Saturday, April 27, 2002 8:58 AM
  Subject: Re: mod_webapp.so socketpool changes..
  
  
  Cavan Morris [EMAIL PROTECTED] wrote:
  
  Hey Guys,
  I reported bug 8433 and am looking for a way to solve it.  I thought that
  the
  socketpool work you're doing might solve the problem but wasn't able to
  compile the latest from cvs.
  My question is do you think that what you're working on could fix the bug?
  This is a very severe problem on my system.  I have to restart both apache
  and
  tomcat if I get 2 concurrent requests.  If you've got any ideas on the java
  side I can look into it, but have no idea what to do with the C.
  
  Yes, that's right... Can you send out why it doesn't compile? That code uses
  ATOMIC, but given that atomics are going to go away in a short time from
  APR, we'll need to change it to use intra-process mutexes...
  
  It doesn't compile on OS/X as well... :(
  
  Pier
  
  --
  I think that it's extremely foolish to name a server after the current U.S.
  President. B.W. Fitzpatrick
  
  
  
  --
  To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
  For additional commands, e-mail: mailto:[EMAIL PROTECTED]
  
  
  
  --
  To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
  For additional commands, e-mail: mailto:[EMAIL PROTECTED]
 
 --
 I think that it's extremely foolish to name a server after the current U.S.
 President. B.W. Fitzpatrick
 
 
 
 --
 To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
 For additional commands, e-mail: mailto:[EMAIL PROTECTED]
 


--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]




Re: mod_webapp.so socketpool changes..

2002-04-26 Thread jean-frederic clere

[EMAIL PROTECTED] wrote:
 Hi John-Frederic,
 
 I've only just seen your mail as I've been away for a few days...
 
 Is there any update on this?  Have you had a chance to look at the updates I made?

Yep... But it needs more when having a not threaded Apache-1.3 for example.
The code still does not compile on my Unixes it needs more #ifdef. I will try to 
  find time for it...

 
 If you want me to do any more work on this just let me know...  BTW I've been 
testing this on an NT intranet web server running Apache 1.3.24 and more recently 2.0 
and so far it's seems to be stable...
 
 Simon.
 
 
 [EMAIL PROTECTED] wrote:
 
 
Pier Fumagalli wrote:

[EMAIL PROTECTED] [EMAIL PROTECTED] wrote:



Hi,

As mentioned in my last email I have updated the socketpool changes to
incorporate the changes recommended.

I'm not sure if my mail was missed or not but I haven't received any feedback
on the latest changes.  What's the process to get this code into the codebase?
I believe I would need cvs access to commit this (if deemed acceptable) into
the source myself.  Is this the case?  Can someone fill me in

Attached are the changes made.


Simon,
both I and JF have been pretty busy this last few days... I wanted to
it out also with Apache 2.0's worker MPM before committing...

I have been out for a week... But I will try to find some time today ;-)


Pier


--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]






--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]


 
 
 --
 To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
 For additional commands, e-mail: mailto:[EMAIL PROTECTED]
 
 
 




--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]




Re: mod_webapp.so socketpool changes..

2002-04-26 Thread jean-frederic clere

Clere, Jean-Frederic wrote:
 [EMAIL PROTECTED] wrote:
 
 Hi John-Frederic,

 I've only just seen your mail as I've been away for a few days...

 Is there any update on this?  Have you had a chance to look at the 
 updates I made?
 
 
 Yep... But it needs more when having a not threaded Apache-1.3 for example.
 The code still does not compile on my Unixes it needs more #ifdef. I 
 will try to  find time for it...

Was not #ifdef but #if (Sorry my bad)

I have committed after quickly arranging it so that it compiles and works with 
Apache-1.3 on Linux, now I will check on the other Unixes and with Apache-2.0.

 

 If you want me to do any more work on this just let me know...  BTW 
 I've been testing this on an NT intranet web server running Apache 
 1.3.24 and more recently 2.0 and so far it's seems to be stable...

 Simon.


 [EMAIL PROTECTED] wrote:


 Pier Fumagalli wrote:

 [EMAIL PROTECTED] [EMAIL PROTECTED] wrote:



 Hi,

 As mentioned in my last email I have updated the socketpool changes to
 incorporate the changes recommended.

 I'm not sure if my mail was missed or not but I haven't received 
 any feedback
 on the latest changes.  What's the process to get this code into 
 the codebase?
 I believe I would need cvs access to commit this (if deemed 
 acceptable) into
 the source myself.  Is this the case?  Can someone fill me in

 Attached are the changes made.



 Simon,
both I and JF have been pretty busy this last few days... I 
 wanted to
 it out also with Apache 2.0's worker MPM before committing...


 I have been out for a week... But I will try to find some time today ;-)


Pier


 -- 
 To unsubscribe, e-mail:   
 mailto:[EMAIL PROTECTED]
 For additional commands, e-mail: 
 mailto:[EMAIL PROTECTED]






 -- 
 To unsubscribe, e-mail:   
 mailto:[EMAIL PROTECTED]
 For additional commands, e-mail: 
 mailto:[EMAIL PROTECTED]




 -- 
 To unsubscribe, e-mail:   
 mailto:[EMAIL PROTECTED]
 For additional commands, e-mail: 
 mailto:[EMAIL PROTECTED]



 
 
 
 
 -- 
 To unsubscribe, e-mail:   
 mailto:[EMAIL PROTECTED]
 For additional commands, e-mail: 
 mailto:[EMAIL PROTECTED]
 
 
 




--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]




Re: mod_webapp.so socketpool changes..

2002-04-26 Thread Cavan Morris

Hey Guys,
I reported bug 8433 and am looking for a way to solve it.  I thought that the 
socketpool work you're doing might solve the problem but wasn't able to compile the 
latest from cvs.
My question is do you think that what you're working on could fix the bug?  This is a 
very severe problem on my system.  I have to restart both apache and tomcat if I get 2 
concurrent requests.  If you've got any ideas on the java side I can look into it, but 
have no idea what to do with the C.

Thanks for your time.
-Cavan Morris

- Original Message - 
From: jean-frederic clere [EMAIL PROTECTED]
To: Tomcat Developers List [EMAIL PROTECTED]
Sent: Friday, April 26, 2002 6:18 AM
Subject: Re: mod_webapp.so socketpool changes..


 Clere, Jean-Frederic wrote:
  [EMAIL PROTECTED] wrote:
  
  Hi John-Frederic,
 
  I've only just seen your mail as I've been away for a few days...
 
  Is there any update on this?  Have you had a chance to look at the 
  updates I made?
  
  
  Yep... But it needs more when having a not threaded Apache-1.3 for example.
  The code still does not compile on my Unixes it needs more #ifdef. I 
  will try to  find time for it...
 
 Was not #ifdef but #if (Sorry my bad)
 
 I have committed after quickly arranging it so that it compiles and works with 
 Apache-1.3 on Linux, now I will check on the other Unixes and with Apache-2.0.
 
  
 
  If you want me to do any more work on this just let me know...  BTW 
  I've been testing this on an NT intranet web server running Apache 
  1.3.24 and more recently 2.0 and so far it's seems to be stable...
 
  Simon.
 
 
  [EMAIL PROTECTED] wrote:
 
 
  Pier Fumagalli wrote:
 
  [EMAIL PROTECTED] [EMAIL PROTECTED] wrote:
 
 
 
  Hi,
 
  As mentioned in my last email I have updated the socketpool changes to
  incorporate the changes recommended.
 
  I'm not sure if my mail was missed or not but I haven't received 
  any feedback
  on the latest changes.  What's the process to get this code into 
  the codebase?
  I believe I would need cvs access to commit this (if deemed 
  acceptable) into
  the source myself.  Is this the case?  Can someone fill me in
 
  Attached are the changes made.
 
 
 
  Simon,
 both I and JF have been pretty busy this last few days... I 
  wanted to
  it out also with Apache 2.0's worker MPM before committing...
 
 
  I have been out for a week... But I will try to find some time today ;-)
 
 
 Pier
 
 
  -- 
  To unsubscribe, e-mail:   
  mailto:[EMAIL PROTECTED]
  For additional commands, e-mail: 
  mailto:[EMAIL PROTECTED]
 
 
 
 
 
 
  -- 
  To unsubscribe, e-mail:   
  mailto:[EMAIL PROTECTED]
  For additional commands, e-mail: 
  mailto:[EMAIL PROTECTED]
 
 
 
 
  -- 
  To unsubscribe, e-mail:   
  mailto:[EMAIL PROTECTED]
  For additional commands, e-mail: 
  mailto:[EMAIL PROTECTED]
 
 
 
  
  
  
  
  -- 
  To unsubscribe, e-mail:   
  mailto:[EMAIL PROTECTED]
  For additional commands, e-mail: 
  mailto:[EMAIL PROTECTED]
  
  
  
 
 
 
 
 --
 To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
 For additional commands, e-mail: mailto:[EMAIL PROTECTED]
 
 


--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]




Re: mod_webapp.so socketpool changes..

2002-04-25 Thread Pier Fumagalli

jean-frederic clere [EMAIL PROTECTED] wrote:

 Simon,
 both I and JF have been pretty busy this last few days... I wanted to
 it out also with Apache 2.0's worker MPM before committing...
 
 I have been out for a week... But I will try to find some time today ;-)

Same here, this week was crazy... Hopefully I'll get around to get some time
to do some OSS work this week-end...

Pier (now with a _real_real_ job)

--
I think that it's extremely foolish to name a server after the current U.S.
President. B.W. Fitzpatrick



--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]




Re: mod_webapp.so socketpool changes..

2002-04-25 Thread simonkeary


Hi John-Frederic,

I've only just seen your mail as I've been away for a few days...

Is there any update on this?  Have you had a chance to look at the updates I made?

If you want me to do any more work on this just let me know...  BTW I've been testing 
this on an NT intranet web server running Apache 1.3.24 and more recently 2.0 and so 
far it's seems to be stable...

Simon.


[EMAIL PROTECTED] wrote:

 
 Pier Fumagalli wrote:
  [EMAIL PROTECTED] [EMAIL PROTECTED] wrote:
  
  
 Hi,
 
 As mentioned in my last email I have updated the socketpool changes to
 incorporate the changes recommended.
 
 I'm not sure if my mail was missed or not but I haven't received any feedback
 on the latest changes.  What's the process to get this code into the codebase?
 I believe I would need cvs access to commit this (if deemed acceptable) into
 the source myself.  Is this the case?  Can someone fill me in
 
 Attached are the changes made.
  
  
  Simon,
  both I and JF have been pretty busy this last few days... I wanted to
  it out also with Apache 2.0's worker MPM before committing...
 
 I have been out for a week... But I will try to find some time today ;-)
 
  
  Pier
  
  
  --
  To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
  For additional commands, e-mail: mailto:[EMAIL PROTECTED]
  
  
  
 
 
 
 
 --
 To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
 For additional commands, e-mail: mailto:[EMAIL PROTECTED]
 
 


--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]




Re: mod_webapp.so socketpool changes..

2002-04-25 Thread Pier Fumagalli

[EMAIL PROTECTED] [EMAIL PROTECTED] wrote:

 
 Hi John-Frederic,
 
 I've only just seen your mail as I've been away for a few days...
 
 Is there any update on this?  Have you had a chance to look at the updates I
 made?
 
 If you want me to do any more work on this just let me know...  BTW I've been
 testing this on an NT intranet web server running Apache 1.3.24 and more
 recently 2.0 and so far it's seems to be stable...

Simon, are you using 2.0 on WinNT as well???

Pier

--
I think that it's extremely foolish to name a server after the current U.S.
President. B.W. Fitzpatrick



--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]




Re: mod_webapp.so socketpool changes..

2002-04-22 Thread jean-frederic clere

Pier Fumagalli wrote:
 [EMAIL PROTECTED] [EMAIL PROTECTED] wrote:
 
 
Hi,

As mentioned in my last email I have updated the socketpool changes to
incorporate the changes recommended.

I'm not sure if my mail was missed or not but I haven't received any feedback
on the latest changes.  What's the process to get this code into the codebase?
I believe I would need cvs access to commit this (if deemed acceptable) into
the source myself.  Is this the case?  Can someone fill me in

Attached are the changes made.
 
 
 Simon,
 both I and JF have been pretty busy this last few days... I wanted to
 it out also with Apache 2.0's worker MPM before committing...

I have been out for a week... But I will try to find some time today ;-)

 
 Pier
 
 
 --
 To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
 For additional commands, e-mail: mailto:[EMAIL PROTECTED]
 
 
 




--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]




Re: mod_webapp.so socketpool changes..

2002-04-17 Thread Pier Fumagalli

[EMAIL PROTECTED] [EMAIL PROTECTED] wrote:

 Hi,
 
 As mentioned in my last email I have updated the socketpool changes to
 incorporate the changes recommended.
 
 I'm not sure if my mail was missed or not but I haven't received any feedback
 on the latest changes.  What's the process to get this code into the codebase?
 I believe I would need cvs access to commit this (if deemed acceptable) into
 the source myself.  Is this the case?  Can someone fill me in
 
 Attached are the changes made.

Simon,
both I and JF have been pretty busy this last few days... I wanted to
it out also with Apache 2.0's worker MPM before committing...

Pier


--
To unsubscribe, e-mail:   mailto:[EMAIL PROTECTED]
For additional commands, e-mail: mailto:[EMAIL PROTECTED]