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