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: Sat, 10 Aug 2013 22:34:58 -0400 (EDT) From: Vince Weaver <vince@...ter.net> To: Sukadev Bhattiprolu <sukadev@...ux.vnet.ibm.com> cc: linux-kernel@...r.kernel.org, linuxppc-dev@...abs.org, Stephane Eranian <eranian@...gle.com>, Paul Mackerras <paulus@...ba.org>, Anton Blanchard <anton@....ibm.com>, Michael Ellerman <michael@...erman.id.au>, Anshuman Khandual <khandual@...ux.vnet.ibm.com> Subject: Re: [PATCH 5/7] powerpc/perf: Define big-endian version of perf_mem_data_src On Sat, 10 Aug 2013, Sukadev Bhattiprolu wrote: > > include/uapi/linux/perf_event.h | 55 +++++++++++++++++++++++++++++++++++++++ > 1 files changed, 55 insertions(+), 0 deletions(-) > +#define __PERF_LE 1234 > +#define __PERF_BE 4321 > + > +#if defined(__KERNEL__) I could be wrong, but I thought files under uapi weren't supposed to contain __KERNEL__ code. Wasn't that the whole point of uapi? Also having the perf_event interface depend on endianess just seems like a complicated mess. Can't we just declare the interface to be a certain endianess and have the kernel byte-swap as necessary? Vince -- 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