tools: Adopt __packed from kernel sources
authorArnaldo Carvalho de Melo <acme@redhat.com>
Fri, 16 Jun 2017 14:39:15 +0000 (11:39 -0300)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Mon, 19 Jun 2017 18:27:07 +0000 (15:27 -0300)
To have a more compact way to ask the compiler to not insert alignment
paddings in a struct, making tools/ look more like kernel source code.

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: David Ahern <dsahern@gmail.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/n/tip-byp46nr7hsxvvyc9oupfb40q@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/include/linux/compiler-gcc.h
tools/perf/util/genelf_debug.c

index c13e6f7..0f57a48 100644 (file)
@@ -22,6 +22,8 @@
 
 #define  noinline      __attribute__((noinline))
 
+#define __packed       __attribute__((packed))
+
 #define __noreturn     __attribute__((noreturn))
 
 #define __printf(a, b) __attribute__((format(printf, a, b)))
index 5980f7d..40789d8 100644 (file)
@@ -11,6 +11,7 @@
  * @remark Copyright 2007 OProfile authors
  * @author Philippe Elie
  */
+#include <linux/compiler.h>
 #include <sys/types.h>
 #include <stdio.h>
 #include <getopt.h>
@@ -125,7 +126,7 @@ struct debug_line_header {
         * and filesize, last entry is followed by en empty string.
         */
        /* follow the first program statement */
-} __attribute__((packed));
+} __packed;
 
 /* DWARF 2 spec talk only about one possible compilation unit header while
  * binutils can handle two flavours of dwarf 2, 32 and 64 bits, this is not
@@ -138,7 +139,7 @@ struct compilation_unit_header {
        uhalf version;
        uword debug_abbrev_offset;
        ubyte pointer_size;
-} __attribute__((packed));
+} __packed;
 
 #define DW_LNS_num_opcode (DW_LNS_set_isa + 1)