* xfs_iomap.c:undefined reference to `put_dax'
@ 2017-05-13 23:21 kbuild test robot
2017-05-15 13:25 ` Arnd Bergmann
0 siblings, 1 reply; 6+ messages in thread
From: kbuild test robot @ 2017-05-13 23:21 UTC (permalink / raw)
To: Dan Williams; +Cc: kbuild-all, linux-kernel, Jan Kara
[-- Attachment #1: Type: text/plain, Size: 1085 bytes --]
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: cd636458904a04de2349c728323c5d2af1203bdf
commit: ef51042472f55b325fd7f2b26a2e29fd89757234 block, dax: move "select DAX" from BLOCK to FS_DAX
date: 5 days ago
config: ia64-bigsur_defconfig (attached as .config)
compiler: ia64-linux-gcc (GCC) 6.2.0
reproduce:
wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
git checkout ef51042472f55b325fd7f2b26a2e29fd89757234
# save the attached .config to linux build tree
make.cross ARCH=ia64
All errors (new ones prefixed by >>):
fs/built-in.o: In function `xfs_file_iomap_end':
>> xfs_iomap.c:(.text+0x2e3022): undefined reference to `put_dax'
fs/built-in.o: In function `xfs_file_iomap_begin':
>> xfs_iomap.c:(.text+0x2e4502): undefined reference to `dax_get_by_host'
---
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: 16335 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: xfs_iomap.c:undefined reference to `put_dax'
2017-05-13 23:21 xfs_iomap.c:undefined reference to `put_dax' kbuild test robot
@ 2017-05-15 13:25 ` Arnd Bergmann
2017-05-15 13:38 ` Dan Williams
0 siblings, 1 reply; 6+ messages in thread
From: Arnd Bergmann @ 2017-05-15 13:25 UTC (permalink / raw)
To: kbuild test robot
Cc: Dan Williams, kbuild-all, Linux Kernel Mailing List, Jan Kara
On Sun, May 14, 2017 at 1:21 AM, kbuild test robot
<fengguang.wu@intel.com> wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: cd636458904a04de2349c728323c5d2af1203bdf
> commit: ef51042472f55b325fd7f2b26a2e29fd89757234 block, dax: move "select DAX" from BLOCK to FS_DAX
> date: 5 days ago
> config: ia64-bigsur_defconfig (attached as .config)
> compiler: ia64-linux-gcc (GCC) 6.2.0
> reproduce:
> wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> chmod +x ~/bin/make.cross
> git checkout ef51042472f55b325fd7f2b26a2e29fd89757234
> # save the attached .config to linux build tree
> make.cross ARCH=ia64
>
> All errors (new ones prefixed by >>):
>
> fs/built-in.o: In function `xfs_file_iomap_end':
>>> xfs_iomap.c:(.text+0x2e3022): undefined reference to `put_dax'
> fs/built-in.o: In function `xfs_file_iomap_begin':
>>> xfs_iomap.c:(.text+0x2e4502): undefined reference to `dax_get_by_host'
This is caused by CONFIG_DAX now being allowed to be '=m' while used by
a built-in file system. This only happens for XFS at the moment though, as the
other file systems check for CONFIG_FS_DAX already.
I have a patch that I'm testing at the moment.
Arnd
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: xfs_iomap.c:undefined reference to `put_dax'
2017-05-15 13:25 ` Arnd Bergmann
@ 2017-05-15 13:38 ` Dan Williams
2017-05-15 13:59 ` Arnd Bergmann
0 siblings, 1 reply; 6+ messages in thread
From: Dan Williams @ 2017-05-15 13:38 UTC (permalink / raw)
To: Arnd Bergmann
Cc: kbuild test robot, kbuild-all, Linux Kernel Mailing List, Jan Kara
On Mon, May 15, 2017 at 6:25 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Sun, May 14, 2017 at 1:21 AM, kbuild test robot
> <fengguang.wu@intel.com> wrote:
>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
>> head: cd636458904a04de2349c728323c5d2af1203bdf
>> commit: ef51042472f55b325fd7f2b26a2e29fd89757234 block, dax: move "select DAX" from BLOCK to FS_DAX
>> date: 5 days ago
>> config: ia64-bigsur_defconfig (attached as .config)
>> compiler: ia64-linux-gcc (GCC) 6.2.0
>> reproduce:
>> wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>> chmod +x ~/bin/make.cross
>> git checkout ef51042472f55b325fd7f2b26a2e29fd89757234
>> # save the attached .config to linux build tree
>> make.cross ARCH=ia64
>>
>> All errors (new ones prefixed by >>):
>>
>> fs/built-in.o: In function `xfs_file_iomap_end':
>>>> xfs_iomap.c:(.text+0x2e3022): undefined reference to `put_dax'
>> fs/built-in.o: In function `xfs_file_iomap_begin':
>>>> xfs_iomap.c:(.text+0x2e4502): undefined reference to `dax_get_by_host'
>
> This is caused by CONFIG_DAX now being allowed to be '=m' while used by
> a built-in file system. This only happens for XFS at the moment though, as the
> other file systems check for CONFIG_FS_DAX already.
>
> I have a patch that I'm testing at the moment.
>
Hi Arnd,
I have this patch out for testing.
https://patchwork.kernel.org/patch/9725515/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: xfs_iomap.c:undefined reference to `put_dax'
2017-05-15 13:38 ` Dan Williams
@ 2017-05-15 13:59 ` Arnd Bergmann
2017-05-15 14:05 ` Dan Williams
0 siblings, 1 reply; 6+ messages in thread
From: Arnd Bergmann @ 2017-05-15 13:59 UTC (permalink / raw)
To: Dan Williams
Cc: kbuild test robot, kbuild-all, Linux Kernel Mailing List, Jan Kara
On Mon, May 15, 2017 at 3:38 PM, Dan Williams <dan.j.williams@intel.com> wrote:
> On Mon, May 15, 2017 at 6:25 AM, Arnd Bergmann <arnd@arndb.de> wrote:
>> On Sun, May 14, 2017 at 1:21 AM, kbuild test robot
>> <fengguang.wu@intel.com> wrote:
>>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
>>> head: cd636458904a04de2349c728323c5d2af1203bdf
>>> commit: ef51042472f55b325fd7f2b26a2e29fd89757234 block, dax: move "select DAX" from BLOCK to FS_DAX
>>> date: 5 days ago
>>> config: ia64-bigsur_defconfig (attached as .config)
>>> compiler: ia64-linux-gcc (GCC) 6.2.0
>>> reproduce:
>>> wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>>> chmod +x ~/bin/make.cross
>>> git checkout ef51042472f55b325fd7f2b26a2e29fd89757234
>>> # save the attached .config to linux build tree
>>> make.cross ARCH=ia64
>>>
>>> All errors (new ones prefixed by >>):
>>>
>>> fs/built-in.o: In function `xfs_file_iomap_end':
>>>>> xfs_iomap.c:(.text+0x2e3022): undefined reference to `put_dax'
>>> fs/built-in.o: In function `xfs_file_iomap_begin':
>>>>> xfs_iomap.c:(.text+0x2e4502): undefined reference to `dax_get_by_host'
>>
>> This is caused by CONFIG_DAX now being allowed to be '=m' while used by
>> a built-in file system. This only happens for XFS at the moment though, as the
>> other file systems check for CONFIG_FS_DAX already.
>>
>> I have a patch that I'm testing at the moment.
>>
>
> Hi Arnd,
>
> I have this patch out for testing.
>
> https://patchwork.kernel.org/patch/9725515/
That looks good for the file system parts, but what about drivers/md/dm.c,
do you have another patch for that one? With just the one patch you
quoted, that would break when dax_get_by_host() gets removed.
Arnd
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: xfs_iomap.c:undefined reference to `put_dax'
2017-05-15 13:59 ` Arnd Bergmann
@ 2017-05-15 14:05 ` Dan Williams
2017-05-15 14:23 ` Arnd Bergmann
0 siblings, 1 reply; 6+ messages in thread
From: Dan Williams @ 2017-05-15 14:05 UTC (permalink / raw)
To: Arnd Bergmann
Cc: kbuild test robot, kbuild-all, Linux Kernel Mailing List, Jan Kara
On Mon, May 15, 2017 at 6:59 AM, Arnd Bergmann <arnd@arndb.de> wrote:
> On Mon, May 15, 2017 at 3:38 PM, Dan Williams <dan.j.williams@intel.com> wrote:
>> On Mon, May 15, 2017 at 6:25 AM, Arnd Bergmann <arnd@arndb.de> wrote:
>>> On Sun, May 14, 2017 at 1:21 AM, kbuild test robot
>>> <fengguang.wu@intel.com> wrote:
>>>> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
>>>> head: cd636458904a04de2349c728323c5d2af1203bdf
>>>> commit: ef51042472f55b325fd7f2b26a2e29fd89757234 block, dax: move "select DAX" from BLOCK to FS_DAX
>>>> date: 5 days ago
>>>> config: ia64-bigsur_defconfig (attached as .config)
>>>> compiler: ia64-linux-gcc (GCC) 6.2.0
>>>> reproduce:
>>>> wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>>>> chmod +x ~/bin/make.cross
>>>> git checkout ef51042472f55b325fd7f2b26a2e29fd89757234
>>>> # save the attached .config to linux build tree
>>>> make.cross ARCH=ia64
>>>>
>>>> All errors (new ones prefixed by >>):
>>>>
>>>> fs/built-in.o: In function `xfs_file_iomap_end':
>>>>>> xfs_iomap.c:(.text+0x2e3022): undefined reference to `put_dax'
>>>> fs/built-in.o: In function `xfs_file_iomap_begin':
>>>>>> xfs_iomap.c:(.text+0x2e4502): undefined reference to `dax_get_by_host'
>>>
>>> This is caused by CONFIG_DAX now being allowed to be '=m' while used by
>>> a built-in file system. This only happens for XFS at the moment though, as the
>>> other file systems check for CONFIG_FS_DAX already.
>>>
>>> I have a patch that I'm testing at the moment.
>>>
>>
>> Hi Arnd,
>>
>> I have this patch out for testing.
>>
>> https://patchwork.kernel.org/patch/9725515/
>
> That looks good for the file system parts, but what about drivers/md/dm.c,
> do you have another patch for that one? With just the one patch you
> quoted, that would break when dax_get_by_host() gets removed.
>
BLK_DEV_DM selects DAX so dax_get_by_host() should always be available.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: xfs_iomap.c:undefined reference to `put_dax'
2017-05-15 14:05 ` Dan Williams
@ 2017-05-15 14:23 ` Arnd Bergmann
0 siblings, 0 replies; 6+ messages in thread
From: Arnd Bergmann @ 2017-05-15 14:23 UTC (permalink / raw)
To: Dan Williams
Cc: kbuild test robot, kbuild-all, Linux Kernel Mailing List, Jan Kara
On Mon, May 15, 2017 at 4:05 PM, Dan Williams <dan.j.williams@intel.com> wrote:
> On Mon, May 15, 2017 at 6:59 AM, Arnd Bergmann <arnd@arndb.de> wrote:
>> On Mon, May 15, 2017 at 3:38 PM, Dan Williams <dan.j.williams@intel.com> wrote:
>>>
>>> I have this patch out for testing.
>>>
>>> https://patchwork.kernel.org/patch/9725515/
>>
>> That looks good for the file system parts, but what about drivers/md/dm.c,
>> do you have another patch for that one? With just the one patch you
>> quoted, that would break when dax_get_by_host() gets removed.
>>
>
> BLK_DEV_DM selects DAX so dax_get_by_host() should always be available.
Got it, I just misread your patch and I agree this should work fine.
Arnd
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-05-15 14:23 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-13 23:21 xfs_iomap.c:undefined reference to `put_dax' kbuild test robot
2017-05-15 13:25 ` Arnd Bergmann
2017-05-15 13:38 ` Dan Williams
2017-05-15 13:59 ` Arnd Bergmann
2017-05-15 14:05 ` Dan Williams
2017-05-15 14:23 ` Arnd Bergmann
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).