Does amdgpu driver supports if CONFIG_PREEMPT is enabled

Hello, Can amdgpu driver supports if we enable CONFIG_PREEMPT in configuration? Thanks, Ayyappa ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Re: [PATCH 2/4] sync_file: add replace and export some functionality

On 15 March 2017 at 18:55, Daniel Vetter wrote: > On Wed, Mar 15, 2017 at 02:19:16PM +1000, Dave Airlie wrote: >> > >> > uabi semantics question: Should we wake up everyone when the fence gets >> > replaced? What's the khr semaphore expectation here? >> >> There are no real semantics for this case

[PATCH 4/4] amdgpu: add a new bo va function to support NULL bo

Signed-off-by: Junwei Zhang --- amdgpu/amdgpu.h| 7 +++ amdgpu/amdgpu_bo.c | 40 +++- 2 files changed, 46 insertions(+), 1 deletion(-) diff --git a/amdgpu/amdgpu.h b/amdgpu/amdgpu.h index 84ab688..a9fddd6 100644 --- a/amdgpu/amdgpu.h +++ b/amdgpu/amdg

[PATCH 3/4] amdgpu: add PRT page attribute

Signed-off-by: Junwei Zhang --- include/drm/amdgpu_drm.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/drm/amdgpu_drm.h b/include/drm/amdgpu_drm.h index 34f8282..a73b40a 100644 --- a/include/drm/amdgpu_drm.h +++ b/include/drm/amdgpu_drm.h @@ -423,6 +423,8 @@ struct drm_amdgpu_gem_

[PATCH 2/4] amdgpu: set va flag with input parameter

Signed-off-by: Junwei Zhang --- amdgpu/amdgpu_bo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/amdgpu/amdgpu_bo.c b/amdgpu/amdgpu_bo.c index 6189e5a..800b77a 100644 --- a/amdgpu/amdgpu_bo.c +++ b/amdgpu/amdgpu_bo.c @@ -941,7 +941,7 @@ int amdgpu_bo_va_op(amdgpu_bo_handle b

[PATCH 0/4] [libdrm] support PRT and replace interface

* add PRT flag and new VA op * fix flag setting for current VA op function * Why don't we set flag for current VA op function? * [RFC] implement new VA fun to support NULL bo Junwei Zhang (4): amdgpu: add new VA operations CLEAR and REPLACE amdgpu: set va flag with input parameter amdgpu

[PATCH 1/4] amdgpu: add new VA operations CLEAR and REPLACE

Signed-off-by: Junwei Zhang --- include/drm/amdgpu_drm.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/include/drm/amdgpu_drm.h b/include/drm/amdgpu_drm.h index 69a7340..34f8282 100644 --- a/include/drm/amdgpu_drm.h +++ b/include/drm/amdgpu_drm.h @@ -410,6 +410,8 @@ struct drm_amdgpu_gem_

[PATCH 2/2] drm/amdgpu: fix bo_va tracing for bo unmap

Signed-off-by: Junwei Zhang --- drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_trace.h index d5a00f7..c3182e4 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgp

[PATCH 1/2] drm/amdgpu: insert partial mappings before and after directly

Currently it may miss one page before or after the target mapping Signed-off-by: Junwei Zhang --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c | 26 -- 1 file changed, 8 insertions(+), 18 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c b/drivers/gpu/drm/amd/a

Re: [PATCH 2/2] drm/amdgpu/vi: add missing error handling when setting uvd dclk

Hi Alex, On 16 March 2017 at 02:06, Alex Deucher wrote: > Noticed-by: David Binderman In case it matters - Reported-by or Suggested-by are the more common tags. Quick search in kernel history [since 2007] shows less than ~60 instances of the above with ~20 from yourself. > Signed-off-by: Alex D

RE: [PATCH 4/4] drm/amdgpu: resize VRAM BAR for CPU access

Thanks for your info. I see. Regards, Jerry (Junwei Zhang) Linux Base Graphics SRDC Software Development _ > -Original Message- > From: Alex Deucher [mailto:alexdeuc...@gmail.com] > Sent: Thursday, March 16, 2017 10:25 > To: Zhang, Jerry > Cc: Christi

Re: [PATCH 4/4] drm/amdgpu: resize VRAM BAR for CPU access

On Wed, Mar 15, 2017 at 10:19 PM, Zhang, Jerry wrote: >> -Original Message- >> From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of >> Christian K?nig >> Sent: Wednesday, March 15, 2017 17:29 >> To: Zhou, David(ChunMing); Ayyappa Ch >> Cc: linux-...@vger.kernel.org

RE: [PATCH 4/4] drm/amdgpu: resize VRAM BAR for CPU access

> -Original Message- > From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of > Christian K?nig > Sent: Wednesday, March 15, 2017 17:29 > To: Zhou, David(ChunMing); Ayyappa Ch > Cc: linux-...@vger.kernel.org; linux-ker...@vger.kernel.org; amd- > g...@lists.freedesktop

Re: linux-4.11-rc1/drivers/gpu/drm/amd/amdgpu/vi.c: 3 bugs

On Mon, Mar 6, 2017 at 4:40 AM, David Binderman wrote: > > Hello there, > 1 > > [linux-4.11-rc1/drivers/gpu/drm/amd/amdgpu/vi.c:1041] -> > [linux-4.11-rc1/drivers/gpu/drm/amd/amdgpu/vi.c:1037]: (style) Same > expression on both sides of '|'. > > Maybe the macro AMD_CG_SUPPORT_GFX_MGLS is used tw

[PATCH 2/2] drm/amdgpu/vi: add missing error handling when setting uvd dclk

Noticed-by: David Binderman Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/vi.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/vi.c b/drivers/gpu/drm/amd/amdgpu/vi.c index 28385b8..eff123b 100644 --- a/drivers/gpu/drm/amd/amdgpu/vi.c +++ b/drivers/gp

[PATCH 1/2] drm/amdgpu/vi: remove duplicate CG flags

GFX_MGLS was added twice. Noticed-by: David Binderman Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/vi.c | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/vi.c b/drivers/gpu/drm/amd/amdgpu/vi.c index fca85f8..28385b8 100644 --- a/drivers/gpu/drm/amd/am

Re: [PATCH] drm/amdgpu: increase IH ring buffer size to avoid overflow

On Wed, Mar 15, 2017 at 5:05 PM, Andres Rodriguez wrote: > Hey Alex, Christian, > > On a slightly unrelated note. > > Have you also considered using system_highpri_wq instead of system_wq > for the delayed interrupt work? > > There is a potential for multi-ms latency for systems under high CPU > l

[PATCH 2/2] drm/amdgpu: reinstate oland workaround for sclk

Higher sclks seem to be unstable on some boards. bug: https://bugs.freedesktop.org/show_bug.cgi?id=100222 Signed-off-by: Alex Deucher Cc: sta...@vger.kernel.org --- drivers/gpu/drm/amd/amdgpu/si_dpm.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm

[PATCH 1/2] drm/radeon: reinstate oland workaround for sclk

Higher sclks seem to be unstable on some boards. bug: https://bugs.freedesktop.org/show_bug.cgi?id=100222 Signed-off-by: Alex Deucher Cc: sta...@vger.kernel.org --- drivers/gpu/drm/radeon/si_dpm.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/rad

Re: [PATCH] drm/amdgpu: add macro to retrieve timeline name v2

On Sat, Mar 11, 2017 at 11:18 AM, Christian König wrote: > Am 11.03.2017 um 16:50 schrieb Andres Rodriguez: >> >> This helps de-duplicate a long expression and removes overly long lines. >> >> v2: Rename macro and undef it >> >> Signed-off-by: Andres Rodriguez > > > Reviewed-by: Christian König

[pull] radeon and amdgpu drm-fixes-4.11

Hi Dave, A few fixes for 4.11. The following changes since commit 3f81e1340706e9a7f854808e2f580c3106805d0c: drm: mxsfb: Implement drm_panel handling (2017-03-10 11:11:14 +1000) are available in the git repository at: git://people.freedesktop.org/~agd5f/linux drm-fixes-4.11 for you to fetc

Re: [PATCH] drm/amdgpu: increase IH ring buffer size to avoid overflow

Hey Alex, Christian, On a slightly unrelated note. Have you also considered using system_highpri_wq instead of system_wq for the delayed interrupt work? There is a potential for multi-ms latency for systems under high CPU load. And that is usually the case when users are running games. Regards,

[PATCH] drm/amdgpu: increase IH ring buffer size to avoid overflow

From: "Roger.He" We originally limited the IH to 4k on tonga since it uses bus addresses directly rather than GPU MC addresses, so it needs contigous physical memory. This brings it inline with other asics. Signed-off-by: Roger.He Acked-by: Alex Deucher Signed-off-by: Alex Deucher --- drive

RE: [PATCH 4/4] drm/amdgpu: resize VRAM BAR for CPU access

> -Original Message- > From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf > Of Christian König > Sent: Wednesday, March 15, 2017 3:38 AM > To: Ayyappa Ch > Cc: linux-...@vger.kernel.org; linux-ker...@vger.kernel.org; amd- > g...@lists.freedesktop.org; platform-driver-...

Re: [PATCH 1/2] drm/amdgpu/gfx6: drop gds unrefs

Am 15.03.2017 um 14:56 schrieb Alex Deucher: Leftover from gfx7 code. gfx6 never sets up the gds buffers in the first place. Signed-off-by: Alex Deucher Reviewed-by: Christian König for the series. --- drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c | 4 1 file changed, 4 deletions(-) diff

Re: [RESEND PATCH] drm: amd: remove broken include path

On Tue, Mar 14, 2017 at 5:27 PM, Arnd Bergmann wrote: > The AMD ACP driver adds "-I../acp -I../acp/include" to the gcc command > line, which makes no sense, since these are evaluated relative to the > build directory. When we build with "make W=1", they instead cause > a warning: > > cc1: error: .

[PATCH 2/2] drm/amdgpu: don't init GDS pool if GDS size is 0

SI cards don't expose GDS as a separate pool. The CP manages GDS and the UMDs use special CP packets to allocate GDS memory. bug: https://bugzilla.kernel.org/show_bug.cgi?id=194867 Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 47 - 1

[PATCH 1/2] drm/amdgpu/gfx6: drop gds unrefs

Leftover from gfx7 code. gfx6 never sets up the gds buffers in the first place. Signed-off-by: Alex Deucher --- drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c | 4 1 file changed, 4 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c index 7259d

[PATCH umr] Add ability to write registers by address

Signed-off-by: Tom St Denis --- src/app/main.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/app/main.c b/src/app/main.c index 88b4feaf0058..72c29d6a26d4 100644 --- a/src/app/main.c +++ b/src/app/main.c @@ -216,9 +216,14 @@ int main(int argc, char **argv)

Re: [PATCH 1/2] drm/radeon/si: add dpm quirk for Oland

Am 15.03.2017 um 14:01 schrieb Alex Deucher: OLAND 0x1002:0x6604 0x1028:0x066F 0x00 seems to have problems with higher sclks. Signed-off-by: Alex Deucher Cc: sta...@vger.kernel.org Acked-by: Christian König for both patches. --- drivers/gpu/drm/radeon/si_dpm.c | 6 ++ 1 file changed

[PATCH 2/2] drm/amdgpu/si: add dpm quirk for Oland

OLAND 0x1002:0x6604 0x1028:0x066F 0x00 seems to have problems with higher sclks. Signed-off-by: Alex Deucher Cc: sta...@vger.kernel.org --- drivers/gpu/drm/amd/amdgpu/si_dpm.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/gpu/drm/amd/amdgpu/si_dpm.c b/drivers/gpu/drm/amd/amd

[PATCH 1/2] drm/radeon/si: add dpm quirk for Oland

OLAND 0x1002:0x6604 0x1028:0x066F 0x00 seems to have problems with higher sclks. Signed-off-by: Alex Deucher Cc: sta...@vger.kernel.org --- drivers/gpu/drm/radeon/si_dpm.c | 6 ++ 1 file changed, 6 insertions(+) diff --git a/drivers/gpu/drm/radeon/si_dpm.c b/drivers/gpu/drm/radeon/si_dpm.c

Re: [PATCH 4/4] drm/amdgpu: resize VRAM BAR for CPU access

No, we resize the BAR on the fly during driver load without help from the BIOS or VBIOS. Christian. Am 15.03.2017 um 11:42 schrieb Ayyappa Ch: It also needs any support from VBIOS side ? I mean PCIe large bar support? Thanks, Ayyappa. On Wed, Mar 15, 2017 at 1:07 PM, Christian König wrote:

Re: [PATCH 4/4] drm/amdgpu: resize VRAM BAR for CPU access

It also needs any support from VBIOS side ? I mean PCIe large bar support? Thanks, Ayyappa. On Wed, Mar 15, 2017 at 1:07 PM, Christian König wrote: > Carizzo is an APU and resizing BARs isn't needed nor supported there. The > CPU can access the full stolen VRAM directly on that hardware. > > As

Re: [PATCH libdrm] libdrm/amdgpu: add interface for kernel semaphores

Hi Dave, Barring the other discussions, allow me to put a couple of trivial suggestions: Please re-wrap the long lines to follow existing code style. On 14 March 2017 at 00:50, Dave Airlie wrote: > @@ -882,6 +894,12 @@ int amdgpu_cs_submit(amdgpu_context_handle context, >

Re: [PATCH 4/4] amdgpu: use sync file for shared semaphores

On Wed, Mar 15, 2017 at 10:43:01AM +0100, Christian König wrote: > Am 15.03.2017 um 10:01 schrieb Daniel Vetter: > > On Tue, Mar 14, 2017 at 10:50:54AM +1000, Dave Airlie wrote: > > > From: Dave Airlie > > > > > > This creates a new interface for amdgpu with ioctls to > > > create/destroy/import

Re: [PATCH] drm/amdgpu: fix the clearing wb size

Am 15.03.2017 um 08:30 schrieb Huang Rui: The clearing wb size should be the one that it is assigned. Signed-off-by: Huang Rui Reviewed-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/

Re: [PATCH 4/4] amdgpu: use sync file for shared semaphores

Am 15.03.2017 um 10:01 schrieb Daniel Vetter: On Tue, Mar 14, 2017 at 10:50:54AM +1000, Dave Airlie wrote: From: Dave Airlie This creates a new interface for amdgpu with ioctls to create/destroy/import and export shared semaphores using sem object backed by the sync_file code. The semaphores a

Re: [PATCH] drm/amd/amdgpu: Fix debugfs reg read/write address width

Am 15.03.2017 um 10:35 schrieb Tom St Denis: The MMIO space is wider now so we mask the lower 22 bits instead of 18. Signed-off-by: Tom St Denis Reviewed-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --

Re: [PATCH libdrm] libdrm/amdgpu: add interface for kernel semaphores

Am 15.03.2017 um 09:48 schrieb Daniel Vetter: On Wed, Mar 15, 2017 at 01:01:19AM +0100, Marek Olšák wrote: While it's nice that you are all having fun here, I don't think that's the way to communicate this. The truth is, if AMD had an open source driver using the semaphores (e.g. Vulkan) and if

[PATCH] drm/amd/amdgpu: Fix debugfs reg read/write address width

The MMIO space is wider now so we mask the lower 22 bits instead of 18. Signed-off-by: Tom St Denis --- 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/amdg

Re: [PATCH 4/4] drm/amdgpu: resize VRAM BAR for CPU access

Yes, exactly that. Christian. Am 15.03.2017 um 09:25 schrieb Zhou, David(ChunMing): Does that means we don't need invisible vram later? David -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Wednesday, March 15, 20

Re: [PATCH 4/4] amdgpu: use sync file for shared semaphores

On Tue, Mar 14, 2017 at 10:50:54AM +1000, Dave Airlie wrote: > From: Dave Airlie > > This creates a new interface for amdgpu with ioctls to > create/destroy/import and export shared semaphores using > sem object backed by the sync_file code. The semaphores > are not installed as fd (except for ex

Re: [PATCH 2/4] sync_file: add replace and export some functionality

On Wed, Mar 15, 2017 at 02:19:16PM +1000, Dave Airlie wrote: > > > > uabi semantics question: Should we wake up everyone when the fence gets > > replaced? What's the khr semaphore expectation here? > > There are no real semantics for this case, you either wait the semaphore and > replace it with N

Re: [PATCH libdrm] libdrm/amdgpu: add interface for kernel semaphores

On Wed, Mar 15, 2017 at 01:01:19AM +0100, Marek Olšák wrote: > While it's nice that you are all having fun here, I don't think that's > the way to communicate this. > > The truth is, if AMD had an open source driver using the semaphores > (e.g. Vulkan) and if the libdrm semaphore code was merged,

Re: [rfc] amdgpu/sync_file shared semaphores

On Wed, Mar 15, 2017 at 11:09:39AM +1000, Dave Airlie wrote: > On 14 March 2017 at 18:53, Daniel Vetter wrote: > > On Tue, Mar 14, 2017 at 10:50:49AM +1000, Dave Airlie wrote: > >> This contains one libdrm patch and 4 kernel patches. > >> > >> The goal is to implement the Vulkan KHR_external_semap

RE: [PATCH 4/4] drm/amdgpu: resize VRAM BAR for CPU access

Does that means we don't need invisible vram later? David -Original Message- From: dri-devel [mailto:dri-devel-boun...@lists.freedesktop.org] On Behalf Of Christian K?nig Sent: Wednesday, March 15, 2017 3:38 PM To: Ayyappa Ch Cc: linux-...@vger.kernel.org; linux-ker...@vger.kernel.org;

[PATCH] drm/amdgpu: fix the clearing wb size

The clearing wb size should be the one that it is assigned. Signed-off-by: Huang Rui --- 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 ind

Re: [PATCH 4/4] drm/amdgpu: resize VRAM BAR for CPU access

Carizzo is an APU and resizing BARs isn't needed nor supported there. The CPU can access the full stolen VRAM directly on that hardware. As far as I know ASICs with support for this are Tonga, Fiji and all Polaris variants. Christian. Am 15.03.2017 um 08:23 schrieb Ayyappa Ch: Is it possibl

Re: Linux 4.11: Reported regressions as of Tuesday, 20176-03-14

On 15/03/17 04:33 PM, Christian König wrote: > Am 15.03.2017 um 02:59 schrieb Michel Dänzer: >> [ Moving this sub-thread to the amd-gfx mailing list ] >> >> On 14/03/17 07:02 PM, Thorsten Leemhuis wrote: >>> Hi! Find below my first regression report for Linux 4.11. It lists 9 >>> regressions I'm cu

Re: Linux 4.11: Reported regressions as of Tuesday, 20176-03-14

Am 15.03.2017 um 02:59 schrieb Michel Dänzer: [ Moving this sub-thread to the amd-gfx mailing list ] On 14/03/17 07:02 PM, Thorsten Leemhuis wrote: Hi! Find below my first regression report for Linux 4.11. It lists 9 regressions I'm currently aware of. [...] Desc: DRM BUG while initializing

Re: [PATCH 4/4] drm/amdgpu: resize VRAM BAR for CPU access

Is it possible on Carrizo asics? Or only supports on newer asics? On Mon, Mar 13, 2017 at 6:11 PM, Christian König wrote: > From: Christian König > > Try to resize BAR0 to let CPU access all of VRAM. > > Signed-off-by: Christian König > --- > drivers/gpu/drm/amd/amdgpu/amdgpu.h| 1 + >