From patchwork Fri Dec 16 14:05:27 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Adhemerval Zanella X-Patchwork-Id: 88308 Delivered-To: patch@linaro.org Received: by 10.140.20.101 with SMTP id 92csp1422215qgi; Fri, 16 Dec 2016 06:05:58 -0800 (PST) X-Received: by 10.98.75.11 with SMTP id y11mr3244659pfa.30.1481897158880; Fri, 16 Dec 2016 06:05:58 -0800 (PST) Return-Path: Received: from sourceware.org (server1.sourceware.org. [209.132.180.131]) by mx.google.com with ESMTPS id k11si7933294pgp.150.2016.12.16.06.05.58 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 16 Dec 2016 06:05:58 -0800 (PST) Received-SPF: pass (google.com: domain of libc-alpha-return-75982-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) client-ip=209.132.180.131; Authentication-Results: mx.google.com; dkim=pass header.i=@sourceware.org; spf=pass (google.com: domain of libc-alpha-return-75982-patch=linaro.org@sourceware.org designates 209.132.180.131 as permitted sender) smtp.mailfrom=libc-alpha-return-75982-patch=linaro.org@sourceware.org; dmarc=fail (p=NONE dis=NONE) header.from=linaro.org DomainKey-Signature: a=rsa-sha1; c=nofws; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:subject:to:references:cc:from:message-id:date :mime-version:in-reply-to:content-type :content-transfer-encoding; q=dns; s=default; b=sl4NycK3QQJPoQAs WFySqO/yLpQMERQ5yN28u95OLWL4U4LXOa2S/T2IrCqi7cKYPkCHQd/ZZqZlWqA7 TuIDv+jLJqrW6YyNF2uKwxnB5Jc1rdFdV54cOdCKigEjRowq6clsdof23XDFYb58 SGFXmv65fSgh212ULMqIR0z0ppQ= DKIM-Signature: v=1; a=rsa-sha1; c=relaxed; d=sourceware.org; h=list-id :list-unsubscribe:list-subscribe:list-archive:list-post :list-help:sender:subject:to:references:cc:from:message-id:date :mime-version:in-reply-to:content-type :content-transfer-encoding; s=default; bh=yfknJ6f42bElfOORCxx427 GMKaI=; b=IXpU8essl0jbHf3F0vL1cOHZjaSi3oHMIFVYpseHkpa/b4iIkiu785 a2TBVR/CkdSfW9XJCksnhOkz81ajDaWyH7HpyvIOnz0lIcu7xCDNRnweEyUOnGQz pOcMrjl7TapDZKRRGohFtWMMkipLEpizl9dNbwniG00rWrYQIDSBk= Received: (qmail 120121 invoked by alias); 16 Dec 2016 14:05:47 -0000 Mailing-List: contact libc-alpha-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Unsubscribe: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libc-alpha-owner@sourceware.org Delivered-To: mailing list libc-alpha@sourceware.org Received: (qmail 120102 invoked by uid 89); 16 Dec 2016 14:05:45 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.4 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_NONE, RCVD_IN_SORBS_SPAM, SPF_PASS autolearn=no version=3.3.2 spammy=gid, 1, 41, ipc, IPC X-HELO: mail-yb0-f182.google.com X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:references:cc:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding; bh=QLGodmGUKbSOIWqMZcuBzyPPGOAW+1gcr1YPW6XTdKU=; b=qYoaARloYfotgdbHPp4fUuHn4sh2unUFXnliRCxOmMVNZsmaFkojnyUgNwWW4/uSHV f6qBfVEWMPxFoiYxiz6KSXjFLtnh6YzCG6ejVR2w6B+TwOiiqsKw04oM2fBfxNc40lgr QAIiImqHr7XpPih08w0tJBU4rEwqa9LsIojuMVHrVNFjOray+WUl3p8vKQ/QaL0mSB6I CojC3jh9LpenkgqHzbDl+uYwmqhVvrhFQFg+Ct5QT7tgWITox2SHjT5F+Z2gEGofWd/J HswyBTWanKBjrLHfhnLQW3y0c8LBT0T8Zfu/4Da4xkops3UH8KNlOx1YBuf507hk/5oL whwQ== X-Gm-Message-State: AIkVDXIzN1MYPaB4S1Ee14Br/+iGFIo5uli1PPtlgumCTdGi0LqIEVVQ0kkm6xW+iBmt47yg X-Received: by 10.37.54.17 with SMTP id d17mr1557467yba.81.1481897132932; Fri, 16 Dec 2016 06:05:32 -0800 (PST) Subject: Re: [PATCH 02/17] Refactor Linux ipc_priv header To: Arnd Bergmann References: <1481545990-7247-1-git-send-email-adhemerval.zanella@linaro.org> <741391304.V4HNlYREZJ@wuerfel> <2453765.Bgf9HmvKZ1@wuerfel> <48ba3725-9d3d-c5d9-b51b-34860ccd3fea@linaro.org> Cc: libc-alpha@sourceware.org From: Adhemerval Zanella Message-ID: <5af5e95c-c9c0-993e-8fd0-d8d84642821b@linaro.org> Date: Fri, 16 Dec 2016 12:05:27 -0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <48ba3725-9d3d-c5d9-b51b-34860ccd3fea@linaro.org> On 12/12/2016 12:29, Adhemerval Zanella wrote: > > > On 12/12/2016 11:25, Arnd Bergmann wrote: >> On Monday, December 12, 2016 10:57:51 AM CET Adhemerval Zanella wrote: >>> >>> On 12/12/2016 10:47, Arnd Bergmann wrote: >>>> On Monday, December 12, 2016 10:32:55 AM CET Adhemerval Zanella wrote: >>>> >>>>> Since current IPC_64 default now on kernel is 0x100, some architectures >>>>> require it to 0 (for instance x86_64) while others continue to use a >>>>> non zero default regardless (powerpc). >>>> >>>> I find the explanation is a bit confusing, it's not that the kernel >>>> requires IPC_64 to be 0x100, it's that some architectures support >>>> the the old-style IPC and require IPC_64 to be passed, while new >>>> architectures should default to the new version. >>> >>> Right, I did not want to expose some kernel internal implementation, >>> but I think your explanation should be better. I will change to >>> >>> " >>> Some architectures support the old-style IPC and require IPC_64 >>> equal to 0x100 to be passed along SysV IPC syscalls, while new >>> architectures should default to new IPC version (without the >>> flags being set). >>> >>> This patch refactor current ipc_priv.h Linux headers in two directions: >>> >>> - Remove cross platform references (for instance alpha including powerpc >>> definition) and add required definition for each required port. The >>> idea is to avoid tie one architecture definition with another and make >>> platform change independent. >>> >>> - Move all common definitions (the ipc syscall commands) on a common >>> header, ipc_ops.h. >>> " >>> >>> Do you think it is less confusing? >> >> Sounds good, thanks! >> >>>> >>>>> index 0000000..1a31396 >>>>> --- /dev/null >>>>> +++ b/sysdeps/unix/sysv/linux/aarch64/ipc_priv.h >>>> ... >>>>> + >>>>> +#include /* For __key_t */ >>>>> + >>>>> +#define __IPC_64 0x0 >>>>> + >>>>> +struct __old_ipc_perm >>>>> +{ >>>>> + __key_t __key; /* Key. */ >>>>> + unsigned int uid; /* Owner's user ID. */ >>>>> + unsigned int gid; /* Owner's group ID. */ >>>>> + unsigned int cuid; /* Creator's user ID. */ >>>>> + unsigned int cgid; /* Creator's group ID. */ >>>>> + unsigned int mode; /* Read/write permission. */ >>>>> + unsigned short int __seq; /* Sequence number. */ >>>>> +}; >>>> >>>> I don't understand this part at all: the #define line first says >>>> that the old IPC is not supported, but then you define the structure >>>> anyway? >>>> >>>> What is the structure actually used for in glibc? My interpretation >>>> is that it's only needed to implement the SHLIB_COMPAT version >>>> of the library calls that don't exist on ARM64 either. >>> >>> You are right and it is mainly a limitation on how my code defines the >>> 'union semun' in semctl.c. I will change it to use an opaque type >>> instead of a pointer to __old_ipc_perm. >> >> I guess we can even remove the entire definition of >> struct __old_semid_ds etc on all architectures since we >> just pass a pointer from the compat entry point to the >> kernel without looking at the contents? > > Yeap, this is what I ended up doing in my local branch (remove aarch64 > ipc_priv.h ununsed definitions and cleanup semctl old ipc struct > definitions). This is the updated patch based on previous comments (CL is essentially the same): -- Some architectures support the old-style IPC and require IPC_64 equal to 0x100 to be passed along SysV IPC syscalls, while new architectures should default to new IPC version (without the flags being set). This patch refactor current ipc_priv.h Linux headers in two directions: - Remove cross platform references (for instance alpha including powerpc definition) and add required definition for each required port. The idea is to avoid tie one architecture definition with another and make platform change independent. - Move all common definitions (the ipc syscall commands) on a common header, ipc_ops.h. * sysdeps/unix/sysv/linux/aarch64/ipc_priv.h: New file. * sysdeps/unix/sysv/linux/alpha/ipc_priv.h: Avoid included other arch definition and define its own. * sysdeps/unix/sysv/linux/ipc_ops.h: New file. * sysdeps/unix/sysv/linux/x86_64/ipc_priv.h: Likewise. * sysdeps/unix/sysv/linux/sparc/sparc64/ipc_priv.h: Likewise. * sysdeps/unix/sysv/linux/mips/ipc_priv.h: Remove file. * sysdeps/unix/sysv/linux/mips/mips64/ipc_priv.h: New file. * sysdeps/unix/sysv/linux/ipc_priv.h: Move ipc syscall operation definitions to common header. * sysdeps/unix/sysv/linux/powerpc/ipc_priv.h: Use common syscall operation from ipc_ops.h. --- ChangeLog | 12 +++++++ sysdeps/unix/sysv/linux/aarch64/ipc_priv.h | 21 ++++++++++++ sysdeps/unix/sysv/linux/alpha/ipc_priv.h | 33 ++++++++++++++++++- sysdeps/unix/sysv/linux/ipc_ops.h | 30 +++++++++++++++++ sysdeps/unix/sysv/linux/ipc_priv.h | 23 +++++-------- sysdeps/unix/sysv/linux/mips/ipc_priv.h | 1 - sysdeps/unix/sysv/linux/mips/mips64/ipc_priv.h | 32 ++++++++++++++++++ sysdeps/unix/sysv/linux/powerpc/ipc_priv.h | 23 +++++-------- sysdeps/unix/sysv/linux/sparc/sparc64/ipc_priv.h | 41 ++++++++++++++++++++++++ sysdeps/unix/sysv/linux/x86_64/ipc_priv.h | 32 ++++++++++++++++++ 10 files changed, 216 insertions(+), 32 deletions(-) create mode 100644 sysdeps/unix/sysv/linux/aarch64/ipc_priv.h create mode 100644 sysdeps/unix/sysv/linux/ipc_ops.h delete mode 100644 sysdeps/unix/sysv/linux/mips/ipc_priv.h create mode 100644 sysdeps/unix/sysv/linux/mips/mips64/ipc_priv.h create mode 100644 sysdeps/unix/sysv/linux/sparc/sparc64/ipc_priv.h create mode 100644 sysdeps/unix/sysv/linux/x86_64/ipc_priv.h -- 2.7.4 diff --git a/sysdeps/unix/sysv/linux/aarch64/ipc_priv.h b/sysdeps/unix/sysv/linux/aarch64/ipc_priv.h new file mode 100644 index 0000000..f73530c --- /dev/null +++ b/sysdeps/unix/sysv/linux/aarch64/ipc_priv.h @@ -0,0 +1,21 @@ +/* Old SysV permission definition for Linux. AArch64 version. + Copyright (C) 2016 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#include /* For __key_t */ + +#define __IPC_64 0x0 diff --git a/sysdeps/unix/sysv/linux/alpha/ipc_priv.h b/sysdeps/unix/sysv/linux/alpha/ipc_priv.h index 67883be..06444fc 100644 --- a/sysdeps/unix/sysv/linux/alpha/ipc_priv.h +++ b/sysdeps/unix/sysv/linux/alpha/ipc_priv.h @@ -1 +1,32 @@ -#include +/* Old SysV permission definition for Linux. Alpha version. + Copyright (C) 2016 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#include /* For __key_t */ + +#define __IPC_64 0x100 + +struct __old_ipc_perm +{ + __key_t __key; /* Key. */ + unsigned int uid; /* Owner's user ID. */ + unsigned int gid; /* Owner's group ID. */ + unsigned int cuid; /* Creator's user ID. */ + unsigned int cgid; /* Creator's group ID. */ + unsigned int mode; /* Read/write permission. */ + unsigned short int __seq; /* Sequence number. */ +}; diff --git a/sysdeps/unix/sysv/linux/ipc_ops.h b/sysdeps/unix/sysv/linux/ipc_ops.h new file mode 100644 index 0000000..ea3028c --- /dev/null +++ b/sysdeps/unix/sysv/linux/ipc_ops.h @@ -0,0 +1,30 @@ +/* The codes for the functions to use the ipc syscall multiplexer. + Copyright (C) 2016 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#define IPCOP_semop 1 +#define IPCOP_semget 2 +#define IPCOP_semctl 3 +#define IPCOP_semtimedop 4 +#define IPCOP_msgsnd 11 +#define IPCOP_msgrcv 12 +#define IPCOP_msgget 13 +#define IPCOP_msgctl 14 +#define IPCOP_shmat 21 +#define IPCOP_shmdt 22 +#define IPCOP_shmget 23 +#define IPCOP_shmctl 24 diff --git a/sysdeps/unix/sysv/linux/ipc_priv.h b/sysdeps/unix/sysv/linux/ipc_priv.h index 7ded463..9b97f00 100644 --- a/sysdeps/unix/sysv/linux/ipc_priv.h +++ b/sysdeps/unix/sysv/linux/ipc_priv.h @@ -1,4 +1,5 @@ -/* Copyright (C) 1995-2016 Free Software Foundation, Inc. +/* Old SysV permission definition for Linux. Default version. + Copyright (C) 1995-2016 Free Software Foundation, Inc. This file is part of the GNU C Library. The GNU C Library is free software; you can redistribute it and/or @@ -15,7 +16,7 @@ License along with the GNU C Library; if not, see . */ -#include +#include /* For __key_t */ #define __IPC_64 0x100 @@ -30,17 +31,9 @@ struct __old_ipc_perm unsigned short int __seq; /* Sequence number. */ }; +#define SEMCTL_ARG_ADDRESS(__arg) &__arg.array -/* The codes for the functions to use the ipc syscall multiplexer. */ -#define IPCOP_semop 1 -#define IPCOP_semget 2 -#define IPCOP_semctl 3 -#define IPCOP_semtimedop 4 -#define IPCOP_msgsnd 11 -#define IPCOP_msgrcv 12 -#define IPCOP_msgget 13 -#define IPCOP_msgctl 14 -#define IPCOP_shmat 21 -#define IPCOP_shmdt 22 -#define IPCOP_shmget 23 -#define IPCOP_shmctl 24 +#define MSGRCV_ARGS(__msgp, __msgtyp) \ + ((long int []){ (long int) __msgp, __msgtyp }) + +#include diff --git a/sysdeps/unix/sysv/linux/mips/ipc_priv.h b/sysdeps/unix/sysv/linux/mips/ipc_priv.h deleted file mode 100644 index 67883be..0000000 --- a/sysdeps/unix/sysv/linux/mips/ipc_priv.h +++ /dev/null @@ -1 +0,0 @@ -#include diff --git a/sysdeps/unix/sysv/linux/mips/mips64/ipc_priv.h b/sysdeps/unix/sysv/linux/mips/mips64/ipc_priv.h new file mode 100644 index 0000000..9f47d89 --- /dev/null +++ b/sysdeps/unix/sysv/linux/mips/mips64/ipc_priv.h @@ -0,0 +1,32 @@ +/* Old SysV permission definition for Linux. MIPS64 version. + Copyright (C) 2016 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#include + +#define __IPC_64 0x100 + +struct __old_ipc_perm +{ + __key_t __key; /* Key. */ + int uid; /* Owner's user ID. */ + int gid; /* Owner's group ID. */ + int cuid; /* Creator's user ID. */ + int cgid; /* Creator's group ID. */ + int mode; /* Read/write permission. */ + unsigned short int __seq; /* Sequence number. */ +}; diff --git a/sysdeps/unix/sysv/linux/powerpc/ipc_priv.h b/sysdeps/unix/sysv/linux/powerpc/ipc_priv.h index baae7ab..4f72f58 100644 --- a/sysdeps/unix/sysv/linux/powerpc/ipc_priv.h +++ b/sysdeps/unix/sysv/linux/powerpc/ipc_priv.h @@ -1,4 +1,5 @@ -/* Copyright (C) 1995-2016 Free Software Foundation, Inc. +/* Old SysV permission definition for Linux. PowerPC version. + Copyright (C) 1995-2016 Free Software Foundation, Inc. This file is part of the GNU C Library. The GNU C Library is free software; you can redistribute it and/or @@ -15,7 +16,7 @@ License along with the GNU C Library; if not, see . */ -#include +#include /* For __key_t */ #define __IPC_64 0x100 @@ -30,17 +31,9 @@ struct __old_ipc_perm unsigned short int __seq; /* Sequence number. */ }; +#define SEMCTL_ARG_ADDRESS(__arg) &__arg.array -/* The codes for the functions to use the ipc syscall multiplexer. */ -#define IPCOP_semop 1 -#define IPCOP_semget 2 -#define IPCOP_semctl 3 -#define IPCOP_semtimedop 4 -#define IPCOP_msgsnd 11 -#define IPCOP_msgrcv 12 -#define IPCOP_msgget 13 -#define IPCOP_msgctl 14 -#define IPCOP_shmat 21 -#define IPCOP_shmdt 22 -#define IPCOP_shmget 23 -#define IPCOP_shmctl 24 +#define MSGRCV_ARGS(__msgp, __msgtyp) \ + ((long int []){ (long int) __msgp, __msgtyp }) + +#include diff --git a/sysdeps/unix/sysv/linux/sparc/sparc64/ipc_priv.h b/sysdeps/unix/sysv/linux/sparc/sparc64/ipc_priv.h new file mode 100644 index 0000000..386ff8a --- /dev/null +++ b/sysdeps/unix/sysv/linux/sparc/sparc64/ipc_priv.h @@ -0,0 +1,41 @@ +/* Old SysV permission definition for Linux. x86_64 version. + Copyright (C) 2016 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#include /* For __key_t */ + +#define __IPC_64 0x0 + +struct __old_ipc_perm +{ + __key_t __key; /* Key. */ + unsigned int uid; /* Owner's user ID. */ + unsigned int gid; /* Owner's group ID. */ + unsigned int cuid; /* Creator's user ID. */ + unsigned int cgid; /* Creator's group ID. */ + unsigned int mode; /* Read/write permission. */ + unsigned short int __seq; /* Sequence number. */ +}; + +/* SPARC semctl multiplex syscall expects the union pointed address, not + the union address itself. */ +#define SEMCTL_ARG_ADDRESS(__arg) __arg.array + +/* Also for msgrcv it does not use the kludge on final 2 arguments. */ +#define MSGRCV_ARGS(__msgp, __msgtyp) __msgp, __msgtyp + +#include diff --git a/sysdeps/unix/sysv/linux/x86_64/ipc_priv.h b/sysdeps/unix/sysv/linux/x86_64/ipc_priv.h new file mode 100644 index 0000000..d39db53 --- /dev/null +++ b/sysdeps/unix/sysv/linux/x86_64/ipc_priv.h @@ -0,0 +1,32 @@ +/* Old SysV permission definition for Linux. x86_64 version. + Copyright (C) 2016 Free Software Foundation, Inc. + This file is part of the GNU C Library. + + The GNU C Library is free software; you can redistribute it and/or + modify it under the terms of the GNU Lesser General Public + License as published by the Free Software Foundation; either + version 2.1 of the License, or (at your option) any later version. + + The GNU C Library is distributed in the hope that it will be useful, + but WITHOUT ANY WARRANTY; without even the implied warranty of + MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + Lesser General Public License for more details. + + You should have received a copy of the GNU Lesser General Public + License along with the GNU C Library; if not, see + . */ + +#include /* For __key_t */ + +#define __IPC_64 0x0 + +struct __old_ipc_perm +{ + __key_t __key; /* Key. */ + unsigned short uid; /* Owner's user ID. */ + unsigned short gid; /* Owner's group ID. */ + unsigned short cuid; /* Creator's user ID. */ + unsigned short cgid; /* Creator's group ID. */ + unsigned short mode; /* Read/write permission. */ + unsigned short int __seq; /* Sequence number. */ +};