From: Arnaldo Carvalho de Melo <acme@kernel.org>
To: Li Zhijian <lizhijian@cn.fujitsu.com>
Cc: peterz@infradead.org, mingo@redhat.com,
alexander.shishkin@linux.intel.com, linux-kernel@vger.kernel.org,
philip.li@intel.com
Subject: Re: [PATCH] tool/perf: fix perf compiling errors
Date: Thu, 21 Sep 2017 12:01:38 -0300 [thread overview]
Message-ID: <20170921150138.GB29668@kernel.org> (raw)
In-Reply-To: <1505973673-2424-1-git-send-email-lizhijian@cn.fujitsu.com>
Em Thu, Sep 21, 2017 at 02:01:13PM +0800, Li Zhijian escreveu:
> since f045b8c, we failed to build perf with LIBCLANGLLVM=1, this patch is to
> fix the following compiling errors
> --------------
> lizhijian@haswell-OptiPlex-9020:~/lkp/linux/tools/perf$ make LLVM_CONFIG=/usr/bin/llvm-config-3.9 LIBCLANGLLVM=1
> CXX util/c++/clang-test.o
> In file included from /home/lizj/linux/tools/include/linux/list.h:5:0,
> from /home/lizj/linux/tools/perf/util/namespaces.h:13,
> from /home/lizj/linux/tools/perf/util/util.h:15,
> from /home/lizj/linux/tools/perf/util/util-cxx.h:20,
> from util/c++/clang-c.h:5,
> from util/c++/clang-test.cpp:2:
> /home/lizj/linux/tools/include/linux/list.h: In function ‘void list_del(list_head*)’:
> /home/lizj/linux/tools/include/linux/poison.h:14:31: error: pointer of type ‘void *’ used in arithmetic [-Werror=pointer-arith]
> # define POISON_POINTER_DELTA 0
> ^
> /home/lizj/linux/tools/include/linux/poison.h:22:41: note: in expansion of macro ‘POISON_POINTER_DELTA’
> #define LIST_POISON1 ((void *) 0x100 + POISON_POINTER_DELTA)
> ^
> /home/lizj/linux/tools/include/linux/list.h:107:16: note: in expansion of macro ‘LIST_POISON1’
> entry->next = LIST_POISON1;
> ^
> In file included from /home/lizj/linux/tools/perf/util/namespaces.h:13:0,
> from /home/lizj/linux/tools/perf/util/util.h:15,
> from /home/lizj/linux/tools/perf/util/util-cxx.h:20,
> from util/c++/clang-c.h:5,
> from util/c++/clang-test.cpp:2:
> /home/lizj/linux/tools/include/linux/list.h:107:14: error: invalid conversion from ‘void*’ to ‘list_head*’ [-fpermissive]
> +++ b/tools/perf/util/namespaces.h
> @@ -10,7 +10,6 @@
> #define __PERF_NAMESPACES_H
>
> #include "../perf.h"
> -#include <linux/list.h>
> #include <linux/refcount.h>
Well, you can't do that, as namespaces.h needs list.h, a bit after te
above hunk:
9 #ifndef __PERF_NAMESPACES_H
10 #define __PERF_NAMESPACES_H
11
12 #include "../perf.h"
13 #include <linux/list.h>
14 #include <linux/refcount.h>
15
16 struct namespaces_event;
17
18 struct namespaces {
19 struct list_head list;
20 u64 end_time;
21 struct perf_ns_link_info link_info[];
22 };
Can you try with the following patch instead?
diff --git a/tools/include/linux/poison.h b/tools/include/linux/poison.h
index 51334edec506..f306a7642509 100644
--- a/tools/include/linux/poison.h
+++ b/tools/include/linux/poison.h
@@ -14,6 +14,10 @@
# define POISON_POINTER_DELTA 0
#endif
+#ifdef __cplusplus
+#define LIST_POISON1 NULL
+#define LIST_POISON2 NULL
+#else
/*
* These are non-NULL pointers that will result in page faults
* under normal circumstances, used to verify that nobody uses
@@ -21,6 +25,7 @@
*/
#define LIST_POISON1 ((void *) 0x100 + POISON_POINTER_DELTA)
#define LIST_POISON2 ((void *) 0x200 + POISON_POINTER_DELTA)
+#endif
/********** include/linux/timer.h **********/
/*
next prev parent reply other threads:[~2017-09-21 15:01 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-21 6:01 [PATCH] tool/perf: fix perf compiling errors Li Zhijian
2017-09-21 15:01 ` Arnaldo Carvalho de Melo [this message]
-- strict thread matches above, loose matches on Subject: below --
2017-09-21 5:59 Li Zhijian
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20170921150138.GB29668@kernel.org \
--to=acme@kernel.org \
--cc=alexander.shishkin@linux.intel.com \
--cc=linux-kernel@vger.kernel.org \
--cc=lizhijian@cn.fujitsu.com \
--cc=mingo@redhat.com \
--cc=peterz@infradead.org \
--cc=philip.li@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).