Re: [PATCH] drm/amdgpu: fix build error without CONFIG_HSA_AMD (V2)
On 2019-09-12 2:58 a.m., S, Shirish wrote: > On 9/12/2019 3:29 AM, Kuehling, Felix wrote: >> On 2019-09-11 2:52 a.m., S, Shirish wrote: >>> If CONFIG_HSA_AMD is not set, build fails: >>> >>> drivers/gpu/drm/amd/amdgpu/amdgpu_device.o: In function >>> `amdgpu_device_ip_early_init': >>> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:1626: undefined reference to >>> `sched_policy' >>> >>> Use CONFIG_HSA_AMD to guard this. >>> >>> Fixes: 1abb680ad371 ("drm/amdgpu: disable gfxoff while use no H/W >>> scheduling policy") >>> >>> V2: declare sched_policy in amdgpu.h and remove changes in amdgpu_device.c >> Which branch is this for. V1 of this patch was already submitted to >> amd-staging-drm-next. So unless you're planning to revert v1 and submit >> v2, I was expecting to see a change that fixes up the previous patch, >> rather than a patch that replaces it. > Have sent a patch that fixes up previous patch as well. > > Apparently, I did not send the revert but my plan was to revert and only > then submit V2. Reverts must be reviewed too. If you're planning to submit a revert, please do include it in code review. That also avoids this type of confusion. I'll review your other patch. Thanks, Felix > > Anyways both work for me as long as the kernel builds. > > Regards, > > Shirish S > >> Regards, >> Felix >> >> >>> Signed-off-by: Shirish S >>> --- >>> drivers/gpu/drm/amd/amdgpu/amdgpu.h | 4 >>> 1 file changed, 4 insertions(+) >>> >>> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h >>> b/drivers/gpu/drm/amd/amdgpu/amdgpu.h >>> index 1030cb3..6ff02bb 100644 >>> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h >>> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h >>> @@ -169,7 +169,11 @@ extern int amdgpu_discovery; >>> extern int amdgpu_mes; >>> extern int amdgpu_noretry; >>> extern int amdgpu_force_asic_type; >>> +#ifdef CONFIG_HSA_AMD >>> extern int sched_policy; >>> +#else >>> +static const int sched_policy = KFD_SCHED_POLICY_HWS; >>> +#endif >>> >>> #ifdef CONFIG_DRM_AMDGPU_SI >>> extern int amdgpu_si_support; ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH] drm/amdgpu: fix build error without CONFIG_HSA_AMD (V2)
On 9/12/2019 3:29 AM, Kuehling, Felix wrote: > On 2019-09-11 2:52 a.m., S, Shirish wrote: >> If CONFIG_HSA_AMD is not set, build fails: >> >> drivers/gpu/drm/amd/amdgpu/amdgpu_device.o: In function >> `amdgpu_device_ip_early_init': >> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:1626: undefined reference to >> `sched_policy' >> >> Use CONFIG_HSA_AMD to guard this. >> >> Fixes: 1abb680ad371 ("drm/amdgpu: disable gfxoff while use no H/W scheduling >> policy") >> >> V2: declare sched_policy in amdgpu.h and remove changes in amdgpu_device.c > Which branch is this for. V1 of this patch was already submitted to > amd-staging-drm-next. So unless you're planning to revert v1 and submit > v2, I was expecting to see a change that fixes up the previous patch, > rather than a patch that replaces it. Have sent a patch that fixes up previous patch as well. Apparently, I did not send the revert but my plan was to revert and only then submit V2. Anyways both work for me as long as the kernel builds. Regards, Shirish S > Regards, > Felix > > >> Signed-off-by: Shirish S >> --- >>drivers/gpu/drm/amd/amdgpu/amdgpu.h | 4 >>1 file changed, 4 insertions(+) >> >> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h >> b/drivers/gpu/drm/amd/amdgpu/amdgpu.h >> index 1030cb3..6ff02bb 100644 >> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h >> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h >> @@ -169,7 +169,11 @@ extern int amdgpu_discovery; >>extern int amdgpu_mes; >>extern int amdgpu_noretry; >>extern int amdgpu_force_asic_type; >> +#ifdef CONFIG_HSA_AMD >>extern int sched_policy; >> +#else >> +static const int sched_policy = KFD_SCHED_POLICY_HWS; >> +#endif >> >>#ifdef CONFIG_DRM_AMDGPU_SI >>extern int amdgpu_si_support; -- Regards, Shirish S ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH] drm/amdgpu: fix build error without CONFIG_HSA_AMD (V2)
On 2019-09-11 2:52 a.m., S, Shirish wrote: > If CONFIG_HSA_AMD is not set, build fails: > > drivers/gpu/drm/amd/amdgpu/amdgpu_device.o: In function > `amdgpu_device_ip_early_init': > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:1626: undefined reference to > `sched_policy' > > Use CONFIG_HSA_AMD to guard this. > > Fixes: 1abb680ad371 ("drm/amdgpu: disable gfxoff while use no H/W scheduling > policy") > > V2: declare sched_policy in amdgpu.h and remove changes in amdgpu_device.c Which branch is this for. V1 of this patch was already submitted to amd-staging-drm-next. So unless you're planning to revert v1 and submit v2, I was expecting to see a change that fixes up the previous patch, rather than a patch that replaces it. Regards, Felix > > Signed-off-by: Shirish S > --- > drivers/gpu/drm/amd/amdgpu/amdgpu.h | 4 > 1 file changed, 4 insertions(+) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h > b/drivers/gpu/drm/amd/amdgpu/amdgpu.h > index 1030cb3..6ff02bb 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h > @@ -169,7 +169,11 @@ extern int amdgpu_discovery; > extern int amdgpu_mes; > extern int amdgpu_noretry; > extern int amdgpu_force_asic_type; > +#ifdef CONFIG_HSA_AMD > extern int sched_policy; > +#else > +static const int sched_policy = KFD_SCHED_POLICY_HWS; > +#endif > > #ifdef CONFIG_DRM_AMDGPU_SI > extern int amdgpu_si_support; ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
RE: [PATCH] drm/amdgpu: fix build error without CONFIG_HSA_AMD
Agree, have sent V2. My patch was actually in line to already up streamed patch: https://lkml.org/lkml/2019/8/26/201 Regards, Shirish S -Original Message- From: Kuehling, Felix Sent: Wednesday, September 11, 2019 9:09 AM To: Huang, Ray ; S, Shirish ; Deucher, Alexander ; Koenig, Christian Cc: amd-gfx@lists.freedesktop.org Subject: Re: [PATCH] drm/amdgpu: fix build error without CONFIG_HSA_AMD This is pretty ugly. See a suggestion inline. On 2019-09-10 4:12 a.m., Huang, Ray wrote: >> -Original Message- >> From: S, Shirish >> Sent: Tuesday, September 10, 2019 3:54 PM >> To: Deucher, Alexander ; Koenig, Christian >> ; Huang, Ray >> Cc: amd-gfx@lists.freedesktop.org; S, Shirish >> Subject: [PATCH] drm/amdgpu: fix build error without CONFIG_HSA_AMD >> >> If CONFIG_HSA_AMD is not set, build fails: >> >> drivers/gpu/drm/amd/amdgpu/amdgpu_device.o: In function >> `amdgpu_device_ip_early_init': >> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:1626: undefined reference >> to `sched_policy' >> >> Use CONFIG_HSA_AMD to guard this. >> >> Fixes: 1abb680ad371 ("drm/amdgpu: disable gfxoff while use no H/W >> scheduling policy") >> >> Signed-off-by: Shirish S > + Felix for his awareness. > > Reviewed-by: Huang Rui > >> --- >> drivers/gpu/drm/amd/amdgpu/amdgpu.h| 2 ++ >> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 6 +- >> 2 files changed, 7 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h >> b/drivers/gpu/drm/amd/amdgpu/amdgpu.h >> index 1030cb30720c..a1516a3ae9a8 100644 >> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h >> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h >> @@ -169,7 +169,9 @@ extern int amdgpu_discovery; extern int >> amdgpu_mes; extern int amdgpu_noretry; extern int >> amdgpu_force_asic_type; >> +#ifdef CONFIG_HSA_AMD >> extern int sched_policy; #else static const int sched_policy = KFD_SCHED_POLICY_HWS; #endif This way you don't need another set of ugly #ifdefs in amdgpu_device.c. Regards, Felix >> +#endif >> >> #ifdef CONFIG_DRM_AMDGPU_SI >> extern int amdgpu_si_support; >> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> index bd423dd64e18..2535db27f821 100644 >> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> @@ -1623,7 +1623,11 @@ static int amdgpu_device_ip_early_init(struct >> amdgpu_device *adev) >> } >> >> adev->pm.pp_feature = amdgpu_pp_feature_mask; >> -if (amdgpu_sriov_vf(adev) || sched_policy == >> KFD_SCHED_POLICY_NO_HWS) >> +if (amdgpu_sriov_vf(adev) >> +#ifdef CONFIG_HSA_AMD >> +|| sched_policy == KFD_SCHED_POLICY_NO_HWS >> +#endif >> +) >> adev->pm.pp_feature &= ~PP_GFXOFF_MASK; >> >> for (i = 0; i < adev->num_ip_blocks; i++) { >> -- >> 2.20.1 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH] drm/amdgpu: fix build error without CONFIG_HSA_AMD
This is pretty ugly. See a suggestion inline. On 2019-09-10 4:12 a.m., Huang, Ray wrote: >> -Original Message- >> From: S, Shirish >> Sent: Tuesday, September 10, 2019 3:54 PM >> To: Deucher, Alexander ; Koenig, Christian >> ; Huang, Ray >> Cc: amd-gfx@lists.freedesktop.org; S, Shirish >> Subject: [PATCH] drm/amdgpu: fix build error without CONFIG_HSA_AMD >> >> If CONFIG_HSA_AMD is not set, build fails: >> >> drivers/gpu/drm/amd/amdgpu/amdgpu_device.o: In function >> `amdgpu_device_ip_early_init': >> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:1626: undefined >> reference to `sched_policy' >> >> Use CONFIG_HSA_AMD to guard this. >> >> Fixes: 1abb680ad371 ("drm/amdgpu: disable gfxoff while use no H/W >> scheduling policy") >> >> Signed-off-by: Shirish S > + Felix for his awareness. > > Reviewed-by: Huang Rui > >> --- >> drivers/gpu/drm/amd/amdgpu/amdgpu.h| 2 ++ >> drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 6 +- >> 2 files changed, 7 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h >> b/drivers/gpu/drm/amd/amdgpu/amdgpu.h >> index 1030cb30720c..a1516a3ae9a8 100644 >> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h >> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h >> @@ -169,7 +169,9 @@ extern int amdgpu_discovery; extern int >> amdgpu_mes; extern int amdgpu_noretry; extern int >> amdgpu_force_asic_type; >> +#ifdef CONFIG_HSA_AMD >> extern int sched_policy; #else static const int sched_policy = KFD_SCHED_POLICY_HWS; #endif This way you don't need another set of ugly #ifdefs in amdgpu_device.c. Regards, Felix >> +#endif >> >> #ifdef CONFIG_DRM_AMDGPU_SI >> extern int amdgpu_si_support; >> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> index bd423dd64e18..2535db27f821 100644 >> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c >> @@ -1623,7 +1623,11 @@ static int amdgpu_device_ip_early_init(struct >> amdgpu_device *adev) >> } >> >> adev->pm.pp_feature = amdgpu_pp_feature_mask; >> -if (amdgpu_sriov_vf(adev) || sched_policy == >> KFD_SCHED_POLICY_NO_HWS) >> +if (amdgpu_sriov_vf(adev) >> +#ifdef CONFIG_HSA_AMD >> +|| sched_policy == KFD_SCHED_POLICY_NO_HWS >> +#endif >> +) >> adev->pm.pp_feature &= ~PP_GFXOFF_MASK; >> >> for (i = 0; i < adev->num_ip_blocks; i++) { >> -- >> 2.20.1 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
RE: [PATCH] drm/amdgpu: fix build error without CONFIG_HSA_AMD
> -Original Message- > From: S, Shirish > Sent: Tuesday, September 10, 2019 3:54 PM > To: Deucher, Alexander ; Koenig, Christian > ; Huang, Ray > Cc: amd-gfx@lists.freedesktop.org; S, Shirish > Subject: [PATCH] drm/amdgpu: fix build error without CONFIG_HSA_AMD > > If CONFIG_HSA_AMD is not set, build fails: > > drivers/gpu/drm/amd/amdgpu/amdgpu_device.o: In function > `amdgpu_device_ip_early_init': > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c:1626: undefined > reference to `sched_policy' > > Use CONFIG_HSA_AMD to guard this. > > Fixes: 1abb680ad371 ("drm/amdgpu: disable gfxoff while use no H/W > scheduling policy") > > Signed-off-by: Shirish S + Felix for his awareness. Reviewed-by: Huang Rui > --- > drivers/gpu/drm/amd/amdgpu/amdgpu.h| 2 ++ > drivers/gpu/drm/amd/amdgpu/amdgpu_device.c | 6 +- > 2 files changed, 7 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h > b/drivers/gpu/drm/amd/amdgpu/amdgpu.h > index 1030cb30720c..a1516a3ae9a8 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h > @@ -169,7 +169,9 @@ extern int amdgpu_discovery; extern int > amdgpu_mes; extern int amdgpu_noretry; extern int > amdgpu_force_asic_type; > +#ifdef CONFIG_HSA_AMD > extern int sched_policy; > +#endif > > #ifdef CONFIG_DRM_AMDGPU_SI > extern int amdgpu_si_support; > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > index bd423dd64e18..2535db27f821 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c > @@ -1623,7 +1623,11 @@ static int amdgpu_device_ip_early_init(struct > amdgpu_device *adev) > } > > adev->pm.pp_feature = amdgpu_pp_feature_mask; > - if (amdgpu_sriov_vf(adev) || sched_policy == > KFD_SCHED_POLICY_NO_HWS) > + if (amdgpu_sriov_vf(adev) > + #ifdef CONFIG_HSA_AMD > + || sched_policy == KFD_SCHED_POLICY_NO_HWS > + #endif > + ) > adev->pm.pp_feature &= ~PP_GFXOFF_MASK; > > for (i = 0; i < adev->num_ip_blocks; i++) { > -- > 2.20.1 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx