On 2018-04-18 12:16 PM, Christian König wrote:
> Am 18.04.2018 um 17:29 schrieb Samuel Li:
>>
>> On 2018-04-18 12:14 AM, Alex Deucher wrote:
>>> On Tue, Apr 17, 2018 at 8:40 PM, Samuel Li <Samuel.Li at amd.com> wrote:
>>>> It's auto by default. For CZ/ST, auto setting enables sg display
>>>> when vram size is small; otherwise still uses vram.
>>>> This patch fixed some potention hang issue introduced by change
>>>> "allow framebuffer in GART memory as well" due to CZ/ST hardware
>>>> limitation.
>>>>
>>>
>> OK.
>>
[...]
>
> Mhm, for developer testing we can easily modify amdgpu_display_supported_domains().
>
> The real question is should we give an end user the ability to modify the behavior? I currently can't think of a reason for that.
Yes, we do. For example, there are cases that user prefers GTT(VRAM can run out), also there are cases user prefers VRAM for performance.
Regards,
Sam
>
> Regards,
> Christian.
>
>>
>> Regards,
>> Samuel Li
>>
>>> Alex
>>>
>>>> +Â Â Â Â Â Â }
>>>> +#endif
>>>>
>>>> Â Â Â Â Â Â Â Â if (bo->pin_count) {
>>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â uint32_t mem_type = bo->tbo.mem.mem_type;
>>>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_prime.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_prime.c
>>>> index 4b584cb7..cf0749f 100644
>>>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_prime.c
>>>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_prime.c
>>>> @@ -209,7 +209,7 @@ static int amdgpu_gem_begin_cpu_access(struct dma_buf *dma_buf,
>>>> Â Â Â Â Â Â Â Â struct amdgpu_bo *bo = gem_to_amdgpu_bo(dma_buf->priv);
>>>> Â Â Â Â Â Â Â Â struct amdgpu_device *adev = amdgpu_ttm_adev(bo->tbo.bdev);
>>>> Â Â Â Â Â Â Â Â struct ttm_operation_ctx ctx = { true, false };
>>>> -Â Â Â Â Â Â u32 domain = amdgpu_display_framebuffer_domains(adev);
>>>> +Â Â Â Â Â Â u32 domain = amdgpu_display_supported_domains(adev);
>>>> Â Â Â Â Â Â Â Â int ret;
>>>> Â Â Â Â Â Â Â Â bool reads = (direction == DMA_BIDIRECTIONAL ||
>>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â direction == DMA_FROM_DEVICE);
>>>> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
>>>> index 6f92a19..1f5603a 100644
>>>> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
>>>> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
>>>> @@ -3109,12 +3109,11 @@ static int dm_plane_helper_prepare_fb(struct drm_plane *plane,
>>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â return r;
>>>>
>>>> Â Â Â Â Â Â Â Â if (plane->type != DRM_PLANE_TYPE_CURSOR)
>>>> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â domain = amdgpu_display_framebuffer_domains(adev);
>>>> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â domain = amdgpu_display_supported_domains(adev);
>>>> Â Â Â Â Â Â Â Â else
>>>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â domain = AMDGPU_GEM_DOMAIN_VRAM;
>>>>
>>>> Â Â Â Â Â Â Â Â r = amdgpu_bo_pin(rbo, domain, &afb->address);
>>>> -
>>>> Â Â Â Â Â Â Â Â amdgpu_bo_unreserve(rbo);
>>>>
>>>> Â Â Â Â Â Â Â Â if (unlikely(r != 0)) {
>>>> --
>>>> 2.7.4
>>>>
>>>> _______________________________________________
>>>> 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
>
