Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1492257 > unrolled thread

[PATCH] perf tools: Fix building in 32 bit platform

Started byWang Nan <wangnan0@huawei.com>
First post2016-09-28 06:00 +0200
Last post2016-09-29 20:20 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] perf tools: Fix building in 32 bit platform Wang Nan <wangnan0@huawei.com> - 2016-09-28 06:00 +0200
    Re: [PATCH] perf tools: Fix building in 32 bit platform Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-09-28 15:40 +0200
    [tip:perf/core] perf data: Fix building in 32 bit platform with  libbabeltrace tip-bot for Wang Nan <tipbot@zytor.com> - 2016-09-29 20:20 +0200

#1492257 — [PATCH] perf tools: Fix building in 32 bit platform

FromWang Nan <wangnan0@huawei.com>
Date2016-09-28 06:00 +0200
Subject[PATCH] perf tools: Fix building in 32 bit platform
Message-ID<smech-1D0-1@gated-at.bofh.it>
On ARM32 building it report following error:

util/data-convert-bt.c: In function 'add_bpf_output_values':
util/data-convert-bt.c:440:3: error: format '%lu' expects argument of type 'long unsigned int', but argument 5 has type 'unsigned int' [-Werror=format]
cc1: all warnings being treated as errors

Fix it by changing %lu to %zu.

Signed-off-by: Wang Nan <wangnan0@huawei.com>
Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
Cc: Jiri Olsa <jolsa@kernel.org>
---
 tools/perf/util/data-convert-bt.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/perf/util/data-convert-bt.c b/tools/perf/util/data-convert-bt.c
index 4f979bb..7123f4d 100644
--- a/tools/perf/util/data-convert-bt.c
+++ b/tools/perf/util/data-convert-bt.c
@@ -437,7 +437,7 @@ add_bpf_output_values(struct bt_ctf_event_class *event_class,
 	int ret;
 
 	if (nr_elements * sizeof(u32) != raw_size)
-		pr_warning("Incorrect raw_size (%u) in bpf output event, skip %lu bytes\n",
+		pr_warning("Incorrect raw_size (%u) in bpf output event, skip %zu bytes\n",
 			   raw_size, nr_elements * sizeof(u32) - raw_size);
 
 	len_type = bt_ctf_event_class_get_field_by_name(event_class, "raw_len");
-- 
1.8.3.4

[toc] | [next] | [standalone]


#1492601

FromArnaldo Carvalho de Melo <acme@kernel.org>
Date2016-09-28 15:40 +0200
Message-ID<smnfz-7mO-7@gated-at.bofh.it>
In reply to#1492257
Em Wed, Sep 28, 2016 at 03:58:46AM +0000, Wang Nan escreveu:
> On ARM32 building it report following error:
> 
> util/data-convert-bt.c: In function 'add_bpf_output_values':
> util/data-convert-bt.c:440:3: error: format '%lu' expects argument of type 'long unsigned int', but argument 5 has type 'unsigned int' [-Werror=format]
> cc1: all warnings being treated as errors
> 
> Fix it by changing %lu to %zu.

Thanks, applied.

- Arnaldo
 
> Signed-off-by: Wang Nan <wangnan0@huawei.com>
> Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
> Cc: Jiri Olsa <jolsa@kernel.org>
> ---
>  tools/perf/util/data-convert-bt.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tools/perf/util/data-convert-bt.c b/tools/perf/util/data-convert-bt.c
> index 4f979bb..7123f4d 100644
> --- a/tools/perf/util/data-convert-bt.c
> +++ b/tools/perf/util/data-convert-bt.c
> @@ -437,7 +437,7 @@ add_bpf_output_values(struct bt_ctf_event_class *event_class,
>  	int ret;
>  
>  	if (nr_elements * sizeof(u32) != raw_size)
> -		pr_warning("Incorrect raw_size (%u) in bpf output event, skip %lu bytes\n",
> +		pr_warning("Incorrect raw_size (%u) in bpf output event, skip %zu bytes\n",
>  			   raw_size, nr_elements * sizeof(u32) - raw_size);
>  
>  	len_type = bt_ctf_event_class_get_field_by_name(event_class, "raw_len");
> -- 
> 1.8.3.4

[toc] | [prev] | [next] | [standalone]


#1493642 — [tip:perf/core] perf data: Fix building in 32 bit platform with libbabeltrace

Fromtip-bot for Wang Nan <tipbot@zytor.com>
Date2016-09-29 20:20 +0200
Subject[tip:perf/core] perf data: Fix building in 32 bit platform with libbabeltrace
Message-ID<smO65-7D0-17@gated-at.bofh.it>
In reply to#1492257
Commit-ID:  f2c8852e6e990fcab0d9e68de9d86e5fbea0b5dc
Gitweb:     http://git.kernel.org/tip/f2c8852e6e990fcab0d9e68de9d86e5fbea0b5dc
Author:     Wang Nan <wangnan0@huawei.com>
AuthorDate: Wed, 28 Sep 2016 03:58:46 +0000
Committer:  Arnaldo Carvalho de Melo <acme@redhat.com>
CommitDate: Wed, 28 Sep 2016 10:38:20 -0300

perf data: Fix building in 32 bit platform with libbabeltrace

On ARM32 building it report following error when we build with
libbabeltrace:

  util/data-convert-bt.c: In function 'add_bpf_output_values':
  util/data-convert-bt.c:440:3: error: format '%lu' expects argument of type 'long unsigned int', but argument 5 has type 'unsigned int' [-Werror=format]
  cc1: all warnings being treated as errors

Fix it by changing %lu to %zu.

Signed-off-by: Wang Nan <wangnan0@huawei.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Zefan Li <lizefan@huawei.com>
Cc: pi3orama@163.com
Fixes: 6122d57e9f7c ("perf data: Support converting data from bpf_perf_event_output()")
Link: http://lkml.kernel.org/r/1475035126-146587-1-git-send-email-wangnan0@huawei.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/perf/util/data-convert-bt.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/perf/util/data-convert-bt.c b/tools/perf/util/data-convert-bt.c
index 4f979bb..7123f4d 100644
--- a/tools/perf/util/data-convert-bt.c
+++ b/tools/perf/util/data-convert-bt.c
@@ -437,7 +437,7 @@ add_bpf_output_values(struct bt_ctf_event_class *event_class,
 	int ret;
 
 	if (nr_elements * sizeof(u32) != raw_size)
-		pr_warning("Incorrect raw_size (%u) in bpf output event, skip %lu bytes\n",
+		pr_warning("Incorrect raw_size (%u) in bpf output event, skip %zu bytes\n",
 			   raw_size, nr_elements * sizeof(u32) - raw_size);
 
 	len_type = bt_ctf_event_class_get_field_by_name(event_class, "raw_len");

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web