| Message ID | 20241002-adreno-smmu-aparture-v1-2-e9a63c9ccef5@oss.qualcomm.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | drm/msm/adreno: Setup SMMU aparture | expand |
On 3.10.2024 5:01 AM, Bjorn Andersson wrote: > Support for per-process page tables requires the SMMU aparture to be > setup such that the GPU can make updates with the SMMU. On some targets > this is done statically in firmware, on others it's expected to be > requested in runtime by the driver, through a SCM call. > > One place where configuration is expected to be done dynamically is the > QCS6490 rb3gen2. > > The downstream driver does this unconditioanlly on any A6xx and newer, > so follow suite and make the call. > > Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com> > --- Not all A6xx targets support PPPT (e.g. A619 on SM6375 - but A619 on SM6350 does..). We already print some error messages when that's the case, I think this may add one more. Nonetheless, I think that sticks to the accepted status quo where lacking PPPT is a bug, so.. Tested-by: Konrad Dybcio <konradybcio@kernel.org> # FP5 Reviewed-by: Konrad Dybcio <konradybcio@kernel.org> Konrad
On 10/2/2024 8:01 PM, Bjorn Andersson wrote: > Support for per-process page tables requires the SMMU aparture to be > setup such that the GPU can make updates with the SMMU. On some targets > this is done statically in firmware, on others it's expected to be > requested in runtime by the driver, through a SCM call. > > One place where configuration is expected to be done dynamically is the > QCS6490 rb3gen2. > > The downstream driver does this unconditioanlly on any A6xx and newer, > so follow suite and make the call. > > Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com> Hi Bjorn, Tested-by: Jessica Zhang <quic_jesszhan@quicinc.com> # Trogdor (sc7180) Thanks, Jessica Zhang > --- > drivers/gpu/drm/msm/adreno/adreno_gpu.c | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/drivers/gpu/drm/msm/adreno/adreno_gpu.c b/drivers/gpu/drm/msm/adreno/adreno_gpu.c > index 465a4cd14a43..5b06f7a04fe6 100644 > --- a/drivers/gpu/drm/msm/adreno/adreno_gpu.c > +++ b/drivers/gpu/drm/msm/adreno/adreno_gpu.c > @@ -572,8 +572,18 @@ struct drm_gem_object *adreno_fw_create_bo(struct msm_gpu *gpu, > > int adreno_hw_init(struct msm_gpu *gpu) > { > + struct adreno_gpu *adreno_gpu = to_adreno_gpu(gpu); > + int ret; > + > VERB("%s", gpu->name); > > + if (adreno_gpu->info->family >= ADRENO_6XX_GEN1) { > + /* We currently always use context bank 0, so hard code this */ > + ret = qcom_scm_set_gpu_smmu_aperture(0); > + if (ret) > + DRM_DEV_ERROR(gpu->dev->dev, "unable to set SMMU aperture: %d\n", ret); > + } > + > for (int i = 0; i < gpu->nr_rings; i++) { > struct msm_ringbuffer *ring = gpu->rb[i]; > > > -- > 2.45.2 > >
diff --git a/drivers/gpu/drm/msm/adreno/adreno_gpu.c b/drivers/gpu/drm/msm/adreno/adreno_gpu.c index 465a4cd14a43..5b06f7a04fe6 100644 --- a/drivers/gpu/drm/msm/adreno/adreno_gpu.c +++ b/drivers/gpu/drm/msm/adreno/adreno_gpu.c @@ -572,8 +572,18 @@ struct drm_gem_object *adreno_fw_create_bo(struct msm_gpu *gpu, int adreno_hw_init(struct msm_gpu *gpu) { + struct adreno_gpu *adreno_gpu = to_adreno_gpu(gpu); + int ret; + VERB("%s", gpu->name); + if (adreno_gpu->info->family >= ADRENO_6XX_GEN1) { + /* We currently always use context bank 0, so hard code this */ + ret = qcom_scm_set_gpu_smmu_aperture(0); + if (ret) + DRM_DEV_ERROR(gpu->dev->dev, "unable to set SMMU aperture: %d\n", ret); + } + for (int i = 0; i < gpu->nr_rings; i++) { struct msm_ringbuffer *ring = gpu->rb[i];
Support for per-process page tables requires the SMMU aparture to be setup such that the GPU can make updates with the SMMU. On some targets this is done statically in firmware, on others it's expected to be requested in runtime by the driver, through a SCM call. One place where configuration is expected to be done dynamically is the QCS6490 rb3gen2. The downstream driver does this unconditioanlly on any A6xx and newer, so follow suite and make the call. Signed-off-by: Bjorn Andersson <bjorn.andersson@oss.qualcomm.com> --- drivers/gpu/drm/msm/adreno/adreno_gpu.c | 10 ++++++++++ 1 file changed, 10 insertions(+)