On Fri, 2011-01-21 at 16:24 -0500, John W. Linville wrote: > On Wed, Jan 19, 2011 at 12:50:59PM +0100, Johannes Berg wrote: > > From: Johannes Berg <johannes.berg@xxxxxxxxx> > > > > For events that include only the local struct as > > their parameter, we can use DECLARE_EVENT_CLASS > > and save quite some binary size across segments > > as well lines of code. > > > > text data bss dec hex filename > > 375745 19296 916 395957 60ab5 mac80211.ko.before > > 367473 17888 916 386277 5e4e5 mac80211.ko.after > > -8272 -1408 0 -9680 -25d0 delta > > > > Some more tracepoints with identical arguments > > could be combined like this but for now this is > > the one that benefits most. > > > > Signed-off-by: Johannes Berg <johannes.berg@xxxxxxxxx> > > Is this patch complete? It seems to break the build with undefined symbols... Yes, I believe it is, I'll resend the version I just compile tested against wireless-testing. johannes -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html