lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 08 Mar 2011 22:39:51 -0800 (PST) From: David Miller <davem@...emloft.net> To: rostedt@...dmis.org Cc: linux-kernel@...r.kernel.org, mrubin@...gle.com, dhsharp@...gle.com Subject: Re: [PATCH 04/15] ftrace: pack event structures. From: Steven Rostedt <rostedt@...dmis.org> Date: Tue, 08 Mar 2011 18:30:03 -0500 > I'm going back through this patch set and looking to incorporate it into > 2.6.39. But I know how touchy sparc is with "packed" data structures. Do > you see this patch as hurting sparc? > > I wonder if we should change this to something like: > > #ifdef CONFIG_HAVE_EFFICIENT_UNALIGNED_ACCESS > # define do_event_packed __attirbute__((packed)) > #else > # define do_event_packed > #endif > > and use "do_event_packed" instead? I think you should elide packed at all costs, and instead tell the compiler what your intentions are by exposing the real types using unions or similar instead of hiding it in an opaque way behind final char[] arrays. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists