Re: [PATCH 5/7] powerpc/perf: Define big-endian version ofperf_mem_data_src
From: Vince Weaver
Date: Sat Aug 10 2013 - 22:34:25 EST
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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/