Hi Chanho, On Mon, Dec 7, 2020 at 8:58 PM Chanho Park <chanho61.p...@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. Thanks, Namhyung