Re: BISECTED- amd-staging-drm-next, xorg-server segfault A6-6310 APU - R4 Mullins.

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

 



On 2019-01-10 6:56 p.m., Przemek Socha wrote:
>
> [  147.846148] [drm:amdgpu_display_user_framebuffer_create [amdgpu]] Invalid 
> pitch: expecting 10752 but got 10624
> [  147.846155] [drm:drm_internal_framebuffer_create] could not create 
> framebuffer"

Thanks, this confirms that the check is too strict. I've sent a patch
reverting this as well.


Yu, I like the idea behind your changes, but unfortunately it's more
complicated than that. If you want to work on similar checks which
accurately reflect the hardware constraints, people on the amd-gfx list
should be able to help with that.


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
index 70a816dd8b4d..99b646c16311 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_display.c
@@ -537,8 +537,11 @@ amdgpu_display_user_framebuffer_create(struct drm_device *dev,
 
 	pitch = amdgpu_align_pitch(adev, pitch, cpp, false);
 	if (mode_cmd->pitches[0] != pitch) {
-		DRM_DEBUG_KMS("Invalid pitch: expecting %d but got %d\n",
-			      pitch, mode_cmd->pitches[0]);
+		struct drm_format_name_buf format_name;
+
+		DRM_ERROR("Invalid pitch: expecting %d but got %d, format %s => cpp=%d\n",
+			  pitch, mode_cmd->pitches[0],
+			  drm_get_format_name(mode_cmd->pixel_format, &format_name), cpp);
 		return ERR_PTR(-EINVAL);
 	}
 

Attachment: signature.asc
Description: OpenPGP digital signature

_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
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