Re: [PATCH] tools/perf: Remove duplicate header

[Souptick Joarder]

On Tue, Jan 15, 2019 at 7:29 PM Brajeswar Ghosh
<brajeswar.linux@xxxxxxxxx> wrote:
>
> Remove duplicate headers which are included more than once
>
> Signed-off-by: Brajeswar Ghosh <brajeswar.linux@xxxxxxxxx>

Acked-by: Souptick Joarder <jrdr.linux@xxxxxxxxx>

> ---
>  tools/perf/builtin-stat.c     | 1 -
>  tools/perf/tests/bp_account.c | 1 -
>  tools/perf/util/dso.h         | 1 -
>  tools/perf/util/zlib.c        | 1 -
>  4 files changed, 4 deletions(-)
>
> diff --git a/tools/perf/builtin-stat.c b/tools/perf/builtin-stat.c
> index d1028d7755bb..f919ffc4b3b3 100644
> --- a/tools/perf/builtin-stat.c
> +++ b/tools/perf/builtin-stat.c
> @@ -83,7 +83,6 @@
>  #include <unistd.h>
>  #include <sys/time.h>
>  #include <sys/resource.h>
> -#include <sys/wait.h>
>
>  #include "sane_ctype.h"
>
> diff --git a/tools/perf/tests/bp_account.c b/tools/perf/tests/bp_account.c
> index a20cbc445426..57fc544aedb0 100644
> --- a/tools/perf/tests/bp_account.c
> +++ b/tools/perf/tests/bp_account.c
> @@ -15,7 +15,6 @@
>  #include <sys/mman.h>
>  #include <linux/compiler.h>
>  #include <linux/hw_breakpoint.h>
> -#include <sys/ioctl.h>
>
>  #include "tests.h"
>  #include "debug.h"
> diff --git a/tools/perf/util/dso.h b/tools/perf/util/dso.h
> index c5380500bed4..722d67a407e2 100644
> --- a/tools/perf/util/dso.h
> +++ b/tools/perf/util/dso.h
> @@ -8,7 +8,6 @@
>  #include <sys/types.h>
>  #include <stdbool.h>
>  #include "rwsem.h"
> -#include <linux/types.h>
>  #include <linux/bitops.h>
>  #include "map.h"
>  #include "namespaces.h"
> diff --git a/tools/perf/util/zlib.c b/tools/perf/util/zlib.c
> index 902ce6384f57..512ad7c09b13 100644
> --- a/tools/perf/util/zlib.c
> +++ b/tools/perf/util/zlib.c
> @@ -6,7 +6,6 @@
>  #include <sys/mman.h>
>  #include <zlib.h>
>  #include <linux/compiler.h>
> -#include <unistd.h>
>
>  #include "util/compress.h"
>  #include "util/util.h"
> --
> 2.17.1
>