<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"> <meta name="Generator" content="Microsoft Exchange Server"> <style></style> </head> <body> <div> <div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black"> Thanks. </div> <div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black"> <div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black"> (typed on mobile, kindly ignore typos) </div> <div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black"> Regards, </div> <div dir="auto" style="direction:ltr; margin:0; padding:0; font-family:sans-serif; font-size:11pt; color:black"> Shirish S</div> </div> <hr tabindex="-1" style="display:inline-block; width:98%"> <div id="x_divRplyFwdMsg" dir="ltr">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' <div> </div> </div> </div> <div class="PlainText">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 >>> <a href="https://lists.01.org/pipermail/kbuild-all">https://lists.01.org/pipermail/kbuild-all</a> Intel Corporation >>> >> </div> </body> </html>