Re: cvs commit: jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11 Http11AprProcessor.java

2005-07-02 Thread Natasha Hasmani
Thank-you for your e-mail.

Please note that i will be away from the office starting Wednesday June
29th, returning Thursday July 7th, with no access to email.  In my absence,
kindly contact Cheri Dueck at [EMAIL PROTECTED]

Kind Regards,

Natasha Hasmani
Senior Event Manager 



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



Re: cvs commit: jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11 Http11AprProcessor.java

2005-07-02 Thread Natasha Hasmani
Thank-you for your e-mail.

Please note that i will be away from the office starting Wednesday June
29th, returning Thursday July 7th, with no access to email.  In my absence,
kindly contact Cheri Dueck at [EMAIL PROTECTED]

Kind Regards,

Natasha Hasmani
Senior Event Manager 



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



Re: cvs commit: jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11 Http11AprProcessor.java

2005-05-14 Thread Remy Maucherat
Bill Barker wrote:
Peter's test was stressing my poor little box a lot (which it really 
shouldn't, since it is a high-concurrency/low-traffic test).  It was one 
of the things I wanted to look into when I had time to run more tests.

Also, I've been running the test with sendfile disabled.  It seems that 
Solaris7 doesn't have this particular feature :(It was added to 
Solaris8). With sendfile enabled, I get a nice core-dump (since there is 
no apr_socket_sendfile in my libapr.so :).  I'm guessing that a for 
production release we need something like this in AprEndpoint.init:
  if(useSendfile  !Apr.hasFeature(Apr.SENDFILE)) {
log.warn(disabling sendfile, since your apr version doesn't 
support it);
useSendfile = false;
  }
I didn't know that. Yes, the check is needed then.
He still needs to remove some syncs in the native code, apparently (= 
the hacks he added on top of APR, but which didn't quite work).
It seems I need to look for a few more patches before I can usefully test.
Which changes do you suggest besides fixing sendfile ?
Rémy
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


Re: cvs commit: jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11 Http11AprProcessor.java

2005-05-14 Thread Mladen Turk
Remy Maucherat wrote:
Bill Barker wrote:
  if(useSendfile  !Apr.hasFeature(Apr.SENDFILE)) {
log.warn(disabling sendfile, since your apr version doesn't 
support it);
useSendfile = false;
  }
I didn't know that. Yes, the check is needed then.
Yes seems it's needed.
You already have the :
Library.APR_HAS_SENDFILE boolean, that is meant to be used
for that (checking if the OS supports sendfile).
Regards,
Mladen.
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


Re: cvs commit: jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11 Http11AprProcessor.java

2005-05-13 Thread Bill Barker

- Original Message -
From: [EMAIL PROTECTED]
To: [EMAIL PROTECTED]
Sent: Wednesday, May 11, 2005 4:23 AM
Subject: cvs commit:
jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11
Http11AprProcessor.java


 remm2005/05/11 04:23:26

   Modified:util/java/org/apache/tomcat/util/net AprEndpoint.java
http11/src/java/org/apache/coyote/http11
 Http11AprProcessor.java
   Log:
   - Should fix thread safety issue reported by Bill (needs testing).

Peter's test case does finish with this on my Solaris box.  I'll see if I
can do more testing next week.



This message is intended only for the use of the person(s) listed above as the 
intended recipient(s), and may contain information that is PRIVILEGED and 
CONFIDENTIAL.  If you are not an intended recipient, you may not read, copy, or 
distribute this message or any attachment. If you received this communication 
in error, please notify us immediately by e-mail and then delete all copies of 
this message and any attachments.

In addition you should be aware that ordinary (unencrypted) e-mail sent through 
the Internet is not secure. Do not send confidential or sensitive information, 
such as social security numbers, account numbers, personal identification 
numbers and passwords, to us via ordinary (unencrypted) e-mail.


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

Re: cvs commit: jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11 Http11AprProcessor.java

2005-05-13 Thread Remy Maucherat
Bill Barker wrote:
remm2005/05/11 04:23:26
 Modified:util/java/org/apache/tomcat/util/net AprEndpoint.java
  http11/src/java/org/apache/coyote/http11
   Http11AprProcessor.java
 Log:
 - Should fix thread safety issue reported by Bill (needs testing).
Peter's test case does finish with this on my Solaris box.  I'll see if I
can do more testing next week.
Mladen told me that the sync was badly done, and that I should be using 
a queue instead (too bad, it took one week).

He still needs to remove some syncs in the native code, apparently (= 
the hacks he added on top of APR, but which didn't quite work).

Rémy
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


Re: cvs commit: jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11 Http11AprProcessor.java

2005-05-13 Thread Bill Barker
- Original Message - 
From: Remy Maucherat [EMAIL PROTECTED]
To: Tomcat Developers List tomcat-dev@jakarta.apache.org
Sent: Friday, May 13, 2005 9:29 PM
Subject: Re: cvs commit: 
jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11 
Http11AprProcessor.java


Bill Barker wrote:
remm2005/05/11 04:23:26
 Modified:util/java/org/apache/tomcat/util/net AprEndpoint.java
  http11/src/java/org/apache/coyote/http11
   Http11AprProcessor.java
 Log:
 - Should fix thread safety issue reported by Bill (needs testing).
Peter's test case does finish with this on my Solaris box.  I'll see if I
can do more testing next week.
Mladen told me that the sync was badly done, and that I should be using a 
queue instead (too bad, it took one week).
Peter's test was stressing my poor little box a lot (which it really 
shouldn't, since it is a high-concurrency/low-traffic test).  It was one of 
the things I wanted to look into when I had time to run more tests.

Also, I've been running the test with sendfile disabled.  It seems that 
Solaris7 doesn't have this particular feature :(It was added to Solaris8). 
With sendfile enabled, I get a nice core-dump (since there is no 
apr_socket_sendfile in my libapr.so :).  I'm guessing that a for production 
release we need something like this in AprEndpoint.init:
  if(useSendfile  !Apr.hasFeature(Apr.SENDFILE)) {
log.warn(disabling sendfile, since your apr version doesn't 
support it);
useSendfile = false;
  }

He still needs to remove some syncs in the native code, apparently (= the 
hacks he added on top of APR, but which didn't quite work).

It seems I need to look for a few more patches before I can usefully test.
Rémy

This message is intended only for the use of the person(s) listed above as 
the intended recipient(s), and may contain information that is PRIVILEGED and 
CONFIDENTIAL.  If you are not an intended recipient, you may not read, copy, or 
distribute this message or any attachment. If you received this communication 
in error, please notify us immediately by e-mail and then delete all copies of 
this message and any attachments.
In addition you should be aware that ordinary (unencrypted) e-mail sent through 
the Internet is not secure. Do not send confidential or sensitive information, 
such as social security numbers, account numbers, personal identification 
numbers and passwords, to us via ordinary (unencrypted) e-mail.

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