>> + if (amdgpu_display_framebuffer_domains(adev) == AMDGPU_GEM_DOMAIN_GTT)
>
> This check should probably be & instead of ==.
Thought about that as well. For mixed VRAM/GTT display case, since it has some complications, I prefer to do it later.
Sam
On 2018-03-03 08:42 AM, Christian König wrote:
> Am 03.03.2018 um 00:26 schrieb Samuel Li:
>> ---
>> Â drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 3 +++
>>  include/uapi/drm/amdgpu_drm.h          | 1 +
>> Â 2 files changed, 4 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
>> index 51a4b08..2cb344f 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c
>> @@ -31,6 +31,7 @@
>> Â #include "amdgpu_sched.h"
>> Â #include "amdgpu_uvd.h"
>> Â #include "amdgpu_vce.h"
>> +#include "amdgpu_display.h"
>> Â Â #include <linux/vga_switcheroo.h>
>> Â #include <linux/slab.h>
>> @@ -578,6 +579,8 @@ static int amdgpu_info_ioctl(struct drm_device *dev, void *data, struct drm_file
>> Â Â Â Â Â Â Â Â Â Â Â Â Â dev_info.ids_flags |= AMDGPU_IDS_FLAGS_FUSION;
>> Â Â Â Â Â Â Â Â Â if (amdgpu_sriov_vf(adev))
>> Â Â Â Â Â Â Â Â Â Â Â Â Â dev_info.ids_flags |= AMDGPU_IDS_FLAGS_PREEMPTION;
>> +Â Â Â Â Â Â Â if (amdgpu_display_framebuffer_domains(adev) == AMDGPU_GEM_DOMAIN_GTT)
>
> This check should probably be & instead of ==.
>
> Christian.
>
>> +Â Â Â Â Â Â Â Â Â Â Â dev_info.ids_flags |= AMDGPU_IDS_FLAGS_SGDISPLAY;
>> Â Â Â Â Â Â Â Â Â Â vm_size = adev->vm_manager.max_pfn * AMDGPU_GPU_PAGE_SIZE;
>> Â Â Â Â Â Â Â Â Â vm_size -= AMDGPU_VA_RESERVED_SIZE;
>> diff --git a/include/uapi/drm/amdgpu_drm.h b/include/uapi/drm/amdgpu_drm.h
>> index fe17b67..2f30b98 100644
>> --- a/include/uapi/drm/amdgpu_drm.h
>> +++ b/include/uapi/drm/amdgpu_drm.h
>> @@ -584,6 +584,7 @@ struct drm_amdgpu_cs_chunk_data {
>> Â Â */
>> Â #define AMDGPU_IDS_FLAGS_FUSIONÂ Â Â Â Â Â Â Â 0x1
>> Â #define AMDGPU_IDS_FLAGS_PREEMPTIONÂ Â Â Â 0x2
>> +#define AMDGPU_IDS_FLAGS_SGDISPLAYÂ Â Â Â Â 0x4
>> Â Â /* indicate if acceleration can be working */
>> Â #define AMDGPU_INFO_ACCEL_WORKINGÂ Â Â Â Â Â Â 0x00
>
