[PATCH 1/1] amdgpu: Enable KFD on POWER systems

[Felix Kuehling]

Hi Timothy,

Thank you for the patch and for confirming that it works. We did some 
experimental work on Power8 a few years ago. I see that Talos II is Power9.

At the time we were working on Power8 we had to add some #ifdef 
CONFIG_ACPI guards around some ACPI-specific code in KFD. Do you know to 
what extent ACPI is available and working on the Power architecture?

Another problem we ran into with Power, is the physical address map. 
System memory can be a physical addresses outside the range accessible 
by the GPU. Vega has 44-bit physical addressing. Older Polaris GPUs only 
have 40-bits. Did you run into any such problems? Do you need an IOMMU 
to make system memory accessible to the GPU?

Regards,
   Felix

On 2019-11-24 2:15 p.m., Timothy Pearson wrote:
> KFD has been verified to function on POWER systems (Talos II / Vega 64).
> It should be available as a kernel configuration option on these systems.
>
> Signed-off-by: Timothy Pearson <tpearson at raptorengineering.com>
> ---
>   drivers/gpu/drm/amd/amdkfd/Kconfig | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/amd/amdkfd/Kconfig b/drivers/gpu/drm/amd/amdkfd/Kconfig
> index a1a35d4d594b..ba0e68057a89 100644
> --- a/drivers/gpu/drm/amd/amdkfd/Kconfig
> +++ b/drivers/gpu/drm/amd/amdkfd/Kconfig
> @@ -5,7 +5,7 @@
>   
>   config HSA_AMD
>   	bool "HSA kernel driver for AMD GPU devices"
> -	depends on DRM_AMDGPU && (X86_64 || ARM64)
> +	depends on DRM_AMDGPU && (X86_64 || ARM64 || PPC64)
>   	imply AMD_IOMMU_V2 if X86_64
>   	select MMU_NOTIFIER
>   	help