If core DMA-buf framework forgets to call dma_fence_enable_signaling()
before calling the dma_fence_is_signaled(). To handle this scenario on
debug kernel the DMA_FENCE_FLAG_ENABLE_SIGNAL_BIT needs to be checked
before checking the actual signaling status.
Signed-off-by: Arvind Yadav
calling dma_fence_is_signaled(). To avoid this scenario
on debug kernel, check the DMA_FENCE_FLAG_ENABLE_SIGNAL_BIT bit
status before checking the MA_FENCE_FLAG_SIGNALED_BIT bit status
to confirm that software signaling is enabled.
Arvind Yadav (4):
dma-buf: Check status of enable-signaling bit
bit status to confirm that enable_signaling
is enabled.
Signed-off-by: Arvind Yadav
---
include/linux/dma-fence.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/include/linux/dma-fence.h b/include/linux/dma-fence.h
index 775cdc0b4f24..60c0e935c0b5 100644
--- a/include/linux/dma-fence.h
Here's on debug adding an enable_signaling callback for finished
fences and enabling software signaling for finished fence.
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/scheduler/sched_fence.c | 12
drivers/gpu/drm/scheduler/sched_main.c | 4 +++-
2 files change
meout.
Signed-off-by: Arvind Yadav
---
drivers/dma-buf/dma-fence.c | 17 +
1 file changed, 17 insertions(+)
diff --git a/drivers/dma-buf/dma-fence.c b/drivers/dma-buf/dma-fence.c
index 066400ed8841..0a67af945ef8 100644
--- a/drivers/dma-buf/dma-fence.c
+++ b/drivers/dma-buf/dma-fe
Here's on debug adding an enable_signaling callback for fences
and enabling software signaling for selftest.
Signed-off-by: Arvind Yadav
---
drivers/dma-buf/st-dma-fence-chain.c | 17 +
drivers/dma-buf/st-dma-fence-unwrap.c | 54 +++
drivers/dma-buf/s
calling dma_fence_is_signaled(). To avoid this scenario
on debug kernel, check the DMA_FENCE_FLAG_ENABLE_SIGNAL_BIT bit
status before checking the MA_FENCE_FLAG_SIGNALED_BIT bit status
to confirm that software signaling is enabled.
Arvind Yadav (4):
[PATCH v2 1/4] drm/sched: Enable signaling for
Here's enabling software signaling for finished fence.
Signed-off-by: Arvind Yadav
---
Changes in v1 :
1- Addressing Christian's comment to remove CONFIG_DEBUG_FS check from
this patch.
2- The version of this patch is also changed and previously
it was [PATCH 2/4]
---
drive
Here's on debug enabling software signaling for the stub fence
which is always signaled. This fence should enable software
signaling otherwise the AMD GPU scheduler will cause a GPU reset
due to a GPU scheduler cleanup activity timeout.
Signed-off-by: Arvind Yadav
---
Changes in v
Here's on debug enabling software signaling for selftest.
Signed-off-by: Arvind Yadav
---
Changes in v1 :
1- Addressing Christian's comment to remove unnecessary callback.
2- Replacing CONFIG_DEBUG_WW_MUTEX_SLOWPATH instead of CONFIG_DEBUG_FS.
3- The version of this patch is also c
bit status to confirm that enable_signaling
is enabled.
Signed-off-by: Arvind Yadav
---
Changes in v1 :
1- Addressing Christian's comment to replace
CONFIG_DEBUG_WW_MUTEX_SLOWPATH instead of CONFIG_DEBUG_FS.
2- As per Christian's comment moving this patch at last so
The version of thi
resulting in not
signaled fences much more obvious.
Arvind Yadav (6):
[PATCH v3 1/6] dma-buf: Remove the signaled bit status check
[PATCH v3 2/6] dma-buf: set signaling bit for the stub fence
[PATCH v3 3/6] dma-buf: Enable signaling on fence for selftests
[PATCH v3 4/6] drm/amdgpu: Enable
Remove the signaled bit status check because it is returning
early when the fence is already signaled and
__dma_fence_enable_signaling is checking the status of signaled
bit again.
Signed-off-by: Arvind Yadav
---
Changes in v1, v2: This new patch was not part of previous series.
---
drivers
Here's setting software signaling bit for the stub fence
which is always signaled. If this fence signaling bit is
not set then the AMD GPU scheduler will cause a GPU reset
due to a GPU scheduler cleanup activity timeout.
Signed-off-by: Arvind Yadav
---
Changes in v1 :
1- Addressing Christ
Here's enabling software signaling on fence for selftest.
Signed-off-by: Arvind Yadav
---
Changes in v1 :
1- Addressing Christian's comment to remove unnecessary callback.
2- Replacing CONFIG_DEBUG_WW_MUTEX_SLOWPATH instead of CONFIG_DEBUG_FS.
3- The version of this patch is also c
Here's enabling software signaling on fence because
amdgpu_ctx_add_fence() is checking the status of fence
and emits warning.
Signed-off-by: Arvind Yadav
---
Changes in v1, v2: This new patch was not part of previous series.
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c | 2 ++
1 file ch
Using the parent fence instead of the finished fence
to get the job status. This change is to avoid GPU
scheduler timeout error which can cause GPU reset.
Signed-off-by: Arvind Yadav
---
changes in v1,v2 - Enable signaling for finished fence in sche_main()
is removed
---
drivers/gpu/drm
resulting in not
signaled fences much more obvious.
Signed-off-by: Arvind Yadav
---
Changes in v1,v2 :
1- Addressing Christian's comment to replace
CONFIG_DEBUG_WW_MUTEX_SLOWPATH instead of CONFIG_DEBUG_FS.
2- As per Christian's comment moving this patch at last so
The version of this pat
resulting in not
signaled fences much more obvious.
Arvind Yadav (6):
[PATCH v4 1/6] dma-buf: Remove the signaled bit status check
[PATCH v4 2/6] dma-buf: set signaling bit for the stub fence
[PATCH v4 3/6] dma-buf: Enable signaling on fence for selftests
[PATCH v4 4/6] dma-buf: dma_fence_wait
Remove the signaled bit status check because it is returning
early when the fence is already signaled and
__dma_fence_enable_signaling is checking the status of signaled
bit again.
Signed-off-by: Arvind Yadav
Reviewed-by: Christian König
---
Changes in v1, v2: This new patch was not part of
Here's setting software signaling bit for the stub fence
which is always signaled. If this fence signaling bit is
not set then the AMD GPU scheduler will cause a GPU reset
due to a GPU scheduler cleanup activity timeout.
Signed-off-by: Arvind Yadav
Reviewed-by: Christian König
---
Chang
dma_fence_wait() should always enable signaling even
when the fence is already signaled.
Signed-off-by: Arvind Yadav
---
Changes in v1..v3: This new patch was not part of previous series.
---
drivers/dma-buf/dma-fence.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a
Here's enabling software signaling on fence for selftest.
Signed-off-by: Arvind Yadav
Reviewed-by: Christian König
---
Changes in v1 :
1- Addressing Christian's comment to remove unnecessary callback.
2- Replacing CONFIG_DEBUG_WW_MUTEX_SLOWPATH instead of CONFIG_DEBUG_FS.
3- The
resulting in not
signaled fences much more obvious.
Signed-off-by: Arvind Yadav
---
Changes in v1,v2 :
1- Addressing Christian's comment to replace
CONFIG_DEBUG_WW_MUTEX_SLOWPATH instead of CONFIG_DEBUG_FS.
2- As per Christian's comment moving this patch at last so
The version of thi
Using the parent fence instead of the finished fence
to get the job status. This change is to avoid GPU
scheduler timeout error which can cause GPU reset.
Signed-off-by: Arvind Yadav
Reviewed-by: Andrey Grodzovsky
---
changes in v1,v2 - Enable signaling for finished fence in sche_main()
is
make any implementation bugs resulting in not
signaled fences much more obvious.
Arvind Yadav (3):
[PATCH 1/3] dma-buf: Remove the signaled bit status check
[PATCH 2/3] dma-buf: Enable signaling on fence for sw_sync
[PATCH 3/3] dma-buf: Check status of enable-signaling bit on debug
drivers
bugs resulting in not
signaled fences much more obvious.
Signed-off-by: Arvind Yadav
---
include/linux/dma-fence.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/include/linux/dma-fence.h b/include/linux/dma-fence.h
index 775cdc0b4f24..5156dc6be0a6 100644
--- a/include/linux/dma
Here's enabling software signaling on fence for sw_sync.
Signed-off-by: Arvind Yadav
---
drivers/dma-buf/sw_sync.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/dma-buf/sw_sync.c b/drivers/dma-buf/sw_sync.c
index 348b3a9170fa..d2a52ceac14e 100644
--- a/drivers/dma-buf/sw_s
Remove the extra signaled bit status check because
it is returning early when the fence is already signaled and
__dma_fence_enable_signaling is checking the status of signaled
bit again.
Signed-off-by: Arvind Yadav
---
drivers/dma-buf/dma-fence.c | 5 -
1 file changed, 5 deletions(-)
diff
NULL check is added for s_fence->parent.
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/scheduler/sched_main.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/scheduler/sched_main.c
b/drivers/gpu/drm/scheduler/sched_main.c
index 4f2395d1a791..6684d8846
NULL check is added for s_fence->parent.
Signed-off-by: Arvind Yadav
---
changes in v1 : subject 'drm/sched:' was missing.
drivers/gpu/drm/scheduler/sched_main.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/scheduler/sched_main.c
b/dr
amdgpu]
amdgpu_ih_process+0x86/0x100 [amdgpu]
amdgpu_irq_handler+0x24/0x60 [amdgpu]
__handle_irq_event_percpu+0x4b/0x190
handle_irq_event_percpu+0x15/0x50
handle_irq_event+0x39/0x60
handle_edge_irq+0xaf/0x210
__common_interrupt+0x6e/0x110
common_interrupt+0xc1/0xe0
Signed-off-by: Arvind
-by: Arvind Yadav
---
drivers/dma-buf/dma-fence-array.c | 6 ++
include/linux/dma-fence-array.h | 2 ++
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/dma-buf/dma-fence-array.c
b/drivers/dma-buf/dma-fence-array.c
index 5c8a7084577b..3ebb6c5fa287 100644
--- a/drivers
x24/0x60 [amdgpu]
__handle_irq_event_percpu+0x4b/0x190
handle_irq_event_percpu+0x15/0x50
handle_irq_event+0x39/0x60
handle_edge_irq+0xaf/0x210
__common_interrupt+0x6e/0x110
common_interrupt+0xc1/0xe0
Signed-off-by: Arvind Yadav
---
Changes in v2: Moving 'dma_fence_cb' callback from
jo
e_irq+0xaf/0x210
__common_interrupt+0x6e/0x110
common_interrupt+0xc1/0xe0
Signed-off-by: Arvind Yadav
---
Changes in v2: Moving 'dma_fence_cb' callback from
job(struct drm_sched_job) to scheduler fence(struct drm_sched_fence)
instead of adding NULL check for s_fe
b7/0x210 [amdgpu]
amdgpu_ih_process+0x86/0x100 [amdgpu]
amdgpu_irq_handler+0x24/0x60 [amdgpu]
__handle_irq_event_percpu+0x4b/0x190
handle_irq_event_percpu+0x15/0x50
handle_irq_event+0x39/0x60
handle_edge_irq+0xaf/0x210
__common_interrupt+0x6e/0x110
common_interrupt+0xc1/0xe0
Signe
Here's enabling software signaling on fence for syncobj.
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/drm_syncobj.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/drm_syncobj.c b/drivers/gpu/drm/drm_syncobj.c
index 0c2be8360525..2ab7a1a9eeb4 100644
--- a/driver
ater.
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/scheduler/sched_main.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/gpu/drm/scheduler/sched_main.c
b/drivers/gpu/drm/scheduler/sched_main.c
index 820c0c5544e1..ea7bfa99d6c9 100644
--- a/drivers/gpu/drm/sched
submitted job. This can optimize the power
performance when the particular workload is on.
Arvind Yadav (3):
drm/amdgpu: Add new api to switch on/off power profile mode
drm/amdgpu: Switch on/off GPU workload profile
Revert "drm/amd/amdgpu: switch on/off vcn power profile mode"
drive
This patch adds a function which will allow to
change the GPU power profile based on a submitted job.
This can optimize the power performance when the
workload is on.
Cc: Shashank Sharma
Cc: Christian Koenig
Cc: Alex Deucher
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/Makefile
This patch is to switch the GPU workload profile based
on the submitted job. The workload profile is reset to
default when the job is done.
Cc: Shashank Sharma
Cc: Christian Koenig
Cc: Alex Deucher
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | 5 +
1 file
-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 14 ++
1 file changed, 2 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
index 2d94f1b63bd6..70777fcfa626 100644
--- a/drivers/gpu/drm/amd
.
Arvind Yadav (7):
drm/amdgpu: Added init/fini functions for workload
drm/amdgpu: Add new function to set GPU power profile
drm/amdgpu: Add new function to put GPU power profile
drm/amdgpu: Add suspend function to clear the GPU power profile.
drm/amdgpu: Switch on/off GPU workload profile
The'struct amdgpu_smu_workload' initialization/cleanup
functions is added by this patch.
v2:
- Splitting big patch into separate patches.
- Added new fini function.
Cc: Shashank Sharma
Cc: Christian Koenig
Cc: Alex Deucher
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgp
Cc: Christian Koenig
Cc: Alex Deucher
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/amdgpu_workload.c | 56 +++
drivers/gpu/drm/amd/include/amdgpu_workload.h | 3 +
2 files changed, 59 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_workload.c
b
function
will cancel this work and set the GPU power profile based on
preferences.
v2:
- Splitting workload_profile_set and workload_profile_put
into two separate patches.
- Addressed review comment.
Cc: Shashank Sharma
Cc: Christian Koenig
Cc: Alex Deucher
Signed-off-by: Arvind Yadav
This patch adds a suspend function that will clear the GPU
power profile before going into suspend state.
v2:
- Add the new suspend function based on review comment.
Cc: Shashank Sharma
Cc: Christian Koenig
Cc: Alex Deucher
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu
This patch is to switch the GPU workload profile based
on the submitted job. The workload profile is reset to
default when the job is done.
Cc: Shashank Sharma
Cc: Christian Koenig
Cc: Alex Deucher
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | 5 +
1 file
This patch switches the GPU workload mode to/from
compute mode, while submitting compute workload.
Cc: Christian Koenig
Signed-off-by: Alex Deucher
Signed-off-by: Shashank Sharma
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/amdgpu_amdkfd.c | 8 +---
1 file changed, 5
Deucher
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 14 ++
1 file changed, 2 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
index 2d94f1b63bd6..70777fcfa626 100644
--- a/drivers
Free memory mapping, if hdmi_probe is not successful.
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/exynos/exynos_hdmi.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c
b/drivers/gpu/drm/exynos/exynos_hdmi.c
index 2275efe..9b7857b 100644
--- a
Free memory mapping, if hdmi_probe is not successful.
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/exynos/exynos_hdmi.c |5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/gpu/drm/exynos/exynos_hdmi.c
b/drivers/gpu/drm/exynos/exynos_hdmi.c
index 2275efe..ba28dec 100644
--- a
Here, If ioremap_nocache will fail. It will return NULL.
Kernel can run into a NULL-pointer dereference.
This error check will avoid NULL pointer dereference.
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers
Here, If ioremap_nocache will fail. It will return NULL.
Kernel can run into a NULL-pointer dereference.
This error check will avoid NULL pointer dereference.
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/radeon/radeon_ttm.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu
/drm/exynos/exynos_hdmi.o
File size after constify hdmi_match_types.
textdata bss dec hex filename
13318 176 0 1349434b6 drivers/gpu/drm/exynos/exynos_hdmi.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/exynos/exynos_hdmi.c | 2 +-
1 file changed, 1
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/exynos/exynos_mixer.c | 10 +-
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/exynos/exynos_mixer.c
b/drivers/gpu/drm/exynos/exynos_mixer.c
index 25edb63..778cae2 100644
--- a/drivers/gpu/drm/exynos
/drm/tilcdc/tilcdc_drv.o
File size after constify tilcdc_of_match.
textdata bss dec hex filename
5445 800 062451865 drivers/gpu/drm/tilcdc/tilcdc_drv.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/tilcdc/tilcdc_drv.c | 4 +---
1 file changed, 1 insertion
!= %d)\n",
^
drivers/gpu/drm/sti/sti_hqvdp.c:930:3: warning: format ‘%d’ expects argument of
type ‘int’, but argument 9 has type ‘size_t’ [-Wformat=]
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/sti/sti_hqvdp.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/
/drm/sti/sti_hqvdp.o
File size after constify hqvdp_of_match:
textdata bss dec hex filename
16229 224 0 164534045 drivers/gpu/drm/sti/sti_hqvdp.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/sti/sti_hqvdp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
/drm/sti/sti_dvo.o
File size after constify dvo_of_match:
textdata bss dec hex filename
4638 248 048861316 drivers/gpu/drm/sti/sti_dvo.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/sti/sti_dvo.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
Hi,
On Tuesday 20 June 2017 04:59 PM, Benjamin Gaignard wrote:
2017-06-20 6:55 GMT+02:00 Arvind Yadav :
Replace '%d' by '%zu' and '%lu' to fix the following compilation warning:-
drivers/gpu/drm/sti/sti_hqvdp.c: In function ‘sti_hqvdp_start_xp70’:
drivers
/drm/armada/armada_crtc.o
File size after constify armada_lcd_of_match:
textdata bss dec hex filename
9220 328 09548254c drivers/gpu/drm/armada/armada_crtc.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/armada/armada_crtc.c | 2 +-
1 file changed, 1
Undo preparation of a clock source, if sti_hqvdp_start_xp70 and
sti_hqvdp_atomic_check are not successful.
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/sti/sti_hqvdp.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/sti/sti_hqvdp.c b/drivers/gpu/drm/sti/sti_hqvdp.c
/drm/tilcdc/tilcdc_tfp410.o
File size after constify:
textdata bss dec hex filename
1880 176 02056 808 drivers/gpu/drm/tilcdc/tilcdc_tfp410.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/tilcdc/tilcdc_tfp410.c | 4 +---
1 file changed, 1 insertion(+), 3
/drm/tilcdc/tilcdc_panel.o
File size after constify:
textdata bss dec hex filename
1915 176 02091 82b drivers/gpu/drm/tilcdc/tilcdc_panel.o
Signed-off-by: Arvind Yadav
---
Changes in v1:
Subject was not correct.
drivers/gpu/drm/tilcdc
/drm/tilcdc/tilcdc_panel.o
File size after constify:
textdata bss dec hex filename
1915 176 02091 82b drivers/gpu/drm/tilcdc/tilcdc_panel.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/tilcdc/tilcdc_panel.c | 2 +-
1 file changed, 1 insertion(+), 1
/drm/tilcdc/tilcdc_tfp410.o
File size after constify:
textdata bss dec hex filename
1880 176 02056 808 drivers/gpu/drm/tilcdc/tilcdc_tfp410.o
Signed-off-by: Arvind Yadav
---
Changes in v1:
Subject was not correct.
drivers/gpu/drm/tilcdc
/udl/udl_dmabuf.o
File size After adding 'const':
textdata bss dec hex filename
2114 0 02114 842 drivers/gpu/drm/udl/udl_dmabuf.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/udl/udl_dmabuf.c | 2 +-
1 file changed, 1 insertion(+), 1 deletio
48e4 drivers/gpu/drm/radeon/radeon_display.o
File size After adding 'const':
textdata bss dec hex filename
18660 0 0 1866048e4 drivers/gpu/drm/radeon/radeon_display.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/radeon/radeon_disp
eba drivers/gpu/drm/armada/armada_overlay.o
File size After adding 'const':
textdata bss dec hex filename
3722 48 03770 eba drivers/gpu/drm/armada/armada_overlay.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/armada/armada_overlay.c | 2
/vmwgfx_buffer.o
File size After adding 'const':
textdata bss dec hex filename
3456 512 163984 f90 drivers/gpu/drm/vmwgfx/vmwgfx_buffer.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/vmwgfx/vmwgfx_buffer.c | 24
1 file c
/virtgpu_ttm.o
File size After adding 'const':
textdata bss dec hex filename
2347 152 02499 9c3 drivers/gpu/drm/virtio/virtgpu_ttm.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/virtio/virtgpu_ttm.c | 2 +-
1 file changed, 1 insertion(+),
2885 drivers/gpu/drm/armada/armada_crtc.o
File size After adding 'const':
textdata bss dec hex filename
9757 616 0 103732885 drivers/gpu/drm/armada/armada_crtc.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/armada/armada_crtc.c | 4 ++--
1 file
/omapdrm/omap_gem_dmabuf.o
File size After adding 'const':
textdata bss dec hex filename
1352 0 01352 548
drivers/gpu/drm/omapdrm/omap_gem_dmabuf.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/omapdrm/omap_gem_dmabuf.c | 2 +-
1 file
/radeon_ttm.o
File size After adding 'const':
textdata bss dec hex filename
9267 312 136971525f3 drivers/gpu/drm/radeon/radeon_ttm.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/radeon/radeon_ttm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletio
ttm_place are not supposed to change at runtime. All functions
working with ttm_place provided by work
with const ttm_place. So mark the non-const structs as const.
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion
/qxl_ttm.o
File size After adding 'const':
textdata bss dec hex filename
3501 152 2643917 f4d drivers/gpu/drm/qxl/qxl_ttm.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/qxl/qxl_ttm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
panel-dsi-cm.o
File size After adding 'const':
textdata bss dec hex filename
4725 400 051251405 panel-dsi-cm.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/omapdrm/displays/panel-dsi-cm.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
di
drivers/gpu/drm/i915/i915_sysfs.o
File size After adding 'const':
textdata bss dec hex filename
4196 928 051241404 drivers/gpu/drm/i915/i915_sysfs.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/i915/i915_sysfs.c | 6 +++---
1 file changed, 3
1021 3fd drivers/gpu/drm/sun4i/sun4i_layer.o
File size After adding 'const':
textdata bss dec hex filename
1021 0 01021 3fd drivers/gpu/drm/sun4i/sun4i_layer.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/sun4i/sun4i_layer.c | 2
xtdata bss dec hex filename
4002 184 04186105a drivers/gpu/drm/drm_fb_cma_helper.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/drm_fb_cma_helper.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/drm_fb_cma_helper.c
b/drive
panel-sony-acx565akm.o
File size After adding 'const':
textdata bss dec hex filename
5211 496 05707164b panel-sony-acx565akm.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/omapdrm/displays/panel-sony-acx565akm.c | 2 +-
1 file changed, 1 inser
066681a0c atmel_hlcdc_plane.o
File size After adding 'const':
textdata bss dec hex filename
6218 436 0665419fe atmel_hlcdc_plane.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_plane.c | 4 ++--
1 file
panel-tpo-td043mtea1.o
File size After adding 'const':
textdata bss dec hex filename
2721 304 03025 bd1 panel-tpo-td043mtea1.o
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/omapdrm/displays/panel-tpo-td043mtea1.c | 2 +-
1 file changed, 1 inser
Here, pci_iomap can fail, handle this case and return -ENOMEM.
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/mgag200/mgag200_main.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/mgag200/mgag200_main.c
b/drivers/gpu/drm/mgag200/mgag200_main.c
index e79cbc2..fb03e30
Here, If devm_ioremap_nocache will fail. It will return NULL.
Kernel can run into a NULL-pointer dereference. This error check
will avoid NULL pointer dereference.
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/sti/sti_vtac.c |4
1 file changed, 4 insertions(+)
diff --git a/drivers
Here, If devm_ioremap_nocache will fail. It will return NULL.
Kernel can run into a NULL-pointer dereference. This error check
will avoid NULL pointer dereference.
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/sti/sti_vtg.c |4
1 file changed, 4 insertions(+)
diff --git a/drivers
.
v3:
- Addressed all review comment.
- Changed the function name from *_set() to *_get().
- Now clearing all the profile in work handler.
- Added *_clear_all function to clear all the power profile.
Arvind Yadav (7):
drm/amdgpu: Added init/fini functions for workload
drm/amdgpu: Add new function
Cc: Christian Koenig
Cc: Alex Deucher
Reviewed-by: Shashank Sharma
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/Makefile | 2 +-
drivers/gpu/drm/amd/amdgpu/amdgpu.h | 3 ++
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c| 4 ++
drivers/gpu/drm/amd/
the review comment.
- Changing the function name from *_set() to *_get().
- Now setting a power profile when refcount is zero.
Cc: Shashank Sharma
Cc: Christian Koenig
Cc: Alex Deucher
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/amdgpu_workload.c | 59
.
- Added *_clear_all function to clear all the power profile.
- scheduling delay work to clear the power profile when refcount
becomes zero.
Cc: Shashank Sharma
Cc: Christian Koenig
Cc: Alex Deucher
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/amdgpu_workload.c | 118
: Alex Deucher
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c| 2 ++
drivers/gpu/drm/amd/amdgpu/amdgpu_workload.c | 7 +++
drivers/gpu/drm/amd/include/amdgpu_workload.h | 2 ++
3 files changed, 11 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu
Sharma
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
index c3d9d75143f4..c5032762d497 100644
--- a/drivers/gpu/drm/amd/amdgpu
This patch switches the GPU workload mode to/from
compute mode, while submitting compute workload.
v3:
- Addressed the review comment about changing the
function name from *_set() to *_get().
Cc: Christian Koenig
Signed-off-by: Alex Deucher
Reviewed-by: Shashank Sharma
Signed-off-by: Arvind
Sharma
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 14 ++
1 file changed, 2 insertions(+), 12 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
index 2d94f1b63bd6..70777fcfa626 100644
--- a
32/64 bit.
Arvind Yadav (1):
drm/amdkfd: Fix unaligned doorbell absolute offset for gfx8
drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c | 11 ++-
1 file changed, 10 insertions(+), 1 deletion(-)
--
2.34.1
This patch is to adjust the absolute doorbell offset
against the doorbell id considering the doorbell
size of 32/64 bit.
Cc: Christian Koenig
Cc: Alex Deucher
Signed-off-by: Shashank Sharma
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c | 11
/64 bit.
v2:
- Addressed the review comment from Felix.
Arvind Yadav (1):
drm/amdkfd: Fix unaligned doorbell absolute offset for gfx8
drivers/gpu/drm/amd/amdkfd/kfd_device_queue_manager.c | 9 -
1 file changed, 8 insertions(+), 1 deletion(-)
--
2.34.1
This patch is to adjust the absolute doorbell offset
against the doorbell id considering the doorbell
size of 32/64 bit.
v2:
- Addressed the review comment from Felix.
Cc: Christian Koenig
Cc: Alex Deucher
Signed-off-by: Shashank Sharma
Signed-off-by: Arvind Yadav
---
drivers/gpu/drm/amd
sizes.
So that doorbell offset will be aligned based on the doorbell
size.
v2:
- Addressed the review comment from Felix.
v3:
- Adding doorbell_size as parameter to get db absolute offset.
Arvind Yadav (2):
drm/amdgpu: Adding db_size to get doorbell absolute offset
drm/amdkfd: get doorb
1 - 100 of 139 matches
Mail list logo