Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on Polaris
Is the switching automatically by driver or manually by user? Regards, Eric On 2018-02-08 11:54 AM, Zhu, Rex wrote: >In case of compute, we have auto switching mechanism. when compute ring is coming, it switches to the default compute profile or user set profile, >and switches back when it's done. So VR should be doing the same thing with compute. Auto wattman will be disabled when profile auto switching >happens. Rex: Ok, the logic is same. when computer ring begins, set manual dpm mode(pp do nothing except disable auto wattman feature), and set compute profile or custom profile. set clock range. and set auto dpm mode, switch back Best Regards Rex *From:* amd-gfx on behalf of Eric Huang *Sent:* Friday, February 9, 2018 12:47 AM *To:* amd-gfx@lists.freedesktop.org *Subject:* Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on Polaris On 2018-02-08 11:35 AM, Zhu, Rex wrote: >But in auto mode, we may want to force a particular profile within the >driver for a specific use case. E.g., for VR, we may want to select >the VR profile when the user sets the priority to high in the context >ioctl, and then switch it back to autowattman or the default profile >when the context if freed or the priority is changes. We may want to >do something similar for video when the user submits work to the >multi-media engines. > I guess it should just be >a matter of disabling autowattman and switching to a profile when the >driver needs it and then switching back when it's done. Rex: For MM, we can disable auto wattman (if enabled), and set video profile mode when enable uvd/vce dpm on smu7. restore to default when disable uvd/vce dpm when engine idle. VR, may be similar to compute case Eric have implemented, need user ctrl through sysfs. In case of compute, we have auto switching mechanism. when compute ring is coming, it switches to the default compute profile or user set profile, and switches back when it's done. So VR should be doing the same thing with compute. Auto wattman will be disabled when profile auto switching happens. Regards, Eric Enter manual dpm mode(driver disable auto wattman), and select prefered profile mode, set sclk/mclk range(smu7). when enter auto dpm mode, restore default value. >It's possible autowattman will perform ok and we >don't need to manually change the profile in the driver, but we'll >have to do some profiling to sorhat out. Rex: Idealy, if autowan feature enabled, the proflling mode should be adjusted based on workload with some time delay caused by the work queue. But need some time to verify. Best Regards Rex *From:* Alex Deucher <mailto:alexdeuc...@gmail.com> *Sent:* Thursday, February 8, 2018 11:28 PM *To:* Zhu, Rex *Cc:* amd-gfx list *Subject:* Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on Polaris On Thu, Feb 8, 2018 at 10:14 AM, Zhu, Rex <mailto:rex@amd.com> wrote: > if user select manual dpm mode,we will disabe autowattman. > When switch to auto, we will restart the auto wattman on Polaris, > Restore to default profile mode on other ASICS. > But in auto mode, we may want to force a particular profile within the driver for a specific use case. E.g., for VR, we may want to select the VR profile when the user sets the priority to high in the context ioctl, and then switch it back to autowattman or the default profile when the context if freed or the priority is changes. We may want to do something similar for video when the user submits work to the multi-media engines. It's possible autowattman will perform ok and we don't need to manually change the profile in the driver, but we'll have to do some profiling to sort that out. I guess it should just be a matter of disabling autowattman and switching to a profile when the driver needs it and then switching back when it's done. Alex > > Best Regards > Rex > > From: Alex Deucher <mailto:alexdeuc...@gmail.com> > Sent: Thursday, February 8, 2018 11:02:21 PM > To: Zhu, Rex > Cc: amd-gfx list > Subject: Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on > Polaris > > On Thu, Feb 8, 2018 at 4:14 AM, Rex Zhu <mailto:rex@amd.com> wrote: >> Change-Id: I69b24ce65ddb361a89e5ac9b197ae6df9b60a9e5 >> Signed-off-by: Rex Zhu <mailto:rex@amd.com> > > Would be good to give a brief description of what autowattman actually > does in the patch description. Also, we may need to manually select a > profile in some cases (e.g., VR). Would that be a problem with > autowattman? > > Alex >
Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on Polaris
>In case of compute, we have auto switching mechanism. when compute ring is >coming, it switches to the default compute profile or user set profile, >and >switches back when it's done. So VR should be doing the same thing with >compute. Auto wattman will be disabled when profile auto switching >happens. Rex: Ok, the logic is same. when computer ring begins, set manual dpm mode(pp do nothing except disable auto wattman feature), and set compute profile or custom profile. set clock range. and set auto dpm mode, switch back Best Regards Rex From: amd-gfx on behalf of Eric Huang Sent: Friday, February 9, 2018 12:47 AM To: amd-gfx@lists.freedesktop.org Subject: Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on Polaris On 2018-02-08 11:35 AM, Zhu, Rex wrote: >But in auto mode, we may want to force a particular profile within the >driver for a specific use case. E.g., for VR, we may want to select >the VR profile when the user sets the priority to high in the context >ioctl, and then switch it back to autowattman or the default profile >when the context if freed or the priority is changes. We may want to >do something similar for video when the user submits work to the >multi-media engines. > I guess it should just be >a matter of disabling autowattman and switching to a profile when the >driver needs it and then switching back when it's done. Rex: For MM, we can disable auto wattman (if enabled), and set video profile mode when enable uvd/vce dpm on smu7. restore to default when disable uvd/vce dpm when engine idle. VR, may be similar to compute case Eric have implemented, need user ctrl through sysfs. In case of compute, we have auto switching mechanism. when compute ring is coming, it switches to the default compute profile or user set profile, and switches back when it's done. So VR should be doing the same thing with compute. Auto wattman will be disabled when profile auto switching happens. Regards, Eric Enter manual dpm mode(driver disable auto wattman), and select prefered profile mode, set sclk/mclk range(smu7). when enter auto dpm mode, restore default value. >It's possible autowattman will perform ok and we >don't need to manually change the profile in the driver, but we'll >have to do some profiling to sorhat out. Rex: Idealy, if autowan feature enabled, the proflling mode should be adjusted based on workload with some time delay caused by the work queue. But need some time to verify. Best Regards Rex From: Alex Deucher <mailto:alexdeuc...@gmail.com> Sent: Thursday, February 8, 2018 11:28 PM To: Zhu, Rex Cc: amd-gfx list Subject: Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on Polaris On Thu, Feb 8, 2018 at 10:14 AM, Zhu, Rex <mailto:rex@amd.com> wrote: > if user select manual dpm mode,we will disabe autowattman. > When switch to auto, we will restart the auto wattman on Polaris, > Restore to default profile mode on other ASICS. > But in auto mode, we may want to force a particular profile within the driver for a specific use case. E.g., for VR, we may want to select the VR profile when the user sets the priority to high in the context ioctl, and then switch it back to autowattman or the default profile when the context if freed or the priority is changes. We may want to do something similar for video when the user submits work to the multi-media engines. It's possible autowattman will perform ok and we don't need to manually change the profile in the driver, but we'll have to do some profiling to sort that out. I guess it should just be a matter of disabling autowattman and switching to a profile when the driver needs it and then switching back when it's done. Alex > > Best Regards > Rex > > From: Alex Deucher <mailto:alexdeuc...@gmail.com> > Sent: Thursday, February 8, 2018 11:02:21 PM > To: Zhu, Rex > Cc: amd-gfx list > Subject: Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on > Polaris > > On Thu, Feb 8, 2018 at 4:14 AM, Rex Zhu > <mailto:rex@amd.com> wrote: >> Change-Id: I69b24ce65ddb361a89e5ac9b197ae6df9b60a9e5 >> Signed-off-by: Rex Zhu <mailto:rex@amd.com> > > Would be good to give a brief description of what autowattman actually > does in the patch description. Also, we may need to manually select a > profile in some cases (e.g., VR). Would that be a problem with > autowattman? > > Alex > >> --- >> drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | 4 >> 1 file changed, 4 insertions(+) >> >> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c >> b/drivers/
Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on Polaris
On 2018-02-08 11:35 AM, Zhu, Rex wrote: >But in auto mode, we may want to force a particular profile within the >driver for a specific use case. E.g., for VR, we may want to select >the VR profile when the user sets the priority to high in the context >ioctl, and then switch it back to autowattman or the default profile >when the context if freed or the priority is changes. We may want to >do something similar for video when the user submits work to the >multi-media engines. > I guess it should just be >a matter of disabling autowattman and switching to a profile when the >driver needs it and then switching back when it's done. Rex: For MM, we can disable auto wattman (if enabled), and set video profile mode when enable uvd/vce dpm on smu7. restore to default when disable uvd/vce dpm when engine idle. VR, may be similar to compute case Eric have implemented, need user ctrl through sysfs. In case of compute, we have auto switching mechanism. when compute ring is coming, it switches to the default compute profile or user set profile, and switches back when it's done. So VR should be doing the same thing with compute. Auto wattman will be disabled when profile auto switching happens. Regards, Eric Enter manual dpm mode(driver disable auto wattman), and select prefered profile mode, set sclk/mclk range(smu7). when enter auto dpm mode, restore default value. >It's possible autowattman will perform ok and we >don't need to manually change the profile in the driver, but we'll >have to do some profiling to sorhat out. Rex: Idealy, if autowan feature enabled, the proflling mode should be adjusted based on workload with some time delay caused by the work queue. But need some time to verify. Best Regards Rex *From:* Alex Deucher *Sent:* Thursday, February 8, 2018 11:28 PM *To:* Zhu, Rex *Cc:* amd-gfx list *Subject:* Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on Polaris On Thu, Feb 8, 2018 at 10:14 AM, Zhu, Rex wrote: > if user select manual dpm mode,we will disabe autowattman. > When switch to auto, we will restart the auto wattman on Polaris, > Restore to default profile mode on other ASICS. > But in auto mode, we may want to force a particular profile within the driver for a specific use case. E.g., for VR, we may want to select the VR profile when the user sets the priority to high in the context ioctl, and then switch it back to autowattman or the default profile when the context if freed or the priority is changes. We may want to do something similar for video when the user submits work to the multi-media engines. It's possible autowattman will perform ok and we don't need to manually change the profile in the driver, but we'll have to do some profiling to sort that out. I guess it should just be a matter of disabling autowattman and switching to a profile when the driver needs it and then switching back when it's done. Alex > > Best Regards > Rex > > From: Alex Deucher > Sent: Thursday, February 8, 2018 11:02:21 PM > To: Zhu, Rex > Cc: amd-gfx list > Subject: Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on > Polaris > > On Thu, Feb 8, 2018 at 4:14 AM, Rex Zhu wrote: >> Change-Id: I69b24ce65ddb361a89e5ac9b197ae6df9b60a9e5 >> Signed-off-by: Rex Zhu > > Would be good to give a brief description of what autowattman actually > does in the patch description. Also, we may need to manually select a > profile in some cases (e.g., VR). Would that be a problem with > autowattman? > > Alex > >> --- >> drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | 4 >> 1 file changed, 4 insertions(+) >> >> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c >> b/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c >> index f50b6cd..549e830 100644 >> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c >> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c >> @@ -1023,6 +1023,10 @@ int polaris_set_asic_special_caps(struct pp_hwmgr >> *hwmgr) >> phm_cap_set(hwmgr->platform_descriptor.platformCaps, >> >> PHM_PlatformCaps_TCPRamping); >> } >> + >> + if (hwmgr->feature_mask & PP_AUTOWATTMAN_MASK) >> + hwmgr->autowattman_enabled = true; /* currently only >> enabled on polaris */ >> + >> return 0; >> } >> >> -- >> 1.9.1 >> >> ___ >> amd-gfx mailing list >> amd-gfx@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/amd-gfx amd-gfx Inf
Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on Polaris
>But in auto mode, we may want to force a particular profile within the >driver for a specific use case. E.g., for VR, we may want to select >the VR profile when the user sets the priority to high in the context >ioctl, and then switch it back to autowattman or the default profile >when the context if freed or the priority is changes. We may want to >do something similar for video when the user submits work to the >multi-media engines. > I guess it should just be >a matter of disabling autowattman and switching to a profile when the >driver needs it and then switching back when it's done. Rex: For MM, we can disable auto wattman (if enabled), and set video profile mode when enable uvd/vce dpm on smu7. restore to default when disable uvd/vce dpm when engine idle. VR, may be similar to compute case Eric have implemented, need user ctrl through sysfs. Enter manual dpm mode(driver disable auto wattman), and select prefered profile mode, set sclk/mclk range(smu7). when enter auto dpm mode, restore default value. >It's possible autowattman will perform ok and we >don't need to manually change the profile in the driver, but we'll >have to do some profiling to sorhat out. Rex: Idealy, if autowan feature enabled, the proflling mode should be adjusted based on workload with some time delay caused by the work queue. But need some time to verify. Best Regards Rex From: Alex Deucher Sent: Thursday, February 8, 2018 11:28 PM To: Zhu, Rex Cc: amd-gfx list Subject: Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on Polaris On Thu, Feb 8, 2018 at 10:14 AM, Zhu, Rex wrote: > if user select manual dpm mode,we will disabe autowattman. > When switch to auto, we will restart the auto wattman on Polaris, > Restore to default profile mode on other ASICS. > But in auto mode, we may want to force a particular profile within the driver for a specific use case. E.g., for VR, we may want to select the VR profile when the user sets the priority to high in the context ioctl, and then switch it back to autowattman or the default profile when the context if freed or the priority is changes. We may want to do something similar for video when the user submits work to the multi-media engines. It's possible autowattman will perform ok and we don't need to manually change the profile in the driver, but we'll have to do some profiling to sort that out. I guess it should just be a matter of disabling autowattman and switching to a profile when the driver needs it and then switching back when it's done. Alex > > Best Regards > Rex > > From: Alex Deucher > Sent: Thursday, February 8, 2018 11:02:21 PM > To: Zhu, Rex > Cc: amd-gfx list > Subject: Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on > Polaris > > On Thu, Feb 8, 2018 at 4:14 AM, Rex Zhu wrote: >> Change-Id: I69b24ce65ddb361a89e5ac9b197ae6df9b60a9e5 >> Signed-off-by: Rex Zhu > > Would be good to give a brief description of what autowattman actually > does in the patch description. Also, we may need to manually select a > profile in some cases (e.g., VR). Would that be a problem with > autowattman? > > Alex > >> --- >> drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | 4 >> 1 file changed, 4 insertions(+) >> >> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c >> b/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c >> index f50b6cd..549e830 100644 >> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c >> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c >> @@ -1023,6 +1023,10 @@ int polaris_set_asic_special_caps(struct pp_hwmgr >> *hwmgr) >> phm_cap_set(hwmgr->platform_descriptor.platformCaps, >> >> PHM_PlatformCaps_TCPRamping); >> } >> + >> + if (hwmgr->feature_mask & PP_AUTOWATTMAN_MASK) >> + hwmgr->autowattman_enabled = true; /* currently only >> enabled on polaris */ >> + >> return 0; >> } >> >> -- >> 1.9.1 >> >> ___ >> amd-gfx mailing list >> amd-gfx@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/amd-gfx amd-gfx Info Page - freedesktop.org<https://lists.freedesktop.org/mailman/listinfo/amd-gfx> lists.freedesktop.org Subscribing to amd-gfx: Subscribe to amd-gfx by filling out the following form. Use of all freedesktop.org lists is subject to our Code of ... ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on Polaris
On Thu, Feb 8, 2018 at 10:14 AM, Zhu, Rex wrote: > if user select manual dpm mode,we will disabe autowattman. > When switch to auto, we will restart the auto wattman on Polaris, > Restore to default profile mode on other ASICS. > But in auto mode, we may want to force a particular profile within the driver for a specific use case. E.g., for VR, we may want to select the VR profile when the user sets the priority to high in the context ioctl, and then switch it back to autowattman or the default profile when the context if freed or the priority is changes. We may want to do something similar for video when the user submits work to the multi-media engines. It's possible autowattman will perform ok and we don't need to manually change the profile in the driver, but we'll have to do some profiling to sort that out. I guess it should just be a matter of disabling autowattman and switching to a profile when the driver needs it and then switching back when it's done. Alex > > Best Regards > Rex > > From: Alex Deucher > Sent: Thursday, February 8, 2018 11:02:21 PM > To: Zhu, Rex > Cc: amd-gfx list > Subject: Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on > Polaris > > On Thu, Feb 8, 2018 at 4:14 AM, Rex Zhu wrote: >> Change-Id: I69b24ce65ddb361a89e5ac9b197ae6df9b60a9e5 >> Signed-off-by: Rex Zhu > > Would be good to give a brief description of what autowattman actually > does in the patch description. Also, we may need to manually select a > profile in some cases (e.g., VR). Would that be a problem with > autowattman? > > Alex > >> --- >> drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | 4 >> 1 file changed, 4 insertions(+) >> >> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c >> b/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c >> index f50b6cd..549e830 100644 >> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c >> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c >> @@ -1023,6 +1023,10 @@ int polaris_set_asic_special_caps(struct pp_hwmgr >> *hwmgr) >> phm_cap_set(hwmgr->platform_descriptor.platformCaps, >> >> PHM_PlatformCaps_TCPRamping); >> } >> + >> + if (hwmgr->feature_mask & PP_AUTOWATTMAN_MASK) >> + hwmgr->autowattman_enabled = true; /* currently only >> enabled on polaris */ >> + >> return 0; >> } >> >> -- >> 1.9.1 >> >> ___ >> amd-gfx mailing list >> amd-gfx@lists.freedesktop.org >> https://lists.freedesktop.org/mailman/listinfo/amd-gfx ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on Polaris
if user select manual dpm mode,we will disabe autowattman. When switch to auto, we will restart the auto wattman on Polaris, Restore to default profile mode on other ASICS. Best Regards Rex From: Alex Deucher Sent: Thursday, February 8, 2018 11:02:21 PM To: Zhu, Rex Cc: amd-gfx list Subject: Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on Polaris On Thu, Feb 8, 2018 at 4:14 AM, Rex Zhu wrote: > Change-Id: I69b24ce65ddb361a89e5ac9b197ae6df9b60a9e5 > Signed-off-by: Rex Zhu Would be good to give a brief description of what autowattman actually does in the patch description. Also, we may need to manually select a profile in some cases (e.g., VR). Would that be a problem with autowattman? Alex > --- > drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c > b/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c > index f50b6cd..549e830 100644 > --- a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c > +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c > @@ -1023,6 +1023,10 @@ int polaris_set_asic_special_caps(struct pp_hwmgr > *hwmgr) > phm_cap_set(hwmgr->platform_descriptor.platformCaps, > > PHM_PlatformCaps_TCPRamping); > } > + > + if (hwmgr->feature_mask & PP_AUTOWATTMAN_MASK) > + hwmgr->autowattman_enabled = true; /* currently only enabled > on polaris */ > + > return 0; > } > > -- > 1.9.1 > > ___ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH 10/10] drm/amd/pp: Enable auto automan feature on Polaris
On Thu, Feb 8, 2018 at 4:14 AM, Rex Zhu wrote: > Change-Id: I69b24ce65ddb361a89e5ac9b197ae6df9b60a9e5 > Signed-off-by: Rex Zhu Would be good to give a brief description of what autowattman actually does in the patch description. Also, we may need to manually select a profile in some cases (e.g., VR). Would that be a problem with autowattman? Alex > --- > drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | 4 > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c > b/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c > index f50b6cd..549e830 100644 > --- a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c > +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c > @@ -1023,6 +1023,10 @@ int polaris_set_asic_special_caps(struct pp_hwmgr > *hwmgr) > phm_cap_set(hwmgr->platform_descriptor.platformCaps, > > PHM_PlatformCaps_TCPRamping); > } > + > + if (hwmgr->feature_mask & PP_AUTOWATTMAN_MASK) > + hwmgr->autowattman_enabled = true; /* currently only enabled > on polaris */ > + > return 0; > } > > -- > 1.9.1 > > ___ > amd-gfx mailing list > amd-gfx@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/amd-gfx ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx