All of lore.kernel.org
 help / color / mirror / Atom feed
* [linux-stable-rc:linux-5.6.y 1567/1864] drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:2623:29: warning: variable 'stream_status' set but not used
@ 2020-06-17 13:18 kernel test robot
  0 siblings, 0 replies; 4+ messages in thread
From: kernel test robot @ 2020-06-17 13:18 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 7147 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.6.y
head:   b60e06c9887321691c8d341e86c085ed3a6a4138
commit: 5fc20b04d12c0827dbf498532e8a736a1f88ad6f [1567/1864] drm/amd/display: Split program front end part that occur outside lock
config: i386-randconfig-s001-20200617 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-13) 9.3.0
reproduce:
        # apt-get install sparse
        # sparse version: v0.6.2-rc1-6-g78f577f8-dirty
        git checkout 5fc20b04d12c0827dbf498532e8a736a1f88ad6f
        # save the attached .config to linux build tree
        make W=1 C=1 ARCH=i386 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>, old ones prefixed by <<):

drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:72:6: warning: no previous prototype for 'print_microsec' [-Wmissing-prototypes]
72 | void print_microsec(struct dc_context *dc_ctx,
|      ^~~~~~~~~~~~~~
drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:126:6: warning: no previous prototype for 'dcn10_log_hubbub_state' [-Wmissing-prototypes]
126 | void dcn10_log_hubbub_state(struct dc *dc, struct dc_log_buffer_ctx *log_ctx)
|      ^~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:1878:6: warning: no previous prototype for 'dcn10_program_pte_vm' [-Wmissing-prototypes]
1878 | void dcn10_program_pte_vm(struct dce_hwseq *hws, struct hubp *hubp)
|      ^~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c: In function 'dcn10_post_unlock_program_front_end':
>> drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:2623:29: warning: variable 'stream_status' set but not used [-Wunused-but-set-variable]
2623 |    struct dc_stream_status *stream_status = NULL;
|                             ^~~~~~~~~~~~~
In file included from drivers/gpu/drm/amd/amdgpu/../display/dc/inc/core_types.h:85,
from drivers/gpu/drm/amd/amdgpu/../display/dc/basics/dc_common.h:29,
from drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:28:
At top level:
drivers/gpu/drm/amd/amdgpu/../display/dc/inc/hw/dpp.h:50:42: warning: 'dpp_input_csc_matrix' defined but not used [-Wunused-const-variable=]
50 | static const struct dpp_input_csc_matrix dpp_input_csc_matrix[] = {
|                                          ^~~~~~~~~~~~~~~~~~~~
In file included from drivers/gpu/drm/amd/amdgpu/../display/dc/inc/core_types.h:32,
from drivers/gpu/drm/amd/amdgpu/../display/dc/basics/dc_common.h:29,
from drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:28:
drivers/gpu/drm/amd/amdgpu/../display/include/ddc_service_types.h:124:22: warning: 'DP_DVI_CONVERTER_ID_4' defined but not used [-Wunused-const-variable=]
124 | static const uint8_t DP_DVI_CONVERTER_ID_4[] = "m2DVIa";
|                      ^~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/amd/amdgpu/../display/include/ddc_service_types.h:122:22: warning: 'DP_VGA_LVDS_CONVERTER_ID_3' defined but not used [-Wunused-const-variable=]
122 | static const uint8_t DP_VGA_LVDS_CONVERTER_ID_3[] = "dnomlA";
|                      ^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/amd/amdgpu/../display/include/ddc_service_types.h:120:22: warning: 'DP_VGA_LVDS_CONVERTER_ID_2' defined but not used [-Wunused-const-variable=]
120 | static const uint8_t DP_VGA_LVDS_CONVERTER_ID_2[] = "sivarT";
|                      ^~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from drivers/gpu/drm/amd/amdgpu/../display/dc/dc_types.h:33,
from drivers/gpu/drm/amd/amdgpu/../display/dc/dm_services_types.h:30,
from drivers/gpu/drm/amd/amdgpu/../display/dc/dm_services.h:37,
from drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:27:
drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:76:32: warning: 'dc_fixpt_ln2_div_2' defined but not used [-Wunused-const-variable=]
76 | static const struct fixed31_32 dc_fixpt_ln2_div_2 = { 1488522236LL };
|                                ^~~~~~~~~~~~~~~~~~
drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:75:32: warning: 'dc_fixpt_ln2' defined but not used [-Wunused-const-variable=]
75 | static const struct fixed31_32 dc_fixpt_ln2 = { 2977044471LL };
|                                ^~~~~~~~~~~~
drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:74:32: warning: 'dc_fixpt_e' defined but not used [-Wunused-const-variable=]
74 | static const struct fixed31_32 dc_fixpt_e = { 11674931555LL };
|                                ^~~~~~~~~~
drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:73:32: warning: 'dc_fixpt_two_pi' defined but not used [-Wunused-const-variable=]
73 | static const struct fixed31_32 dc_fixpt_two_pi = { 26986075409LL };
|                                ^~~~~~~~~~~~~~~
drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:72:32: warning: 'dc_fixpt_pi' defined but not used [-Wunused-const-variable=]
72 | static const struct fixed31_32 dc_fixpt_pi = { 13493037705LL };
|                                ^~~~~~~~~~~
drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:67:32: warning: 'dc_fixpt_zero' defined but not used [-Wunused-const-variable=]
67 | static const struct fixed31_32 dc_fixpt_zero = { 0 };
|                                ^~~~~~~~~~~~~

vim +/stream_status +2623 drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c

  2608	
  2609	void dcn10_post_unlock_program_front_end(
  2610			struct dc *dc,
  2611			struct dc_state *context)
  2612	{
  2613		int i, j;
  2614	
  2615		DC_LOGGER_INIT(dc->ctx->logger);
  2616	
  2617		for (i = 0; i < dc->res_pool->pipe_count; i++) {
  2618			struct pipe_ctx *pipe_ctx = &context->res_ctx.pipe_ctx[i];
  2619	
  2620			if (!pipe_ctx->top_pipe &&
  2621				!pipe_ctx->prev_odm_pipe &&
  2622				pipe_ctx->stream) {
> 2623				struct dc_stream_status *stream_status = NULL;
  2624				struct timing_generator *tg = pipe_ctx->stream_res.tg;
  2625	
  2626				for (j = 0; j < context->stream_count; j++) {
  2627					if (pipe_ctx->stream == context->streams[j])
  2628						stream_status = &context->stream_status[j];
  2629				}
  2630	
  2631				if (context->stream_status[i].plane_count == 0)
  2632					false_optc_underflow_wa(dc, pipe_ctx->stream, tg);
  2633			}
  2634		}
  2635	
  2636		for (i = 0; i < dc->res_pool->pipe_count; i++)
  2637			if (context->res_ctx.pipe_ctx[i].update_flags.bits.disable)
  2638				dc->hwss.disable_plane(dc, &dc->current_state->res_ctx.pipe_ctx[i]);
  2639	
  2640		for (i = 0; i < dc->res_pool->pipe_count; i++)
  2641			if (context->res_ctx.pipe_ctx[i].update_flags.bits.disable) {
  2642				dc->hwss.optimize_bandwidth(dc, context);
  2643				break;
  2644			}
  2645	
  2646		if (dc->hwseq->wa.DEGVIDCN10_254)
  2647			hubbub1_wm_change_req_wa(dc->res_pool->hubbub);
  2648	}
  2649	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 34252 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [linux-stable-rc:linux-5.6.y 1567/1864] drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:2623:29: warning: variable 'stream_status' set but not used
@ 2020-07-14  3:32 kernel test robot
  2020-07-14  6:25 ` Greg Kroah-Hartman
  0 siblings, 1 reply; 4+ messages in thread
From: kernel test robot @ 2020-07-14  3:32 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 7264 bytes --]

Hi Anthony,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.6.y
head:   61aba373f5708f2aebc3f72078e51949a068aa6f
commit: 5fc20b04d12c0827dbf498532e8a736a1f88ad6f [1567/1864] drm/amd/display: Split program front end part that occur outside lock
config: i386-allmodconfig (attached as .config)
compiler: gcc-9 (Debian 9.3.0-14) 9.3.0

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

   drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:72:6: warning: no previous prototype for 'print_microsec' [-Wmissing-prototypes]
      72 | void print_microsec(struct dc_context *dc_ctx,
         |      ^~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:126:6: warning: no previous prototype for 'dcn10_log_hubbub_state' [-Wmissing-prototypes]
     126 | void dcn10_log_hubbub_state(struct dc *dc, struct dc_log_buffer_ctx *log_ctx)
         |      ^~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:1878:6: warning: no previous prototype for 'dcn10_program_pte_vm' [-Wmissing-prototypes]
    1878 | void dcn10_program_pte_vm(struct dce_hwseq *hws, struct hubp *hubp)
         |      ^~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c: In function 'dcn10_post_unlock_program_front_end':
>> drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:2623:29: warning: variable 'stream_status' set but not used [-Wunused-but-set-variable]
    2623 |    struct dc_stream_status *stream_status = NULL;
         |                             ^~~~~~~~~~~~~
   In file included from drivers/gpu/drm/amd/amdgpu/../display/dc/inc/core_types.h:85,
                    from drivers/gpu/drm/amd/amdgpu/../display/dc/basics/dc_common.h:29,
                    from drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:28:
   At top level:
   drivers/gpu/drm/amd/amdgpu/../display/dc/inc/hw/dpp.h:50:42: warning: 'dpp_input_csc_matrix' defined but not used [-Wunused-const-variable=]
      50 | static const struct dpp_input_csc_matrix dpp_input_csc_matrix[] = {
         |                                          ^~~~~~~~~~~~~~~~~~~~
   In file included from drivers/gpu/drm/amd/amdgpu/../display/dc/inc/core_types.h:32,
                    from drivers/gpu/drm/amd/amdgpu/../display/dc/basics/dc_common.h:29,
                    from drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:28:
   drivers/gpu/drm/amd/amdgpu/../display/include/ddc_service_types.h:124:22: warning: 'DP_DVI_CONVERTER_ID_4' defined but not used [-Wunused-const-variable=]
     124 | static const uint8_t DP_DVI_CONVERTER_ID_4[] = "m2DVIa";
         |                      ^~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/../display/include/ddc_service_types.h:122:22: warning: 'DP_VGA_LVDS_CONVERTER_ID_3' defined but not used [-Wunused-const-variable=]
     122 | static const uint8_t DP_VGA_LVDS_CONVERTER_ID_3[] = "dnomlA";
         |                      ^~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/../display/include/ddc_service_types.h:120:22: warning: 'DP_VGA_LVDS_CONVERTER_ID_2' defined but not used [-Wunused-const-variable=]
     120 | static const uint8_t DP_VGA_LVDS_CONVERTER_ID_2[] = "sivarT";
         |                      ^~~~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/gpu/drm/amd/amdgpu/../display/dc/dc_types.h:33,
                    from drivers/gpu/drm/amd/amdgpu/../display/dc/dm_services_types.h:30,
                    from drivers/gpu/drm/amd/amdgpu/../display/dc/dm_services.h:37,
                    from drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:27:
   drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:76:32: warning: 'dc_fixpt_ln2_div_2' defined but not used [-Wunused-const-variable=]
      76 | static const struct fixed31_32 dc_fixpt_ln2_div_2 = { 1488522236LL };
         |                                ^~~~~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:75:32: warning: 'dc_fixpt_ln2' defined but not used [-Wunused-const-variable=]
      75 | static const struct fixed31_32 dc_fixpt_ln2 = { 2977044471LL };
         |                                ^~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:74:32: warning: 'dc_fixpt_e' defined but not used [-Wunused-const-variable=]
      74 | static const struct fixed31_32 dc_fixpt_e = { 11674931555LL };
         |                                ^~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:73:32: warning: 'dc_fixpt_two_pi' defined but not used [-Wunused-const-variable=]
      73 | static const struct fixed31_32 dc_fixpt_two_pi = { 26986075409LL };
         |                                ^~~~~~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:72:32: warning: 'dc_fixpt_pi' defined but not used [-Wunused-const-variable=]
      72 | static const struct fixed31_32 dc_fixpt_pi = { 13493037705LL };
         |                                ^~~~~~~~~~~
   drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:67:32: warning: 'dc_fixpt_zero' defined but not used [-Wunused-const-variable=]
      67 | static const struct fixed31_32 dc_fixpt_zero = { 0 };
         |                                ^~~~~~~~~~~~~

vim +/stream_status +2623 drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c

  2608	
  2609	void dcn10_post_unlock_program_front_end(
  2610			struct dc *dc,
  2611			struct dc_state *context)
  2612	{
  2613		int i, j;
  2614	
  2615		DC_LOGGER_INIT(dc->ctx->logger);
  2616	
  2617		for (i = 0; i < dc->res_pool->pipe_count; i++) {
  2618			struct pipe_ctx *pipe_ctx = &context->res_ctx.pipe_ctx[i];
  2619	
  2620			if (!pipe_ctx->top_pipe &&
  2621				!pipe_ctx->prev_odm_pipe &&
  2622				pipe_ctx->stream) {
> 2623				struct dc_stream_status *stream_status = NULL;
  2624				struct timing_generator *tg = pipe_ctx->stream_res.tg;
  2625	
  2626				for (j = 0; j < context->stream_count; j++) {
  2627					if (pipe_ctx->stream == context->streams[j])
  2628						stream_status = &context->stream_status[j];
  2629				}
  2630	
  2631				if (context->stream_status[i].plane_count == 0)
  2632					false_optc_underflow_wa(dc, pipe_ctx->stream, tg);
  2633			}
  2634		}
  2635	
  2636		for (i = 0; i < dc->res_pool->pipe_count; i++)
  2637			if (context->res_ctx.pipe_ctx[i].update_flags.bits.disable)
  2638				dc->hwss.disable_plane(dc, &dc->current_state->res_ctx.pipe_ctx[i]);
  2639	
  2640		for (i = 0; i < dc->res_pool->pipe_count; i++)
  2641			if (context->res_ctx.pipe_ctx[i].update_flags.bits.disable) {
  2642				dc->hwss.optimize_bandwidth(dc, context);
  2643				break;
  2644			}
  2645	
  2646		if (dc->hwseq->wa.DEGVIDCN10_254)
  2647			hubbub1_wm_change_req_wa(dc->res_pool->hubbub);
  2648	}
  2649	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 72651 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [linux-stable-rc:linux-5.6.y 1567/1864] drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:2623:29: warning: variable 'stream_status' set but not used
  2020-07-14  3:32 [linux-stable-rc:linux-5.6.y 1567/1864] drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:2623:29: warning: variable 'stream_status' set but not used kernel test robot
@ 2020-07-14  6:25 ` Greg Kroah-Hartman
  2020-07-14  6:41   ` Philip Li
  0 siblings, 1 reply; 4+ messages in thread
From: Greg Kroah-Hartman @ 2020-07-14  6:25 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 255 bytes --]

On Tue, Jul 14, 2020 at 11:32:33AM +0800, kernel test robot wrote:
> Hi Anthony,
> 
> FYI, the error/warning still remains.

Any specific reason you are testing/reporting issues on a kernel branch
that is end-of-life and can not be modified anymore?

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [linux-stable-rc:linux-5.6.y 1567/1864] drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:2623:29: warning: variable 'stream_status' set but not used
  2020-07-14  6:25 ` Greg Kroah-Hartman
@ 2020-07-14  6:41   ` Philip Li
  0 siblings, 0 replies; 4+ messages in thread
From: Philip Li @ 2020-07-14  6:41 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 685 bytes --]

On Tue, Jul 14, 2020 at 08:25:35AM +0200, Greg Kroah-Hartman wrote:
> On Tue, Jul 14, 2020 at 11:32:33AM +0800, kernel test robot wrote:
> > Hi Anthony,
> > 
> > FYI, the error/warning still remains.
> 
> Any specific reason you are testing/reporting issues on a kernel branch
> that is end-of-life and can not be modified anymore?
sorry about this, the EOL is not monitored in time and automatically, we
will take this into consideration in future and enhance the bot, to avoid
this kind of report.

> _______________________________________________
> kbuild-all mailing list -- kbuild-all(a)lists.01.org
> To unsubscribe send an email to kbuild-all-leave(a)lists.01.org

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2020-07-14  6:41 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-07-14  3:32 [linux-stable-rc:linux-5.6.y 1567/1864] drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hw_sequencer.c:2623:29: warning: variable 'stream_status' set but not used kernel test robot
2020-07-14  6:25 ` Greg Kroah-Hartman
2020-07-14  6:41   ` Philip Li
  -- strict thread matches above, loose matches on Subject: below --
2020-06-17 13:18 kernel test robot

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.