From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <carlos@baldric.uwo.ca>
Received: from fep04-mail.bloor.is.net.cable.rogers.com
	(fep04-mail.bloor.is.net.cable.rogers.com [66.185.86.74])
	by dsl2.external.hp.com (Postfix) with ESMTP id 85B11484B
	for <parisc-linux@lists.parisc-linux.org>;
	Mon, 26 Jan 2004 13:34:11 -0700 (MST)
Received: from systemhalted ([24.43.33.92])
	by fep04-mail.bloor.is.net.cable.rogers.com
	(InterMail vM.5.01.05.12 201-253-122-126-112-20020820) with ESMTP id
	<20040126203121.PRWL337997.fep04-mail.bloor.is.net.cable.rogers.com@systemhalted>
	for <parisc-linux@lists.parisc-linux.org>;
	Mon, 26 Jan 2004 15:31:21 -0500
Date: Mon, 26 Jan 2004 15:34:30 -0500
From: Carlos O'Donell <carlos@baldric.uwo.ca>
To: parisc-linux@lists.parisc-linux.org
Subject: Re: [parisc-linux] semid64_ds compiled on a 32-bit system has wrong
	expected size for sem_ctime
Message-ID: <20040126203429.GJ5351@baldric.uwo.ca>
References: <20040126192659.GH5351@baldric.uwo.ca>
	<20040126194400.GI5351@baldric.uwo.ca>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
In-Reply-To: <20040126194400.GI5351@baldric.uwo.ca>
List-Id: parisc-linux developers list <parisc-linux.lists.parisc-linux.org>
List-Unsubscribe: <http://lists.parisc-linux.org/mailman/listinfo/parisc-linux>,
	<mailto:parisc-linux-request@lists.parisc-linux.org?subject=unsubscribe>
List-Archive: <http://lists.parisc-linux.org/pipermail/parisc-linux>
List-Post: <mailto:parisc-linux@lists.parisc-linux.org>
List-Help: <mailto:parisc-linux-request@lists.parisc-linux.org?subject=help>
List-Subscribe: <http://lists.parisc-linux.org/mailman/listinfo/parisc-linux>,
	<mailto:parisc-linux-request@lists.parisc-linux.org?subject=subscribe>

On Mon, Jan 26, 2004 at 02:44:00PM -0500, Carlos O'Donell wrote:
> > struct semid64_ds {
> >         struct ipc64_perm sem_perm;             /* permissions .. see ipc.h */
> > #ifndef __LP64__
> >         unsigned int    __pad1; 
> > #endif
> >         __kernel_time_t sem_otime;              /* last semop time */
> > #ifndef __LP64__
> >         unsigned int    __pad2; 
> > #endif
> >         __kernel_time_t sem_ctime;              /* last change time */
> >         unsigned int    sem_nsems;              /* no. of semaphores in array */
> >         unsigned int    __unused1;
> >         unsigned int    __unused2;
> > };
> 
> Our 32-bit userspace will see different values for this depending on
> running a 32-bit kernel or 64-bit. That last sem_ctime should probably
> have a padding so our 32-bit userspace sees the right thing all the
> time.

Sorry, I got confused with sem_perm, __pad2 is the padding for ctime,
thanks for pointing that out Matthew. Changing the base posix type for
32-bit builds to "long" fixes all the compiler complaints.

False alarm :)

c.