<html> <head> <meta http-equiv="Content-Type" content="text/html; charset=windows-1252"> </head> <body text="#000000" bgcolor="#FFFFFF"> <div class="moz-cite-prefix">That's a completely different issue. I'm running into it with just a Tonga in my system. According to my bisecting the problematic patch seems to be: <blockquote type="cite">commit 3fdadbfe1edf4168b2515083af5651c95aeb299d (refs/bisect/bad) Author: Rex Zhu <a class="moz-txt-link-rfc2396E" href="mailto:rex.zhu@amd.com"><rex.zhu@amd.com></a> Date: Thu Jul 5 19:22:50 2018 +0800 drm/amd/pp: Implement get_performance_level for legacy dgpu display can get clock info through this function. implement this function for vega10 and old asics. from vega12, there is no power state management, so need to add new interface to notify display the clock info Reviewed-by: Alex Deucher <a class="moz-txt-link-rfc2396E" href="mailto:alexander.deucher@amd.com"><alexander.deucher@amd.com></a> Signed-off-by: Rex Zhu <a class="moz-txt-link-rfc2396E" href="mailto:Rex.Zhu@amd.com"><Rex.Zhu@amd.com></a> </blockquote> Please fix and/or revert immediately, cause that is quite a show stopper for my testing. Thanks, Christian. Am 07.07.2018 um 03:51 schrieb Zhu, Rex: </div> <blockquote type="cite" cite="mid:CY4PR12MB1687241A8A0FF74C10E93985FB460@CY4PR12MB1687.namprd12.prod.outlook.com"> <meta http-equiv="Content-Type" content="text/html; charset=windows-1252"> <style type="text/css" style="display:none;"></style> <div id="divtagdefaultwrapper" style="font-size: 12pt; color: rgb(0, 0, 0); font-family: Calibri, Helvetica, sans-serif, EmojiFont, "Apple Color Emoji", "Segoe UI Emoji", NotoColorEmoji, "Segoe UI Symbol", "Android Emoji", EmojiSymbols;" dir="ltr"> Add Michel, seems Michel found a similar issue. Please see the kern.log. Best Regards Rex <div style="color: rgb(0, 0, 0);"> <hr style="display:inline-block;width:98%" tabindex="-1"> <div id="divRplyFwdMsg" dir="ltr">From: amd-gfx <a class="moz-txt-link-rfc2396E" href="mailto:amd-gfx-bounces@lists.freedesktop.org"><amd-gfx-bounces@lists.freedesktop.org></a> on behalf of Harry Wentland <a class="moz-txt-link-rfc2396E" href="mailto:harry.wentland@amd.com"><harry.wentland@amd.com></a> Sent: Friday, July 6, 2018 9:32 PM To: Christian König; <a class="moz-txt-link-abbreviated" href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> Cc: Lipski, Mikita Subject: Re: [PATCH 2/2] drm/amd/display: make function tables const <div> </div> </div> <div class="BodyFragment"> <div class="PlainText">On 2018-07-06 08:22 AM, Christian König wrote: > It is good practice to make global function tables const to avoid > accidental override. > > Signed-off-by: Christian König <a class="moz-txt-link-rfc2396E" href="mailto:christian.koenig@amd.com"><christian.koenig@amd.com></a> Well, this is embarassing. Can't believe I didn't spot this. Series is Reviewed-by: Harry Wentland <a class="moz-txt-link-rfc2396E" href="mailto:harry.wentland@amd.com"><harry.wentland@amd.com></a> Now I wonder where else we might have this issue. Time to review all of our other function pointers. Harry > --- > drivers/gpu/drm/amd/display/dc/dce/dce_mem_input.c | 6 +++--- > drivers/gpu/drm/amd/display/dc/dce110/dce110_mem_input_v.c | 2 +- > drivers/gpu/drm/amd/display/dc/inc/hw/mem_input.h | 2 +- > 3 files changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/dc/dce/dce_mem_input.c b/drivers/gpu/drm/amd/display/dc/dce/dce_mem_input.c > index bae752332a9f..85686d917636 100644 > --- a/drivers/gpu/drm/amd/display/dc/dce/dce_mem_input.c > +++ b/drivers/gpu/drm/amd/display/dc/dce/dce_mem_input.c > @@ -729,7 +729,7 @@ static bool dce_mi_program_surface_flip_and_addr( > return true; > } > > -static struct mem_input_funcs dce_mi_funcs = { > +static const struct mem_input_funcs dce_mi_funcs = { > .mem_input_program_display_marks = dce_mi_program_display_marks, > .allocate_mem_input = dce_mi_allocate_dmif, > .free_mem_input = dce_mi_free_dmif, > @@ -741,7 +741,7 @@ static struct mem_input_funcs dce_mi_funcs = { > .mem_input_is_flip_pending = dce_mi_is_flip_pending > }; > > -static struct mem_input_funcs dce112_mi_funcs = { > +static const struct mem_input_funcs dce112_mi_funcs = { > .mem_input_program_display_marks = dce112_mi_program_display_marks, > .allocate_mem_input = dce_mi_allocate_dmif, > .free_mem_input = dce_mi_free_dmif, > @@ -753,7 +753,7 @@ static struct mem_input_funcs dce112_mi_funcs = { > .mem_input_is_flip_pending = dce_mi_is_flip_pending > }; > > -static struct mem_input_funcs dce120_mi_funcs = { > +static const struct mem_input_funcs dce120_mi_funcs = { > .mem_input_program_display_marks = dce120_mi_program_display_marks, > .allocate_mem_input = dce_mi_allocate_dmif, > .free_mem_input = dce_mi_free_dmif, > diff --git a/drivers/gpu/drm/amd/display/dc/dce110/dce110_mem_input_v.c b/drivers/gpu/drm/amd/display/dc/dce110/dce110_mem_input_v.c > index 0564c8e31252..9b9fc3d96c07 100644 > --- a/drivers/gpu/drm/amd/display/dc/dce110/dce110_mem_input_v.c > +++ b/drivers/gpu/drm/amd/display/dc/dce110/dce110_mem_input_v.c > @@ -1011,7 +1011,7 @@ void dce110_free_mem_input_v( > { > } > > -static struct mem_input_funcs dce110_mem_input_v_funcs = { > +static const struct mem_input_funcs dce110_mem_input_v_funcs = { > .mem_input_program_display_marks = > dce_mem_input_v_program_display_marks, > .mem_input_program_chroma_display_marks = > diff --git a/drivers/gpu/drm/amd/display/dc/inc/hw/mem_input.h b/drivers/gpu/drm/amd/display/dc/inc/hw/mem_input.h > index 47f1dc5a43b7..da89c2edb07c 100644 > --- a/drivers/gpu/drm/amd/display/dc/inc/hw/mem_input.h > +++ b/drivers/gpu/drm/amd/display/dc/inc/hw/mem_input.h > @@ -64,7 +64,7 @@ struct stutter_modes { > }; > > struct mem_input { > - struct mem_input_funcs *funcs; > + const struct mem_input_funcs *funcs; > struct dc_context *ctx; > struct dc_plane_address request_address; > struct dc_plane_address current_address; > _______________________________________________ amd-gfx mailing list <a class="moz-txt-link-abbreviated" href="mailto:amd-gfx@lists.freedesktop.org">amd-gfx@lists.freedesktop.org</a> <a href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx" id="LPlnk757106" class="OWAAutoLink" previewremoved="true" moz-do-not-send="true">https://lists.freedesktop.org/mailman/listinfo/amd-gfx</a> <div id="LPBorder_GT_15309280054370.5139627939483356" style="margin-bottom: 20px; overflow: auto; width: 100%; text-indent: 0px;"> <table id="LPContainer_15309280054340.885055010626419" role="presentation" style="width: 90%; background-color: rgb(255, 255, 255); position: relative; overflow: auto; padding-top: 20px; padding-bottom: 20px; margin-top: 20px; border-top: 1px dotted rgb(200, 200, 200); border-bottom: 1px dotted rgb(200, 200, 200);" cellspacing="0"> <tbody> <tr style="border-spacing: 0px;" valign="top"> <td id="TextCell_15309280054350.09207141975106015" colspan="2" style="vertical-align: top; position: relative; padding: 0px; display: table-cell;"> <div id="LPTitle_15309280054360.5322745038679628" style="top: 0px; color: rgb(0, 120, 215); font-weight: normal; font-size: 21px; font-family: wf_segoe-ui_light, "Segoe UI Light", "Segoe WP Light", "Segoe UI", "Segoe WP", Tahoma, Arial, sans-serif; line-height: 21px;"> <a id="LPUrlAnchor_15309280054360.6263051860853015" href="https://lists.freedesktop.org/mailman/listinfo/amd-gfx" target="_blank" style="text-decoration: none;" moz-do-not-send="true">amd-gfx Info Page - freedesktop.org</a></div> <div id="LPMetadata_15309280054360.6664147003914225" style="margin: 10px 0px 16px; color: rgb(102, 102, 102); font-weight: normal; font-family: wf_segoe-ui_normal, "Segoe UI", "Segoe WP", Tahoma, Arial, sans-serif; font-size: 14px; line-height: 14px;"> lists.freedesktop.org</div> <div id="LPDescription_15309280054370.5356504301440415" style="display: block; color: rgb(102, 102, 102); font-weight: normal; font-family: wf_segoe-ui_normal, "Segoe UI", "Segoe WP", Tahoma, Arial, sans-serif; font-size: 14px; line-height: 20px; max-height: 100px; overflow: hidden;"> Subscribing to amd-gfx: Subscribe to amd-gfx by filling out the following form. Use of all freedesktop.org lists is subject to our Code of Conduct.</div> </td> </tr> </tbody> </table> </div> </div> </div> </div> </div> </blockquote> </body> </html>