Reviewed-by: Huang Rui <ray.hu...@amd.com>
> -----Original Message----- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > Of Deng, Emily > Sent: Wednesday, December 26, 2018 10:27 AM > To: amd-gfx@lists.freedesktop.org > Subject: RE: [PATCH 2/2] drm/amdgpu/virtual_dce: No need to pin the cursor > bo > > Ping...... > > Best wishes > Emily Deng > > > > >-----Original Message----- > >From: Deng, Emily <emily.d...@amd.com> > >Sent: Tuesday, December 25, 2018 11:53 AM > >To: Deng, Emily <emily.d...@amd.com>; amd-gfx@lists.freedesktop.org > >Subject: RE: [PATCH 2/2] drm/amdgpu/virtual_dce: No need to pin the > >cursor bo > > > >Ping...... > >Please help to review. This issue will gate promotion to mainline, as > >startx will have call trace when use virtual display. > > > >>-----Original Message----- > >>From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> On Behalf Of > >>Emily Deng > >>Sent: Monday, December 24, 2018 2:09 PM > >>To: amd-gfx@lists.freedesktop.org > >>Cc: Deng, Emily <emily.d...@amd.com> > >>Subject: [PATCH 2/2] drm/amdgpu/virtual_dce: No need to pin the cursor > >>bo > >> > >>For virtual display feature, no need to pin cursor bo. > >> > >>Signed-off-by: Emily Deng <emily.d...@amd.com> > >>--- > >> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 4 ++-- > >> 1 file changed, 2 insertions(+), 2 deletions(-) > >> > >>diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > >>b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > >>index 8a078f4..98df8e4 100644 > >>--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > >>+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > >>@@ -2798,7 +2798,7 @@ int amdgpu_device_suspend(struct drm_device > >>*dev, bool suspend, bool fbcon) > >> struct drm_framebuffer *fb = crtc->primary->fb; > >> struct amdgpu_bo *robj; > >> > >>- if (amdgpu_crtc->cursor_bo) { > >>+ if (amdgpu_crtc->cursor_bo && !adev- > >>>enable_virtual_display) { > >> struct amdgpu_bo *aobj = > >>gem_to_amdgpu_bo(amdgpu_crtc->cursor_bo); > >> r = amdgpu_bo_reserve(aobj, true); > >> if (r == 0) { > >>@@ -2906,7 +2906,7 @@ int amdgpu_device_resume(struct drm_device > *dev, > >>bool resume, bool fbcon) > >> list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) > { > >> struct amdgpu_crtc *amdgpu_crtc = > >>to_amdgpu_crtc(crtc); > >> > >>- if (amdgpu_crtc->cursor_bo) { > >>+ if (amdgpu_crtc->cursor_bo && !adev- > >>>enable_virtual_display) { > >> struct amdgpu_bo *aobj = > >>gem_to_amdgpu_bo(amdgpu_crtc->cursor_bo); > >> r = amdgpu_bo_reserve(aobj, true); > >> if (r == 0) { > >>-- > >>2.7.4 > >> > >>_______________________________________________ > >>amd-gfx mailing list > >>amd-gfx@lists.freedesktop.org > >>https://lists.freedesktop.org/mailman/listinfo/amd-gfx > _______________________________________________ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
