[PATCH 8/8] firmware: mark some structs as packed
Harald Welte
laforge at gnumonks.org
Thu Nov 24 09:40:19 CET 2011
Hi Sylvain and Alexander,
On Thu, Nov 24, 2011 at 07:58:33AM +0100, Sylvain Munaut wrote:
> First, thanks for this series of patch, I'll be sure to review and test them.
Thanks for testing, Sylvain.
> > -};
> > +} __packed;
>
> I'm not conviced by this one : Why should we use packed for structures
> not used as 'communication packets' ?
I agree with Sylvain here. this will just cause the compiler to
generate much less efficient code at no other benefit...
--
- Harald Welte <laforge at gnumonks.org> http://laforge.gnumonks.org/
============================================================================
"Privacy in residential applications is a desirable marketing option."
(ETSI EN 300 175-7 Ch. A6)
More information about the baseband-devel
mailing list