[PATCH 4/6] drm/amdgpu: implement PRT for GFX6 v2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jerry,

thanks for the defines. I accidentally compiled the patch without SI 
support, so I didn't noted that they are missing.

Going to integrate them and send out the patch set once more.

Regards,
Christian.

Am 06.02.2017 um 09:51 schrieb Zhang, Jerry:
> Hi all,
>
> We also need below for SI(gmc v6) support.
>
> {{{
> diff --git a/drivers/gpu/drm/amd/include/asic_reg/gmc/gmc_6_0_sh_mask.h b/drivers/gpu/drm/amd/include/asic_reg/
> index 0f6c6c8..7155312 100644
> --- a/drivers/gpu/drm/amd/include/asic_reg/gmc/gmc_6_0_sh_mask.h
> +++ b/drivers/gpu/drm/amd/include/asic_reg/gmc/gmc_6_0_sh_mask.h
> @@ -11891,5 +11891,9 @@
>   #define VM_PRT_CNTL__L1_TLB_STORE_INVALID_ENTRIES__SHIFT 0x00000003
>   #define VM_PRT_CNTL__L2_CACHE_STORE_INVALID_ENTRIES_MASK 0x00000004L
>   #define VM_PRT_CNTL__L2_CACHE_STORE_INVALID_ENTRIES__SHIFT 0x00000002
> +#define VM_PRT_CNTL__CB_DISABLE_FAULT_ON_UNMAPPED_ACCESS_MASK 0x00000001L
> +#define VM_PRT_CNTL__CB_DISABLE_FAULT_ON_UNMAPPED_ACCESS__SHIFT 0x00000000
> +#define VM_PRT_CNTL__TC_DISABLE_FAULT_ON_UNMAPPED_ACCESS_MASK 0x00000002L
> +#define VM_PRT_CNTL__TC_DISABLE_FAULT_ON_UNMAPPED_ACCESS__SHIFT 0x00000001
> }}}
>
> Regards,
> Jerry (Junwei Zhang)
>
> Linux Base Graphics
> SRDC Software Development
> _____________________________________
>
>
>> -----Original Message-----
>> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf Of
>> Nicolai H?hnle
>> Sent: Friday, February 03, 2017 22:37
>> To: Christian König; amd-gfx at lists.freedesktop.org
>> Cc: bas at basnieuwenhuizen.nl
>> Subject: Re: [PATCH 4/6] drm/amdgpu: implement PRT for GFX6 v2
>>
>> On 02.02.2017 11:25, Christian König wrote:
>>> From: Christian König <christian.koenig at amd.com>
>>>
>>> Enable/disable the handling globally for now and print a warning when
>>> we enable it for the first time.
>>>
>>> v2: write to the correct register, adjust bits to that hw generation
>>>
>>> Signed-off-by: Christian König <christian.koenig at amd.com>
>>> ---
>>>   drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c | 55
>>> +++++++++++++++++++++++++++++++++++
>>>   1 file changed, 55 insertions(+)
>>>
>>> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c
>>> b/drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c
>>> index e2b0b16..b9b5c24 100644
>>> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c
>>> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c
>>> @@ -398,6 +398,60 @@ static void gmc_v6_0_set_fault_enable_default(struct
>> amdgpu_device *adev,
>>>   	WREG32(mmVM_CONTEXT1_CNTL, tmp);
>>>   }
>>>
>>> + /**
>>> +   + * gmc_v8_0_set_prt - set PRT VM fault
>>> +   + *
>>> +   + * @adev: amdgpu_device pointer
>>> +   + * @enable: enable/disable VM fault handling for PRT
>>> +   +*/
>>> +static void gmc_v6_0_set_prt(struct amdgpu_device *adev, bool enable)
>>> +{
>>> +	u32 tmp;
>>> +
>>> +	if (enable && !adev->mc.prt_warning) {
>>> +		dev_warn(adev->dev, "Disabling VM faults because of PRT
>> request!\n");
>>> +		adev->mc.prt_warning = true;
>>> +	}
>>> +
>>> +	tmp = RREG32(mmVM_PRT_CNTL);
>>> +	tmp = REG_SET_FIELD(tmp, VM_PRT_CNTL,
>>> +			    CB_DISABLE_FAULT_ON_UNMAPPED_ACCESS,
>>> +			    enable);
>> I get:
>>
>> drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c: In function â??gmc_v6_0_set_prtâ??:
>> drivers/gpu/drm/amd/amdgpu/gmc_v6_0.c:419:27: error:
>> â??VM_PRT_CNTL__CB_DISABLE_FAULT_ON_UNMAPPED_ACCESS_MASKâ??
>> undeclared (first use in this function)
>>     tmp = REG_SET_FIELD(tmp, VM_PRT_CNTL,
>>                              ^
>> and similar compiler errors here. The other patches compile fine.
>>
>> Nicolai
>>
>>> +	tmp = REG_SET_FIELD(tmp, VM_PRT_CNTL,
>>> +			    TC_DISABLE_FAULT_ON_UNMAPPED_ACCESS,
>>> +			    enable);
>>> +	tmp = REG_SET_FIELD(tmp, VM_PRT_CNTL,
>>> +			    L2_CACHE_STORE_INVALID_ENTRIES,
>>> +			    enable);
>>> +	tmp = REG_SET_FIELD(tmp, VM_PRT_CNTL,
>>> +			    L1_TLB_STORE_INVALID_ENTRIES,
>>> +			    enable);
>>> +	WREG32(mmVM_PRT_CNTL, tmp);
>>> +
>>> +	if (enable) {
>>> +		uint32_t low = AMDGPU_VA_RESERVED_SIZE >>
>> AMDGPU_GPU_PAGE_SHIFT;
>>> +		uint32_t high = adev->vm_manager.max_pfn;
>>> +
>>> +		WREG32(mmVM_PRT_APERTURE0_LOW_ADDR, low);
>>> +		WREG32(mmVM_PRT_APERTURE1_LOW_ADDR, low);
>>> +		WREG32(mmVM_PRT_APERTURE2_LOW_ADDR, low);
>>> +		WREG32(mmVM_PRT_APERTURE3_LOW_ADDR, low);
>>> +		WREG32(mmVM_PRT_APERTURE0_HIGH_ADDR, high);
>>> +		WREG32(mmVM_PRT_APERTURE1_HIGH_ADDR, high);
>>> +		WREG32(mmVM_PRT_APERTURE2_HIGH_ADDR, high);
>>> +		WREG32(mmVM_PRT_APERTURE3_HIGH_ADDR, high);
>>> +	} else {
>>> +		WREG32(mmVM_PRT_APERTURE0_LOW_ADDR, 0xfffffff);
>>> +		WREG32(mmVM_PRT_APERTURE1_LOW_ADDR, 0xfffffff);
>>> +		WREG32(mmVM_PRT_APERTURE2_LOW_ADDR, 0xfffffff);
>>> +		WREG32(mmVM_PRT_APERTURE3_LOW_ADDR, 0xfffffff);
>>> +		WREG32(mmVM_PRT_APERTURE0_HIGH_ADDR, 0x0);
>>> +		WREG32(mmVM_PRT_APERTURE1_HIGH_ADDR, 0x0);
>>> +		WREG32(mmVM_PRT_APERTURE2_HIGH_ADDR, 0x0);
>>> +		WREG32(mmVM_PRT_APERTURE3_HIGH_ADDR, 0x0);
>>> +	}
>>> +}
>>> +
>>>   static int gmc_v6_0_gart_enable(struct amdgpu_device *adev)  {
>>>   	int r, i;
>>> @@ -1080,6 +1134,7 @@ static const struct amd_ip_funcs
>>> gmc_v6_0_ip_funcs = {  static const struct amdgpu_gart_funcs
>> gmc_v6_0_gart_funcs = {
>>>   	.flush_gpu_tlb = gmc_v6_0_gart_flush_gpu_tlb,
>>>   	.set_pte_pde = gmc_v6_0_gart_set_pte_pde,
>>> +	.set_prt = gmc_v6_0_set_prt,
>>>   };
>>>
>>>   static const struct amdgpu_irq_src_funcs gmc_v6_0_irq_funcs = {
>>>
>> _______________________________________________
>> amd-gfx mailing list
>> amd-gfx at lists.freedesktop.org
>> https://lists.freedesktop.org/mailman/listinfo/amd-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux