Fixes coccicheck warning:
drivers/gpu/drm/amd/amdgpu/psp_v3_1.c:182:40-41: WARNING: Use ARRAY_SIZE
Reported-by: Hulk Robot
Signed-off-by: zhengbin
---
drivers/gpu/drm/amd/amdgpu/psp_v3_1.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/psp_v3_1.c
Fixes coccicheck warning:
drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c:4946:28-29: WARNING: Use
ARRAY_SIZE
Reported-by: Hulk Robot
Signed-off-by: zhengbin
---
drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/d
zhengbin (2):
drm/amd/powerplay: Use ARRAY_SIZE for smu7_profiling
drm/amdgpu: Use ARRAY_SIZE for sos_old_versions
drivers/gpu/drm/amd/amdgpu/psp_v3_1.c| 2 +-
drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
--
2.7.4
[AMD Official Use Only - Internal Distribution Only]
On Thu, Nov 21, 2019 at 11:47:15PM +0800, Zhu, Changfeng wrote:
> From: changzhu
>
> It may lose gpuvm invalidate acknowldege state across power-gating off
> cycle. To avoid this issue in gmc9/gmc10 invalidation, add semaphore acquire
> before
On Fri, Nov 22, 2019 at 7:37 AM Gerd Hoffmann wrote:
>
> Use the shared address space of the drm device (see drm_open() in
> drm_file.c) for dma-bufs too. That removes a difference betweem drm
> device mmap vmas and dma-buf mmap vmas and fixes corner cases like
> dropping ptes (using madvise(DONT
Thanks, Ray
I 'll submit the patch and continue to see the gfxhub semaphore problem.
BR,
Changfeng.
-Original Message-
From: Huang, Ray
Sent: Friday, November 22, 2019 5:16 PM
To: Zhu, Changfeng
Cc: Koenig, Christian ; Xiao, Jack
; Zhou1, Tao ; Huang, Shimmer
; amd-gfx@lists.freede
On 2019-11-22 1:33 a.m., Louis Li wrote:
> On Thu, Nov 21, 2019 at 08:47:50AM -0500, Kazlauskas, Nicholas wrote:
>> On 2019-11-21 8:40 a.m., Kazlauskas, Nicholas wrote:
>>> On 2019-11-21 3:31 a.m., Li, Ching-shih (Louis) wrote:
Hi reviewers,
What is the review result for this patch
On Wed, Oct 16, 2019 at 3:24 AM Daniel Drake wrote:
>
> On Wed, Oct 16, 2019 at 2:43 AM Alex Deucher wrote:
> > Is s2idle actually powering down the GPU?
>
> My understanding is that s2idle (at a high level) just calls all
> devices suspend routines and then puts the CPU into its deepest
> runnin
Ping?
On Tue, Nov 19, 2019 at 3:56 PM Alex Deucher wrote:
>
> dm_pp_get_clock_levels_by_type needs to add the default clocks
> to the powerplay case as well. This was accidently dropped.
>
> Fixes: b3ea88fef321de ("drm/amd/powerplay: add get_clock_by_type interface
> for display")
> Bug: https:
Pushed. Thanks!
Yong
On 2019-11-21 11:21 p.m., Liu, Zhan wrote:
Looks good to me.
Reviewed-by: Zhan Liu
-Original Message-
From: amd-gfx On Behalf Of
Yong Zhao
Sent: 2019/November/21, Thursday 4:25 PM
To: amd-gfx@lists.freedesktop.org
Cc: Zhao, Yong
Subject: [PATCH] drm/amdkfd: Re
On Fri, Nov 22, 2019 at 4:17 AM Daniel Vetter wrote:
>
> On Fri, Nov 22, 2019 at 7:37 AM Gerd Hoffmann wrote:
> >
> > Use the shared address space of the drm device (see drm_open() in
> > drm_file.c) for dma-bufs too. That removes a difference betweem drm
> > device mmap vmas and dma-buf mmap vm
Applied the series. Thanks!
Alex
On Fri, Nov 22, 2019 at 4:01 AM zhengbin wrote:
>
> zhengbin (2):
> drm/amd/powerplay: Use ARRAY_SIZE for smu7_profiling
> drm/amdgpu: Use ARRAY_SIZE for sos_old_versions
>
> drivers/gpu/drm/amd/amdgpu/psp_v3_1.c| 2 +-
> drivers/gpu/drm/amd/pow
From: Mikita Lipski
We shouldn't compare int with unsigned long to find the max value
and since we are not expecting negative value returned from
compute_offset we should make this function return unsigned long
so we can compare the values when computing rc parameters.
v2: Modified function para
No longer works on the driver.
Signed-off-by: Alex Deucher
---
MAINTAINERS | 1 -
1 file changed, 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index b63c291ad029..d518588b9879 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -856,7 +856,6 @@ S: Maintained
F: drivers/i2c/busses/i2c-am
Reviewed-by: Nicholas Kazlauskas
On 2019-11-22 1:09 p.m., Alex Deucher wrote:
Ping?
On Tue, Nov 19, 2019 at 3:56 PM Alex Deucher wrote:
dm_pp_get_clock_levels_by_type needs to add the default clocks
to the powerplay case as well. This was accidently dropped.
Fixes: b3ea88fef321de ("drm/am
Config the translation retry behavior according to noretry
kernel parameter
Change-Id: I5b91ea77715137cf8cb84e258ccdfbb19c7a4ed1
Signed-off-by: Oak Zeng
Suggested-by: Jay Cornwall
---
drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 4 +++-
drivers/gpu/drm/amd/amdgpu/mmhub_v9_4.c | 5 +++--
2 files ch
Hi Dave, Daniel,
Fixes for 5.5.
The following changes since commit 17eee668b3cad423a47c090fe2275733c55db910:
Merge tag 'drm-misc-next-fixes-2019-11-20' of
git://anongit.freedesktop.org/drm/drm-misc into drm-next (2019-11-21 10:19:45
+1000)
are available in the Git repository at:
git://pe
On 2019-11-22 3:23 p.m., Oak Zeng wrote:
Config the translation retry behavior according to noretry
kernel parameter
Change-Id: I5b91ea77715137cf8cb84e258ccdfbb19c7a4ed1
Signed-off-by: Oak Zeng
Suggested-by: Jay Cornwall
---
drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 4 +++-
drivers/gpu/drm
I'm not sure about this one. Looks like the interface is getting
needlessly more complicated. Now the caller has to keep track of the
runlist IB address and size just to pass those to another function. I
could understand this if there was a use case that needs to separate the
allocation of the
So that we can power down the GPU and audio to save power.
Reviewed-by: Takashi Iwai
Acked-by: Evan Quan
Signed-off-by: Alex Deucher
---
sound/pci/hda/patch_hdmi.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/pci/hda/patch_hdmi.c b/sound/pci/hda/patch_hdmi.c
index 795cbda32cbb..87
These are needed so we can enable runtime pm in a subsequent
patch.
Reviewed-by: Takashi Iwai
Signed-off-by: Alex Deucher
---
sound/pci/hda/hda_intel.c | 14 ++
1 file changed, 14 insertions(+)
diff --git a/sound/pci/hda/hda_intel.c b/sound/pci/hda/hda_intel.c
index 3ebc7b2a897f..7
These patches were originally part of a larger set of patches
to enabled runtime pm support on the GPU side[1]. However, the
patches are useful on their own there are really no dependencies,
other than the fact that you need both for runtime pm to kick in
on the GPU side. The GPU side will be lan
Only enable the vga_switcheroo logic on systems with the
ATPX ACPI method. This logic is not needed for asics
that are not part of a PX (PowerXpress)/HG (Hybrid Graphics)
platform.
Reviewed-by: Takashi Iwai
Acked-by: Evan Quan
Signed-off-by: Alex Deucher
---
sound/pci/hda/hda_intel.c | 45 +++
We are able to power down the GPU and audio via the GPU driver
so flag these asics as supporting runtime pm.
Reviewed-by: Takashi Iwai
Acked-by: Evan Quan
Signed-off-by: Alex Deucher
---
sound/pci/hda/hda_intel.c | 35 +++
1 file changed, 23 insertions(+), 12 de
[AMD Official Use Only - Internal Distribution Only]
Hi Felix,
There is no big picture unfortunately, just some improvements that I came to
when navigating the code.
Regarding your suggestion, I have a concern. With the original code in
unmap_queues_cpsch(), if amdkfd_fence_wait_timeout() fail
Config the translation retry behavior according to noretry
kernel parameter
Change-Id: I5b91ea77715137cf8cb84e258ccdfbb19c7a4ed1
Signed-off-by: Oak Zeng
Suggested-by: Jay Cornwall
---
drivers/gpu/drm/amd/amdgpu/mmhub_v9_4.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git
I didn't have new information about the retry_mode setting for gfx10. I just
saw the noretry kernel parameter wasn't applied to gfx10 but I didn't know
gfx10 is excluded purposely. Let me drop the gfx10 part for now.
Oak
-Original Message-
From: Kuehling, Felix
Sent: Friday, November
On 2019-11-22 5:55 p.m., Oak Zeng wrote:
Config the translation retry behavior according to noretry
kernel parameter
Change-Id: I5b91ea77715137cf8cb84e258ccdfbb19c7a4ed1
Signed-off-by: Oak Zeng
Suggested-by: Jay Cornwall
Reviewed-by: Felix Kuehling
---
drivers/gpu/drm/amd/amdgpu/mmhub_
On 11/13/19 8:46 AM, Jason Gunthorpe wrote:
On Wed, Nov 13, 2019 at 05:59:52AM -0800, Christoph Hellwig wrote:
+int mmu_interval_notifier_insert(struct mmu_interval_notifier *mni,
+ struct mm_struct *mm, unsigned long start,
+
29 matches
Mail list logo