RE: [PATCH] Regarding semun definition in proc_mutex.h

2002-06-27 Thread MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1)
'just wondering if somebody could spend 5 seconds to give a thought to it :-). -Madhu -Original Message- From: MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1) [mailto:[EMAIL PROTECTED] Sent: Wednesday, June 26, 2002 9:35 AM To: 'dev@apr.apache.org' Subject: [PATCH] Regarding semun definition

Re: [PATCH] Regarding semun definition in proc_mutex.h

2002-06-27 Thread Jeff Trawick
MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1) [EMAIL PROTECTED] writes: 'just wondering if somebody could spend 5 seconds to give a thought to it :-). yes, this is an good fix (but you knew that already)... I'll commit later today if nobody beats me to it (staring down gun barrels at the moment)

Re: [PATCH] Regarding semun definition in proc_mutex.h

2002-06-27 Thread Jeff Trawick
MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1) [EMAIL PROTECTED] writes: On platforms where the semun union is not available, the APR defines the semun structure - but I believe the first parameter should be a int val and not long val. I verified it on HP-UX and Linux (both 32-bit and 64-bit

Re: [PATCH] Regarding semun definition in proc_mutex.h

2002-06-27 Thread Cliff Woolley
On 27 Jun 2002, Jeff Trawick wrote: So I guess sizeof(long) is 8 bytes on 64-bit HP-UX? sizeof(long) is still 4 on 64-bit AIX. $ uname -a HP-UX nova126 B.11.00 U 9000/785 2010844465 unlimited-user license $ cat test.c #include stdio.h int main() {

Re: [PATCH] Regarding semun definition in proc_mutex.h

2002-06-27 Thread Jim Jagielski
Jeff Trawick wrote: MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1) [EMAIL PROTECTED] writes: On platforms where the semun union is not available, the APR defines the semun structure - but I believe the first parameter should be a int val and not long val. I verified it on HP-UX and Linux

RE: [PATCH] Regarding semun definition in proc_mutex.h

2002-06-27 Thread MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1)
was that the listener_thread hangs while doing a SAFE_ACCEPT. Thanks, -Madhu -Original Message- From: Jeff Trawick [mailto:[EMAIL PROTECTED] Sent: Thursday, June 27, 2002 10:57 AM To: MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1) Cc: 'dev@apr.apache.org' Subject: Re: [PATCH] Regarding semun definition in proc_mutex.h

RE: [PATCH] Regarding semun definition in proc_mutex.h

2002-06-27 Thread MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1)
Yep. we compile with the +DD64 option for ansi-c. -Madhu -Original Message- From: Cliff Woolley [mailto:[EMAIL PROTECTED] Sent: Thursday, June 27, 2002 10:49 AM To: Jeff Trawick Cc: MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1); 'dev@apr.apache.org' Subject: Re: [PATCH] Regarding semun

RE: [PATCH] Regarding semun definition in proc_mutex.h

2002-06-27 Thread William A. Rowe, Jr.
At 01:00 PM 6/27/2002, MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1) wrote: Yep - with HP-UX running on 64-bit Itanium, and using the 64-bit data model. The side effect of having the long val in the union definition is that it'll have corrupted the mutex/semaphore while creating the proc_mutex itself

Re: [PATCH] Regarding semun definition in proc_mutex.h

2002-06-27 Thread Jeff Trawick
Jim Jagielski [EMAIL PROTECTED] writes: Jeff Trawick wrote: MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1) [EMAIL PROTECTED] writes: On platforms where the semun union is not available, the APR defines the semun structure - but I believe the first parameter should be a int val and not

RE: [PATCH] Regarding semun definition in proc_mutex.h

2002-06-27 Thread Jim Jagielski
I'm still slogging through tons of email, but was the issue that it was simply 'long' instead of 'int' (and trusting that the platform/OS correctly handles things) or that we require a specific physical size? FWIW, the current 2.0 implementation is copied from 1.3, so if it's wrong in one, it's

RE: [PATCH] Regarding semun definition in proc_mutex.h

2002-06-27 Thread William A. Rowe, Jr.
At 02:17 PM 6/27/2002, Jim Jagielski wrote: I'm still slogging through tons of email, but was the issue that it was simply 'long' instead of 'int' (and trusting that the platform/OS correctly handles things) or that we require a specific physical size? Ours is simply a HANDLE. End of issue for

[PATCH] Regarding semun definition in proc_mutex.h

2002-06-26 Thread MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1)
On platforms where the semun union is not available, the APR defines the semun structure - but I believe the first parameter should be a int val and not long val. I verified it on HP-UX and Linux (both 32-bit and 64-bit systems). BTW, we were able to get Apache (2.0.39) to work on the HP-UX /