subject:"\[radeon\-alex\:upstream\-4.14\-drm\-next\-amd\-dc\-staging\-chrome 4\/16\] drivers\/gpu\/\/drm\/ttm\/ttm_page_alloc.c\:923\:5\: error\: redefinition of 'ttm_populate_and_map

Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 'ttm_populate_and_map_pages'

2017-11-14 Thread S, Shirish

Do you have a standard such reference for uniformity, else chrome keyword will 
do.

(typed on mobile, kindly ignore typos)

Regards,
Shirish S


From: Fengguang Wu <fengguang...@intel.com>
Sent: Tuesday, November 14, 2017 6:41:41 PM
To: S, Shirish
Cc: StDenis, Tom; kbuild-...@01.org; dri-devel@lists.freedesktop.org; Koenig, 
Christian; Deucher, Alexander
Subject: Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] 
drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
'ttm_populate_and_map_pages'

Hi Shirish,

Sorry for the noise! Do you have any branch pattern that I can add to
blacklist? The regex could be 'chrome$' to match it in the end or just
'chrome' to match it anywhere in the branch name.

Thanks,
Fengguang

On Tue, Nov 14, 2017 at 01:07:07PM +, S, Shirish wrote:
>Please ignore all the kbuild bot related mails on this branch.
>As the purpose of this branch is chrome specific and in a very specific build 
>environment.
>
>
>
>Regards,
>Shirish S
>
>
>From: StDenis, Tom
>Sent: Tuesday, November 14, 2017 4:58:48 PM
>To: kbuild test robot
>Cc: kbuild-...@01.org; dri-devel@lists.freedesktop.org; S, Shirish; Koenig, 
>Christian; Deucher, Alexander
>Subject: Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] 
>drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
>'ttm_populate_and_map_pages'
>
>Is this:
>
>commit 7a9667ae197460e6c9c3bb432fe68c708fce6259
>Refs: v4.13-rc5-1195-g7a9667ae1974
>Author: Tom St Denis <tom.stde...@amd.com>
>AuthorDate: Tue Sep 5 07:30:59 2017 -0400
>Commit: Alex Deucher <alexander.deuc...@amd.com>
>CommitDate: Tue Sep 12 14:22:55 2017 -0400
>
> drm/ttm: Fix configuration error around populate_and_map() functions
>
> Fixed kbuild errors when IOMMU/SWIOTLB are disabled.
>
> Signed-off-by: Tom St Denis <tom.stde...@amd.com>
> Reviewed-by: Christian König <christian.koe...@amd.com>
> Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
>
>Not part of this series because we literally went through this before :-)
>
>Tom
>
>
>On 13/11/17 05:25 PM, kbuild test robot wrote:
>> tree:   git://people.freedesktop.org/~agd5f/linux.git 
>> upstream-4.14-drm-next-amd-dc-staging-chrome
>> head:   4448b9a68413462529d018050cd246bc33957bd6
>> commit: ed285b98008b667978d7faf348a22000b8a1c6b9 [4/16] drm/ttm: Add helper 
>> functions to populate/map in one call (v2)
>> config: i386-randconfig-s0-201746 (attached as .config)
>> compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026
>> reproduce:
>>  git checkout ed285b98008b667978d7faf348a22000b8a1c6b9
>>  # save the attached .config to linux build tree
>>  make ARCH=i386
>>
>> All errors (new ones prefixed by >>):
>>
>>>> drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
>>>> 'ttm_populate_and_map_pages'
>>  int ttm_populate_and_map_pages(struct device *dev, struct ttm_dma_tt 
>> *tt)
>>  ^~
>> In file included from drivers/gpu//drm/ttm/ttm_page_alloc.c:49:0:
>> include/drm/ttm/ttm_page_alloc.h:120:19: note: previous definition of 
>> 'ttm_populate_and_map_pages' was here
>>  static inline int ttm_populate_and_map_pages(struct device *dev, struct 
>> ttm_dma_tt *tt)
>>^~
>>>> drivers/gpu//drm/ttm/ttm_page_alloc.c:950:6: error: redefinition of 
>>>> 'ttm_unmap_and_unpopulate_pages'
>>  void ttm_unmap_and_unpopulate_pages(struct device *dev, struct 
>> ttm_dma_tt *tt)
>>   ^~
>> In file included from drivers/gpu//drm/ttm/ttm_page_alloc.c:49:0:
>> include/drm/ttm/ttm_page_alloc.h:125:20: note: previous definition of 
>> 'ttm_unmap_and_unpopulate_pages' was here
>>  static inline void ttm_unmap_and_unpopulate_pages(struct device *dev, 
>> struct ttm_dma_tt *tt)
>> ^~
>>
>> vim +/ttm_populate_and_map_pages +923 drivers/gpu//drm/ttm/ttm_page_alloc.c
>>
>> 922
>>   > 923   int ttm_populate_and_map_pages(struct device *dev, struct 
>> ttm_dma_tt *tt)
>> 924   {
>> 925   unsigned i;
>> 926   int r;
>> 927
>> 928   r = ttm_pool_populate(>ttm);
>> 929   if (r)
>> 930   return r;
>> 931
>> 932   for (i = 0; i < tt->ttm.num_pages; i++) {
&

Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 'ttm_populate_and_map_pages'

2017-11-14 Thread S, Shirish

Thanks.

(typed on mobile, kindly ignore typos)

Regards,
Shirish S


From: Fengguang Wu <fengguang...@intel.com>
Sent: Tuesday, November 14, 2017 6:52:03 PM
To: S, Shirish
Cc: StDenis, Tom; kbuild-...@01.org; dri-devel@lists.freedesktop.org; Koenig, 
Christian; Deucher, Alexander
Subject: Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] 
drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
'ttm_populate_and_map_pages'

OK I'll blacklist all branches containing the "chrome" string.

It's "grep -E" patterns. Here are some examples:

wfg /c/lkp-tests% grep -hr blacklist repo
blacklist_branch: master|revert-
blacklist_branch: .*
blacklist_branch: .*
blacklist_branch: .*
blacklist_branch: for-epmt
blacklist_branch: master
blacklist_branch: pending-rebases
blacklist_branch: for_3.16/wip/net-qmss5
blacklist_branch: auto-next
blacklist_branch: .*
blacklist_branch: .*
blacklist_branch: cpufreq-tixy
blacklist_branch: .*\<stable\>
blacklist_branch: akpm|akpm-base|stable
blacklist_branch: dev
blacklist_branch: .*
blacklist_branch: auto-.*|tmp-.*|base-.*|test.*|.*-for-linus
blacklist_branch: nx-emu
blacklist_branch: .*
blacklist_branch: master
blacklist_branch: master
blacklist_branch: unionmount
blacklist_branch: check
blacklist_branch: devel/.*
blacklist_branch: rerere-cache|maintainer-tools
blacklist_branch: .*
blacklist_branch: auto-next
blacklist_branch: .*
blacklist_branch: .*
blacklist_branch: openrisc-4.12-smp
blacklist_branch: master|revert-
blacklist_branch: master|revert-
blacklist_branch: master|revert-
blacklist_branch: master|revert-
blacklist_branch: master|revert-
blacklist_branch: master|revert-

Thanks,
Fengguang


on Tue, Nov 14, 2017 at 01:13:07PM +, S, Shirish wrote:
>Do you have a standard such reference for uniformity, else chrome keyword will 
>do.
>
>(typed on mobile, kindly ignore typos)
>
>Regards,
>Shirish S
>
>
>From: Fengguang Wu <fengguang...@intel.com>
>Sent: Tuesday, November 14, 2017 6:41:41 PM
>To: S, Shirish
>Cc: StDenis, Tom; kbuild-...@01.org; dri-devel@lists.freedesktop.org; Koenig, 
>Christian; Deucher, Alexander
>Subject: Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] 
>drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
>'ttm_populate_and_map_pages'
>
>Hi Shirish,
>
>Sorry for the noise! Do you have any branch pattern that I can add to
>blacklist? The regex could be 'chrome$' to match it in the end or just
>'chrome' to match it anywhere in the branch name.
>
>Thanks,
>Fengguang
>
>On Tue, Nov 14, 2017 at 01:07:07PM +, S, Shirish wrote:
>>Please ignore all the kbuild bot related mails on this branch.
>>As the purpose of this branch is chrome specific and in a very specific build 
>>environment.
>>
>>
>>
>>Regards,
>>Shirish S
>>
>>
>>From: StDenis, Tom
>>Sent: Tuesday, November 14, 2017 4:58:48 PM
>>To: kbuild test robot
>>Cc: kbuild-...@01.org; dri-devel@lists.freedesktop.org; S, Shirish; Koenig, 
>>Christian; Deucher, Alexander
>>Subject: Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] 
>>drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
>>'ttm_populate_and_map_pages'
>>
>>Is this:
>>
>>commit 7a9667ae197460e6c9c3bb432fe68c708fce6259
>>Refs: v4.13-rc5-1195-g7a9667ae1974
>>Author: Tom St Denis <tom.stde...@amd.com>
>>AuthorDate: Tue Sep 5 07:30:59 2017 -0400
>>Commit: Alex Deucher <alexander.deuc...@amd.com>
>>CommitDate: Tue Sep 12 14:22:55 2017 -0400
>>
>> drm/ttm: Fix configuration error around populate_and_map() functions
>>
>> Fixed kbuild errors when IOMMU/SWIOTLB are disabled.
>>
>> Signed-off-by: Tom St Denis <tom.stde...@amd.com>
>> Reviewed-by: Christian König <christian.koe...@amd.com>
>> Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
>>
>>Not part of this series because we literally went through this before :-)
>>
>>Tom
>>
>>
>>On 13/11/17 05:25 PM, kbuild test robot wrote:
>>> tree:   git://people.freedesktop.org/~agd5f/linux.git 
>>> upstream-4.14-drm-next-amd-dc-staging-chrome
>>> head:   4448b9a68413462529d018050cd246bc33957bd6
>>> commit: ed285b98008b667978d7faf348a22000b8a1c6b9 [4/16] drm/ttm: Add helper 
>>> functions to populate/map in one call (v2)
>>> config: i386-randconfig-s0-201746 (attached as .config)
>>> compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026
>>> reproduce:
>>>  git checkout ed285b98008b667978d7faf348a220

Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 'ttm_populate_and_map_pages'

2017-11-14 Thread Fengguang Wu


OK I'll blacklist all branches containing the "chrome" string.

It's "grep -E" patterns. Here are some examples:

wfg /c/lkp-tests% grep -hr blacklist repo
blacklist_branch: master|revert-
blacklist_branch: .*
blacklist_branch: .*
blacklist_branch: .*
blacklist_branch: for-epmt
blacklist_branch: master
blacklist_branch: pending-rebases
blacklist_branch: for_3.16/wip/net-qmss5
blacklist_branch: auto-next
blacklist_branch: .*
blacklist_branch: .*
blacklist_branch: cpufreq-tixy
blacklist_branch: .*\<stable\>
blacklist_branch: akpm|akpm-base|stable
blacklist_branch: dev
blacklist_branch: .*
blacklist_branch: auto-.*|tmp-.*|base-.*|test.*|.*-for-linus
blacklist_branch: nx-emu
blacklist_branch: .*
blacklist_branch: master
blacklist_branch: master
blacklist_branch: unionmount
blacklist_branch: check
blacklist_branch: devel/.*
blacklist_branch: rerere-cache|maintainer-tools
blacklist_branch: .*
blacklist_branch: auto-next
blacklist_branch: .*
blacklist_branch: .*
blacklist_branch: openrisc-4.12-smp
blacklist_branch: master|revert-
blacklist_branch: master|revert-
blacklist_branch: master|revert-
blacklist_branch: master|revert-
blacklist_branch: master|revert-
blacklist_branch: master|revert-

Thanks,
Fengguang


on Tue, Nov 14, 2017 at 01:13:07PM +, S, Shirish wrote:

Do you have a standard such reference for uniformity, else chrome keyword will 
do.

(typed on mobile, kindly ignore typos)

Regards,
Shirish S


From: Fengguang Wu <fengguang...@intel.com>
Sent: Tuesday, November 14, 2017 6:41:41 PM
To: S, Shirish
Cc: StDenis, Tom; kbuild-...@01.org; dri-devel@lists.freedesktop.org; Koenig, 
Christian; Deucher, Alexander
Subject: Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] 
drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
'ttm_populate_and_map_pages'

Hi Shirish,

Sorry for the noise! Do you have any branch pattern that I can add to
blacklist? The regex could be 'chrome$' to match it in the end or just
'chrome' to match it anywhere in the branch name.

Thanks,
Fengguang

On Tue, Nov 14, 2017 at 01:07:07PM +, S, Shirish wrote:

Please ignore all the kbuild bot related mails on this branch.
As the purpose of this branch is chrome specific and in a very specific build 
environment.



Regards,
Shirish S


From: StDenis, Tom
Sent: Tuesday, November 14, 2017 4:58:48 PM
To: kbuild test robot
Cc: kbuild-...@01.org; dri-devel@lists.freedesktop.org; S, Shirish; Koenig, 
Christian; Deucher, Alexander
Subject: Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] 
drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
'ttm_populate_and_map_pages'

Is this:

commit 7a9667ae197460e6c9c3bb432fe68c708fce6259
Refs: v4.13-rc5-1195-g7a9667ae1974
Author: Tom St Denis <tom.stde...@amd.com>
AuthorDate: Tue Sep 5 07:30:59 2017 -0400
Commit: Alex Deucher <alexander.deuc...@amd.com>
CommitDate: Tue Sep 12 14:22:55 2017 -0400

drm/ttm: Fix configuration error around populate_and_map() functions

Fixed kbuild errors when IOMMU/SWIOTLB are disabled.

Signed-off-by: Tom St Denis <tom.stde...@amd.com>
Reviewed-by: Christian König <christian.koe...@amd.com>
Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>

Not part of this series because we literally went through this before :-)

Tom


On 13/11/17 05:25 PM, kbuild test robot wrote:

tree:   git://people.freedesktop.org/~agd5f/linux.git 
upstream-4.14-drm-next-amd-dc-staging-chrome
head:   4448b9a68413462529d018050cd246bc33957bd6
commit: ed285b98008b667978d7faf348a22000b8a1c6b9 [4/16] drm/ttm: Add helper 
functions to populate/map in one call (v2)
config: i386-randconfig-s0-201746 (attached as .config)
compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026
reproduce:
 git checkout ed285b98008b667978d7faf348a22000b8a1c6b9
 # save the attached .config to linux build tree
 make ARCH=i386

All errors (new ones prefixed by >>):


drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
'ttm_populate_and_map_pages'

 int ttm_populate_and_map_pages(struct device *dev, struct ttm_dma_tt *tt)
 ^~
In file included from drivers/gpu//drm/ttm/ttm_page_alloc.c:49:0:
include/drm/ttm/ttm_page_alloc.h:120:19: note: previous definition of 
'ttm_populate_and_map_pages' was here
 static inline int ttm_populate_and_map_pages(struct device *dev, struct 
ttm_dma_tt *tt)
   ^~

drivers/gpu//drm/ttm/ttm_page_alloc.c:950:6: error: redefinition of 
'ttm_unmap_and_unpopulate_pages'

 void ttm_unmap_and_unpopulate_pages(struct device *dev, struct ttm_dma_tt 
*tt)
  ^~
In file included from drivers/gpu//drm/ttm/ttm_page_alloc.c:49:0:
include/drm/ttm/ttm_page_alloc.h:125:20: note: previous definition o

Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 'ttm_populate_and_map_pages'

2017-11-14 Thread Fengguang Wu


Hi Shirish,

Sorry for the noise! Do you have any branch pattern that I can add to
blacklist? The regex could be 'chrome$' to match it in the end or just
'chrome' to match it anywhere in the branch name.

Thanks,
Fengguang

On Tue, Nov 14, 2017 at 01:07:07PM +, S, Shirish wrote:

Please ignore all the kbuild bot related mails on this branch.
As the purpose of this branch is chrome specific and in a very specific build 
environment.



Regards,
Shirish S


From: StDenis, Tom
Sent: Tuesday, November 14, 2017 4:58:48 PM
To: kbuild test robot
Cc: kbuild-...@01.org; dri-devel@lists.freedesktop.org; S, Shirish; Koenig, 
Christian; Deucher, Alexander
Subject: Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] 
drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
'ttm_populate_and_map_pages'

Is this:

commit 7a9667ae197460e6c9c3bb432fe68c708fce6259
Refs: v4.13-rc5-1195-g7a9667ae1974
Author: Tom St Denis <tom.stde...@amd.com>
AuthorDate: Tue Sep 5 07:30:59 2017 -0400
Commit: Alex Deucher <alexander.deuc...@amd.com>
CommitDate: Tue Sep 12 14:22:55 2017 -0400

drm/ttm: Fix configuration error around populate_and_map() functions

Fixed kbuild errors when IOMMU/SWIOTLB are disabled.

Signed-off-by: Tom St Denis <tom.stde...@amd.com>
Reviewed-by: Christian König <christian.koe...@amd.com>
Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>

Not part of this series because we literally went through this before :-)

Tom


On 13/11/17 05:25 PM, kbuild test robot wrote:

tree:   git://people.freedesktop.org/~agd5f/linux.git 
upstream-4.14-drm-next-amd-dc-staging-chrome
head:   4448b9a68413462529d018050cd246bc33957bd6
commit: ed285b98008b667978d7faf348a22000b8a1c6b9 [4/16] drm/ttm: Add helper 
functions to populate/map in one call (v2)
config: i386-randconfig-s0-201746 (attached as .config)
compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026
reproduce:
 git checkout ed285b98008b667978d7faf348a22000b8a1c6b9
 # save the attached .config to linux build tree
 make ARCH=i386

All errors (new ones prefixed by >>):


drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
'ttm_populate_and_map_pages'

 int ttm_populate_and_map_pages(struct device *dev, struct ttm_dma_tt *tt)
 ^~
In file included from drivers/gpu//drm/ttm/ttm_page_alloc.c:49:0:
include/drm/ttm/ttm_page_alloc.h:120:19: note: previous definition of 
'ttm_populate_and_map_pages' was here
 static inline int ttm_populate_and_map_pages(struct device *dev, struct 
ttm_dma_tt *tt)
   ^~

drivers/gpu//drm/ttm/ttm_page_alloc.c:950:6: error: redefinition of 
'ttm_unmap_and_unpopulate_pages'

 void ttm_unmap_and_unpopulate_pages(struct device *dev, struct ttm_dma_tt 
*tt)
  ^~
In file included from drivers/gpu//drm/ttm/ttm_page_alloc.c:49:0:
include/drm/ttm/ttm_page_alloc.h:125:20: note: previous definition of 
'ttm_unmap_and_unpopulate_pages' was here
 static inline void ttm_unmap_and_unpopulate_pages(struct device *dev, 
struct ttm_dma_tt *tt)
^~

vim +/ttm_populate_and_map_pages +923 drivers/gpu//drm/ttm/ttm_page_alloc.c

922
  > 923   int ttm_populate_and_map_pages(struct device *dev, struct 
ttm_dma_tt *tt)
924   {
925   unsigned i;
926   int r;
927
928   r = ttm_pool_populate(>ttm);
929   if (r)
930   return r;
931
932   for (i = 0; i < tt->ttm.num_pages; i++) {
933   tt->dma_address[i] = dma_map_page(dev, 
tt->ttm.pages[i],
934 0, PAGE_SIZE,
935 
DMA_BIDIRECTIONAL);
936   if (dma_mapping_error(dev, tt->dma_address[i])) {
937   while (i--) {
938   dma_unmap_page(dev, 
tt->dma_address[i],
939  PAGE_SIZE, 
DMA_BIDIRECTIONAL);
940   tt->dma_address[i] = 0;
941   }
942   ttm_pool_unpopulate(>ttm);
943   return -EFAULT;
944   }
945   }
946   return 0;
947   }
948   EXPORT_SYMBOL(ttm_populate_and_map_pages);
949
  > 950   void ttm_unmap_and_unpopulate_pages(struct device *dev, struct 
ttm_dma_tt *tt)
951   {
952   unsigned i;
953
954   for (i = 0; i < tt->ttm.num_pa

Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 'ttm_populate_and_map_pages'

2017-11-14 Thread S, Shirish

Please ignore all the kbuild bot related mails on this branch.
As the purpose of this branch is chrome specific and in a very specific build 
environment.



Regards,
Shirish S


From: StDenis, Tom
Sent: Tuesday, November 14, 2017 4:58:48 PM
To: kbuild test robot
Cc: kbuild-...@01.org; dri-devel@lists.freedesktop.org; S, Shirish; Koenig, 
Christian; Deucher, Alexander
Subject: Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] 
drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
'ttm_populate_and_map_pages'

Is this:

commit 7a9667ae197460e6c9c3bb432fe68c708fce6259
Refs: v4.13-rc5-1195-g7a9667ae1974
Author: Tom St Denis <tom.stde...@amd.com>
AuthorDate: Tue Sep 5 07:30:59 2017 -0400
Commit: Alex Deucher <alexander.deuc...@amd.com>
CommitDate: Tue Sep 12 14:22:55 2017 -0400

 drm/ttm: Fix configuration error around populate_and_map() functions

 Fixed kbuild errors when IOMMU/SWIOTLB are disabled.

 Signed-off-by: Tom St Denis <tom.stde...@amd.com>
 Reviewed-by: Christian König <christian.koe...@amd.com>
 Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>

Not part of this series because we literally went through this before :-)

Tom


On 13/11/17 05:25 PM, kbuild test robot wrote:
> tree:   git://people.freedesktop.org/~agd5f/linux.git 
> upstream-4.14-drm-next-amd-dc-staging-chrome
> head:   4448b9a68413462529d018050cd246bc33957bd6
> commit: ed285b98008b667978d7faf348a22000b8a1c6b9 [4/16] drm/ttm: Add helper 
> functions to populate/map in one call (v2)
> config: i386-randconfig-s0-201746 (attached as .config)
> compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026
> reproduce:
>  git checkout ed285b98008b667978d7faf348a22000b8a1c6b9
>  # save the attached .config to linux build tree
>  make ARCH=i386
>
> All errors (new ones prefixed by >>):
>
>>> drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
>>> 'ttm_populate_and_map_pages'
>  int ttm_populate_and_map_pages(struct device *dev, struct ttm_dma_tt *tt)
>  ^~
> In file included from drivers/gpu//drm/ttm/ttm_page_alloc.c:49:0:
> include/drm/ttm/ttm_page_alloc.h:120:19: note: previous definition of 
> 'ttm_populate_and_map_pages' was here
>  static inline int ttm_populate_and_map_pages(struct device *dev, struct 
> ttm_dma_tt *tt)
>^~
>>> drivers/gpu//drm/ttm/ttm_page_alloc.c:950:6: error: redefinition of 
>>> 'ttm_unmap_and_unpopulate_pages'
>  void ttm_unmap_and_unpopulate_pages(struct device *dev, struct 
> ttm_dma_tt *tt)
>   ^~
> In file included from drivers/gpu//drm/ttm/ttm_page_alloc.c:49:0:
> include/drm/ttm/ttm_page_alloc.h:125:20: note: previous definition of 
> 'ttm_unmap_and_unpopulate_pages' was here
>  static inline void ttm_unmap_and_unpopulate_pages(struct device *dev, 
> struct ttm_dma_tt *tt)
> ^~
>
> vim +/ttm_populate_and_map_pages +923 drivers/gpu//drm/ttm/ttm_page_alloc.c
>
> 922
>   > 923   int ttm_populate_and_map_pages(struct device *dev, struct 
> ttm_dma_tt *tt)
> 924   {
> 925   unsigned i;
> 926   int r;
> 927
> 928   r = ttm_pool_populate(>ttm);
> 929   if (r)
> 930   return r;
> 931
> 932   for (i = 0; i < tt->ttm.num_pages; i++) {
> 933   tt->dma_address[i] = dma_map_page(dev, 
> tt->ttm.pages[i],
> 934 0, PAGE_SIZE,
> 935 
> DMA_BIDIRECTIONAL);
> 936   if (dma_mapping_error(dev, tt->dma_address[i])) 
> {
> 937   while (i--) {
> 938   dma_unmap_page(dev, 
> tt->dma_address[i],
> 939  PAGE_SIZE, 
> DMA_BIDIRECTIONAL);
> 940   tt->dma_address[i] = 0;
> 941   }
> 942   ttm_pool_unpopulate(>ttm);
> 943   return -EFAULT;
> 944   }
> 945   }
> 946   return 0;
> 947   }
> 948   EXPORT_SYMBOL(ttm_populate_and_map_pages);
> 949
>   > 950   void ttm_unmap_and_unpopulate_pages(struct device *dev, struct 
> ttm_dma_tt *tt)
> 951   {
> 952

Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 'ttm_populate_and_map_pages'

2017-11-14 Thread Tom St Denis


Is this:

commit 7a9667ae197460e6c9c3bb432fe68c708fce6259
Refs: v4.13-rc5-1195-g7a9667ae1974
Author: Tom St Denis 
AuthorDate: Tue Sep 5 07:30:59 2017 -0400
Commit: Alex Deucher 
CommitDate: Tue Sep 12 14:22:55 2017 -0400

drm/ttm: Fix configuration error around populate_and_map() functions

Fixed kbuild errors when IOMMU/SWIOTLB are disabled.

Signed-off-by: Tom St Denis 
Reviewed-by: Christian König 
Signed-off-by: Alex Deucher 

Not part of this series because we literally went through this before :-)

Tom


On 13/11/17 05:25 PM, kbuild test robot wrote:

tree:   git://people.freedesktop.org/~agd5f/linux.git 
upstream-4.14-drm-next-amd-dc-staging-chrome
head:   4448b9a68413462529d018050cd246bc33957bd6
commit: ed285b98008b667978d7faf348a22000b8a1c6b9 [4/16] drm/ttm: Add helper 
functions to populate/map in one call (v2)
config: i386-randconfig-s0-201746 (attached as .config)
compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026
reproduce:
 git checkout ed285b98008b667978d7faf348a22000b8a1c6b9
 # save the attached .config to linux build tree
 make ARCH=i386

All errors (new ones prefixed by >>):


drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
'ttm_populate_and_map_pages'

 int ttm_populate_and_map_pages(struct device *dev, struct ttm_dma_tt *tt)
 ^~
In file included from drivers/gpu//drm/ttm/ttm_page_alloc.c:49:0:
include/drm/ttm/ttm_page_alloc.h:120:19: note: previous definition of 
'ttm_populate_and_map_pages' was here
 static inline int ttm_populate_and_map_pages(struct device *dev, struct 
ttm_dma_tt *tt)
   ^~

drivers/gpu//drm/ttm/ttm_page_alloc.c:950:6: error: redefinition of 
'ttm_unmap_and_unpopulate_pages'

 void ttm_unmap_and_unpopulate_pages(struct device *dev, struct ttm_dma_tt 
*tt)
  ^~
In file included from drivers/gpu//drm/ttm/ttm_page_alloc.c:49:0:
include/drm/ttm/ttm_page_alloc.h:125:20: note: previous definition of 
'ttm_unmap_and_unpopulate_pages' was here
 static inline void ttm_unmap_and_unpopulate_pages(struct device *dev, 
struct ttm_dma_tt *tt)
^~

vim +/ttm_populate_and_map_pages +923 drivers/gpu//drm/ttm/ttm_page_alloc.c

922 
  > 923  int ttm_populate_and_map_pages(struct device *dev, struct 
ttm_dma_tt *tt)
924 {
925 unsigned i;
926 int r;
927 
928 r = ttm_pool_populate(>ttm);
929 if (r)
930 return r;
931 
932 for (i = 0; i < tt->ttm.num_pages; i++) {
933 tt->dma_address[i] = dma_map_page(dev, tt->ttm.pages[i],
934   0, PAGE_SIZE,
935   DMA_BIDIRECTIONAL);
936 if (dma_mapping_error(dev, tt->dma_address[i])) {
937 while (i--) {
938 dma_unmap_page(dev, tt->dma_address[i],
939PAGE_SIZE, 
DMA_BIDIRECTIONAL);
940 tt->dma_address[i] = 0;
941 }
942 ttm_pool_unpopulate(>ttm);
943 return -EFAULT;
944 }
945 }
946 return 0;
947 }
948 EXPORT_SYMBOL(ttm_populate_and_map_pages);
949 
  > 950  void ttm_unmap_and_unpopulate_pages(struct device *dev, struct 
ttm_dma_tt *tt)
951 {
952 unsigned i;
953 
954 for (i = 0; i < tt->ttm.num_pages; i++) {
955 if (tt->dma_address[i]) {
956 dma_unmap_page(dev, tt->dma_address[i],
957PAGE_SIZE, DMA_BIDIRECTIONAL);
958 }
959 }
960 ttm_pool_unpopulate(>ttm);
961 }
962 EXPORT_SYMBOL(ttm_unmap_and_unpopulate_pages);
963 

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



___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 'ttm_populate_and_map_pages'

2017-11-13 Thread kbuild test robot

tree:   git://people.freedesktop.org/~agd5f/linux.git 
upstream-4.14-drm-next-amd-dc-staging-chrome
head:   4448b9a68413462529d018050cd246bc33957bd6
commit: ed285b98008b667978d7faf348a22000b8a1c6b9 [4/16] drm/ttm: Add helper 
functions to populate/map in one call (v2)
config: i386-randconfig-s0-201746 (attached as .config)
compiler: gcc-6 (Debian 6.4.0-9) 6.4.0 20171026
reproduce:
git checkout ed285b98008b667978d7faf348a22000b8a1c6b9
# save the attached .config to linux build tree
make ARCH=i386 

All errors (new ones prefixed by >>):

>> drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 
>> 'ttm_populate_and_map_pages'
int ttm_populate_and_map_pages(struct device *dev, struct ttm_dma_tt *tt)
^~
   In file included from drivers/gpu//drm/ttm/ttm_page_alloc.c:49:0:
   include/drm/ttm/ttm_page_alloc.h:120:19: note: previous definition of 
'ttm_populate_and_map_pages' was here
static inline int ttm_populate_and_map_pages(struct device *dev, struct 
ttm_dma_tt *tt)
  ^~
>> drivers/gpu//drm/ttm/ttm_page_alloc.c:950:6: error: redefinition of 
>> 'ttm_unmap_and_unpopulate_pages'
void ttm_unmap_and_unpopulate_pages(struct device *dev, struct ttm_dma_tt 
*tt)
 ^~
   In file included from drivers/gpu//drm/ttm/ttm_page_alloc.c:49:0:
   include/drm/ttm/ttm_page_alloc.h:125:20: note: previous definition of 
'ttm_unmap_and_unpopulate_pages' was here
static inline void ttm_unmap_and_unpopulate_pages(struct device *dev, 
struct ttm_dma_tt *tt)
   ^~

vim +/ttm_populate_and_map_pages +923 drivers/gpu//drm/ttm/ttm_page_alloc.c

   922  
 > 923  int ttm_populate_and_map_pages(struct device *dev, struct ttm_dma_tt 
 > *tt)
   924  {
   925  unsigned i;
   926  int r;
   927  
   928  r = ttm_pool_populate(>ttm);
   929  if (r)
   930  return r;
   931  
   932  for (i = 0; i < tt->ttm.num_pages; i++) {
   933  tt->dma_address[i] = dma_map_page(dev, tt->ttm.pages[i],
   9340, PAGE_SIZE,
   935DMA_BIDIRECTIONAL);
   936  if (dma_mapping_error(dev, tt->dma_address[i])) {
   937  while (i--) {
   938  dma_unmap_page(dev, tt->dma_address[i],
   939 PAGE_SIZE, 
DMA_BIDIRECTIONAL);
   940  tt->dma_address[i] = 0;
   941  }
   942  ttm_pool_unpopulate(>ttm);
   943  return -EFAULT;
   944  }
   945  }
   946  return 0;
   947  }
   948  EXPORT_SYMBOL(ttm_populate_and_map_pages);
   949  
 > 950  void ttm_unmap_and_unpopulate_pages(struct device *dev, struct 
 > ttm_dma_tt *tt)
   951  {
   952  unsigned i;
   953  
   954  for (i = 0; i < tt->ttm.num_pages; i++) {
   955  if (tt->dma_address[i]) {
   956  dma_unmap_page(dev, tt->dma_address[i],
   957 PAGE_SIZE, DMA_BIDIRECTIONAL);
   958  }
   959  }
   960  ttm_pool_unpopulate(>ttm);
   961  }
   962  EXPORT_SYMBOL(ttm_unmap_and_unpopulate_pages);
   963  

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


.config.gz
Description: application/gzip
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 'ttm_populate_and_map_pages'

Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 'ttm_populate_and_map_pages'

Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 'ttm_populate_and_map_pages'

Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 'ttm_populate_and_map_pages'

Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 'ttm_populate_and_map_pages'

Re: [radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 'ttm_populate_and_map_pages'

[radeon-alex:upstream-4.14-drm-next-amd-dc-staging-chrome 4/16] drivers/gpu//drm/ttm/ttm_page_alloc.c:923:5: error: redefinition of 'ttm_populate_and_map_pages'

7 matches

Site Navigation

Mail list logo

Footer information