diff mbox series

[bpf] samples/bpf: fix build for task_fd_query_user.c

Message ID 20191001112249.27341-1-bjorn.topel@gmail.com
State Accepted
Delegated to: BPF Maintainers
Headers show
Series [bpf] samples/bpf: fix build for task_fd_query_user.c | expand

Commit Message

Björn Töpel Oct. 1, 2019, 11:22 a.m. UTC 
From: Björn Töpel <bjorn.topel@intel.com>

Add missing "linux/perf_event.h" include file.

Signed-off-by: Björn Töpel <bjorn.topel@intel.com>
---
 samples/bpf/task_fd_query_user.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Song Liu Oct. 2, 2019, 6:46 p.m. UTC | #1 
On Tue, Oct 1, 2019 at 4:26 AM Björn Töpel <bjorn.topel@gmail.com> wrote:
>
> From: Björn Töpel <bjorn.topel@intel.com>
>
> Add missing "linux/perf_event.h" include file.
>
> Signed-off-by: Björn Töpel <bjorn.topel@intel.com>

Acked-by: Song Liu <songliubraving@fb.com>
KP Singh Oct. 2, 2019, 9:02 p.m. UTC | #2 
On Wed, Oct 2, 2019 at 8:46 PM Song Liu <liu.song.a23@gmail.com> wrote:
>
> On Tue, Oct 1, 2019 at 4:26 AM Björn Töpel <bjorn.topel@gmail.com> wrote:
> >
> > From: Björn Töpel <bjorn.topel@intel.com>
> >
> > Add missing "linux/perf_event.h" include file.
> >
> > Signed-off-by: Björn Töpel <bjorn.topel@intel.com>
>
> Acked-by: Song Liu <songliubraving@fb.com>

Tested-by: KP Singh <kpsingh@google.com>

(https://lore.kernel.org/bpf/20191002185233.GA3650@chromium.org/T/#t)
Daniel Borkmann Oct. 3, 2019, 2:54 p.m. UTC | #3 
On Wed, Oct 02, 2019 at 11:02:09PM +0200, KP Singh wrote:
> On Wed, Oct 2, 2019 at 8:46 PM Song Liu <liu.song.a23@gmail.com> wrote:
> > On Tue, Oct 1, 2019 at 4:26 AM Björn Töpel <bjorn.topel@gmail.com> wrote:
> > >
> > > From: Björn Töpel <bjorn.topel@intel.com>
> > >
> > > Add missing "linux/perf_event.h" include file.
> > >
> > > Signed-off-by: Björn Töpel <bjorn.topel@intel.com>
> >
> > Acked-by: Song Liu <songliubraving@fb.com>
> 
> Tested-by: KP Singh <kpsingh@google.com>
> 
> (https://lore.kernel.org/bpf/20191002185233.GA3650@chromium.org/T/#t)

Applied, thanks! Given also KP and Florent sent a similar patch earlier,
I've added Reported-by and improved the commit message a bit to something
more useful with regards to *why* we suddenly need to add the include.
diff mbox series

Patch

diff --git a/samples/bpf/task_fd_query_user.c b/samples/bpf/task_fd_query_user.c
index e39938058223..4c31b305e6ef 100644
--- a/samples/bpf/task_fd_query_user.c
+++ b/samples/bpf/task_fd_query_user.c
@@ -13,6 +13,7 @@ 
 #include <sys/resource.h>
 #include <sys/types.h>
 #include <sys/stat.h>
+#include <linux/perf_event.h>
 
 #include "libbpf.h"
 #include "bpf_load.h"