| Message ID | 20240521022616.45240-1-mpe@ellerman.id.au (mailing list archive) |
|---|---|
| State | Handled Elsewhere |
| Headers | show |
| Series | selftests/overlayfs: Fix build error on ppc64 | expand |
| Context | Check | Description |
|---|---|---|
| snowpatch_ozlabs/github-powerpc_selftests | success | Successfully ran 8 jobs. |
| snowpatch_ozlabs/github-powerpc_clang | success | Successfully ran 6 jobs. |
| snowpatch_ozlabs/github-powerpc_ppctests | success | Successfully ran 8 jobs. |
| snowpatch_ozlabs/github-powerpc_sparse | success | Successfully ran 4 jobs. |
| snowpatch_ozlabs/github-powerpc_kernel_qemu | success | Successfully ran 23 jobs. |
On 5/20/24 20:26, Michael Ellerman wrote: > Fix build error on ppc64: > dev_in_maps.c: In function ‘get_file_dev_and_inode’: > dev_in_maps.c:60:59: error: format ‘%llu’ expects argument of type > ‘long long unsigned int *’, but argument 7 has type ‘__u64 *’ {aka ‘long > unsigned int *’} [-Werror=format=] > > By switching to unsigned long long for u64 for ppc64 builds. > > Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> > --- > tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c b/tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c > index 759f86e7d263..2862aae58b79 100644 > --- a/tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c > +++ b/tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c > @@ -1,5 +1,6 @@ > // SPDX-License-Identifier: GPL-2.0 > #define _GNU_SOURCE > +#define __SANE_USERSPACE_TYPES__ // Use ll64 > > #include <inttypes.h> > #include <unistd.h> Applied to linux-kselftest fixes for the next rc. Michael, If you want to take this through, let me know, I can drop this. thanks, -- Shuah
Shuah Khan <skhan@linuxfoundation.org> writes: > On 5/20/24 20:26, Michael Ellerman wrote: >> Fix build error on ppc64: >> dev_in_maps.c: In function ‘get_file_dev_and_inode’: >> dev_in_maps.c:60:59: error: format ‘%llu’ expects argument of type >> ‘long long unsigned int *’, but argument 7 has type ‘__u64 *’ {aka ‘long >> unsigned int *’} [-Werror=format=] >> >> By switching to unsigned long long for u64 for ppc64 builds. >> >> Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> >> --- >> tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c b/tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c >> index 759f86e7d263..2862aae58b79 100644 >> --- a/tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c >> +++ b/tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c >> @@ -1,5 +1,6 @@ >> // SPDX-License-Identifier: GPL-2.0 >> #define _GNU_SOURCE >> +#define __SANE_USERSPACE_TYPES__ // Use ll64 >> >> #include <inttypes.h> >> #include <unistd.h> > > Applied to linux-kselftest fixes for the next rc. Thanks. > Michael, If you want to take this through, let me know, I can drop this. I'm happy for you to take this one and the others. cheers
diff --git a/tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c b/tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c index 759f86e7d263..2862aae58b79 100644 --- a/tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c +++ b/tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c @@ -1,5 +1,6 @@ // SPDX-License-Identifier: GPL-2.0 #define _GNU_SOURCE +#define __SANE_USERSPACE_TYPES__ // Use ll64 #include <inttypes.h> #include <unistd.h>
Fix build error on ppc64: dev_in_maps.c: In function ‘get_file_dev_and_inode’: dev_in_maps.c:60:59: error: format ‘%llu’ expects argument of type ‘long long unsigned int *’, but argument 7 has type ‘__u64 *’ {aka ‘long unsigned int *’} [-Werror=format=] By switching to unsigned long long for u64 for ppc64 builds. Signed-off-by: Michael Ellerman <mpe@ellerman.id.au> --- tools/testing/selftests/filesystems/overlayfs/dev_in_maps.c | 1 + 1 file changed, 1 insertion(+)