On Mon, Dec 07, 2020 at 09:31:06PM +0900, Namhyung Kim wrote:
Hi Chanho,
On Mon, Dec 7, 2020 at 8:58 PM Chanho Park chanho61.park@samsung.com wrote:
Hi Will,
Looks like other files just include this unconditionally, but have a comment explaining why. See util/branch.h and util/event.h. Maybe we should do the same for util/pmu.h, which is already included here?
I found below files which perf includes <linux/perf_event.h>. Instead of doing same for all, we'd better put this only for tools/include/uapi/linux/perf_event.h.
It's a copy of the kernel header, I'm not sure we want to add something there.
right, we want to copy that directly from kernel uapi so let's not do any changes in here
jirka