On systems with STACKTRACE_SUPPORT=n, we get:
WARNING: unmet direct dependencies detected for STACKTRACE
Depends on [n]: STACKTRACE_SUPPORT
Selected by [y]:
- STACKDEPOT [=y]
and build errors such as:
m68k-linux-ld: kernel/stacktrace.o: in function `stack_trace_save':
(.text+0x11c): undefined reference to `save_stack_trace'
Add the missing deendency on STACKTRACE_SUPPORT.
Fixes: 12a280c72868 ("drm/dp_mst: Add topology ref history tracking for debugging")
Cc: Lyude Paul <[email protected]>
Cc: Sean Paul <[email protected]>
Signed-off-by: Guenter Roeck <[email protected]>
---
drivers/gpu/drm/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig
index 1168351267fd..bfdadc3667e0 100644
--- a/drivers/gpu/drm/Kconfig
+++ b/drivers/gpu/drm/Kconfig
@@ -95,6 +95,7 @@ config DRM_KMS_FB_HELPER
config DRM_DEBUG_DP_MST_TOPOLOGY_REFS
bool "Enable refcount backtrace history in the DP MST helpers"
+ depends on STACKTRACE_SUPPORT
select STACKDEPOT
depends on DRM_KMS_HELPER
depends on DEBUG_KERNEL
--
2.17.1
On Mon, Dec 2, 2019 at 2:41 PM Guenter Roeck <[email protected]> wrote:
> On systems with STACKTRACE_SUPPORT=n, we get:
>
> WARNING: unmet direct dependencies detected for STACKTRACE
> Depends on [n]: STACKTRACE_SUPPORT
> Selected by [y]:
> - STACKDEPOT [=y]
>
> and build errors such as:
>
> m68k-linux-ld: kernel/stacktrace.o: in function `stack_trace_save':
> (.text+0x11c): undefined reference to `save_stack_trace'
>
> Add the missing deendency on STACKTRACE_SUPPORT.
>
> Fixes: 12a280c72868 ("drm/dp_mst: Add topology ref history tracking for debugging")
> Cc: Lyude Paul <[email protected]>
> Cc: Sean Paul <[email protected]>
> Signed-off-by: Guenter Roeck <[email protected]>
Acked-by: Geert Uytterhoeven <[email protected]>
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
Reviewed-by: Lyude Paul <[email protected]>
I'll go ahead and push this to drm-misc-next, thanks!
On Mon, 2019-12-02 at 16:20 +0100, Geert Uytterhoeven wrote:
> On Mon, Dec 2, 2019 at 2:41 PM Guenter Roeck <[email protected]> wrote:
> > On systems with STACKTRACE_SUPPORT=n, we get:
> >
> > WARNING: unmet direct dependencies detected for STACKTRACE
> > Depends on [n]: STACKTRACE_SUPPORT
> > Selected by [y]:
> > - STACKDEPOT [=y]
> >
> > and build errors such as:
> >
> > m68k-linux-ld: kernel/stacktrace.o: in function `stack_trace_save':
> > (.text+0x11c): undefined reference to `save_stack_trace'
> >
> > Add the missing deendency on STACKTRACE_SUPPORT.
> >
> > Fixes: 12a280c72868 ("drm/dp_mst: Add topology ref history tracking for
> > debugging")
> > Cc: Lyude Paul <[email protected]>
> > Cc: Sean Paul <[email protected]>
> > Signed-off-by: Guenter Roeck <[email protected]>
>
> Acked-by: Geert Uytterhoeven <[email protected]>
>
> Gr{oetje,eeting}s,
>
> Geert
>
--
Cheers,
Lyude Paul
On Mon, Dec 02, 2019 at 01:49:47PM -0500, Lyude Paul wrote:
> Reviewed-by: Lyude Paul <[email protected]>
>
> I'll go ahead and push this to drm-misc-next, thanks!
drm-misc-next-fixes since it's in the merge window. drm-misc-next is for
5.6 already.
-Daniel
>
> On Mon, 2019-12-02 at 16:20 +0100, Geert Uytterhoeven wrote:
> > On Mon, Dec 2, 2019 at 2:41 PM Guenter Roeck <[email protected]> wrote:
> > > On systems with STACKTRACE_SUPPORT=n, we get:
> > >
> > > WARNING: unmet direct dependencies detected for STACKTRACE
> > > Depends on [n]: STACKTRACE_SUPPORT
> > > Selected by [y]:
> > > - STACKDEPOT [=y]
> > >
> > > and build errors such as:
> > >
> > > m68k-linux-ld: kernel/stacktrace.o: in function `stack_trace_save':
> > > (.text+0x11c): undefined reference to `save_stack_trace'
> > >
> > > Add the missing deendency on STACKTRACE_SUPPORT.
> > >
> > > Fixes: 12a280c72868 ("drm/dp_mst: Add topology ref history tracking for
> > > debugging")
> > > Cc: Lyude Paul <[email protected]>
> > > Cc: Sean Paul <[email protected]>
> > > Signed-off-by: Guenter Roeck <[email protected]>
> >
> > Acked-by: Geert Uytterhoeven <[email protected]>
> >
> > Gr{oetje,eeting}s,
> >
> > Geert
> >
> --
> Cheers,
> Lyude Paul
>
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
On Tue, 2019-12-03 at 10:33 +0100, Daniel Vetter wrote:
> On Mon, Dec 02, 2019 at 01:49:47PM -0500, Lyude Paul wrote:
> > Reviewed-by: Lyude Paul <[email protected]>
> >
> > I'll go ahead and push this to drm-misc-next, thanks!
>
> drm-misc-next-fixes since it's in the merge window. drm-misc-next is for
> 5.6 already.
> -Daniel
Ahh, that explains things :). Pushed, thanks for the patches!
>
> > On Mon, 2019-12-02 at 16:20 +0100, Geert Uytterhoeven wrote:
> > > On Mon, Dec 2, 2019 at 2:41 PM Guenter Roeck <[email protected]> wrote:
> > > > On systems with STACKTRACE_SUPPORT=n, we get:
> > > >
> > > > WARNING: unmet direct dependencies detected for STACKTRACE
> > > > Depends on [n]: STACKTRACE_SUPPORT
> > > > Selected by [y]:
> > > > - STACKDEPOT [=y]
> > > >
> > > > and build errors such as:
> > > >
> > > > m68k-linux-ld: kernel/stacktrace.o: in function `stack_trace_save':
> > > > (.text+0x11c): undefined reference to `save_stack_trace'
> > > >
> > > > Add the missing deendency on STACKTRACE_SUPPORT.
> > > >
> > > > Fixes: 12a280c72868 ("drm/dp_mst: Add topology ref history tracking
> > > > for
> > > > debugging")
> > > > Cc: Lyude Paul <[email protected]>
> > > > Cc: Sean Paul <[email protected]>
> > > > Signed-off-by: Guenter Roeck <[email protected]>
> > >
> > > Acked-by: Geert Uytterhoeven <[email protected]>
> > >
> > > Gr{oetje,eeting}s,
> > >
> > > Geert
> > >
> > --
> > Cheers,
> > Lyude Paul
> >
--
Cheers,
Lyude Paul