Re: Subtleties of __attribute__((packed))

On Wed, Dec 06, 2006 at 04:54:39PM +0100, Jan Blunck wrote:
> Maybe the arm backend is somehow broken. AFAIK (and I verfied it on S390 and
> i386) the alignment shouldn't change.

Please read the info pages:

`packed'
     This attribute, attached to an `enum', `struct', or `union' type
     definition, specifies that the minimum required memory be used to
     represent the type.

     Specifying this attribute for `struct' and `union' types is
     equivalent to specifying the `packed' attribute on each of the
     structure or union members.  Specifying the `-fshort-enums' flag
     on the line is equivalent to specifying the `packed' attribute on
     all `enum' definitions.

Note that it says *nothing* about alignment.  It says "minimum required
memory be used to represent the type." which implies that the internals
of the structure are packed together as tightly as possible.

It does not say "and as such the struct may be aligned to any alignment".

-- 
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

---

• Follow-Ups:
  • Re: Subtleties of __attribute__((packed))
    • From: Jan Blunck <[email protected]>
  • Re: Subtleties of __attribute__((packed))
    • From: David Miller <[email protected]>

• References:
  • Re: Subtleties of __attribute__((packed))
    • From: "Jan Blunck" <[email protected]>
  • Re: Subtleties of __attribute__((packed))
    • From: "Phil Endecott" <[email protected]>
  • Re: Subtleties of __attribute__((packed))
    • From: Jan Blunck <[email protected]>

• Prev by Date: Re: [PATCH] Export current_is_keventd() for libphy
• Next by Date: [PATCH] WorkStruct: Implement generic UP cmpxchg() where an arch doesn't support it [try #2]
• Previous by thread: Re: Subtleties of __attribute__((packed))
• Next by thread: Re: Subtleties of __attribute__((packed))
• Index(es):
  • Date
  • Thread

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]