[RFC] Revert "drm/amdgpu/gfx8: Fix SET_RESOURCES packet"

Deucher, Alexander Alexander.Deucher at amd.com
Tue Apr 4 20:03:53 UTC 2017


> -----Original Message-----
> From: Andres Rodriguez [mailto:andresx7 at gmail.com]
> Sent: Tuesday, April 04, 2017 4:01 PM
> To: Tom St Denis; Deucher, Alexander; StDenis, Tom; amd-
> gfx at lists.freedesktop.org
> Subject: Re: [RFC] Revert "drm/amdgpu/gfx8: Fix SET_RESOURCES packet"
> 
> This is my info in case that is useful:
> 
>      umr.version == ea8e49bb15ed
> 
>      fw.VCE == .feature==0 .firmware==0x34040300
>      fw.UVD == .feature==0 .firmware==0x014f1000
>      fw.MC == .feature==0 .firmware==0x00000000
>      fw.ME == .feature==37 .firmware==0x00000094
>      fw.PFP == .feature==37 .firmware==0x000000dc
>      fw.CE == .feature==37 .firmware==0x00000080
>      fw.RLC == .feature==1 .firmware==0x0000010e
>      fw.MEC == .feature==37 .firmware==0x000002a2
>      fw.MEC2 == .feature==37 .firmware==0x000002a2
>      fw.SOS == .feature==0 .firmware==0x00000000
>      fw.ASD == .feature==0 .firmware==0x00000000
>      fw.SMC == .feature==0 .firmware==0x00170f00
>      fw.SDMA0 == .feature==31 .firmware==0x00000036
>      fw.SDMA1 == .feature==0 .firmware==0x00000036
> 
>      asic.instance == 0
> 
> 
>      gfx.max_shader_engines == 4
>      gfx.max_tile_pipes == 8
>      gfx.max_cu_per_sh == 9
>      gfx.max_sh_per_se == 1
>      gfx.max_backends_per_se == 2
>      gfx.max_texture_channel_caches == 8
>      gfx.max_gprs == 256
>      gfx.max_gs_threads == 32
>      gfx.max_hw_contexts == 8
>      gfx.sc_prim_fifo_size_frontend == 32
>      gfx.sc_prim_fifo_size_backend == 256
>      gfx.sc_hiz_tile_fifo_size == 48
>      gfx.sc_earlyz_tile_fifo_size == 304
>      gfx.num_tile_pipes == 8
>      gfx.backend_enable_mask == 255
>      gfx.mem_max_burst_length_bytes == 256
>      gfx.mem_row_size_in_kb == 4
>      gfx.shader_engine_tile_size == 32
>      gfx.num_gpus == 1
>      gfx.multi_gpu_tile_size == 64
>      gfx.mc_arb_ramcfg == 24738
>      gfx.gb_addr_config == 570494979
>      gfx.num_rbs == 8
>      gfx.family = 130, Volcanic Islands
>      gfx.rev_id == 00000001
>      gfx.external_rev_id == 00000051
>      gfx.cg_flags == 003fffcd
>          AMD_CG_SUPPORT_GFX_MGCG
>          AMD_CG_SUPPORT_GFX_CGCG
>          AMD_CG_SUPPORT_GFX_CGLS
>          AMD_CG_SUPPORT_GFX_CP_LS
>          AMD_CG_SUPPORT_GFX_RLC_LS
>          AMD_CG_SUPPORT_MC_LS
>          AMD_CG_SUPPORT_MC_MGCG
>          AMD_CG_SUPPORT_SDMA_LS
>          AMD_CG_SUPPORT_SDMA_MGCG
>          AMD_CG_SUPPORT_BIF_LS
>          AMD_CG_SUPPORT_UVD_MGCG
>          AMD_CG_SUPPORT_VCE_MGCG
>          AMD_CG_SUPPORT_HDP_LS
>          AMD_CG_SUPPORT_HDP_MGCG
>          AMD_CG_SUPPORT_ROM_MGCG
>      gfx.pg_flags == 00000000

Is this the stock firmware from upstream or the stuff I pointed you to yesterday on my fdo site?

Alex

> 
> Regards,
> Andres
> 
> On 2017-04-04 03:59 PM, Tom St Denis wrote:
> > I'm using the latest from all open (our internal FC24 repo).  I'll fetch
> > the exact later with "umr -c".
> >
> > Tom
> >
> > On Tue, Apr 4, 2017 at 3:48 PM Deucher, Alexander
> <Alexander.Deucher at amd.com>
> > wrote:
> >
> >>> -----Original Message-----
> >>> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On
> Behalf
> >>> Of Andres Rodriguez
> >>> Sent: Tuesday, April 04, 2017 3:39 PM
> >>> To: StDenis, Tom; amd-gfx at lists.freedesktop.org
> >>> Subject: Re: [RFC] Revert "drm/amdgpu/gfx8: Fix SET_RESOURCES
> packet"
> >>>
> >>>
> >>>
> >>> On 2017-04-04 08:27 AM, Tom St Denis wrote:
> >>>> On 03/04/17 04:25 PM, Andres Rodriguez wrote:
> >>>>> Commit e579f56 results in a KCQ initialization error for polaris10
> >>>>> cards. Providing the full SET_RESOURCES packet instead of the
> >> truncated
> >>>>> version fixes the problem.
> >>>>>
> >>>>> I think this patch may have been one of the stabs at fixing the KCQ
> >>>>> suspend/resume errors. I *think* it may no longer be needed if that
> is
> >>>>> the case.
> >>>>>
> >>>>> Sending this mostly as an FYI to get the patch removed from the
> >> 4.12-wip
> >>>>> This reverts commit e579f564c333a7eaddf7e12b1b8414410c36e80f.
> >>>>> ---
> >>>>>   drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 6 +++++-
> >>>>>   1 file changed, 5 insertions(+), 1 deletion(-)
> >>>>>
> >>>>> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> >>>>> b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> >>>>> index 2d8d7bb..4c3bf51 100644
> >>>>> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> >>>>> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> >>>>> @@ -4647,10 +4647,14 @@ static int gfx_v8_0_kiq_kcq_enable(struct
> >>>>> amdgpu_device *adev)
> >>>>>           return r;
> >>>>>       }
> >>>>>       /* set resources */
> >>>>> -    amdgpu_ring_write(kiq_ring, PACKET3(PACKET3_SET_RESOURCES,
> 2));
> >>>>> +    amdgpu_ring_write(kiq_ring,
> PACKET3(PACKET3_SET_RESOURCES,
> >>> 6));
> >>>>>       amdgpu_ring_write(kiq_ring, 0);    /* vmid_mask:0 queue_type:0
> >>>>> (KIQ) */
> >>>>>       amdgpu_ring_write(kiq_ring, 0x000000FF);    /* queue mask lo */
> >>>>>       amdgpu_ring_write(kiq_ring, 0);    /* queue mask hi */
> >>>>> +    amdgpu_ring_write(kiq_ring, 0);    /* gws mask lo */
> >>>>> +    amdgpu_ring_write(kiq_ring, 0);    /* gws mask hi */
> >>>>> +    amdgpu_ring_write(kiq_ring, 0);    /* oac mask */
> >>>>> +    amdgpu_ring_write(kiq_ring, 0);    /* gds heap base:0, gds heap
> >>>>> size:0 */
> >>>>>       for (i = 0; i < adev->gfx.num_compute_rings; i++) {
> >>>>>           struct amdgpu_ring *ring = &adev->gfx.compute_ring[i];
> >>>>>           uint64_t mqd_addr = amdgpu_bo_gpu_offset(ring->mqd_obj);
> >>>>>
> >>>> This fixes init issues with my polaris board but there's still a clock
> >>>> problem (CP is pegged at 100% as read by umr) resulting in the GPU
> >> clock
> >>>> staying at 1200MHz the entire time.
> >>>>
> >>> Yeah I sent this out because I was having KCQ init issues on my
> >>> polaris10 card. However, I'm not seeing the CP pegged at 100% as you
> >>> mentioned. Everything sitting quietly at 0% here.
> >> Which firmware are you using?
> >>
> >> Alex
> >>
> >>>
> >>>> Tom
> >>>> _______________________________________________
> >>>> amd-gfx mailing list
> >>>> amd-gfx at lists.freedesktop.org
> >>>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> >>> _______________________________________________
> >>> amd-gfx mailing list
> >>> amd-gfx at lists.freedesktop.org
> >>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> >> _______________________________________________
> >> amd-gfx mailing list
> >> amd-gfx at lists.freedesktop.org
> >> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> >>



More information about the amd-gfx mailing list