>> domain = amdgpu_display_framebuffer_domains(adev);
>> + if (domain == AMDGPU_GEM_DOMAIN_GTT) {
>> + DRM_INFO("Scatter gather display: enabled\n");
>> + } else if (domain & AMDGPU_GEM_DOMAIN_GTT) {
>> + DRM_INFO("Scatter gather display: auto\n");
>> + }
>
> Dito and printing that here is not a good idea as far as I can see.
>
> Christian.
The intention here is to print out when fb is created. You have other suggestions?
Sam
On 2018-03-03 08:41 AM, Christian König wrote:
> Am 03.03.2018 um 00:25 schrieb Samuel Li:
>> It's enabled by default. -1 is auto, to allow both vram and gtt
>> memory be available, for testing purpose only.
>> ---
>>  drivers/gpu/drm/amd/amdgpu/amdgpu.h        | 1 +
>> Â drivers/gpu/drm/amd/amdgpu/amdgpu_display.c | 9 +++++++--
>>  drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c    | 4 ++++
>>  drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c     | 5 +++++
>> Â 4 files changed, 17 insertions(+), 2 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
>> index 292c7e7..6b0ee34 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
>> @@ -129,6 +129,7 @@ extern int amdgpu_lbpw;
>> Â extern int amdgpu_compute_multipipe;
>> Â extern int amdgpu_gpu_recovery;
>> Â extern int amdgpu_emu_mode;
>> +extern int amdgpu_sg_display;
>> Â Â #ifdef CONFIG_DRM_AMDGPU_SI
>> Â extern int amdgpu_si_support;
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
>> index 5495b29..dfa11b1 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
>> @@ -513,8 +513,13 @@ uint32_t amdgpu_display_framebuffer_domains(struct amdgpu_device *adev)
>> Â #if defined(CONFIG_DRM_AMD_DC)
>> Â Â Â Â Â if (adev->asic_type >= CHIP_CARRIZO && adev->asic_type < CHIP_RAVEN &&
>> Â Â Â Â Â Â Â Â Â adev->flags & AMD_IS_APU &&
>> -Â Â Â Â Â Â Â amdgpu_device_asic_has_dc_support(adev->asic_type))
>> -Â Â Â Â Â Â Â domain |= AMDGPU_GEM_DOMAIN_GTT;
>> +Â Â Â Â Â Â Â amdgpu_device_asic_has_dc_support(adev->asic_type)) {
>> +Â Â Â Â Â Â Â if (amdgpu_sg_display == 1) {
>> +Â Â Â Â Â Â Â Â Â Â Â domain = AMDGPU_GEM_DOMAIN_GTT;
>> +Â Â Â Â Â Â Â } else if (amdgpu_sg_display == -1) {
>> +Â Â Â Â Â Â Â Â Â Â Â domain |= AMDGPU_GEM_DOMAIN_GTT;
>> +Â Â Â Â Â Â Â }
>
> Coding style, that if shouldn't use "{" and "}".
>
>> +Â Â Â }
>> Â #endif
>> Â Â Â Â Â Â return domain;
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
>> index e670936..f0ada24 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
>> @@ -132,6 +132,7 @@ int amdgpu_lbpw = -1;
>> Â int amdgpu_compute_multipipe = -1;
>> Â int amdgpu_gpu_recovery = -1; /* auto */
>> Â int amdgpu_emu_mode = 0;
>> +int amdgpu_sg_display = 1;
>> Â Â MODULE_PARM_DESC(vramlimit, "Restrict VRAM for testing, in megabytes");
>> Â module_param_named(vramlimit, amdgpu_vram_limit, int, 0600);
>> @@ -290,6 +291,9 @@ module_param_named(gpu_recovery, amdgpu_gpu_recovery, int, 0444);
>> Â MODULE_PARM_DESC(emu_mode, "Emulation mode, (1 = enable, 0 = disable)");
>> Â module_param_named(emu_mode, amdgpu_emu_mode, int, 0444);
>> Â +MODULE_PARM_DESC(sg_display, "Enable scatter gather display, (1 = enable, 0 = disable, -1 = auto");
>> +module_param_named(sg_display, amdgpu_sg_display, int, 0444);
>> +
>> Â #ifdef CONFIG_DRM_AMDGPU_SI
>> Â Â #if defined(CONFIG_DRM_RADEON) || defined(CONFIG_DRM_RADEON_MODULE)
>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
>> index 1206301..10f1f4f 100644
>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
>> @@ -138,6 +138,11 @@ static int amdgpufb_create_pinned_object(struct amdgpu_fbdev *rfbdev,
>> Â Â Â Â Â mode_cmd->pitches[0] = amdgpu_align_pitch(adev, mode_cmd->width, cpp,
>> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â fb_tiled);
>> Â Â Â Â Â domain = amdgpu_display_framebuffer_domains(adev);
>> +Â Â Â if (domain == AMDGPU_GEM_DOMAIN_GTT) {
>> +Â Â Â Â Â Â Â DRM_INFO("Scatter gather display: enabled\n");
>> +Â Â Â } else if (domain & AMDGPU_GEM_DOMAIN_GTT) {
>> +Â Â Â Â Â Â Â DRM_INFO("Scatter gather display: auto\n");
>> +Â Â Â }
>
> Dito and printing that here is not a good idea as far as I can see.
>
> Christian.
>
>> Â Â Â Â Â Â height = ALIGN(mode_cmd->height, 8);
>> Â Â Â Â Â size = mode_cmd->pitches[0] * height;
>
