apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Trawick <traw...@attglobal.net>
Subject Re: [PATCH] Regarding semun definition in proc_mutex.h
Date Thu, 27 Jun 2002 17:46:19 GMT
"MATHIHALLI,MADHUSUDAN (HP-Cupertino,ex1)" <madhusudan_mathihalli@hp.com> 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)

http://www.opengroup.org/onlinepubs/7908799/xsh/semctl.html

http://www.unet.univie.ac.at/aix/libs/basetrf2/semctl.htm

solaris man page:
union semun {
             int             val;
             struct semid_ds *buf;
             ushort_t        *array;
     } arg ;

...

> 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 / Itanium
> architecture without any other problems. Great work !!.
> 
> -Madhu
> 
> Index: proc_mutex.h
> ===================================================================
> RCS file: /home/cvspublic/apr/include/arch/unix/proc_mutex.h,v
> retrieving revision 1.8
> diff -u -r1.8 proc_mutex.h
> --- proc_mutex.h        7 Jun 2002 14:04:34 -0000       1.8
> +++ proc_mutex.h        26 Jun 2002 16:29:52 -0000
> @@ -142,7 +142,7 @@
>  
>  #if !APR_HAVE_UNION_SEMUN && defined(APR_HAS_SYSVSEM_SERIALIZE)
>  union semun {
> -    long val;
> +    int val;
>      struct semid_ds *buf;
>      unsigned short *array;
>  };
> 

-- 
Jeff Trawick | trawick@attglobal.net
Born in Roswell... married an alien...

Mime
View raw message