diff options
| author | Yang Jihong <yangjihong1@huawei.com> | 2022-11-08 09:58:57 +0800 | 
|---|---|---|
| committer | Martin KaFai Lau <martin.lau@kernel.org> | 2022-11-09 12:49:45 -0800 | 
| commit | 5704bc7e8991164b14efb748b5afa0715c25fac3 (patch) | |
| tree | ef0fe59c8b79d0b2855c14b0d3a6b0696aa88deb | |
| parent | 0811664da064c6d7ca64c02f5579f758a007e52d (diff) | |
selftests/bpf: Fix test_progs compilation failure in 32-bit arch
test_progs fails to be compiled in the 32-bit arch, log is as follows:
  test_progs.c:1013:52: error: format '%ld' expects argument of type 'long int', but argument 3 has type 'size_t' {aka 'unsigned int'} [-Werror=format=]
   1013 |                 sprintf(buf, "MSG_TEST_LOG (cnt: %ld, last: %d)",
        |                                                  ~~^
        |                                                    |
        |                                                    long int
        |                                                  %d
   1014 |                         strlen(msg->test_log.log_buf),
        |                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
        |                         |
        |                         size_t {aka unsigned int}
Fix it.
Fixes: 91b2c0afd00c ("selftests/bpf: Add parallelism to test_progs")
Signed-off-by: Yang Jihong <yangjihong1@huawei.com>
Acked-by: Yonghong Song <yhs@fb.com>
Link: https://lore.kernel.org/r/20221108015857.132457-1-yangjihong1@huawei.com
Signed-off-by: Martin KaFai Lau <martin.lau@kernel.org>
| -rw-r--r-- | tools/testing/selftests/bpf/test_progs.c | 2 | 
1 files changed, 1 insertions, 1 deletions
| diff --git a/tools/testing/selftests/bpf/test_progs.c b/tools/testing/selftests/bpf/test_progs.c index 0e9a47f97890..3fef451d8831 100644 --- a/tools/testing/selftests/bpf/test_progs.c +++ b/tools/testing/selftests/bpf/test_progs.c @@ -1010,7 +1010,7 @@ static inline const char *str_msg(const struct msg *msg, char *buf)  			msg->subtest_done.have_log);  		break;  	case MSG_TEST_LOG: -		sprintf(buf, "MSG_TEST_LOG (cnt: %ld, last: %d)", +		sprintf(buf, "MSG_TEST_LOG (cnt: %zu, last: %d)",  			strlen(msg->test_log.log_buf),  			msg->test_log.is_last);  		break; | 
