[mmotm:master 227/248] lima_gem.c:undefined reference to `vmf_insert_mixed'

[mmotm:master 227/248] lima_gem.c:undefined reference to `vmf_insert_mixed'

[kbuild test robot]

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

Hi Andrew,

It's probably a bug fix that unveils the link errors.

tree:   git://git.cmpxchg.org/linux-mmotm.git master
head:   b09c000f671826e6f073a7f89b266e4ac998952b
commit: 39a08f353e1f30f7ba2e8b751a9034010a99666c [227/248] linux-next-git-rejects
config: sh-allyesconfig (attached as .config)
compiler: sh4-linux-gnu-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 39a08f353e1f30f7ba2e8b751a9034010a99666c
        # save the attached .config to linux build tree
        GCC_VERSION=7.2.0 make.cross ARCH=sh 

All errors (new ones prefixed by >>):

   arch/sh/kernel/cpu/sh2/clock-sh7619.o:(.data+0x1c): undefined reference to `followparent_recalc'
   drivers/gpu/drm/lima/lima_gem.o: In function `lima_gem_fault':
>> lima_gem.c:(.text+0x6c): undefined reference to `vmf_insert_mixed'

---
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: 51701 bytes --]

Re: [mmotm:master 227/248] lima_gem.c:undefined reference to `vmf_insert_mixed'

[Randy Dunlap]

On 4/5/19 11:47 PM, kbuild test robot wrote:
> Hi Andrew,
> 
> It's probably a bug fix that unveils the link errors.
> 
> tree:   git://git.cmpxchg.org/linux-mmotm.git master
> head:   b09c000f671826e6f073a7f89b266e4ac998952b
> commit: 39a08f353e1f30f7ba2e8b751a9034010a99666c [227/248] linux-next-git-rejects
> config: sh-allyesconfig (attached as .config)
> compiler: sh4-linux-gnu-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 39a08f353e1f30f7ba2e8b751a9034010a99666c
>         # save the attached .config to linux build tree
>         GCC_VERSION=7.2.0 make.cross ARCH=sh 
> 
> All errors (new ones prefixed by >>):
> 
>    arch/sh/kernel/cpu/sh2/clock-sh7619.o:(.data+0x1c): undefined reference to `followparent_recalc'
>    drivers/gpu/drm/lima/lima_gem.o: In function `lima_gem_fault':
>>> lima_gem.c:(.text+0x6c): undefined reference to `vmf_insert_mixed'


vmf_insert_mixed() is only built for MMU configs, and the attached config
does not set/enable MMU.
Maybe this driver should depend on MMU, like several other drm drivers do.


Also, lima_gem.c needs this line to be added to it:

--- mmotm-2019-0405-1828.orig/drivers/gpu/drm/lima/lima_gem.c
+++ mmotm-2019-0405-1828/drivers/gpu/drm/lima/lima_gem.c
@@ -1,6 +1,7 @@
 // SPDX-License-Identifier: GPL-2.0 OR MIT
 /* Copyright 2017-2019 Qiang Yu <yuq825@gmail.com> */
 
+#include <linux/mm.h>
 #include <linux/sync_file.h>
 #include <linux/pfn_t.h>
 


-- 
~Randy

Re: [mmotm:master 227/248] lima_gem.c:undefined reference to `vmf_insert_mixed'

[Qiang Yu]

Thanks Randy, I can add these.

Where should I send/submit the patch to in this case? Still drm-misc?

Regards,
Qiang


On Mon, Apr 8, 2019 at 3:08 AM Randy Dunlap <rdunlap@infradead.org> wrote:
>
> On 4/5/19 11:47 PM, kbuild test robot wrote:
> > Hi Andrew,
> >
> > It's probably a bug fix that unveils the link errors.
> >
> > tree:   git://git.cmpxchg.org/linux-mmotm.git master
> > head:   b09c000f671826e6f073a7f89b266e4ac998952b
> > commit: 39a08f353e1f30f7ba2e8b751a9034010a99666c [227/248] linux-next-git-rejects
> > config: sh-allyesconfig (attached as .config)
> > compiler: sh4-linux-gnu-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 39a08f353e1f30f7ba2e8b751a9034010a99666c
> >         # save the attached .config to linux build tree
> >         GCC_VERSION=7.2.0 make.cross ARCH=sh
> >
> > All errors (new ones prefixed by >>):
> >
> >    arch/sh/kernel/cpu/sh2/clock-sh7619.o:(.data+0x1c): undefined reference to `followparent_recalc'
> >    drivers/gpu/drm/lima/lima_gem.o: In function `lima_gem_fault':
> >>> lima_gem.c:(.text+0x6c): undefined reference to `vmf_insert_mixed'
>
>
> vmf_insert_mixed() is only built for MMU configs, and the attached config
> does not set/enable MMU.
> Maybe this driver should depend on MMU, like several other drm drivers do.
>
>
> Also, lima_gem.c needs this line to be added to it:
>
> --- mmotm-2019-0405-1828.orig/drivers/gpu/drm/lima/lima_gem.c
> +++ mmotm-2019-0405-1828/drivers/gpu/drm/lima/lima_gem.c
> @@ -1,6 +1,7 @@
>  // SPDX-License-Identifier: GPL-2.0 OR MIT
>  /* Copyright 2017-2019 Qiang Yu <yuq825@gmail.com> */
>
> +#include <linux/mm.h>
>  #include <linux/sync_file.h>
>  #include <linux/pfn_t.h>
>
>
>
> --
> ~Randy

Re: [mmotm:master 227/248] lima_gem.c:undefined reference to `vmf_insert_mixed'

[Randy Dunlap]

On 4/7/19 6:30 PM, Qiang Yu wrote:
> Thanks Randy, I can add these.
> 
> Where should I send/submit the patch to in this case? Still drm-misc?

Sounds good to me.

Thanks.

> Regards,
> Qiang
> 
> 
> On Mon, Apr 8, 2019 at 3:08 AM Randy Dunlap <rdunlap@infradead.org> wrote:
>>
>> On 4/5/19 11:47 PM, kbuild test robot wrote:
>>> Hi Andrew,
>>>
>>> It's probably a bug fix that unveils the link errors.
>>>
>>> tree:   git://git.cmpxchg.org/linux-mmotm.git master
>>> head:   b09c000f671826e6f073a7f89b266e4ac998952b
>>> commit: 39a08f353e1f30f7ba2e8b751a9034010a99666c [227/248] linux-next-git-rejects
>>> config: sh-allyesconfig (attached as .config)
>>> compiler: sh4-linux-gnu-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 39a08f353e1f30f7ba2e8b751a9034010a99666c
>>>         # save the attached .config to linux build tree
>>>         GCC_VERSION=7.2.0 make.cross ARCH=sh
>>>
>>> All errors (new ones prefixed by >>):
>>>
>>>    arch/sh/kernel/cpu/sh2/clock-sh7619.o:(.data+0x1c): undefined reference to `followparent_recalc'
>>>    drivers/gpu/drm/lima/lima_gem.o: In function `lima_gem_fault':
>>>>> lima_gem.c:(.text+0x6c): undefined reference to `vmf_insert_mixed'
>>
>>
>> vmf_insert_mixed() is only built for MMU configs, and the attached config
>> does not set/enable MMU.
>> Maybe this driver should depend on MMU, like several other drm drivers do.
>>
>>
>> Also, lima_gem.c needs this line to be added to it:
>>
>> --- mmotm-2019-0405-1828.orig/drivers/gpu/drm/lima/lima_gem.c
>> +++ mmotm-2019-0405-1828/drivers/gpu/drm/lima/lima_gem.c
>> @@ -1,6 +1,7 @@
>>  // SPDX-License-Identifier: GPL-2.0 OR MIT
>>  /* Copyright 2017-2019 Qiang Yu <yuq825@gmail.com> */
>>
>> +#include <linux/mm.h>
>>  #include <linux/sync_file.h>
>>  #include <linux/pfn_t.h>
>>
>>
>>
>> --
>> ~Randy
> 


-- 
~Randy