Re: [Intel-gfx] [PATCH v7 1/3] gpu: drm: separate panel orientation property creating and value setting

2022-02-07 Thread Ville Syrjälä
On Tue, Feb 08, 2022 at 03:37:12PM +0800, Hsin-Yi Wang wrote: > +int drm_connector_init_panel_orientation_property( > + struct drm_connector *connector) > +{ > + struct drm_device *dev = connector->dev; > + struct drm_property *prop; > + > + prop = drm_property_create_enum(dev, DRM_

Re: [PATCH 5/8] mm: simplify freeing of devmap managed pages

2022-02-07 Thread Chaitanya Kulkarni
> -static inline bool page_is_devmap_managed(struct page *page) > +bool __put_devmap_managed_page(struct page *page); > +static inline bool put_devmap_managed_page(struct page *page) > { > if (!static_branch_unlikely(&devmap_managed_key)) > return false; > if (!is_zone_d

[PATCH] drm/amd/pm: correct hwmon power lable name

2022-02-07 Thread Yang Wang
only vangogh has 2 types of hwmon power node: "fastPPT" and "slowPPT", the other asic only has 1 type of hwmon power node: "PPT". Signed-off-by: Yang Wang --- drivers/gpu/drm/amd/pm/amdgpu_pm.c | 10 +++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/amd/pm/

Re: [PATCH 4/8] mm: move free_devmap_managed_page to memremap.c

2022-02-07 Thread Chaitanya Kulkarni
On 2/6/22 10:32 PM, Christoph Hellwig wrote: > free_devmap_managed_page has nothing to do with the code in swap.c, > move it to live with the rest of the code for devmap handling. > > Signed-off-by: Christoph Hellwig True, the only devmap code is present in the swap.c is couple of calls in my tre

Re: [PATCH 3/8] mm: remove pointless includes from

2022-02-07 Thread Chaitanya Kulkarni
On 2/6/22 10:32 PM, Christoph Hellwig wrote: > hmm.h pulls in the world for no good reason at all. Remove the > includes and push a few ones into the users instead. > > Signed-off-by: Christoph Hellwig Looks good. Reviewed-by: Chaitanya Kulkarni

Re: [PATCH v2 2/2] drm/radeon/uvd: Fix forgotten unmap buffer objects

2022-02-07 Thread Christian König
Am 08.02.22 um 04:14 schrieb zhanglianjie: after the buffer object is successfully mapped, call radeon_bo_kunmap before the function returns. Signed-off-by: zhanglianjie Reviewed-by: Christian König diff --git a/drivers/gpu/drm/radeon/radeon_uvd.c b/drivers/gpu/drm/radeon/radeon_uvd.c i

Re: [PATCH 2/8] mm: remove the __KERNEL__ guard from

2022-02-07 Thread Chaitanya Kulkarni
On 2/6/22 10:32 PM, Christoph Hellwig wrote: > __KERNEL__ ifdefs don't make sense outside of include/uapi/. > > Signed-off-by: Christoph Hellwig > --- > include/linux/mm.h | 4 > 1 file changed, 4 deletions(-) > > diff --git a/include/linux/mm.h b/include/linux/mm.h > index 213cc569b1922

Re: [PATCH 1/8] mm: remove a pointless CONFIG_ZONE_DEVICE check in memremap_pages

2022-02-07 Thread Chaitanya Kulkarni
On 2/6/22 10:32 PM, Christoph Hellwig wrote: > memremap.c is only built when CONFIG_ZONE_DEVICE is set, so remove > the superflous extra check. > > Signed-off-by: Christoph Hellwig > --- > mm/memremap.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/mm/memremap.c b

[PATCH v2] drm/amd/pm: fix hwmon node of power1_label create issue

2022-02-07 Thread Yang Wang
it will cause hwmon node of power1_label is not created. v2: the hwmon node of "power1_lable" is always needed for all ASICs. and the patch will remove ASIC type check for "power1_label". Fixes: ae07970a06 ("drm/amd/pm: add support for hwmon control of slow and fast PPT limit on vangogh") Signe

RE: [PATCH] drm/amd/pm: fix hwmon node of power1_label create issue

2022-02-07 Thread Feng, Kenneth
Reviewed-by: Kenneth Feng -Original Message- From: Wang, Yang(Kevin) Sent: Tuesday, February 8, 2022 2:35 PM To: amd-gfx@lists.freedesktop.org Cc: Feng, Kenneth ; Lazar, Lijo ; Hou, Xiaomeng (Matthew) ; Wang, Yang(Kevin) Subject: [PATCH] drm/amd/pm: fix hwmon node of power1_label cr

Re: [PATCH 6/8] mm: don't include in

2022-02-07 Thread Christoph Hellwig
On Mon, Feb 07, 2022 at 04:19:29PM -0500, Felix Kuehling wrote: > > Am 2022-02-07 um 01:32 schrieb Christoph Hellwig: >> Move the check for the actual pgmap types that need the free at refcount >> one behavior into the out of line helper, and thus avoid the need to >> pull memremap.h into mm.h. >>

[PATCH] drm/amd/pm: fix hwmon node of power1_label create issue

2022-02-07 Thread Yang Wang
fix typo issue about "power1_label", it will cause hwmon node of power1_label is not created. Fixes: ae07970a06 ("drm/amd/pm: add support for hwmon control of slow and fast PPT limit on vangogh") Signed-off-by: Yang Wang --- drivers/gpu/drm/amd/pm/amdgpu_pm.c | 2 +- 1 file changed, 1 insertio

Re: [RFC v3 06/12] drm/amdgpu: Drop hive->in_reset

2022-02-07 Thread Lazar, Lijo
On 1/26/2022 4:07 AM, Andrey Grodzovsky wrote: Since we serialize all resets no need to protect from concurrent resets. Signed-off-by: Andrey Grodzovsky Reviewed-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 19 +-- drivers/gpu/drm/amd/amdgpu/amdgpu

RE: [PATCH] drm/amd/pm: disable GetPptLimit message in sriov mode

2022-02-07 Thread Feng, Kenneth
Reviewed-by: Kenneth Feng -Original Message- From: Wang, Yang(Kevin) Sent: Tuesday, February 8, 2022 12:30 PM To: amd-gfx@lists.freedesktop.org Cc: Feng, Kenneth ; Lazar, Lijo ; Nikolic, Marina ; Wang, Yang(Kevin) Subject: [PATCH] drm/amd/pm: disable GetPptLimit message in sriov mod

Re: [PATCH] drm/amd/pm: disable GetPptLimit message in sriov mode

2022-02-07 Thread Lazar, Lijo
On 2/8/2022 10:00 AM, Yang Wang wrote: the pmfw is not allowed GetPptLimit message in virtualzation mode. Maybe rephrase as - "PPT limit cannot be queried from VF". Reviewed-by: Lijo Lazar Thanks, Lijo Fixes: 3e4a01689daa ("drm/amd/pm: Enable sysfs required by rocm-smi toolfor One VF

[PATCH] drm/amd/pm: disable GetPptLimit message in sriov mode

2022-02-07 Thread Yang Wang
the pmfw is not allowed GetPptLimit message in virtualzation mode. Fixes: 3e4a01689daa ("drm/amd/pm: Enable sysfs required by rocm-smi toolfor One VF mode") Signed-off-by: Yang Wang --- drivers/gpu/drm/amd/pm/swsmu/smu11/sienna_cichlid_ppt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-

[PATCH 4/7] drm/amd/pm: fulfill Navi1x implementations for DriverSmuConfig setting

2022-02-07 Thread Evan Quan
Fulfill the implementations for DriverSmuConfig setting on Navi1x. Signed-off-by: Evan Quan Change-Id: I244766a893b4070dfdf171451f6338d33572ec1d --- .../gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c | 50 +++ 1 file changed, 50 insertions(+) diff --git a/drivers/gpu/drm/amd/pm/swsmu

[PATCH 6/7] drm/amdgpu: disable MMHUB PG for Picasso

2022-02-07 Thread Evan Quan
MMHUB PG needs to be disabled for Picasso for stability reasons. Signed-off-by: Evan Quan Change-Id: Iea0ec757582a764ab5a000d7cc411fb814ffb19f --- drivers/gpu/drm/amd/amdgpu/soc15.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/soc15.c b/driv

[PATCH 7/7] drm/amd/pm: fix some OEM SKU specific stability issues

2022-02-07 Thread Evan Quan
Add a quirk in sienna_cichlid_ppt.c to fix some OEM SKU specific stability issues. Signed-off-by: Evan Quan Change-Id: I172c6429c54253788dbf28f7acf877375f2bfc5b --- .../amd/pm/swsmu/smu11/sienna_cichlid_ppt.c | 32 ++- 1 file changed, 31 insertions(+), 1 deletion(-) diff --git

[PATCH 5/7] drm/amd/pm: fulfill Sienna_Cichlid implementations for DriverSmuConfig setting

2022-02-07 Thread Evan Quan
Fulfill the implementations for DriverSmuConfig setting on Sienna_Cichlid. Signed-off-by: Evan Quan Change-Id: Ic519c8d4fcfeefdda79ba9ed01b235824d76e40f --- .../amd/pm/swsmu/smu11/sienna_cichlid_ppt.c | 54 +++ 1 file changed, 54 insertions(+) diff --git a/drivers/gpu/drm/amd/

[PATCH 2/7] drm/amd/pm: fulfill the support for DriverSmuConfig table

2022-02-07 Thread Evan Quan
Enable the support for DriverSmuConfig table on Navi1x and Sienna_Cichlid. Signed-off-by: Evan Quan Change-Id: Ie510f8b06b7a4910b1574b6e9affa875805ef868 --- drivers/gpu/drm/amd/pm/swsmu/inc/amdgpu_smu.h | 1 + drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c | 9 + .../gpu/drm

[PATCH 3/7] drm/amd/pm: correct the default DriverSmuConfig table settings

2022-02-07 Thread Evan Quan
For Some ASICs, with the PMFW default settings, we may see the power consumption reported via metrics table is "Very Erratic". With the socket power alpha filter set as 10/100ms, we can correct that issue. Signed-off-by: Evan Quan Change-Id: Ia352579e1cc7a531cb1de5c835fe5bf132d5dd20 --- drivers/

[PATCH 1/7] drm/amd/pm: correct UMD pstate clocks for Dimgrey Cavefish and Beige Goby

2022-02-07 Thread Evan Quan
Correct the UMD pstate profiling clocks for Dimgrey Cavefish and Beige Goby. Signed-off-by: Evan Quan Change-Id: I74fdbcf2cfa11f97ae16e4921449ab7cdb7e43c9 --- .../amd/pm/swsmu/smu11/sienna_cichlid_ppt.c | 26 +++ .../amd/pm/swsmu/smu11/sienna_cichlid_ppt.h | 8 ++ 2 file

Re: [PATCH 7/8] mm: remove the extra ZONE_DEVICE struct page refcount

2022-02-07 Thread Ralph Campbell
On 2/6/22 22:32, Christoph Hellwig wrote: ZONE_DEVICE struct pages have an extra reference count that complicates the code for put_page() and several places in the kernel that need to check the reference count to see that a page is not being used (gup, compaction, migration, etc.). Clean up the c

Re: start sorting out the ZONE_DEVICE refcount mess

2022-02-07 Thread Logan Gunthorpe
On 2022-02-06 11:32 p.m., Christoph Hellwig wrote: > Hi all, > > this series removes the offset by one refcount for ZONE_DEVICE pages > that are freed back to the driver owning them, which is just device > private ones for now, but also the planned device coherent pages > and the ehanced p2p on

Re: [PATCH 6/8] mm: don't include in

2022-02-07 Thread Dan Williams
On Sun, Feb 6, 2022 at 10:33 PM Christoph Hellwig wrote: > > Move the check for the actual pgmap types that need the free at refcount > one behavior into the out of line helper, and thus avoid the need to > pull memremap.h into mm.h. Looks good to me assuming the compile bots agree. Reviewed-by:

Re: [PATCH 5/8] mm: simplify freeing of devmap managed pages

2022-02-07 Thread Dan Williams
On Sun, Feb 6, 2022 at 10:33 PM Christoph Hellwig wrote: > > Make put_devmap_managed_page return if it took charge of the page > or not and remove the separate page_is_devmap_managed helper. Looks good to me: Reviewed-by: Dan Williams

Re: [PATCHv2 3/3] drm/amdkfd: Consolidate MQD manager functions

2022-02-07 Thread Felix Kuehling
Am 2022-02-07 um 14:53 schrieb Mukul Joshi: A few MQD manager functions are duplicated for all versions of MQD manager. Remove this duplication by moving the common functions into kfd_mqd_manager.c file. Signed-off-by: Mukul Joshi Reviewed-by: Felix Kuehling --- v1->v2: - Add "kfd_" pre

Re: [PATCHv2 1/3] drm/amdkfd: Fix TLB flushing in KFD SVM with no HWS

2022-02-07 Thread Felix Kuehling
Am 2022-02-07 um 14:52 schrieb Mukul Joshi: With no HWS, TLB flushing will not work in SVM code. Fix this by calling kfd_flush_tlb() which works for both HWS and no HWS case. Signed-off-by: Mukul Joshi Reviewed-by: Philip Yang --- v1->v2: - Don't pass adev to svm_range_map_to_gpu(). drive

Re: [PATCHv2 2/3] drm/amdkfd: Remove unused old debugger implementation

2022-02-07 Thread Felix Kuehling
Am 2022-02-07 um 14:52 schrieb Mukul Joshi: Cleanup the kfd code by removing the unused old debugger implementation. You can add here, that address watch was only ever implemented in the upstream driver for GFXv7 (Kaveri). The user mode tools runtime using this API was never open-sourced. Wor

Re: [PATCH 6/8] mm: don't include in

2022-02-07 Thread Felix Kuehling
Am 2022-02-07 um 01:32 schrieb Christoph Hellwig: Move the check for the actual pgmap types that need the free at refcount one behavior into the out of line helper, and thus avoid the need to pull memremap.h into mm.h. Signed-off-by: Christoph Hellwig The amdkfd part looks good to me. It l

Re: [PATCH] drm/amd/pm: add missing prototypes to amdgpu_dpm_internal

2022-02-07 Thread Alex Deucher
Applied. Thanks! On Sun, Feb 6, 2022 at 10:30 PM Quan, Evan wrote: > > [AMD Official Use Only] > > Thanks for the fix! > Reviewed-by: Evan Quan > > > -Original Message- > > From: Maíra Canal > > Sent: Thursday, February 3, 2022 8:40 AM > > To: Quan, Evan ; Deucher, Alexander > > ; Koen

Re: [PATCH] drm/amd/pm: fix error handling

2022-02-07 Thread Alex Deucher
Applied. Thanks! Alex On Sun, Feb 6, 2022 at 10:04 PM Quan, Evan wrote: > > [AMD Official Use Only] > > Reviewed-by: Evan Quan > > > -Original Message- > > From: t...@redhat.com > > Sent: Saturday, February 5, 2022 11:00 PM > > To: Quan, Evan ; Deucher, Alexander > > ; Koenig, Christi

[PATCHv2 3/3] drm/amdkfd: Consolidate MQD manager functions

2022-02-07 Thread Mukul Joshi
A few MQD manager functions are duplicated for all versions of MQD manager. Remove this duplication by moving the common functions into kfd_mqd_manager.c file. Signed-off-by: Mukul Joshi --- v1->v2: - Add "kfd_" prefix to functions moved to kfd_mqd_manager.c. - Also, suffix "_cp" to function name

[PATCHv2 2/3] drm/amdkfd: Remove unused old debugger implementation

2022-02-07 Thread Mukul Joshi
Cleanup the kfd code by removing the unused old debugger implementation. Only a small piece of resetting wavefronts is kept and is moved to kfd_device_queue_manager.c Signed-off-by: Mukul Joshi --- v1->v2: - Rename AMDKFD_IOC_DBG_* to AMDKFD_IOC_DBG_*_DEPRECATED. - Cleanup address_watch_disable()

[PATCHv2 1/3] drm/amdkfd: Fix TLB flushing in KFD SVM with no HWS

2022-02-07 Thread Mukul Joshi
With no HWS, TLB flushing will not work in SVM code. Fix this by calling kfd_flush_tlb() which works for both HWS and no HWS case. Signed-off-by: Mukul Joshi Reviewed-by: Philip Yang --- v1->v2: - Don't pass adev to svm_range_map_to_gpu(). drivers/gpu/drm/amd/amdkfd/kfd_svm.c | 24 ++-

Re: [PATCH 8/8] fsdax: depend on ZONE_DEVICE || FS_DAX_LIMITED

2022-02-07 Thread Jason Gunthorpe
On Mon, Feb 07, 2022 at 07:32:49AM +0100, Christoph Hellwig wrote: > Add a depends on ZONE_DEVICE support or the s390-specific limited DAX > support, as one of the two is required at runtime for fsdax code to > actually work. > > Signed-off-by: Christoph Hellwig > --- > fs/Kconfig | 1 + > 1 fil

Re: [PATCH 6/8] mm: don't include in

2022-02-07 Thread Jason Gunthorpe
On Mon, Feb 07, 2022 at 07:32:47AM +0100, Christoph Hellwig wrote: > Move the check for the actual pgmap types that need the free at refcount > one behavior into the out of line helper, and thus avoid the need to > pull memremap.h into mm.h. > > Signed-off-by: Christoph Hellwig > --- > arch/arm6

Re: [PATCH 5/8] mm: simplify freeing of devmap managed pages

2022-02-07 Thread Jason Gunthorpe
On Mon, Feb 07, 2022 at 07:32:46AM +0100, Christoph Hellwig wrote: > Make put_devmap_managed_page return if it took charge of the page > or not and remove the separate page_is_devmap_managed helper. > > Signed-off-by: Christoph Hellwig > --- > include/linux/mm.h | 34 ++--

Re: [PATCH 4/8] mm: move free_devmap_managed_page to memremap.c

2022-02-07 Thread Jason Gunthorpe
On Mon, Feb 07, 2022 at 07:32:45AM +0100, Christoph Hellwig wrote: > free_devmap_managed_page has nothing to do with the code in swap.c, > move it to live with the rest of the code for devmap handling. > > Signed-off-by: Christoph Hellwig > --- > include/linux/mm.h | 1 - > mm/memremap.c |

Re: [PATCH 2/8] mm: remove the __KERNEL__ guard from

2022-02-07 Thread Jason Gunthorpe
On Mon, Feb 07, 2022 at 07:32:43AM +0100, Christoph Hellwig wrote: > __KERNEL__ ifdefs don't make sense outside of include/uapi/. > > Signed-off-by: Christoph Hellwig > --- > include/linux/mm.h | 4 > 1 file changed, 4 deletions(-) Reviewed-by: Jason Gunthorpe Jason

Re: [PATCH 1/8] mm: remove a pointless CONFIG_ZONE_DEVICE check in memremap_pages

2022-02-07 Thread Jason Gunthorpe
On Mon, Feb 07, 2022 at 07:32:42AM +0100, Christoph Hellwig wrote: > memremap.c is only built when CONFIG_ZONE_DEVICE is set, so remove > the superflous extra check. > > Signed-off-by: Christoph Hellwig > --- > mm/memremap.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) Reviewed-by:

Re: [PATCH 7/8] mm: remove the extra ZONE_DEVICE struct page refcount

2022-02-07 Thread Jason Gunthorpe
On Mon, Feb 07, 2022 at 07:32:48AM +0100, Christoph Hellwig wrote: > ZONE_DEVICE struct pages have an extra reference count that complicates > the code for put_page() and several places in the kernel that need to > check the reference count to see that a page is not being used (gup, > compaction, m

Re: [PATCH 4/8] mm: move free_devmap_managed_page to memremap.c

2022-02-07 Thread Dan Williams
On Sun, Feb 6, 2022 at 10:33 PM Christoph Hellwig wrote: > > free_devmap_managed_page has nothing to do with the code in swap.c, > move it to live with the rest of the code for devmap handling. > Looks good. Reviewed-by: Dan Williams

Re: [PATCH 05/23] drm/amd/display: Fix color encoding mismatch

2022-02-07 Thread Harry Wentland
On 2022-02-07 13:57, Harry Wentland wrote: > On 2022-02-07 11:34, Maxime Ripard wrote: >> The amdgpu KMS driver calls drm_plane_create_color_properties() with a >> default encoding set to BT709. >> >> However, the core will ignore it and the driver doesn't force it in its >> plane state reset ho

Re: [PATCH 05/23] drm/amd/display: Fix color encoding mismatch

2022-02-07 Thread Harry Wentland
On 2022-02-07 11:34, Maxime Ripard wrote: > The amdgpu KMS driver calls drm_plane_create_color_properties() with a > default encoding set to BT709. > > However, the core will ignore it and the driver doesn't force it in its > plane state reset hook, so the initial value will be 0, which represents

[PATCH 05/23] drm/amd/display: Fix color encoding mismatch

2022-02-07 Thread Maxime Ripard
The amdgpu KMS driver calls drm_plane_create_color_properties() with a default encoding set to BT709. However, the core will ignore it and the driver doesn't force it in its plane state reset hook, so the initial value will be 0, which represents BT601. Fix the mismatch by using an initial value

Re: [PATCH 6/8] mm: don't include in

2022-02-07 Thread Logan Gunthorpe
On 2022-02-06 11:32 p.m., Christoph Hellwig wrote: > Move the check for the actual pgmap types that need the free at refcount > one behavior into the out of line helper, and thus avoid the need to > pull memremap.h into mm.h. > > Signed-off-by: Christoph Hellwig I've noticed mm/memcontrol.c u

Re: [PATCH 2/8] mm: remove the __KERNEL__ guard from

2022-02-07 Thread Dan Williams
On Sun, Feb 6, 2022 at 10:33 PM Christoph Hellwig wrote: > > __KERNEL__ ifdefs don't make sense outside of include/uapi/. Yes. Reviewed-by: Dan Williams

Re: [PATCH 1/8] mm: remove a pointless CONFIG_ZONE_DEVICE check in memremap_pages

2022-02-07 Thread Dan Williams
On Sun, Feb 6, 2022 at 10:33 PM Christoph Hellwig wrote: > > memremap.c is only built when CONFIG_ZONE_DEVICE is set, so remove > the superflous extra check. Looks good to me. Reviewed-by: Dan Williams

Re: [PATCH 3/3] drm/amdkfd: Consolidate MQD manager functions

2022-02-07 Thread Felix Kuehling
Am 2022-02-07 um 12:50 schrieb Joshi, Mukul: [AMD Official Use Only] -Original Message- From: Kuehling, Felix Sent: Monday, February 7, 2022 10:43 AM To: Joshi, Mukul ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH 3/3] drm/amdkfd: Consolidate MQD manager functions Am 2022-02

RE: [PATCH 3/3] drm/amdkfd: Consolidate MQD manager functions

2022-02-07 Thread Joshi, Mukul
[AMD Official Use Only] > -Original Message- > From: Kuehling, Felix > Sent: Monday, February 7, 2022 10:43 AM > To: Joshi, Mukul ; amd-gfx@lists.freedesktop.org > Subject: Re: [PATCH 3/3] drm/amdkfd: Consolidate MQD manager functions > > > Am 2022-02-04 um 18:45 schrieb Mukul Joshi:

Re: [PATCH 2/3] drm/amdkfd: Remove unused old debugger implementation

2022-02-07 Thread Felix Kuehling
Am 2022-02-07 um 12:10 schrieb Joshi, Mukul: [AMD Official Use Only] -Original Message- From: Kuehling, Felix Sent: Monday, February 7, 2022 10:33 AM To: Joshi, Mukul ; amd-gfx@lists.freedesktop.org Subject: Re: [PATCH 2/3] drm/amdkfd: Remove unused old debugger implementation Am 2

RE: [PATCH 2/3] drm/amdkfd: Remove unused old debugger implementation

2022-02-07 Thread Joshi, Mukul
[AMD Official Use Only] > -Original Message- > From: Kuehling, Felix > Sent: Monday, February 7, 2022 10:33 AM > To: Joshi, Mukul ; amd-gfx@lists.freedesktop.org > Subject: Re: [PATCH 2/3] drm/amdkfd: Remove unused old debugger > implementation > > > Am 2022-02-04 um 18:45 schrieb Muk

[PATCH 1/1] Revert "drm/amd/display: Basic support with device ID"

2022-02-07 Thread Jasdeep Dhillon
This reverts commit 733a212f20dfa14fa20814f21526fb180f25fdd8. --- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 24 ++- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.h | 2 +- .../gpu/drm/amd/display/dc/core/dc_resource.c | 2 +- .../gpu/drm/amd/display/include/dal_asic_id.h | 3

[PATCH 0/1] Revert Patch from Feb 7 2022 Patchset v2

2022-02-07 Thread Jasdeep Dhillon
Reverting patch Jasdeep Dhillon (1): Revert "drm/amd/display: Basic support with device ID" .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 24 ++- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.h | 2 +- .../gpu/drm/amd/display/dc/core/dc_resource.c | 2 +- .../gpu/drm/amd/disp

Re: [PATCH 0/1] Revert Patch from Feb 7 2022 Patchset

2022-02-07 Thread Deucher, Alexander
[AMD Official Use Only] Acked-by: Alex Deucher From: amd-gfx on behalf of Jasdeep Dhillon Sent: Monday, February 7, 2022 11:23 AM To: amd-gfx@lists.freedesktop.org Cc: Wang, Chao-kai (Stylon) ; Li, Sun peng (Leo) ; Wentland, Harry ; Zhuo, Qingqing (Lillian)

Re: Minimal GPU setup

2022-02-07 Thread Deucher, Alexander
[AMD Official Use Only] Most of the register programming in evergreen_gpu_init is required. That code handles things like harvesting (e.g., disabling bad hardware resources) and setting sane asic specific settings in some registers. If you don't do it, work may get scheduled to bad or incorre

Re: [PATCH 13/13] drm/amd/display: Basic support with device ID

2022-02-07 Thread Dhillon, Jasdeep
[AMD Official Use Only] Hi Alex, I already merged the branch but I have sent you the revert patch. Regards, Jasdeep From: Alex Deucher Sent: February 7, 2022 10:58 AM To: Dhillon, Jasdeep Cc: amd-gfx list ; Wang, Chao-kai (Stylon) ; Liu, Charlene ; Logush, Oli

[PATCH 0/1] Revert Patch from Feb 7 2022 Patchset

2022-02-07 Thread Jasdeep Dhillon
Reverting patch Jasdeep Dhillon (1): Revert "drm/amd/display: Basic support with device ID" .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 24 ++- .../gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.h | 2 +- .../gpu/drm/amd/display/dc/core/dc_resource.c | 2 +- .../gpu/drm/amd/disp

Re: [PATCH 13/13] drm/amd/display: Basic support with device ID

2022-02-07 Thread Alex Deucher
On Fri, Feb 4, 2022 at 11:33 PM Jasdeep Dhillon wrote: > > From: Oliver Logush > > [why] > To get the the cyan_skillfish check working NAK. This is still not correct. > > Reviewed-by: Charlene Liu > Reviewed-by: Charlene Liu > Acked-by: Jasdeep Dhillon > Signed-off-by: Oliver Logush > ---

Re: [PATCH 3/3] drm/amdkfd: Consolidate MQD manager functions

2022-02-07 Thread Felix Kuehling
Am 2022-02-04 um 18:45 schrieb Mukul Joshi: A few MQD manager functions are duplicated for all versions of MQD manager. Remove this duplication by moving the common functions into kfd_mqd_manager.c file. Signed-off-by: Mukul Joshi --- drivers/gpu/drm/amd/amdkfd/kfd_mqd_manager.c | 63 +

Re: [PATCH 2/3] drm/amdkfd: Remove unused old debugger implementation

2022-02-07 Thread Felix Kuehling
Am 2022-02-04 um 18:45 schrieb Mukul Joshi: Cleanup the kfd code by removing the unused old debugger implementation. Only a small piece of resetting wavefronts is kept and is moved to kfd_device_queue_manager.c Signed-off-by: Mukul Joshi We should update include/uapi/linux/kfd_ioctl.h as we

Re: [PATCH 1/3] drm/amdkfd: Fix TLB flushing in KFD SVM with no HWS

2022-02-07 Thread philip yang
On 2022-02-04 6:45 p.m., Mukul Joshi wrote: With no HWS, TLB flushing will not work in SVM code. Fix this by calling kfd_flush_tlb() which works for both HWS and no HWS case. Signed-off-by: Mukul Joshi With below change to remove one extra calling

Re: [PATCH] drm/amdkfd: enable heavy-weight TLB flush on Vega20

2022-02-07 Thread Eric Huang
Hi Guchun, SDMA FW team confirms MI50/VG20 doesn't have the same bug as MI100, which cases asic hang issue when running RVS test. If this change makes KFDMemoryTest failed, please fill a Jira and assign to me. Thanks, Eric On 2022-02-07 08:01, Chen, Guchun wrote: [Public] Hi Eric, Are you

RE: [PATCH 00/14] DC Patchset, Feb 7 2022 v2

2022-02-07 Thread Wheeler, Daniel
[Public] Hi all, This week this patchset was tested on the following systems: HP Envy 360, with Ryzen 5 4500U, with the following display types: eDP 1080p 60hz, 4k 60hz (via USB-C to DP/HDMI), 1440p 144hz (via USB-C to DP/HDMI), 1680*1050 60hz (via USB-C to DP and then DP to DVI/VGA) Leno

Re: [PATCH] drm/amdgpu: reserve the pd while cleaning up PRTs

2022-02-07 Thread Felix Kuehling
Am 2022-02-07 um 03:07 schrieb Christian König: We want to have lockdep annotation here, so make sure that we reserve the PD while removing PRTs even if it isn't strictly necessary since the VM object is about to be destroyed anyway. Signed-off-by: Christian König Reviewed-by: Felix Kuehling

Re: [PATCH 2/2] drm/amdkfd: use unmap all queues for poison consumption

2022-02-07 Thread Felix Kuehling
Am 2022-02-07 um 02:27 schrieb Tao Zhou: Replace reset queue for specific PASID with unmap all queues, reset queue could break CP scheduler. Signed-off-by: Tao Zhou The series is Reviewed-by: Felix Kuehling --- drivers/gpu/drm/amd/amdkfd/kfd_int_process_v9.c | 3 +-- 1 file changed,

Re: [PATCH 3/8] mm: remove pointless includes from

2022-02-07 Thread Jason Gunthorpe
On Mon, Feb 07, 2022 at 07:32:44AM +0100, Christoph Hellwig wrote: > hmm.h pulls in the world for no good reason at all. Remove the > includes and push a few ones into the users instead. > > Signed-off-by: Christoph Hellwig > --- > drivers/gpu/drm/amd/amdkfd/kfd_migrate.c | 1 + > drivers/gpu/d

RE: [PATCH] drm/amdkfd: enable heavy-weight TLB flush on Vega20

2022-02-07 Thread Chen, Guchun
[Public] Hi Eric, Are you sure that there is no FW requirement for this patch on Vega20? KFDMemory test failed by this commit. Regards, Guchun -Original Message- From: amd-gfx On Behalf Of Alex Deucher Sent: Tuesday, January 25, 2022 4:08 AM To: Huang, JinHuiEric Cc: amd-gfx list Su

[PATCH v2 1/3] migrate.c: Remove vma check in migrate_vma_setup()

2022-02-07 Thread Alistair Popple
migrate_vma_setup() checks that a valid vma is passed so that the page tables can be walked to find the pfns associated with a given address range. However in some cases the pfns are already known, such as when migrating device coherent pages during pin_user_pages() meaning a valid vma isn't requir

[PATCH] drm/radeon: Avoid open coded arithmetic in memory allocation

2022-02-07 Thread Christophe JAILLET
kmalloc_array()/kcalloc() should be used to avoid potential overflow when a multiplication is needed to compute the size of the requested memory. So turn a kzalloc()+explicit size computation into an equivalent kcalloc(). Signed-off-by: Christophe JAILLET --- drivers/gpu/drm/radeon/radeon_atomb

[PATCH v2 2/3] mm/gup.c: Migrate device coherent pages when pinning instead of failing

2022-02-07 Thread Alistair Popple
Currently any attempts to pin a device coherent page will fail. This is because device coherent pages need to be managed by a device driver, and pinning them would prevent a driver from migrating them off the device. However this is no reason to fail pinning of these pages. These are coherent and

Re: [PATCH 2/3] mm/gup.c: Migrate device coherent pages when pinning instead of failing

2022-02-07 Thread Alistair Popple
On Wednesday, 2 February 2022 2:03:01 AM AEDT Felix Kuehling wrote: > > Am 2022-02-01 um 02:05 schrieb Alistair Popple: > > Currently any attempts to pin a device coherent page will fail. This is > > because device coherent pages need to be managed by a device driver, and > > pinning them would pr

[PATCH v2 0/3] Migrate device coherent pages on get_user_pages()

2022-02-07 Thread Alistair Popple
Device coherent pages represent memory on a coherently attached device such as a GPU which is usually under the control of a driver. These pages should not be pinned as the driver needs to be able to move pages as required. Currently this is enforced by failing any attempt to pin a device coherent

Re: Waiting for fences timed out on MacBook Pro 2019

2022-02-07 Thread Tomasz Moń
On Mon, Jul 12, 2021 at 11:56 AM Tomasz Moń wrote: > I am having trouble getting Linux to run on MacBook Pro 2019 with > Radeon Pro Vega 20 4 GB. Basically as soon as graphical user interface > starts, the whole system freezes. This happens with every Linux kernel > version I have tried over the l

[PATCH v2 3/3] tools: add hmm gup test for long term pinned device pages

2022-02-07 Thread Alistair Popple
From: Alex Sierra The intention is to test device coherent type pages that have been called through get user pages with PIN_LONGTERM flag set. These pages should get migrated back to normal system memory. Signed-off-by: Alex Sierra Signed-off-by: Alistair Popple Reviewed-by: Felix Kuehling -

[PATCH] drm/amd/pm: fix error handling

2022-02-07 Thread trix
From: Tom Rix clang static analysis reports this error amdgpu_smu.c:2289:9: warning: Called function pointer is null (null dereference) return smu->ppt_funcs->emit_clk_levels( ^~~~ There is a logic error in the earlier check of emit_clk_levels

[PATCH] drm/amdgpu: reserve the pd while cleaning up PRTs

2022-02-07 Thread Christian König
We want to have lockdep annotation here, so make sure that we reserve the PD while removing PRTs even if it isn't strictly necessary since the VM object is about to be destroyed anyway. Signed-off-by: Christian König --- drivers/gpu/drm/amd/amdgpu/amdgpu_kms.c | 6 -- 1 file changed, 4 inser

Re: [PATCH] drm/radeon: Avoid open coded arithmetic in memory allocation

2022-02-07 Thread Christian König
Am 05.02.22 um 18:38 schrieb Christophe JAILLET: kmalloc_array()/kcalloc() should be used to avoid potential overflow when a multiplication is needed to compute the size of the requested memory. So turn a kzalloc()+explicit size computation into an equivalent kcalloc(). Signed-off-by: Christoph