Re: [PATCH 5/5] amdgpu: use drm sync objects for shared semaphores (v5)

2017-05-29 Thread Christian König
Am 29.05.2017 um 09:30 schrieb Dave Airlie: From: Dave Airlie This creates a new command submission chunk for amdgpu to add in and out sync objects around the submission. Sync objects are managed via the drm syncobj ioctls. The command submission interface is enhanced

Re: [PATCH 1/3] drm/amdgpu: Really leave SI support disabled by default

2017-05-29 Thread Christian König
Am 29.05.2017 um 11:20 schrieb Michel Dänzer: From: Michel Dänzer The default option value didn't match the help text and intention. Signed-off-by: Michel Dänzer I'm still unsure about the last one. The feature parity is a good argument but

Re: [PATCH 3/3] drm/amdgpu/radeon: Use radeon by default for CIK GPUs

2017-05-29 Thread Kai Wasserbäch
Hey Michel, Michel Dänzer wrote on 29.05.2017 11:20: > From: Michel Dänzer > > Even if CONFIG_DRM_AMDGPU_CIK is enabled. > > There is no feature parity yet for CIK, in particular amdgpu doesn't > support HDMI/DisplayPort without DC. that can't be correct. I'm using

Re: [PATCH] iommu/amd: flush IOTLB for specific domains only (v3)

2017-05-29 Thread Joerg Roedel
Hi Arindam, I met Tom Lendacky last week in Nuremberg last week and he told me he is working on the same area of the code that this patch is for. His reason for touching this code was to solve some locking problems. Maybe you two can work together on a joint approach to improve this? On Mon, May

[PATCH 4/5] amdgpu/cs: split out fence dependency checking

2017-05-29 Thread Dave Airlie
From: Dave Airlie This just splits out the fence depenency checking into it's own function to make it easier to add semaphore dependencies. Reviewed-by: Christian König Signed-off-by: Dave Airlie ---

[PATCH 1/5] drm: introduce sync objects (v3)

2017-05-29 Thread Dave Airlie
From: Dave Airlie Sync objects are new toplevel drm object, that contain a pointer to a fence. This fence can be updated via command submission ioctls via drivers. There is also a generic wait obj API modelled on the vulkan wait API (with code modelled on some amdgpu code).

[PATCH 5/5] amdgpu: use drm sync objects for shared semaphores (v5)

2017-05-29 Thread Dave Airlie
From: Dave Airlie This creates a new command submission chunk for amdgpu to add in and out sync objects around the submission. Sync objects are managed via the drm syncobj ioctls. The command submission interface is enhanced with two new chunks, one for syncobj pre

drm-syncobj - mostly wait changes

2017-05-29 Thread Dave Airlie
The wait patch seemed to get the most discussion last time, so I've overhauled it. The others are mostly unchanged. Dave. ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx

[PATCH 3/5] drm/syncobj: add sync_file interaction. (v1.1)

2017-05-29 Thread Dave Airlie
From: Dave Airlie This interface allows importing the fence from a sync_file into an existing drm sync object, or exporting the fence attached to an existing drm sync object into a new sync file object. This should only be used to interact with sync files where necessary.

[PATCH 2/5] drm/syncobj: add sync obj wait interface. (v4)

2017-05-29 Thread Dave Airlie
From: Dave Airlie This interface will allow sync object to be used to back Vulkan fences. This API is pretty much the vulkan fence waiting API, and I've ported the code from amdgpu. v2: accept relative timeout, pass remaining time back to userspace. v3: return to absolute

[PATCH xf86-video-amdgpu] Use reference counting for tracking KMS framebuffer lifetimes

2017-05-29 Thread Michel Dänzer
From: Michel Dänzer References are held by the pixmaps corresponding to the FBs (so the same KMS FB can be reused as long as the pixmap exists) and by the CRTCs scanning out from them (so a KMS FB is only destroyed once it's not being scanned out anymore, preventing

[PATCH 2/3] drm/radeon: Make si_support and cik_support parameters always available

2017-05-29 Thread Michel Dänzer
From: Michel Dänzer This will allow amdgpu-pro / other out-of-tree amdgpu builds to make use of these options for using the out-of-tree amdgpu driver instead of the in-tree radeon driver in a clean way. Signed-off-by: Michel Dänzer ---

[PATCH 3/3] drm/amdgpu/radeon: Use radeon by default for CIK GPUs

2017-05-29 Thread Michel Dänzer
From: Michel Dänzer Even if CONFIG_DRM_AMDGPU_CIK is enabled. There is no feature parity yet for CIK, in particular amdgpu doesn't support HDMI/DisplayPort without DC. Signed-off-by: Michel Dänzer --- drivers/gpu/drm/amd/amdgpu/Kconfig | 8

[PATCH 1/3] drm/amdgpu: Really leave SI support disabled by default

2017-05-29 Thread Michel Dänzer
From: Michel Dänzer The default option value didn't match the help text and intention. Signed-off-by: Michel Dänzer --- Maybe this can be squashed into the commit adding this option when it goes upstream.

Re: [PATCH 2/5] drm/syncobj: add sync obj wait interface. (v3)

2017-05-29 Thread Dave Airlie
Hopefully addressing most of these in the upcoming set, some comments below. >> +/** >> + * drm_timeout_abs_to_jiffies - calculate jiffies timeout from absolute >> value >> + * >> + * @timeout_ns: timeout in ns >> + * >> + * Calculate the timeout in jiffies from an absolute timeout in ns. >> +

Re: [PATCH libdrm v4 1/1] amdgpu: move asic id table to a separate file

2017-05-29 Thread Michel Dänzer
On 30/05/17 06:16 AM, Samuel Li wrote: > From: Xiaojie Yuan I took a closer look and noticed some details (and some non-details about the amdgpu.ids file at the end). > diff --git a/amdgpu/amdgpu_asic_id.c b/amdgpu/amdgpu_asic_id.c > new file mode 100644 > index

RE: [PATCH libdrm v3 1/1] amdgpu: move asic id table to a separate file

2017-05-29 Thread Li, Samuel
Recently there are some changes internally, I can integrate them into this one. There are still discussions on going regarding the format. Sam -Original Message- From: Michel Dänzer [mailto:mic...@daenzer.net] Sent: Sunday, May 28, 2017 10:11 PM To: Li, Samuel Cc:

Re: [PATCH 3/3] drm/amdgpu/radeon: Use radeon by default for CIK GPUs

2017-05-29 Thread Michel Dänzer
On 29/05/17 11:24 PM, Kai Wasserbäch wrote: > Hey Michel, > Michel Dänzer wrote on 29.05.2017 11:20: >> From: Michel Dänzer >> >> Even if CONFIG_DRM_AMDGPU_CIK is enabled. >> >> There is no feature parity yet for CIK, in particular amdgpu doesn't >> support

Re: [PATCH 1/3] drm/amdgpu: Really leave SI support disabled by default

2017-05-29 Thread Michel Dänzer
On 30/05/17 02:18 AM, Christian König wrote: > Am 29.05.2017 um 11:20 schrieb Michel Dänzer: >> From: Michel Dänzer >> >> The default option value didn't match the help text and intention. >> >> Signed-off-by: Michel Dänzer > > I'm still unsure

Re: [PATCH 3/3] drm/amdgpu/radeon: Use radeon by default for CIK GPUs

2017-05-29 Thread Michel Dänzer
On 29/05/17 06:20 PM, Michel Dänzer wrote: > From: Michel Dänzer > > Even if CONFIG_DRM_AMDGPU_CIK is enabled. > > There is no feature parity yet for CIK, in particular amdgpu doesn't > support HDMI/DisplayPort without DC. > > Signed-off-by: Michel Dänzer

RE: [PATCH libdrm v3 1/1] amdgpu: move asic id table to a separate file

2017-05-29 Thread Li, Samuel
Understood your point. However as discussed internally before, marketing names are there for a lot of reasons; my understanding of the policy is we do not need to touch them as long as there is no error in the names and they are allowed to be public. Regards, Sam -Original Message-

[PATCH libdrm v4 1/1] amdgpu: move asic id table to a separate file

2017-05-29 Thread Samuel Li
From: Xiaojie Yuan v2: fix an off by one error and leading white spaces v3: use thread safe strtok_r(); initialize len before calling getline(); change printf() to drmMsg(); add initial amdgpu.ids v4: integrate some recent internal changes, including format changes

[PATCH] drm/amdgpu: Program ring for vce instance 1 at its own register space

2017-05-29 Thread Leo Liu
when harvest part has only instance 1 available Signed-off-by: Leo Liu --- drivers/gpu/drm/amd/amdgpu/vce_v3_0.c | 61 +++ 1 file changed, 55 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/vce_v3_0.c