On Fri, Oct 05, 2012 at 09:37:38AM +0100, Mans Rullgard wrote:
On 5 October 2012 09:33, Russell King - ARM Linux linux@arm.linux.org.uk wrote:
On Fri, Oct 05, 2012 at 09:33:04AM +0100, Mans Rullgard wrote:
On 5 October 2012 09:24, Russell King - ARM Linux linux@arm.linux.org.uk wrote:
On Fri, Oct 05, 2012 at 09:20:56AM +0100, Mans Rullgard wrote:
On 5 October 2012 08:12, Russell King - ARM Linux linux@arm.linux.org.uk wrote:
On Fri, Oct 05, 2012 at 03:25:16AM +0100, Mans Rullgard wrote: > On 5 October 2012 02:56, Rob Herring robherring2@gmail.com wrote: > > This struct is the IP header, so a struct ptr is just set to the > > beginning of the received data. Since ethernet headers are 14 bytes, > > often the IP header is not aligned unless the NIC can place the frame at > > a 2 byte offset (which is something I need to investigate). So this > > function cannot make any assumptions about the alignment. Does the ABI > > define structs have some minimum alignment? Does the struct need to be > > declared as packed or something? > > The ABI defines the alignment of structs as the maximum alignment of its > members. Since this struct contains 32-bit members, the alignment for the > whole struct becomes 32 bits as well. Declaring it as packed tells gcc it > might be unaligned (in addition to removing any holes within).
This has come up before in the past.
The Linux network folk will _not_ allow - in any shape or form - for this struct to be marked packed (it's the struct which needs to be marked packed) because by doing so, it causes GCC to issue byte loads/ stores on architectures where there isn't a problem, and that decreases the performance of the Linux IP stack unnecessarily.
Which architectures? I have never seen anything like that.
Does it matter?
It matters if we want to fix it.
I can't help you there. Ask the networking people.
I'm asking anyone who is listening. Since you've (apparently) seen the claim being made, perhaps you can point to an email or so with more information. I'd hate to think you're just making up excuses.
Sorry, I'm just not going to bother - especially given your attitude of "you're just making up excuses" - you just accused me of being a liar. Thanks for that.
We have the merge window open at the moment, and I've only just got back from being on holiday for the first half of that - so I have new tree conflicts to deal with, and need to finish preparing my tree for Linus. I'm not going to waste the precious time that the merge window is open for searching lots of mail archives for this when others can do the same, or even try sending the Linux networking people a patch adding __packed to their structures and seeing the response.