linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* linux-next: build warnings after merge of the vfs-brauner tree
@ 2024-10-09  6:29 Stephen Rothwell
  0 siblings, 0 replies; 16+ messages in thread
From: Stephen Rothwell @ 2024-10-09  6:29 UTC (permalink / raw)
  To: Christian Brauner; +Cc: Linux Kernel Mailing List, Linux Next Mailing List

[-- Attachment #1: Type: text/plain, Size: 410 bytes --]

Hi all,

After merging the vfs-brauner tree, today's linux-next build (htmldocs)
produced these warnings:

include/linux/fs.h:1071: warning: Function parameter or struct member 'f_ref' not described in 'file'
include/linux/fs.h:1071: warning: Excess struct member 'f_count' description in 'file'

Introduced by commit

  3488e37e1059 ("fs: port files to file_ref")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 16+ messages in thread

* linux-next: build warnings after merge of the vfs-brauner tree
@ 2024-12-11  3:37 Stephen Rothwell
  2024-12-19  6:31 ` Stephen Rothwell
  0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2024-12-11  3:37 UTC (permalink / raw)
  To: Christian Brauner
  Cc: Jeff Layton, Linux Kernel Mailing List, Linux Next Mailing List

[-- Attachment #1: Type: text/plain, Size: 2905 bytes --]

Hi all,

After merging the vfs-brauner tree, today's linux-next build (powerpc
allyesconfig) produced these warnings:

samples/vfs/mountinfo.c: In function 'show_propagation':
samples/vfs/mountinfo.c:104:36: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
  104 |                 printf(" shared:%llu", sm->mnt_peer_group);
      |                                 ~~~^   ~~~~~~~~~~~~~~~~~~
      |                                    |     |
      |                                    |     __u64 {aka long unsigned int}
      |                                    long long unsigned int
      |                                 %lu
samples/vfs/mountinfo.c:106:36: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
  106 |                 printf(" master:%llu", sm->mnt_master);
      |                                 ~~~^   ~~~~~~~~~~~~~~
      |                                    |     |
      |                                    |     __u64 {aka long unsigned int}
      |                                    long long unsigned int
      |                                 %lu
samples/vfs/mountinfo.c:108:52: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
  108 |                         printf(" propagate_from:%llu", sm->propagate_from);
      |                                                 ~~~^   ~~~~~~~~~~~~~~~~~~
      |                                                    |     |
      |                                                    |     __u64 {aka long unsigned int}
      |                                                    long long unsigned int
      |                                                 %lu
samples/vfs/mountinfo.c: In function 'dump_mountinfo':
samples/vfs/mountinfo.c:144:42: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
  144 |                 printf("0x%lx 0x%lx 0x%llx ", mnt_ns_id, mnt_id, buf->mnt_parent_id);
      |                                       ~~~^                       ~~~~~~~~~~~~~~~~~~
      |                                          |                          |
      |                                          long long unsigned int     __u64 {aka long unsigned int}
      |                                       %lx

Introduced by commits

  0c8f746452aa ("samples: add a mountinfo program to demonstrate statmount()/listmount()")
  ee8c840a4a35 ("samples: fix missing nodiratime option and handle propagate_from correctly")

I am not sure why these warnings only turned up today.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 16+ messages in thread

* linux-next: build warnings after merge of the vfs-brauner tree
@ 2024-12-13  4:27 Stephen Rothwell
  2024-12-19  6:32 ` Stephen Rothwell
  0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2024-12-13  4:27 UTC (permalink / raw)
  To: Christian Brauner; +Cc: Linux Kernel Mailing List, Linux Next Mailing List

[-- Attachment #1: Type: text/plain, Size: 6767 bytes --]

Hi all,

After merging the vfs-brauner tree, today's linux-next build (powerpc
allyesconfig) produced these warnings:

samples/vfs/test-list-all-mounts.c: In function 'main':
samples/vfs/test-list-all-mounts.c:175:58: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
  175 |         printf("Listing %u mounts for mount namespace %llu\n",
      |                                                       ~~~^
      |                                                          |
      |                                                          long long unsigned int
      |                                                       %lu
  176 |                info.nr_mounts, info.mnt_ns_id);
      |                                ~~~~~~~~~~~~~~             
      |                                    |
      |                                    __u64 {aka long unsigned int}
samples/vfs/test-list-all-mounts.c:186:83: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
  186 |                         printf("Finished listing %u mounts for mount namespace %llu\n\n",
      |                                                                                ~~~^
      |                                                                                   |
      |                                                                                   long long unsigned int
      |                                                                                %lu
  187 |                                info.nr_mounts, info.mnt_ns_id);
      |                                                ~~~~~~~~~~~~~~                      
      |                                                    |
      |                                                    __u64 {aka long unsigned int}
samples/vfs/test-list-all-mounts.c:199:74: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
  199 |                         printf("Listing %u mounts for mount namespace %llu\n",
      |                                                                       ~~~^
      |                                                                          |
      |                                                                          long long unsigned int
      |                                                                       %lu
  200 |                                info.nr_mounts, info.mnt_ns_id);
      |                                                ~~~~~~~~~~~~~~             
      |                                                    |
      |                                                    __u64 {aka long unsigned int}
samples/vfs/test-list-all-mounts.c:218:64: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
  218 |                                 printf("Failed to statmount(%llu) in mount namespace(%llu)\n",
      |                                                             ~~~^
      |                                                                |
      |                                                                long long unsigned int
      |                                                             %lu
  219 |                                        last_mnt_id, info.mnt_ns_id);
      |                                        ~~~~~~~~~~~              
      |                                        |
      |                                        __u64 {aka long unsigned int}
samples/vfs/test-list-all-mounts.c:218:89: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
  218 |                                 printf("Failed to statmount(%llu) in mount namespace(%llu)\n",
      |                                                                                      ~~~^
      |                                                                                         |
      |                                                                                         long long unsigned int
      |                                                                                      %lu
  219 |                                        last_mnt_id, info.mnt_ns_id);
      |                                                     ~~~~~~~~~~~~~~                       
      |                                                         |
      |                                                         __u64 {aka long unsigned int}
samples/vfs/test-list-all-mounts.c:223:47: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
  223 |                         printf("mnt_id:\t\t%llu\nmnt_parent_id:\t%llu\nfs_type:\t%s\nmnt_root:\t%s\nmnt_point:\t%s\nmnt_opts:\t%s\n\n",
      |                                            ~~~^
      |                                               |
      |                                               long long unsigned int
      |                                            %lu
  224 |                                stmnt->mnt_id,
      |                                ~~~~~~~~~~~~~   
      |                                     |
      |                                     __u64 {aka long unsigned int}
samples/vfs/test-list-all-mounts.c:223:69: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
  223 |                         printf("mnt_id:\t\t%llu\nmnt_parent_id:\t%llu\nfs_type:\t%s\nmnt_root:\t%s\nmnt_point:\t%s\nmnt_opts:\t%s\n\n",
      |                                                                  ~~~^
      |                                                                     |
      |                                                                     long long unsigned int
      |                                                                  %lu
  224 |                                stmnt->mnt_id,
  225 |                                stmnt->mnt_parent_id,
      |                                ~~~~~~~~~~~~~~~~~~~~                  
      |                                     |
      |                                     __u64 {aka long unsigned int}

Introduced by commit

  7b1da09dc088 ("samples: add test-list-all-mounts")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: linux-next: build warnings after merge of the vfs-brauner tree
  2024-12-11  3:37 Stephen Rothwell
@ 2024-12-19  6:31 ` Stephen Rothwell
  2024-12-19 12:40   ` Jeff Layton
  0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2024-12-19  6:31 UTC (permalink / raw)
  To: Christian Brauner
  Cc: Jeff Layton, Linux Kernel Mailing List, Linux Next Mailing List

[-- Attachment #1: Type: text/plain, Size: 3104 bytes --]

Hi all,

On Wed, 11 Dec 2024 14:37:01 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> After merging the vfs-brauner tree, today's linux-next build (powerpc
> allyesconfig) produced these warnings:
> 
> samples/vfs/mountinfo.c: In function 'show_propagation':
> samples/vfs/mountinfo.c:104:36: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
>   104 |                 printf(" shared:%llu", sm->mnt_peer_group);
>       |                                 ~~~^   ~~~~~~~~~~~~~~~~~~
>       |                                    |     |
>       |                                    |     __u64 {aka long unsigned int}
>       |                                    long long unsigned int
>       |                                 %lu
> samples/vfs/mountinfo.c:106:36: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
>   106 |                 printf(" master:%llu", sm->mnt_master);
>       |                                 ~~~^   ~~~~~~~~~~~~~~
>       |                                    |     |
>       |                                    |     __u64 {aka long unsigned int}
>       |                                    long long unsigned int
>       |                                 %lu
> samples/vfs/mountinfo.c:108:52: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
>   108 |                         printf(" propagate_from:%llu", sm->propagate_from);
>       |                                                 ~~~^   ~~~~~~~~~~~~~~~~~~
>       |                                                    |     |
>       |                                                    |     __u64 {aka long unsigned int}
>       |                                                    long long unsigned int
>       |                                                 %lu
> samples/vfs/mountinfo.c: In function 'dump_mountinfo':
> samples/vfs/mountinfo.c:144:42: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
>   144 |                 printf("0x%lx 0x%lx 0x%llx ", mnt_ns_id, mnt_id, buf->mnt_parent_id);
>       |                                       ~~~^                       ~~~~~~~~~~~~~~~~~~
>       |                                          |                          |
>       |                                          long long unsigned int     __u64 {aka long unsigned int}
>       |                                       %lx
> 
> Introduced by commits
> 
>   0c8f746452aa ("samples: add a mountinfo program to demonstrate statmount()/listmount()")
>   ee8c840a4a35 ("samples: fix missing nodiratime option and handle propagate_from correctly")
> 
> I am not sure why these warnings only turned up today.

I am still seeing these warnings.
-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: linux-next: build warnings after merge of the vfs-brauner tree
  2024-12-13  4:27 Stephen Rothwell
@ 2024-12-19  6:32 ` Stephen Rothwell
  2025-01-14  5:48   ` Stephen Rothwell
  0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2024-12-19  6:32 UTC (permalink / raw)
  To: Christian Brauner; +Cc: Linux Kernel Mailing List, Linux Next Mailing List

[-- Attachment #1: Type: text/plain, Size: 7046 bytes --]

Hi all,

On Fri, 13 Dec 2024 15:27:24 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> After merging the vfs-brauner tree, today's linux-next build (powerpc
> allyesconfig) produced these warnings:
> 
> samples/vfs/test-list-all-mounts.c: In function 'main':
> samples/vfs/test-list-all-mounts.c:175:58: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
>   175 |         printf("Listing %u mounts for mount namespace %llu\n",
>       |                                                       ~~~^
>       |                                                          |
>       |                                                          long long unsigned int
>       |                                                       %lu
>   176 |                info.nr_mounts, info.mnt_ns_id);
>       |                                ~~~~~~~~~~~~~~             
>       |                                    |
>       |                                    __u64 {aka long unsigned int}
> samples/vfs/test-list-all-mounts.c:186:83: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
>   186 |                         printf("Finished listing %u mounts for mount namespace %llu\n\n",
>       |                                                                                ~~~^
>       |                                                                                   |
>       |                                                                                   long long unsigned int
>       |                                                                                %lu
>   187 |                                info.nr_mounts, info.mnt_ns_id);
>       |                                                ~~~~~~~~~~~~~~                      
>       |                                                    |
>       |                                                    __u64 {aka long unsigned int}
> samples/vfs/test-list-all-mounts.c:199:74: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
>   199 |                         printf("Listing %u mounts for mount namespace %llu\n",
>       |                                                                       ~~~^
>       |                                                                          |
>       |                                                                          long long unsigned int
>       |                                                                       %lu
>   200 |                                info.nr_mounts, info.mnt_ns_id);
>       |                                                ~~~~~~~~~~~~~~             
>       |                                                    |
>       |                                                    __u64 {aka long unsigned int}
> samples/vfs/test-list-all-mounts.c:218:64: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
>   218 |                                 printf("Failed to statmount(%llu) in mount namespace(%llu)\n",
>       |                                                             ~~~^
>       |                                                                |
>       |                                                                long long unsigned int
>       |                                                             %lu
>   219 |                                        last_mnt_id, info.mnt_ns_id);
>       |                                        ~~~~~~~~~~~              
>       |                                        |
>       |                                        __u64 {aka long unsigned int}
> samples/vfs/test-list-all-mounts.c:218:89: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
>   218 |                                 printf("Failed to statmount(%llu) in mount namespace(%llu)\n",
>       |                                                                                      ~~~^
>       |                                                                                         |
>       |                                                                                         long long unsigned int
>       |                                                                                      %lu
>   219 |                                        last_mnt_id, info.mnt_ns_id);
>       |                                                     ~~~~~~~~~~~~~~                       
>       |                                                         |
>       |                                                         __u64 {aka long unsigned int}
> samples/vfs/test-list-all-mounts.c:223:47: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
>   223 |                         printf("mnt_id:\t\t%llu\nmnt_parent_id:\t%llu\nfs_type:\t%s\nmnt_root:\t%s\nmnt_point:\t%s\nmnt_opts:\t%s\n\n",
>       |                                            ~~~^
>       |                                               |
>       |                                               long long unsigned int
>       |                                            %lu
>   224 |                                stmnt->mnt_id,
>       |                                ~~~~~~~~~~~~~   
>       |                                     |
>       |                                     __u64 {aka long unsigned int}
> samples/vfs/test-list-all-mounts.c:223:69: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
>   223 |                         printf("mnt_id:\t\t%llu\nmnt_parent_id:\t%llu\nfs_type:\t%s\nmnt_root:\t%s\nmnt_point:\t%s\nmnt_opts:\t%s\n\n",
>       |                                                                  ~~~^
>       |                                                                     |
>       |                                                                     long long unsigned int
>       |                                                                  %lu
>   224 |                                stmnt->mnt_id,
>   225 |                                stmnt->mnt_parent_id,
>       |                                ~~~~~~~~~~~~~~~~~~~~                  
>       |                                     |
>       |                                     __u64 {aka long unsigned int}
> 
> Introduced by commit
> 
>   7b1da09dc088 ("samples: add test-list-all-mounts")

I am still seeing these warnings.
-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: linux-next: build warnings after merge of the vfs-brauner tree
  2024-12-19  6:31 ` Stephen Rothwell
@ 2024-12-19 12:40   ` Jeff Layton
  2024-12-19 20:36     ` Stephen Rothwell
  0 siblings, 1 reply; 16+ messages in thread
From: Jeff Layton @ 2024-12-19 12:40 UTC (permalink / raw)
  To: Stephen Rothwell, Christian Brauner
  Cc: Linux Kernel Mailing List, Linux Next Mailing List

On Thu, 2024-12-19 at 17:31 +1100, Stephen Rothwell wrote:
> Hi all,
> 
> On Wed, 11 Dec 2024 14:37:01 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> > 
> > After merging the vfs-brauner tree, today's linux-next build (powerpc
> > allyesconfig) produced these warnings:
> > 
> > samples/vfs/mountinfo.c: In function 'show_propagation':
> > samples/vfs/mountinfo.c:104:36: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> >   104 |                 printf(" shared:%llu", sm->mnt_peer_group);
> >       |                                 ~~~^   ~~~~~~~~~~~~~~~~~~
> >       |                                    |     |
> >       |                                    |     __u64 {aka long unsigned int}
> >       |                                    long long unsigned int
> >       |                                 %lu
> > samples/vfs/mountinfo.c:106:36: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> >   106 |                 printf(" master:%llu", sm->mnt_master);
> >       |                                 ~~~^   ~~~~~~~~~~~~~~
> >       |                                    |     |
> >       |                                    |     __u64 {aka long unsigned int}
> >       |                                    long long unsigned int
> >       |                                 %lu
> > samples/vfs/mountinfo.c:108:52: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> >   108 |                         printf(" propagate_from:%llu", sm->propagate_from);
> >       |                                                 ~~~^   ~~~~~~~~~~~~~~~~~~
> >       |                                                    |     |
> >       |                                                    |     __u64 {aka long unsigned int}
> >       |                                                    long long unsigned int
> >       |                                                 %lu
> > samples/vfs/mountinfo.c: In function 'dump_mountinfo':
> > samples/vfs/mountinfo.c:144:42: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 4 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> >   144 |                 printf("0x%lx 0x%lx 0x%llx ", mnt_ns_id, mnt_id, buf->mnt_parent_id);
> >       |                                       ~~~^                       ~~~~~~~~~~~~~~~~~~
> >       |                                          |                          |
> >       |                                          long long unsigned int     __u64 {aka long unsigned int}
> >       |                                       %lx
> > 
> > Introduced by commits
> > 
> >   0c8f746452aa ("samples: add a mountinfo program to demonstrate statmount()/listmount()")
> >   ee8c840a4a35 ("samples: fix missing nodiratime option and handle propagate_from correctly")
> > 
> > I am not sure why these warnings only turned up today.
> 
> I am still seeing these warnings.

I'm not sure how to fix this the right way.

The problem seems to be a discrepancy in the __u64 type between arches.
Changing these in a way that would make PPC happy causes these warnings
on x86_64:

   gcc -Wp,-MMD,./.mountinfo.d -Wall -Wmissing-prototypes -Wstrict-prototypes -O2 -fomit-frame-pointer -std=gnu11  -m64 -I usr/include  -m64   -o mountinfo mountinfo.c  
mountinfo.c: In function ‘show_propagation’:
mountinfo.c:104:35: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 2 has type ‘__u64’ {aka ‘long long unsigned int’} [-Wformat=]
  104 |                 printf(" shared:%lu", sm->mnt_peer_group);
      |                                 ~~^   ~~~~~~~~~~~~~~~~~~
      |                                   |     |
      |                                   |     __u64 {aka long long unsigned int}
      |                                   long unsigned int
      |                                 %llu
mountinfo.c:106:35: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 2 has type ‘__u64’ {aka ‘long long unsigned int’} [-Wformat=]
  106 |                 printf(" master:%lu", sm->mnt_master);
      |                                 ~~^   ~~~~~~~~~~~~~~
      |                                   |     |
      |                                   |     __u64 {aka long long unsigned int}
      |                                   long unsigned int
      |                                 %llu
mountinfo.c:108:51: warning: format ‘%lu’ expects argument of type ‘long unsigned int’, but argument 2 has type ‘__u64’ {aka ‘long long unsigned int’} [-Wformat=]
  108 |                         printf(" propagate_from:%lu", sm->propagate_from);
      |                                                 ~~^   ~~~~~~~~~~~~~~~~~~
      |                                                   |     |
      |                                                   |     __u64 {aka long long unsigned int}
      |                                                   long unsigned int
      |                                                 %llu
mountinfo.c: In function ‘dump_mountinfo’:
mountinfo.c:144:41: warning: format ‘%lx’ expects argument of type ‘long unsigned int’, but argument 4 has type ‘__u64’ {aka ‘long long unsigned int’} [-Wformat=]
  144 |                 printf("0x%lx 0x%lx 0x%lx ", mnt_ns_id, mnt_id, buf->mnt_parent_id);
      |                                       ~~^                       ~~~~~~~~~~~~~~~~~~
      |                                         |                          |
      |                                         long unsigned int          __u64 {aka long long unsigned int}
      |                                       %llx


What format specifier should we be using that would make both arches happy?

Thanks,
-- 
Jeff Layton <jlayton@kernel.org>

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: linux-next: build warnings after merge of the vfs-brauner tree
  2024-12-19 12:40   ` Jeff Layton
@ 2024-12-19 20:36     ` Stephen Rothwell
  2025-01-06 13:50       ` Geert Uytterhoeven
  0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2024-12-19 20:36 UTC (permalink / raw)
  To: Jeff Layton
  Cc: Christian Brauner, Linux Kernel Mailing List,
	Linux Next Mailing List

[-- Attachment #1: Type: text/plain, Size: 230 bytes --]

Hi Jeff,

On Thu, 19 Dec 2024 07:40:05 -0500 Jeff Layton <jlayton@kernel.org> wrote:
>
> I'm not sure how to fix this the right way.

Some other samples define __SANE_USERSPACE_TYPES__ ...

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: linux-next: build warnings after merge of the vfs-brauner tree
  2024-12-19 20:36     ` Stephen Rothwell
@ 2025-01-06 13:50       ` Geert Uytterhoeven
  0 siblings, 0 replies; 16+ messages in thread
From: Geert Uytterhoeven @ 2025-01-06 13:50 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Jeff Layton, Christian Brauner, Linux Kernel Mailing List,
	Linux Next Mailing List

Hi Stephen,

On Thu, Dec 19, 2024 at 9:39 PM Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> On Thu, 19 Dec 2024 07:40:05 -0500 Jeff Layton <jlayton@kernel.org> wrote:
> >
> > I'm not sure how to fix this the right way.
>
> Some other samples define __SANE_USERSPACE_TYPES__ ...

Which is now used by samples/vfs/mountinfo.c...
However, it's still broken on 32-bit. I have sent a fix.

https://lore.kernel.org/20250106134802.1019911-1-geert+renesas@glider.be

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: linux-next: build warnings after merge of the vfs-brauner tree
  2024-12-19  6:32 ` Stephen Rothwell
@ 2025-01-14  5:48   ` Stephen Rothwell
  2025-01-20  8:25     ` Stephen Rothwell
  0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2025-01-14  5:48 UTC (permalink / raw)
  To: Christian Brauner; +Cc: Linux Kernel Mailing List, Linux Next Mailing List

[-- Attachment #1: Type: text/plain, Size: 7319 bytes --]

Hi all,

On Thu, 19 Dec 2024 17:32:35 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> On Fri, 13 Dec 2024 15:27:24 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> >
> > After merging the vfs-brauner tree, today's linux-next build (powerpc
> > allyesconfig) produced these warnings:
> > 
> > samples/vfs/test-list-all-mounts.c: In function 'main':
> > samples/vfs/test-list-all-mounts.c:175:58: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> >   175 |         printf("Listing %u mounts for mount namespace %llu\n",
> >       |                                                       ~~~^
> >       |                                                          |
> >       |                                                          long long unsigned int
> >       |                                                       %lu
> >   176 |                info.nr_mounts, info.mnt_ns_id);
> >       |                                ~~~~~~~~~~~~~~             
> >       |                                    |
> >       |                                    __u64 {aka long unsigned int}
> > samples/vfs/test-list-all-mounts.c:186:83: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> >   186 |                         printf("Finished listing %u mounts for mount namespace %llu\n\n",
> >       |                                                                                ~~~^
> >       |                                                                                   |
> >       |                                                                                   long long unsigned int
> >       |                                                                                %lu
> >   187 |                                info.nr_mounts, info.mnt_ns_id);
> >       |                                                ~~~~~~~~~~~~~~                      
> >       |                                                    |
> >       |                                                    __u64 {aka long unsigned int}
> > samples/vfs/test-list-all-mounts.c:199:74: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> >   199 |                         printf("Listing %u mounts for mount namespace %llu\n",
> >       |                                                                       ~~~^
> >       |                                                                          |
> >       |                                                                          long long unsigned int
> >       |                                                                       %lu
> >   200 |                                info.nr_mounts, info.mnt_ns_id);
> >       |                                                ~~~~~~~~~~~~~~             
> >       |                                                    |
> >       |                                                    __u64 {aka long unsigned int}
> > samples/vfs/test-list-all-mounts.c:218:64: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> >   218 |                                 printf("Failed to statmount(%llu) in mount namespace(%llu)\n",
> >       |                                                             ~~~^
> >       |                                                                |
> >       |                                                                long long unsigned int
> >       |                                                             %lu
> >   219 |                                        last_mnt_id, info.mnt_ns_id);
> >       |                                        ~~~~~~~~~~~              
> >       |                                        |
> >       |                                        __u64 {aka long unsigned int}
> > samples/vfs/test-list-all-mounts.c:218:89: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> >   218 |                                 printf("Failed to statmount(%llu) in mount namespace(%llu)\n",
> >       |                                                                                      ~~~^
> >       |                                                                                         |
> >       |                                                                                         long long unsigned int
> >       |                                                                                      %lu
> >   219 |                                        last_mnt_id, info.mnt_ns_id);
> >       |                                                     ~~~~~~~~~~~~~~                       
> >       |                                                         |
> >       |                                                         __u64 {aka long unsigned int}
> > samples/vfs/test-list-all-mounts.c:223:47: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> >   223 |                         printf("mnt_id:\t\t%llu\nmnt_parent_id:\t%llu\nfs_type:\t%s\nmnt_root:\t%s\nmnt_point:\t%s\nmnt_opts:\t%s\n\n",
> >       |                                            ~~~^
> >       |                                               |
> >       |                                               long long unsigned int
> >       |                                            %lu
> >   224 |                                stmnt->mnt_id,
> >       |                                ~~~~~~~~~~~~~   
> >       |                                     |
> >       |                                     __u64 {aka long unsigned int}
> > samples/vfs/test-list-all-mounts.c:223:69: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> >   223 |                         printf("mnt_id:\t\t%llu\nmnt_parent_id:\t%llu\nfs_type:\t%s\nmnt_root:\t%s\nmnt_point:\t%s\nmnt_opts:\t%s\n\n",
> >       |                                                                  ~~~^
> >       |                                                                     |
> >       |                                                                     long long unsigned int
> >       |                                                                  %lu
> >   224 |                                stmnt->mnt_id,
> >   225 |                                stmnt->mnt_parent_id,
> >       |                                ~~~~~~~~~~~~~~~~~~~~                  
> >       |                                     |
> >       |                                     __u64 {aka long unsigned int}
> > 
> > Introduced by commit
> > 
> >   7b1da09dc088 ("samples: add test-list-all-mounts")  
> 
> I am still seeing these warnings.

Still there ...
-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: linux-next: build warnings after merge of the vfs-brauner tree
  2025-01-14  5:48   ` Stephen Rothwell
@ 2025-01-20  8:25     ` Stephen Rothwell
  2025-01-20 12:15       ` Christian Brauner
  0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2025-01-20  8:25 UTC (permalink / raw)
  To: Christian Brauner; +Cc: Linux Kernel Mailing List, Linux Next Mailing List

[-- Attachment #1: Type: text/plain, Size: 7614 bytes --]

Hi all,

On Tue, 14 Jan 2025 16:48:12 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> On Thu, 19 Dec 2024 17:32:35 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> >
> > On Fri, 13 Dec 2024 15:27:24 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:  
> > >
> > > After merging the vfs-brauner tree, today's linux-next build (powerpc
> > > allyesconfig) produced these warnings:
> > > 
> > > samples/vfs/test-list-all-mounts.c: In function 'main':
> > > samples/vfs/test-list-all-mounts.c:175:58: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> > >   175 |         printf("Listing %u mounts for mount namespace %llu\n",
> > >       |                                                       ~~~^
> > >       |                                                          |
> > >       |                                                          long long unsigned int
> > >       |                                                       %lu
> > >   176 |                info.nr_mounts, info.mnt_ns_id);
> > >       |                                ~~~~~~~~~~~~~~             
> > >       |                                    |
> > >       |                                    __u64 {aka long unsigned int}
> > > samples/vfs/test-list-all-mounts.c:186:83: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> > >   186 |                         printf("Finished listing %u mounts for mount namespace %llu\n\n",
> > >       |                                                                                ~~~^
> > >       |                                                                                   |
> > >       |                                                                                   long long unsigned int
> > >       |                                                                                %lu
> > >   187 |                                info.nr_mounts, info.mnt_ns_id);
> > >       |                                                ~~~~~~~~~~~~~~                      
> > >       |                                                    |
> > >       |                                                    __u64 {aka long unsigned int}
> > > samples/vfs/test-list-all-mounts.c:199:74: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> > >   199 |                         printf("Listing %u mounts for mount namespace %llu\n",
> > >       |                                                                       ~~~^
> > >       |                                                                          |
> > >       |                                                                          long long unsigned int
> > >       |                                                                       %lu
> > >   200 |                                info.nr_mounts, info.mnt_ns_id);
> > >       |                                                ~~~~~~~~~~~~~~             
> > >       |                                                    |
> > >       |                                                    __u64 {aka long unsigned int}
> > > samples/vfs/test-list-all-mounts.c:218:64: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> > >   218 |                                 printf("Failed to statmount(%llu) in mount namespace(%llu)\n",
> > >       |                                                             ~~~^
> > >       |                                                                |
> > >       |                                                                long long unsigned int
> > >       |                                                             %lu
> > >   219 |                                        last_mnt_id, info.mnt_ns_id);
> > >       |                                        ~~~~~~~~~~~              
> > >       |                                        |
> > >       |                                        __u64 {aka long unsigned int}
> > > samples/vfs/test-list-all-mounts.c:218:89: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> > >   218 |                                 printf("Failed to statmount(%llu) in mount namespace(%llu)\n",
> > >       |                                                                                      ~~~^
> > >       |                                                                                         |
> > >       |                                                                                         long long unsigned int
> > >       |                                                                                      %lu
> > >   219 |                                        last_mnt_id, info.mnt_ns_id);
> > >       |                                                     ~~~~~~~~~~~~~~                       
> > >       |                                                         |
> > >       |                                                         __u64 {aka long unsigned int}
> > > samples/vfs/test-list-all-mounts.c:223:47: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> > >   223 |                         printf("mnt_id:\t\t%llu\nmnt_parent_id:\t%llu\nfs_type:\t%s\nmnt_root:\t%s\nmnt_point:\t%s\nmnt_opts:\t%s\n\n",
> > >       |                                            ~~~^
> > >       |                                               |
> > >       |                                               long long unsigned int
> > >       |                                            %lu
> > >   224 |                                stmnt->mnt_id,
> > >       |                                ~~~~~~~~~~~~~   
> > >       |                                     |
> > >       |                                     __u64 {aka long unsigned int}
> > > samples/vfs/test-list-all-mounts.c:223:69: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> > >   223 |                         printf("mnt_id:\t\t%llu\nmnt_parent_id:\t%llu\nfs_type:\t%s\nmnt_root:\t%s\nmnt_point:\t%s\nmnt_opts:\t%s\n\n",
> > >       |                                                                  ~~~^
> > >       |                                                                     |
> > >       |                                                                     long long unsigned int
> > >       |                                                                  %lu
> > >   224 |                                stmnt->mnt_id,
> > >   225 |                                stmnt->mnt_parent_id,
> > >       |                                ~~~~~~~~~~~~~~~~~~~~                  
> > >       |                                     |
> > >       |                                     __u64 {aka long unsigned int}
> > > 
> > > Introduced by commit
> > > 
> > >   7b1da09dc088 ("samples: add test-list-all-mounts")    
> > 
> > I am still seeing these warnings.  
> 
> Still there ...

Ping?

Please fix this.
-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: linux-next: build warnings after merge of the vfs-brauner tree
  2025-01-20  8:25     ` Stephen Rothwell
@ 2025-01-20 12:15       ` Christian Brauner
  2025-01-20 20:55         ` Stephen Rothwell
  0 siblings, 1 reply; 16+ messages in thread
From: Christian Brauner @ 2025-01-20 12:15 UTC (permalink / raw)
  To: Stephen Rothwell; +Cc: Linux Kernel Mailing List, Linux Next Mailing List

On Mon, Jan 20, 2025 at 07:25:04PM +1100, Stephen Rothwell wrote:
> Hi all,
> 
> On Tue, 14 Jan 2025 16:48:12 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> >
> > On Thu, 19 Dec 2024 17:32:35 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> > >
> > > On Fri, 13 Dec 2024 15:27:24 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:  
> > > >
> > > > After merging the vfs-brauner tree, today's linux-next build (powerpc
> > > > allyesconfig) produced these warnings:
> > > > 
> > > > samples/vfs/test-list-all-mounts.c: In function 'main':
> > > > samples/vfs/test-list-all-mounts.c:175:58: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> > > >   175 |         printf("Listing %u mounts for mount namespace %llu\n",
> > > >       |                                                       ~~~^
> > > >       |                                                          |
> > > >       |                                                          long long unsigned int
> > > >       |                                                       %lu
> > > >   176 |                info.nr_mounts, info.mnt_ns_id);
> > > >       |                                ~~~~~~~~~~~~~~             
> > > >       |                                    |
> > > >       |                                    __u64 {aka long unsigned int}
> > > > samples/vfs/test-list-all-mounts.c:186:83: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> > > >   186 |                         printf("Finished listing %u mounts for mount namespace %llu\n\n",
> > > >       |                                                                                ~~~^
> > > >       |                                                                                   |
> > > >       |                                                                                   long long unsigned int
> > > >       |                                                                                %lu
> > > >   187 |                                info.nr_mounts, info.mnt_ns_id);
> > > >       |                                                ~~~~~~~~~~~~~~                      
> > > >       |                                                    |
> > > >       |                                                    __u64 {aka long unsigned int}
> > > > samples/vfs/test-list-all-mounts.c:199:74: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> > > >   199 |                         printf("Listing %u mounts for mount namespace %llu\n",
> > > >       |                                                                       ~~~^
> > > >       |                                                                          |
> > > >       |                                                                          long long unsigned int
> > > >       |                                                                       %lu
> > > >   200 |                                info.nr_mounts, info.mnt_ns_id);
> > > >       |                                                ~~~~~~~~~~~~~~             
> > > >       |                                                    |
> > > >       |                                                    __u64 {aka long unsigned int}
> > > > samples/vfs/test-list-all-mounts.c:218:64: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> > > >   218 |                                 printf("Failed to statmount(%llu) in mount namespace(%llu)\n",
> > > >       |                                                             ~~~^
> > > >       |                                                                |
> > > >       |                                                                long long unsigned int
> > > >       |                                                             %lu
> > > >   219 |                                        last_mnt_id, info.mnt_ns_id);
> > > >       |                                        ~~~~~~~~~~~              
> > > >       |                                        |
> > > >       |                                        __u64 {aka long unsigned int}
> > > > samples/vfs/test-list-all-mounts.c:218:89: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> > > >   218 |                                 printf("Failed to statmount(%llu) in mount namespace(%llu)\n",
> > > >       |                                                                                      ~~~^
> > > >       |                                                                                         |
> > > >       |                                                                                         long long unsigned int
> > > >       |                                                                                      %lu
> > > >   219 |                                        last_mnt_id, info.mnt_ns_id);
> > > >       |                                                     ~~~~~~~~~~~~~~                       
> > > >       |                                                         |
> > > >       |                                                         __u64 {aka long unsigned int}
> > > > samples/vfs/test-list-all-mounts.c:223:47: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 2 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> > > >   223 |                         printf("mnt_id:\t\t%llu\nmnt_parent_id:\t%llu\nfs_type:\t%s\nmnt_root:\t%s\nmnt_point:\t%s\nmnt_opts:\t%s\n\n",
> > > >       |                                            ~~~^
> > > >       |                                               |
> > > >       |                                               long long unsigned int
> > > >       |                                            %lu
> > > >   224 |                                stmnt->mnt_id,
> > > >       |                                ~~~~~~~~~~~~~   
> > > >       |                                     |
> > > >       |                                     __u64 {aka long unsigned int}
> > > > samples/vfs/test-list-all-mounts.c:223:69: warning: format '%llu' expects argument of type 'long long unsigned int', but argument 3 has type '__u64' {aka 'long unsigned int'} [-Wformat=]
> > > >   223 |                         printf("mnt_id:\t\t%llu\nmnt_parent_id:\t%llu\nfs_type:\t%s\nmnt_root:\t%s\nmnt_point:\t%s\nmnt_opts:\t%s\n\n",
> > > >       |                                                                  ~~~^
> > > >       |                                                                     |
> > > >       |                                                                     long long unsigned int
> > > >       |                                                                  %lu
> > > >   224 |                                stmnt->mnt_id,
> > > >   225 |                                stmnt->mnt_parent_id,
> > > >       |                                ~~~~~~~~~~~~~~~~~~~~                  
> > > >       |                                     |
> > > >       |                                     __u64 {aka long unsigned int}
> > > > 
> > > > Introduced by commit
> > > > 
> > > >   7b1da09dc088 ("samples: add test-list-all-mounts")    
> > > 
> > > I am still seeing these warnings.  
> > 
> > Still there ...
> 
> Ping?

Thanks for the reminder.
This is now fixed.

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: linux-next: build warnings after merge of the vfs-brauner tree
  2025-01-20 12:15       ` Christian Brauner
@ 2025-01-20 20:55         ` Stephen Rothwell
  0 siblings, 0 replies; 16+ messages in thread
From: Stephen Rothwell @ 2025-01-20 20:55 UTC (permalink / raw)
  To: Christian Brauner; +Cc: Linux Kernel Mailing List, Linux Next Mailing List

[-- Attachment #1: Type: text/plain, Size: 197 bytes --]

Hi Christian,

On Mon, 20 Jan 2025 13:15:07 +0100 Christian Brauner <brauner@kernel.org> wrote:
>
> Thanks for the reminder.
> This is now fixed.

Thanks.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 16+ messages in thread

* linux-next: build warnings after merge of the vfs-brauner tree
@ 2025-03-05 10:07 Stephen Rothwell
  2025-03-05 10:55 ` David Disseldorp
  0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2025-03-05 10:07 UTC (permalink / raw)
  To: Christian Brauner
  Cc: David Disseldorp, Linux Kernel Mailing List,
	Linux Next Mailing List

[-- Attachment #1: Type: text/plain, Size: 1219 bytes --]

Hi all,

After merging the vfs-brauner tree, today's linux-next build (powerpc
allyesconfig) produced these warnings:

WARNING: modpost: vmlinux: section mismatch in reference: initramfs_test_cases+0x0 (section: .data) -> initramfs_test_extract (section: .init.text)
WARNING: modpost: vmlinux: section mismatch in reference: initramfs_test_cases+0x30 (section: .data) -> initramfs_test_fname_overrun (section: .init.text)
WARNING: modpost: vmlinux: section mismatch in reference: initramfs_test_cases+0x60 (section: .data) -> initramfs_test_data (section: .init.text)
WARNING: modpost: vmlinux: section mismatch in reference: initramfs_test_cases+0x90 (section: .data) -> initramfs_test_csum (section: .init.text)
WARNING: modpost: vmlinux: section mismatch in reference: initramfs_test_cases+0xc0 (section: .data) -> initramfs_test_hardlink (section: .init.text)
WARNING: modpost: vmlinux: section mismatch in reference: initramfs_test_cases+0xf0 (section: .data) -> initramfs_test_many (section: .init.text)

Introduced by commit

  b6736cfccb58 ("initramfs_test: kunit tests for initramfs unpacking")

Can __refdata (or __refconst) be used on initramfs_test_cases?

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: linux-next: build warnings after merge of the vfs-brauner tree
  2025-03-05 10:07 Stephen Rothwell
@ 2025-03-05 10:55 ` David Disseldorp
  0 siblings, 0 replies; 16+ messages in thread
From: David Disseldorp @ 2025-03-05 10:55 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Christian Brauner, Linux Kernel Mailing List,
	Linux Next Mailing List

Hi Stephen,

On Wed, 5 Mar 2025 21:07:02 +1100, Stephen Rothwell wrote:

> Hi all,
> 
> After merging the vfs-brauner tree, today's linux-next build (powerpc
> allyesconfig) produced these warnings:
> 
> WARNING: modpost: vmlinux: section mismatch in reference: initramfs_test_cases+0x0 (section: .data) -> initramfs_test_extract (section: .init.text)
> WARNING: modpost: vmlinux: section mismatch in reference: initramfs_test_cases+0x30 (section: .data) -> initramfs_test_fname_overrun (section: .init.text)
> WARNING: modpost: vmlinux: section mismatch in reference: initramfs_test_cases+0x60 (section: .data) -> initramfs_test_data (section: .init.text)
> WARNING: modpost: vmlinux: section mismatch in reference: initramfs_test_cases+0x90 (section: .data) -> initramfs_test_csum (section: .init.text)
> WARNING: modpost: vmlinux: section mismatch in reference: initramfs_test_cases+0xc0 (section: .data) -> initramfs_test_hardlink (section: .init.text)
> WARNING: modpost: vmlinux: section mismatch in reference: initramfs_test_cases+0xf0 (section: .data) -> initramfs_test_many (section: .init.text)
> 
> Introduced by commit
> 
>   b6736cfccb58 ("initramfs_test: kunit tests for initramfs unpacking")

The new warnings are being discussed in a thread at:
https://lore.kernel.org/linux-kselftest/20250305114701.28c0ee0b.ddiss@suse.de/T/#u

Thanks, David

^ permalink raw reply	[flat|nested] 16+ messages in thread

* linux-next: build warnings after merge of the vfs-brauner tree
@ 2025-07-17  7:36 Stephen Rothwell
  2025-07-22 10:24 ` Jann Horn
  0 siblings, 1 reply; 16+ messages in thread
From: Stephen Rothwell @ 2025-07-17  7:36 UTC (permalink / raw)
  To: Christian Brauner
  Cc: Jann Horn, Linux Kernel Mailing List, Linux Next Mailing List

[-- Attachment #1: Type: text/plain, Size: 469 bytes --]

Hi all,

After merging the vfs-brauner tree, today's linux-next build (htmldocs)
produced these warnings:

fs/eventpoll.c:2197: warning: Function parameter or struct member 'ep' not described in 'ep_get_upwards_depth_proc'
fs/eventpoll.c:2197: warning: Function parameter or struct member 'depth' not described in 'ep_get_upwards_depth_proc'

Introduced by commit

  f2e467a48287 ("eventpoll: Fix semi-unbounded recursion")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

^ permalink raw reply	[flat|nested] 16+ messages in thread

* Re: linux-next: build warnings after merge of the vfs-brauner tree
  2025-07-17  7:36 linux-next: build warnings after merge of the vfs-brauner tree Stephen Rothwell
@ 2025-07-22 10:24 ` Jann Horn
  0 siblings, 0 replies; 16+ messages in thread
From: Jann Horn @ 2025-07-22 10:24 UTC (permalink / raw)
  To: Stephen Rothwell
  Cc: Christian Brauner, Linux Kernel Mailing List,
	Linux Next Mailing List

On Thu, Jul 17, 2025 at 9:36 AM Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> After merging the vfs-brauner tree, today's linux-next build (htmldocs)
> produced these warnings:
>
> fs/eventpoll.c:2197: warning: Function parameter or struct member 'ep' not described in 'ep_get_upwards_depth_proc'
> fs/eventpoll.c:2197: warning: Function parameter or struct member 'depth' not described in 'ep_get_upwards_depth_proc'
>
> Introduced by commit
>
>   f2e467a48287 ("eventpoll: Fix semi-unbounded recursion")

Thanks for the heads-up, I've sent
https://lore.kernel.org/all/20250721-epoll-sphinx-fix-v1-1-b695c92bf009@google.com/
to address this.

^ permalink raw reply	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2025-07-22 10:24 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-17  7:36 linux-next: build warnings after merge of the vfs-brauner tree Stephen Rothwell
2025-07-22 10:24 ` Jann Horn
  -- strict thread matches above, loose matches on Subject: below --
2025-03-05 10:07 Stephen Rothwell
2025-03-05 10:55 ` David Disseldorp
2024-12-13  4:27 Stephen Rothwell
2024-12-19  6:32 ` Stephen Rothwell
2025-01-14  5:48   ` Stephen Rothwell
2025-01-20  8:25     ` Stephen Rothwell
2025-01-20 12:15       ` Christian Brauner
2025-01-20 20:55         ` Stephen Rothwell
2024-12-11  3:37 Stephen Rothwell
2024-12-19  6:31 ` Stephen Rothwell
2024-12-19 12:40   ` Jeff Layton
2024-12-19 20:36     ` Stephen Rothwell
2025-01-06 13:50       ` Geert Uytterhoeven
2024-10-09  6:29 Stephen Rothwell

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).