linux-kernel - Re: [tip:locking/core 19/28] include/linux/fs.h:1422:29: error: array type has incomplete element type 'struct percpu_rw

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAMuHMdX2vK4DeGNGqWQ09Z1UyP8+QAREDjH82+u9xeWtY0ayHw@mail.gmail.com>
Date:   Wed, 25 Mar 2020 13:27:27 +0100
From:   Geert Uytterhoeven <geert@...ux-m68k.org>
To:     Thomas Gleixner <tglx@...utronix.de>
Cc:     kbuild test robot <lkp@...el.com>,
        "Peter Zijlstra (Intel)" <peterz@...radead.org>,
        kbuild-all@...ts.01.org,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        "the arch/x86 maintainers" <x86@...nel.org>,
        Greg Ungerer <gerg@...ux-m68k.org>,
        linux-m68k <linux-m68k@...ts.linux-m68k.org>
Subject: Re: [tip:locking/core 19/28] include/linux/fs.h:1422:29: error: array
 type has incomplete element type 'struct percpu_rw_semaphore'

Hi Thomas,

On Sat, Mar 21, 2020 at 8:29 PM Thomas Gleixner <tglx@...utronix.de> wrote:
> kbuild test robot <lkp@...el.com> writes:
> > All errors (new ones prefixed by >>):
> >
> >    In file included from include/linux/huge_mm.h:8,
> >                     from include/linux/mm.h:567,
> >                     from arch/m68k/include/asm/uaccess_no.h:8,
> >                     from arch/m68k/include/asm/uaccess.h:3,
> >                     from include/linux/uaccess.h:11,
> >                     from include/linux/sched/task.h:11,
> >                     from include/linux/sched/signal.h:9,
> >                     from include/linux/rcuwait.h:6,
> >                     from include/linux/percpu-rwsem.h:7,
> >                     from kernel/locking/percpu-rwsem.c:6:
> >>> include/linux/fs.h:1422:29: error: array type has incomplete element type 'struct percpu_rw_semaphore'
> >     1422 |  struct percpu_rw_semaphore rw_sem[SB_FREEZE_LEVELS];
> >          |                             ^~~~~~
>
> Same problem as in the other architectures and same cure.

Indeed it does.

> ---
> Subject: m68knommu: Remove mm.h include from uaccess_no.h
> From: Thomas Gleixner <tglx@...utronix.de>
> Date: Sat, 21 Mar 2020 20:22:10 +0100
>
> In file included
>   from include/linux/huge_mm.h:8,
>   from include/linux/mm.h:567,
>   from arch/m68k/include/asm/uaccess_no.h:8,
>   from arch/m68k/include/asm/uaccess.h:3,
>   from include/linux/uaccess.h:11,
>   from include/linux/sched/task.h:11,
>   from include/linux/sched/signal.h:9,
>   from include/linux/rcuwait.h:6,
>   from include/linux/percpu-rwsem.h:7,
>   from kernel/locking/percpu-rwsem.c:6:
>  include/linux/fs.h:1422:29: error: array type has incomplete element type 'struct percpu_rw_semaphore'
>     1422 |  struct percpu_rw_semaphore rw_sem[SB_FREEZE_LEVELS];
>
> Removing the include of linux/mm.h from the uaccess header solves the problem
> and various build tests of nommu configurations still work.
>
> Fixes: 80fbaf1c3f29 ("rcuwait: Add @state argument to rcuwait_wait_event()")
> Reported-by: kbuild test robot <lkp@...el.com>
> Signed-off-by: Thomas Gleixner <tglx@...utronix.de>

Acked-by: Geert Uytterhoeven <geert@...ux-m68k.org>

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-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