* [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 22:25 kbuild test robot
2017-11-14 11:28 ` Tom St Denis
0 siblings, 1 reply; 7+ messages in thread
From: kbuild test robot @ 2017-11-13 22:25 UTC (permalink / raw)
To: Tom St Denis
Cc: Alex Deucher, Christian König, kbuild-all, dri-devel, Shirish S
[-- Attachment #1: Type: text/plain, Size: 3146 bytes --]
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(&tt->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(&tt->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(&tt->ttm);
961 }
962 EXPORT_SYMBOL(ttm_unmap_and_unpopulate_pages);
963
---
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: 30991 bytes --]
[-- Attachment #3: Type: text/plain, Size: 160 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* 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-13 22:25 [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' kbuild test robot
@ 2017-11-14 11:28 ` Tom St Denis
2017-11-14 13:07 ` S, Shirish
0 siblings, 1 reply; 7+ messages in thread
From: Tom St Denis @ 2017-11-14 11:28 UTC (permalink / raw)
To: kbuild test robot
Cc: Alex Deucher, Christian König, kbuild-all, dri-devel, Shirish S
Is this:
commit 7a9667ae197460e6c9c3bb432fe68c708fce6259
Refs: v4.13-rc5-1195-g7a9667ae1974
Author: Tom St Denis <tom.stdenis@amd.com>
AuthorDate: Tue Sep 5 07:30:59 2017 -0400
Commit: Alex Deucher <alexander.deucher@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.stdenis@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@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(&tt->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(&tt->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(&tt->ttm);
> 961 }
> 962 EXPORT_SYMBOL(ttm_unmap_and_unpopulate_pages);
> 963
>
> ---
> 0-DAY kernel test infrastructure Open 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
^ permalink raw reply [flat|nested] 7+ messages in thread
* 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 11:28 ` Tom St Denis
@ 2017-11-14 13:07 ` S, Shirish
2017-11-14 13:11 ` Fengguang Wu
0 siblings, 1 reply; 7+ messages in thread
From: S, Shirish @ 2017-11-14 13:07 UTC (permalink / raw)
To: kbuild test robot, StDenis, Tom
Cc: Deucher, Alexander, kbuild-all, dri-devel, Koenig, Christian
[-- Attachment #1.1: Type: text/plain, Size: 5436 bytes --]
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-all@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.stdenis@amd.com>
AuthorDate: Tue Sep 5 07:30:59 2017 -0400
Commit: Alex Deucher <alexander.deucher@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.stdenis@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@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(&tt->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(&tt->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(&tt->ttm);
> 961 }
> 962 EXPORT_SYMBOL(ttm_unmap_and_unpopulate_pages);
> 963
>
> ---
> 0-DAY kernel test infrastructure Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all Intel Corporation
>
[-- Attachment #1.2: Type: text/html, Size: 13578 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* 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 13:07 ` S, Shirish
@ 2017-11-14 13:11 ` Fengguang Wu
2017-11-14 13:13 ` S, Shirish
0 siblings, 1 reply; 7+ messages in thread
From: Fengguang Wu @ 2017-11-14 13:11 UTC (permalink / raw)
To: S, Shirish
Cc: StDenis, Tom, Deucher, Alexander, kbuild-all, dri-devel, Koenig,
Christian
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 +0000, 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-all@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.stdenis@amd.com>
>AuthorDate: Tue Sep 5 07:30:59 2017 -0400
>Commit: Alex Deucher <alexander.deucher@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.stdenis@amd.com>
> Reviewed-by: Christian König <christian.koenig@amd.com>
> Signed-off-by: Alex Deucher <alexander.deucher@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(&tt->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(&tt->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(&tt->ttm);
>> 961 }
>> 962 EXPORT_SYMBOL(ttm_unmap_and_unpopulate_pages);
>> 963
>>
>> ---
>> 0-DAY kernel test infrastructure Open 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
^ permalink raw reply [flat|nested] 7+ messages in thread
* 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 13:11 ` Fengguang Wu
@ 2017-11-14 13:13 ` S, Shirish
2017-11-14 13:22 ` Fengguang Wu
0 siblings, 1 reply; 7+ messages in thread
From: S, Shirish @ 2017-11-14 13:13 UTC (permalink / raw)
To: Fengguang Wu
Cc: StDenis, Tom, Deucher, Alexander, kbuild-all, dri-devel, Koenig,
Christian
[-- Attachment #1.1: Type: text/plain, Size: 6422 bytes --]
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.wu@intel.com>
Sent: Tuesday, November 14, 2017 6:41:41 PM
To: S, Shirish
Cc: StDenis, Tom; kbuild-all@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 +0000, 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-all@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.stdenis@amd.com>
>AuthorDate: Tue Sep 5 07:30:59 2017 -0400
>Commit: Alex Deucher <alexander.deucher@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.stdenis@amd.com>
> Reviewed-by: Christian König <christian.koenig@amd.com>
> Signed-off-by: Alex Deucher <alexander.deucher@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(&tt->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(&tt->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(&tt->ttm);
>> 961 }
>> 962 EXPORT_SYMBOL(ttm_unmap_and_unpopulate_pages);
>> 963
>>
>> ---
>> 0-DAY kernel test infrastructure Open Source Technology Center
>> https://lists.01.org/pipermail/kbuild-all Intel Corporation
>>
>
[-- Attachment #1.2: Type: text/html, Size: 14615 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
* 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 13:13 ` S, Shirish
@ 2017-11-14 13:22 ` Fengguang Wu
2017-11-14 13:27 ` S, Shirish
0 siblings, 1 reply; 7+ messages in thread
From: Fengguang Wu @ 2017-11-14 13:22 UTC (permalink / raw)
To: S, Shirish
Cc: StDenis, Tom, Deucher, Alexander, kbuild-all, dri-devel, Koenig,
Christian
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 +0000, 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.wu@intel.com>
>Sent: Tuesday, November 14, 2017 6:41:41 PM
>To: S, Shirish
>Cc: StDenis, Tom; kbuild-all@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 +0000, 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-all@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.stdenis@amd.com>
>>AuthorDate: Tue Sep 5 07:30:59 2017 -0400
>>Commit: Alex Deucher <alexander.deucher@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.stdenis@amd.com>
>> Reviewed-by: Christian König <christian.koenig@amd.com>
>> Signed-off-by: Alex Deucher <alexander.deucher@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(&tt->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(&tt->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(&tt->ttm);
>>> 961 }
>>> 962 EXPORT_SYMBOL(ttm_unmap_and_unpopulate_pages);
>>> 963
>>>
>>> ---
>>> 0-DAY kernel test infrastructure Open 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
^ permalink raw reply [flat|nested] 7+ messages in thread
* 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 13:22 ` Fengguang Wu
@ 2017-11-14 13:27 ` S, Shirish
0 siblings, 0 replies; 7+ messages in thread
From: S, Shirish @ 2017-11-14 13:27 UTC (permalink / raw)
To: Fengguang Wu
Cc: StDenis, Tom, Deucher, Alexander, kbuild-all, dri-devel, Koenig,
Christian
[-- Attachment #1.1: Type: text/plain, Size: 8395 bytes --]
Thanks.
(typed on mobile, kindly ignore typos)
Regards,
Shirish S
________________________________
From: Fengguang Wu <fengguang.wu@intel.com>
Sent: Tuesday, November 14, 2017 6:52:03 PM
To: S, Shirish
Cc: StDenis, Tom; kbuild-all@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 +0000, 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.wu@intel.com>
>Sent: Tuesday, November 14, 2017 6:41:41 PM
>To: S, Shirish
>Cc: StDenis, Tom; kbuild-all@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 +0000, 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-all@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.stdenis@amd.com>
>>AuthorDate: Tue Sep 5 07:30:59 2017 -0400
>>Commit: Alex Deucher <alexander.deucher@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.stdenis@amd.com>
>> Reviewed-by: Christian König <christian.koenig@amd.com>
>> Signed-off-by: Alex Deucher <alexander.deucher@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(&tt->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(&tt->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(&tt->ttm);
>>> 961 }
>>> 962 EXPORT_SYMBOL(ttm_unmap_and_unpopulate_pages);
>>> 963
>>>
>>> ---
>>> 0-DAY kernel test infrastructure Open Source Technology Center
>>> https://lists.01.org/pipermail/kbuild-all Intel Corporation
>>>
>>
[-- Attachment #1.2: Type: text/html, Size: 17285 bytes --]
[-- Attachment #2: Type: text/plain, Size: 160 bytes --]
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-11-14 13:46 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-11-13 22:25 [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' kbuild test robot
2017-11-14 11:28 ` Tom St Denis
2017-11-14 13:07 ` S, Shirish
2017-11-14 13:11 ` Fengguang Wu
2017-11-14 13:13 ` S, Shirish
2017-11-14 13:22 ` Fengguang Wu
2017-11-14 13:27 ` S, Shirish
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.