From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | t-ishii(at)sra(dot)co(dot)jp |
Cc: | hackers(at)postgreSQL(dot)org |
Subject: | Re: [HACKERS] acl problem in NetBSD/m68k |
Date: | 1999-06-29 04:46:03 |
Message-ID: | 19251.930631563@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Tatsuo Ishii <t-ishii(at)sra(dot)co(dot)jp> writes:
>> I do not like this patch at *all*. Why is sizeof(AclItem) not the
>> correct thing to use?
> In NetBSD/m68k sizeof(AclItem) = 6, not 8.
Oh, I see: the struct contains int32, uint8, uint8, and so it will
be padded to a multiple of int32's alignment requirement --- which
is 4 most places but only 2 on m68k.
>> Perhaps the real problem is that the AclItem struct definition needs
>> modification? Or maybe we need a way to put a machine-dependent size
>> into the pg_type entry for type aclitem? The latter seems like a
>> good thing to be able to do on general principles.
>
> Glad to hear you have better idea. Anyway, NetBSD/m68k users need some
> way to fix the problem now, since the problem seems very serious.
There are two ways we could attack this: (1) put a "pad" field into
struct AclItem (prolly two uint8s) to try to ensure that compilers
would think it is 8 bytes long, or (2) make the size field for aclitem
in pg_type.h read "sizeof(AclItem)". I think the latter is a better
long-term solution, because it eliminates having to try to guess
what a compiler will do with a struct declaration. But there are
several possible counterarguments:
* It might require patching the scripts that read pg_type.h --- I
am not sure if they'd work unmodified.
* We'd either need to #include acl.h into pg_type.h or push the
declarations for AclItem into some more-widely-used header.
* In theory this would represent an initdb change and couldn't
be applied before 6.6. In practice, Postgres isn't working right
now on any platform where sizeof(AclItem) != 8, so initdb would
*not* be needed for any working installation.
I don't think any of these counterarguments is a big deal, but
maybe someone else will have a different opinion.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Tatsuo Ishii | 1999-06-29 04:50:48 | Re: [HACKERS] acl problem in NetBSD/m68k |
Previous Message | Bruce Momjian | 1999-06-29 04:30:28 | Re: [HACKERS] 6.5.1 status |