RE: [PATCH] drm/amdgpu:fix exclusive mode game texture blank(v2)

2016-10-07 Thread Liu, Monk
Already pushed to staging because Pro team are kinda hurry for this fix The alloc size isn't need changed, cuz it always count vm_flush() in event no vm_flush needed BR Monk -Original Message- From: Deucher, Alexander Sent: Friday, September 30, 2016 9:23 PM To: Liu, Monk

Re: VRAM manager

2016-10-07 Thread Felix Kuehling
Hi Christian, I'm back from vacation and finally caught up with my email and code-review backlog. I added a comment on patch 1. I don't understand patch 2. I couldn't find where a non-0 lpfn would trigger any reallocation. I think this would have to be somewhere in ttm_bo_validate or

Re: [PATCH 1/6] drm/amdgpu: add AMDGPU_GEM_CREATE_VRAM_CONTIGUOUS flag v3

2016-10-07 Thread Felix Kuehling
On 16-09-27 05:49 AM, Christian König wrote: > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cs.c > @@ -1195,6 +1195,15 @@ int amdgpu_cs_sysvm_access_required(struct > amdgpu_cs_parser *parser) > r = amdgpu_ttm_bind(>tbo, >tbo.mem); >

[PATCH 2/2] drm/amdgpu: clarify UVD/VCE special handling for CG

2016-10-07 Thread Alex Deucher
UVD and VCE CG are handled specially, however the previous fix for this skipped late init for those blocks rather than just CG. Just protect the CG function call. No functional change since UVD and VCE don't currently utilize a late_init function. Signed-off-by: Alex Deucher

Re: [PATCH 2/6] drm/amdgpu: add info ioctl query for vce clock info

2016-10-07 Thread Alex Deucher
No, you can just check if it fails or not (i.e., the ioctl will return -EINVAL if the query is not supported). Alex On Fri, Oct 7, 2016 at 2:33 PM, StDenis, Tom wrote: > Shouldn't there be a version bump if you're adding a new command to the > ioctl? > > > Tom > > >

Re: [PATCH 2/6] drm/amdgpu: add info ioctl query for vce clock info

2016-10-07 Thread StDenis, Tom
Shouldn't there be a version bump if you're adding a new command to the ioctl? Tom From: amd-gfx on behalf of Alex Deucher Sent: Friday, October 7, 2016 14:28 To: amd-gfx@lists.freedesktop.org;

[PATCH 1/6] drm/amdgpu: move dpm related definitions to amdgpu_dpm.h

2016-10-07 Thread Alex Deucher
No intended functional change. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu.h | 449 +--- drivers/gpu/drm/amd/amdgpu/amdgpu_dpm.h | 447 +++ 2 files changed, 448 insertions(+), 448

[PATCH 6/6] drm/amdgpu: fill in vce clock info ioctl query

2016-10-07 Thread Alex Deucher
Returns the vce clock table for the user mode driver. The user mode driver can fill this data into vce clock data packet for optimal VCE DPM. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 8 1 file changed, 8 insertions(+) diff

[PATCH 5/6] drm/amdgpu/powerplay: add an implementation for get_vce_clock_table

2016-10-07 Thread Alex Deucher
Used by the powerplay dpm code. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/powerplay/amd_powerplay.c | 24 1 file changed, 24 insertions(+) diff --git a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c

[PATCH 3/6] drm/amdgpu/dpm: add new callback to fetch vce clock table

2016-10-07 Thread Alex Deucher
Will be used by the new info ioctl query. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_dpm.h | 5 + drivers/gpu/drm/amd/powerplay/inc/amd_powerplay.h | 1 + 2 files changed, 6 insertions(+) diff --git

[PATCH 2/6] drm/amdgpu: add info ioctl query for vce clock info

2016-10-07 Thread Alex Deucher
This is needed to set up the vce clock table in userspace for proper VCE DPM. Signed-off-by: Alex Deucher --- include/uapi/drm/amdgpu_drm.h | 18 ++ 1 file changed, 18 insertions(+) diff --git a/include/uapi/drm/amdgpu_drm.h

Re: [PATCH] drm/i915: Before pageflip, also wait for shared dmabuf fences.

2016-10-07 Thread Mike Lothian
Hi This has discussion has gone a little quiet Was there any agreement about what needed doing to get this working for i965/amdgpu? Regards Mike On Mon, 26 Sep 2016 at 09:04 Daniel Vetter wrote: > > On Mon, Sep 26, 2016 at 09:48:37AM +0900, Michel Dänzer wrote: > > On