linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [vfs:work.termios 6/10] arch/arm/mach-sa1100/assabet.c:550:60: error: 'DOMAIN_IO' undeclared; did you mean 'DOMAIN_BUS_IPI'?
@ 2018-09-10 18:34 kbuild test robot
  2018-09-10 20:26 ` Al Viro
  0 siblings, 1 reply; 2+ messages in thread
From: kbuild test robot @ 2018-09-10 18:34 UTC (permalink / raw)
  To: Al Viro; +Cc: kbuild-all, linux-fsdevel

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

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs.git work.termios
head:   a3318f219be23141d4784902d201e4d8e8d6d0d2
commit: b62802628ae228744508363e28118427904bc338 [6/10] untangle asm/termios.h mess
config: arm-neponset_defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (Debian 7.2.0-11) 7.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout b62802628ae228744508363e28118427904bc338
        # save the attached .config to linux build tree
        GCC_VERSION=7.2.0 make.cross ARCH=arm 

All error/warnings (new ones prefixed by >>):

   In file included from include/linux/const.h:4:0,
                    from include/linux/list.h:8,
                    from include/linux/module.h:9,
                    from arch/arm/mach-sa1100/assabet.c:14:
   arch/arm/mach-sa1100/assabet.c: In function 'map_sa1100_gpio_regs':
>> arch/arm/mach-sa1100/assabet.c:550:60: error: 'DOMAIN_IO' undeclared (first use in this function); did you mean 'DOMAIN_BUS_IPI'?
     int prot = PMD_TYPE_SECT | PMD_SECT_AP_WRITE | PMD_DOMAIN(DOMAIN_IO);
                                                               ^
   include/uapi/linux/const.h:22:23: note: in definition of macro '_AT'
    #define _AT(T,X) ((T)(X))
                          ^
>> arch/arm/mach-sa1100/assabet.c:550:49: note: in expansion of macro 'PMD_DOMAIN'
     int prot = PMD_TYPE_SECT | PMD_SECT_AP_WRITE | PMD_DOMAIN(DOMAIN_IO);
                                                    ^~~~~~~~~~
   arch/arm/mach-sa1100/assabet.c:550:60: note: each undeclared identifier is reported only once for each function it appears in
     int prot = PMD_TYPE_SECT | PMD_SECT_AP_WRITE | PMD_DOMAIN(DOMAIN_IO);
                                                               ^
   include/uapi/linux/const.h:22:23: note: in definition of macro '_AT'
    #define _AT(T,X) ((T)(X))
                          ^
>> arch/arm/mach-sa1100/assabet.c:550:49: note: in expansion of macro 'PMD_DOMAIN'
     int prot = PMD_TYPE_SECT | PMD_SECT_AP_WRITE | PMD_DOMAIN(DOMAIN_IO);
                                                    ^~~~~~~~~~

vim +550 arch/arm/mach-sa1100/assabet.c

^1da177e Linus Torvalds 2005-04-16  539  
^1da177e Linus Torvalds 2005-04-16  540  /*
^1da177e Linus Torvalds 2005-04-16  541   * On Assabet, we must probe for the Neponset board _before_
^1da177e Linus Torvalds 2005-04-16  542   * paging_init() has occurred to actually determine the amount
^1da177e Linus Torvalds 2005-04-16  543   * of RAM available.  To do so, we map the appropriate IO section
^1da177e Linus Torvalds 2005-04-16  544   * in the page table here in order to access GPIO registers.
^1da177e Linus Torvalds 2005-04-16  545   */
^1da177e Linus Torvalds 2005-04-16  546  static void __init map_sa1100_gpio_regs( void )
^1da177e Linus Torvalds 2005-04-16  547  {
^1da177e Linus Torvalds 2005-04-16  548  	unsigned long phys = __PREG(GPLR) & PMD_MASK;
3169663a Russell King   2012-06-06  549  	unsigned long virt = (unsigned long)io_p2v(phys);
^1da177e Linus Torvalds 2005-04-16 @550  	int prot = PMD_TYPE_SECT | PMD_SECT_AP_WRITE | PMD_DOMAIN(DOMAIN_IO);
^1da177e Linus Torvalds 2005-04-16  551  	pmd_t *pmd;
^1da177e Linus Torvalds 2005-04-16  552  
a61c2332 Russell King   2012-01-14  553  	pmd = pmd_offset(pud_offset(pgd_offset_k(virt), virt), virt);
^1da177e Linus Torvalds 2005-04-16  554  	*pmd = __pmd(phys | prot);
^1da177e Linus Torvalds 2005-04-16  555  	flush_pmd_entry(pmd);
^1da177e Linus Torvalds 2005-04-16  556  }
^1da177e Linus Torvalds 2005-04-16  557  

:::::: The code at line 550 was first introduced by commit
:::::: 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 Linux-2.6.12-rc2

:::::: TO: Linus Torvalds <torvalds@ppc970.osdl.org>
:::::: CC: Linus Torvalds <torvalds@ppc970.osdl.org>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 15926 bytes --]

^ permalink raw reply	[flat|nested] 2+ messages in thread
* Re: [vfs:work.termios 6/10] arch/arm/mach-sa1100/assabet.c:550:60: error: 'DOMAIN_IO' undeclared; did you mean 'DOMAIN_BUS_IPI'?
  2018-09-10 18:34 [vfs:work.termios 6/10] arch/arm/mach-sa1100/assabet.c:550:60: error: 'DOMAIN_IO' undeclared; did you mean 'DOMAIN_BUS_IPI'? kbuild test robot
@ 2018-09-10 20:26 ` Al Viro
  0 siblings, 0 replies; 2+ messages in thread
From: Al Viro @ 2018-09-10 20:26 UTC (permalink / raw)
  To: kbuild test robot; +Cc: kbuild-all, linux-fsdevel

On Tue, Sep 11, 2018 at 02:34:32AM +0800, kbuild test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs.git work.termios
> head:   a3318f219be23141d4784902d201e4d8e8d6d0d2
> commit: b62802628ae228744508363e28118427904bc338 [6/10] untangle asm/termios.h mess
> config: arm-neponset_defconfig (attached as .config)
> compiler: arm-linux-gnueabi-gcc (Debian 7.2.0-11) 7.2.0
> reproduce:

Huh?

>    In file included from include/linux/const.h:4:0,
>                     from include/linux/list.h:8,
>                     from include/linux/module.h:9,
>                     from arch/arm/mach-sa1100/assabet.c:14:
>    arch/arm/mach-sa1100/assabet.c: In function 'map_sa1100_gpio_regs':
> >> arch/arm/mach-sa1100/assabet.c:550:60: error: 'DOMAIN_IO' undeclared (first use in this function); did you mean 'DOMAIN_BUS_IPI'?

What in hell does that thing have to do with asm/termios.h?  Especially
on arm, where the only visible change in that commit is that include of
asm/termios.h stopped pulling linux/uacess.h?

Oh, dear...  So we have this:

linux/serial_core.h ->
linux/tty.h ->
[uapi] linux/termios.h ->
[uapi] asm/termios.h -> (due to generic-y)
asm-generic/termios.h ->
linux/uaccess.h and [uapi] asm-generic/termios.h.

with linux/uaccess.h -> asm/uaccess.h -> asm/domain.h being what
used to bring DOMAIN_IO in.  And that change has eliminated the
side-trip into asm-generic/termios.h, replacing it with pulling
uapi asm-generic/termios.h.  With missing indirect include of
asm/domain.h as the result.

Note that *nothing* in linux/serial_core.h or the stuff pulled by
it has any need of linux/uaccess.h.  Wonderful, innit?

As it were, prior to that commit any pull of asm/termios.h (realistically -
linux/termios.h) used to imply the pull of linux/uaccess.h.  Sigh...
Let's see...  Includes of linux/termios.h:
arch/arm/mach-ep93xx/core.c:29:#include <linux/termios.h>
arch/arm/mach-integrator/integrator_ap.c:29:#include <linux/termios.h>
drivers/char/pcmcia/synclink_cs.c:68:#include <linux/termios.h>
drivers/misc/ibmasm/uart.c:25:#include <linux/termios.h>
drivers/net/slip/slhc.c:66:#include <linux/termios.h>
drivers/parisc/superio.c:73:#include <linux/termios.h>
drivers/s390/char/sclp_con.c:15:#include <linux/termios.h>
drivers/tty/isicom.c:119:#include <linux/termios.h>
drivers/tty/serial/icom.c:18:#include <linux/termios.h>
drivers/tty/serial/ifx6x60.c:28:#include <linux/termios.h>
drivers/tty/serial/serial-tegra.c:33:#include <linux/termios.h>
drivers/tty/serial/serial_mctrl_gpio.c:12:#include <linux/termios.h>
drivers/tty/synclink.c:93:#include <linux/termios.h>
drivers/tty/synclink_gt.c:69:#include <linux/termios.h>
drivers/tty/synclinkmp.c:66:#include <linux/termios.h>
drivers/tty/tty_baudrate.c:8:#include <linux/termios.h>
drivers/tty/tty_ioctl.c:11:#include <linux/termios.h>
include/linux/serdev.h:18:#include <linux/termios.h>
include/linux/tty.h:7:#include <linux/termios.h>
include/linux/tty_driver.h:246:#include <linux/termios.h>
net/appletalk/ddp.c:57:#include <linux/termios.h>       /* For TIOCOUTQ/INQ */
net/ax25/af_ax25.c:37:#include <linux/termios.h>        /* For TIOCINQ/OUTQ */
net/ax25/ax25_ip.c:28:#include <linux/termios.h>        /* For TIOCINQ/OUTQ */
net/decnet/dn_nsp_in.c:65:#include <linux/termios.h>
net/decnet/dn_nsp_out.c:56:#include <linux/termios.h>
net/ieee802154/socket.c:25:#include <linux/termios.h>   /* For TIOCOUTQ/INQ */
net/netlink/af_netlink.c:38:#include <linux/termios.h>
net/netrom/af_netrom.c:35:#include <linux/termios.h>    /* For TIOCINQ/OUTQ */
net/netrom/nr_route.c:30:#include <linux/termios.h>     /* For TIOCINQ/OUTQ */
net/qrtr/qrtr.c:17:#include <linux/termios.h>   /* For TIOCINQ/OUTQ */
net/rose/af_rose.c:39:#include <linux/termios.h>
net/rose/rose_route.c:30:#include <linux/termios.h>     /* For TIOCINQ/OUTQ */
net/unix/af_unix.c:97:#include <linux/termios.h>
net/x25/af_x25.c:56:#include <linux/termios.h>  /* For TIOCINQ/OUTQ */

Excluding the ones that have an explicit include of linux/uaccess.h and
net/sock.h (it pulls linux/uaccess.h unconditionally) we are left with

arch/arm/mach-ep93xx/core.c:29:#include <linux/termios.h>
arch/arm/mach-integrator/integrator_ap.c:29:#include <linux/termios.h>
drivers/misc/ibmasm/uart.c:25:#include <linux/termios.h>
drivers/parisc/superio.c:73:#include <linux/termios.h>
drivers/s390/char/sclp_con.c:15:#include <linux/termios.h>
drivers/tty/serial/ifx6x60.c:28:#include <linux/termios.h>
drivers/tty/serial/serial-tegra.c:33:#include <linux/termios.h>
drivers/tty/serial/serial_mctrl_gpio.c:12:#include <linux/termios.h>
drivers/tty/tty_baudrate.c:8:#include <linux/termios.h>
include/linux/serdev.h:18:#include <linux/termios.h>
include/linux/tty.h:7:#include <linux/termios.h>
include/linux/tty_driver.h:246:#include <linux/termios.h>

Hmm... serial_core.h pulls tty.h, which pulls tty_driver.h.  And excluding
those the pull either, we are left with

arch/arm/mach-ep93xx/core.c:29:#include <linux/termios.h>
arch/arm/mach-integrator/integrator_ap.c:29:#include <linux/termios.h>
drivers/s390/char/sclp_con.c:15:#include <linux/termios.h>
include/linux/serdev.h:18:#include <linux/termios.h>

sclp_con.c pulls tty_driver.h via sclp_tty.h, so the minimal solution would
be to shift the include of linux/uaccess.h into tty_driver.h, that pair
of arm files and serdev.h ;-/  Odds are, that's a serious overkill, but...

Sigh.  I really, really hate chain includes ;-/

^ permalink raw reply	[flat|nested] 2+ messages in thread
end of thread, other threads:[~2018-09-11  1:22 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-09-10 18:34 [vfs:work.termios 6/10] arch/arm/mach-sa1100/assabet.c:550:60: error: 'DOMAIN_IO' undeclared; did you mean 'DOMAIN_BUS_IPI'? kbuild test robot
2018-09-10 20:26 ` Al Viro

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