Re: [Freedreno] [PATCH] drm/msm: fix link error without CONFIG_DEBUG_FS
On Tue, Jun 9, 2020 at 4:04 PM Rob Clark wrote: > On Tue, Jun 9, 2020 at 5:48 AM Linus Walleij wrote: > > On Tue, May 5, 2020 at 10:27 AM Linus Walleij > > wrote: > > > On Wed, Apr 8, 2020 at 9:15 PM Arnd Bergmann wrote: > > > > > > > I ran into a randconfig link error with debugfs disabled: > > > > > > > > arm-linux-gnueabi-ld: > > > > drivers/gpu/drm/msm/msm_gpu.o: in function `should_dump': > > > > msm_gpu.c:(.text+0x1cc): undefined reference to `rd_full' > > > > > > > > Change the helper to only look at this variable if debugfs is present. > > > > > > > > Fixes: e515af8d4a6f ("drm/msm: devcoredump should dump > > > > MSM_SUBMIT_BO_DUMP buffers") > > > > Signed-off-by: Arnd Bergmann > > > > > > This fixes a compilation error for me on the APQ8060. > > > Tested-by: Linus Walleij > > > > Could someone be so kind and apply this fix to the MSM DRM tree? > > > > This should be fixed by 20aebe83698feb107d5a66b6cfd1d54459ccdfcf in > msm-next/drm-next, are you still seeing this issue? Oh nice this works too, I was just rebasing my trees and the patch was still there, of course I didn't see that anothe patch fixed the same issue already. Thanks Rob! Linus Walleij ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno
Re: [Freedreno] [PATCH] drm/msm: fix link error without CONFIG_DEBUG_FS
On Tue, Jun 9, 2020 at 5:48 AM Linus Walleij wrote: > > On Tue, May 5, 2020 at 10:27 AM Linus Walleij > wrote: > > On Wed, Apr 8, 2020 at 9:15 PM Arnd Bergmann wrote: > > > > > I ran into a randconfig link error with debugfs disabled: > > > > > > arm-linux-gnueabi-ld: > > > drivers/gpu/drm/msm/msm_gpu.o: in function `should_dump': > > > msm_gpu.c:(.text+0x1cc): undefined reference to `rd_full' > > > > > > Change the helper to only look at this variable if debugfs is present. > > > > > > Fixes: e515af8d4a6f ("drm/msm: devcoredump should dump MSM_SUBMIT_BO_DUMP > > > buffers") > > > Signed-off-by: Arnd Bergmann > > > > This fixes a compilation error for me on the APQ8060. > > Tested-by: Linus Walleij > > Could someone be so kind and apply this fix to the MSM DRM tree? > This should be fixed by 20aebe83698feb107d5a66b6cfd1d54459ccdfcf in msm-next/drm-next, are you still seeing this issue? BR, -R ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno
Re: [Freedreno] [PATCH] drm/msm: fix link error without CONFIG_DEBUG_FS
On Tue, May 5, 2020 at 10:27 AM Linus Walleij wrote: > On Wed, Apr 8, 2020 at 9:15 PM Arnd Bergmann wrote: > > > I ran into a randconfig link error with debugfs disabled: > > > > arm-linux-gnueabi-ld: > > drivers/gpu/drm/msm/msm_gpu.o: in function `should_dump': > > msm_gpu.c:(.text+0x1cc): undefined reference to `rd_full' > > > > Change the helper to only look at this variable if debugfs is present. > > > > Fixes: e515af8d4a6f ("drm/msm: devcoredump should dump MSM_SUBMIT_BO_DUMP > > buffers") > > Signed-off-by: Arnd Bergmann > > This fixes a compilation error for me on the APQ8060. > Tested-by: Linus Walleij Could someone be so kind and apply this fix to the MSM DRM tree? Yours, Linus Walleij ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno
Re: [Freedreno] [PATCH] drm/msm: fix link error without CONFIG_DEBUG_FS
On Wed, Apr 8, 2020 at 9:15 PM Arnd Bergmann wrote: > I ran into a randconfig link error with debugfs disabled: > > arm-linux-gnueabi-ld: > drivers/gpu/drm/msm/msm_gpu.o: in function `should_dump': > msm_gpu.c:(.text+0x1cc): undefined reference to `rd_full' > > Change the helper to only look at this variable if debugfs is present. > > Fixes: e515af8d4a6f ("drm/msm: devcoredump should dump MSM_SUBMIT_BO_DUMP > buffers") > Signed-off-by: Arnd Bergmann This fixes a compilation error for me on the APQ8060. Tested-by: Linus Walleij Yours, Linus Walleij ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno
[Freedreno] [PATCH] drm/msm: fix link error without CONFIG_DEBUG_FS
I ran into a randconfig link error with debugfs disabled: arm-linux-gnueabi-ld: drivers/gpu/drm/msm/msm_gpu.o: in function `should_dump': msm_gpu.c:(.text+0x1cc): undefined reference to `rd_full' Change the helper to only look at this variable if debugfs is present. Fixes: e515af8d4a6f ("drm/msm: devcoredump should dump MSM_SUBMIT_BO_DUMP buffers") Signed-off-by: Arnd Bergmann --- drivers/gpu/drm/msm/msm_gem.h | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/msm/msm_gem.h b/drivers/gpu/drm/msm/msm_gem.h index 30584eaf8cc8..eb4aeaf291ea 100644 --- a/drivers/gpu/drm/msm/msm_gem.h +++ b/drivers/gpu/drm/msm/msm_gem.h @@ -167,7 +167,10 @@ static inline bool should_dump(struct msm_gem_submit *submit, int idx) { extern bool rd_full; - return rd_full || (submit->bos[idx].flags & MSM_SUBMIT_BO_DUMP); + if (IS_ENABLED(CONFIG_DEBUG_FS) && rd_full) + return true; + + return submit->bos[idx].flags & MSM_SUBMIT_BO_DUMP; } #endif /* __MSM_GEM_H__ */ -- 2.26.0 ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno