* [PATCH 1/2] drm/amd/amdgpu: Add VMID to SRBM debugfs bank selection
@ 2019-07-12 13:44 StDenis, Tom
[not found] ` <20190712134406.30374-1-tom.stdenis-5C7GfCeVMHo@public.gmane.org>
0 siblings, 1 reply; 5+ messages in thread
From: StDenis, Tom @ 2019-07-12 13:44 UTC (permalink / raw)
To: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org; +Cc: StDenis, Tom
Add 5 bits to the offset for SRBM selection to handle VMIDs. Also
update the select_me_pipe_q() callback to also select VMID.
Signed-off-by: Tom St Denis <tom.stdenis@amd.com>
---
drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c | 9 +++++----
drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h | 4 ++--
drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c | 2 +-
drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c | 4 ++--
drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 4 ++--
drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 4 ++--
6 files changed, 14 insertions(+), 13 deletions(-)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
index 4997efa09ce7..87b32873046f 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
@@ -103,10 +103,10 @@ static int amdgpu_debugfs_process_reg_op(bool read, struct file *f,
ssize_t result = 0;
int r;
bool pm_pg_lock, use_bank, use_ring;
- unsigned instance_bank, sh_bank, se_bank, me, pipe, queue;
+ unsigned instance_bank, sh_bank, se_bank, me, pipe, queue, vmid;
pm_pg_lock = use_bank = use_ring = false;
- instance_bank = sh_bank = se_bank = me = pipe = queue = 0;
+ instance_bank = sh_bank = se_bank = me = pipe = queue = vmid = 0;
if (size & 0x3 || *pos & 0x3 ||
((*pos & (1ULL << 62)) && (*pos & (1ULL << 61))))
@@ -132,6 +132,7 @@ static int amdgpu_debugfs_process_reg_op(bool read, struct file *f,
me = (*pos & GENMASK_ULL(33, 24)) >> 24;
pipe = (*pos & GENMASK_ULL(43, 34)) >> 34;
queue = (*pos & GENMASK_ULL(53, 44)) >> 44;
+ vmid = (*pos & GENMASK_ULL(48, 45)) >> 54;
use_ring = 1;
} else {
@@ -149,7 +150,7 @@ static int amdgpu_debugfs_process_reg_op(bool read, struct file *f,
sh_bank, instance_bank);
} else if (use_ring) {
mutex_lock(&adev->srbm_mutex);
- amdgpu_gfx_select_me_pipe_q(adev, me, pipe, queue);
+ amdgpu_gfx_select_me_pipe_q(adev, me, pipe, queue, vmid);
}
if (pm_pg_lock)
@@ -182,7 +183,7 @@ static int amdgpu_debugfs_process_reg_op(bool read, struct file *f,
amdgpu_gfx_select_se_sh(adev, 0xffffffff, 0xffffffff, 0xffffffff);
mutex_unlock(&adev->grbm_idx_mutex);
} else if (use_ring) {
- amdgpu_gfx_select_me_pipe_q(adev, 0, 0, 0);
+ amdgpu_gfx_select_me_pipe_q(adev, 0, 0, 0, 0);
mutex_unlock(&adev->srbm_mutex);
}
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
index f96407ba9770..1199b5828b90 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
@@ -195,7 +195,7 @@ struct amdgpu_gfx_funcs {
uint32_t wave, uint32_t start, uint32_t size,
uint32_t *dst);
void (*select_me_pipe_q)(struct amdgpu_device *adev, u32 me, u32 pipe,
- u32 queue);
+ u32 queue, u32 vmid);
};
struct amdgpu_ngg_buf {
@@ -327,7 +327,7 @@ struct amdgpu_gfx {
#define amdgpu_gfx_get_gpu_clock_counter(adev) (adev)->gfx.funcs->get_gpu_clock_counter((adev))
#define amdgpu_gfx_select_se_sh(adev, se, sh, instance) (adev)->gfx.funcs->select_se_sh((adev), (se), (sh), (instance))
-#define amdgpu_gfx_select_me_pipe_q(adev, me, pipe, q) (adev)->gfx.funcs->select_me_pipe_q((adev), (me), (pipe), (q))
+#define amdgpu_gfx_select_me_pipe_q(adev, me, pipe, q, vmid) (adev)->gfx.funcs->select_me_pipe_q((adev), (me), (pipe), (q), (vmid))
/**
* amdgpu_gfx_create_bitmask - create a bitmask
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
index 0745370493f3..8c27c305e692 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
@@ -3041,7 +3041,7 @@ static void gfx_v6_0_read_wave_sgprs(struct amdgpu_device *adev, uint32_t simd,
}
static void gfx_v6_0_select_me_pipe_q(struct amdgpu_device *adev,
- u32 me, u32 pipe, u32 q)
+ u32 me, u32 pipe, u32 q, u32 vm)
{
DRM_INFO("Not implemented\n");
}
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
index bc5ff82565d2..e1e2a44ee13c 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
@@ -4167,9 +4167,9 @@ static void gfx_v7_0_read_wave_sgprs(struct amdgpu_device *adev, uint32_t simd,
}
static void gfx_v7_0_select_me_pipe_q(struct amdgpu_device *adev,
- u32 me, u32 pipe, u32 q)
+ u32 me, u32 pipe, u32 q, u32 vm)
{
- cik_srbm_select(adev, me, pipe, q, 0);
+ cik_srbm_select(adev, me, pipe, q, vm);
}
static const struct amdgpu_gfx_funcs gfx_v7_0_gfx_funcs = {
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
index 13432a9210a3..8c590a554675 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
@@ -3432,9 +3432,9 @@ static void gfx_v8_0_select_se_sh(struct amdgpu_device *adev,
}
static void gfx_v8_0_select_me_pipe_q(struct amdgpu_device *adev,
- u32 me, u32 pipe, u32 q)
+ u32 me, u32 pipe, u32 q, u32 vm)
{
- vi_srbm_select(adev, me, pipe, q, 0);
+ vi_srbm_select(adev, me, pipe, q, vm);
}
static u32 gfx_v8_0_get_rb_active_bitmap(struct amdgpu_device *adev)
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
index ed48def12db1..d20d55199a1c 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
@@ -1309,9 +1309,9 @@ static void gfx_v9_0_read_wave_vgprs(struct amdgpu_device *adev, uint32_t simd,
}
static void gfx_v9_0_select_me_pipe_q(struct amdgpu_device *adev,
- u32 me, u32 pipe, u32 q)
+ u32 me, u32 pipe, u32 q, u32 vm)
{
- soc15_grbm_select(adev, me, pipe, q, 0);
+ soc15_grbm_select(adev, me, pipe, q, vm);
}
static const struct amdgpu_gfx_funcs gfx_v9_0_gfx_funcs = {
--
2.21.0
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 2/2] drm/amd/amdgpu: Add missing select_me_pipe_q() for gfx10
[not found] ` <20190712134406.30374-1-tom.stdenis-5C7GfCeVMHo@public.gmane.org>
@ 2019-07-12 13:44 ` StDenis, Tom
[not found] ` <20190712134406.30374-2-tom.stdenis-5C7GfCeVMHo@public.gmane.org>
2019-07-16 9:18 ` [PATCH 1/2] drm/amd/amdgpu: Add VMID to SRBM debugfs bank selection Christian König
1 sibling, 1 reply; 5+ messages in thread
From: StDenis, Tom @ 2019-07-12 13:44 UTC (permalink / raw)
To: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org; +Cc: StDenis, Tom
Signed-off-by: Tom St Denis <tom.stdenis@amd.com>
---
drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
index 0d94c812df1b..7d5207bbe382 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
@@ -1061,6 +1061,12 @@ static void gfx_v10_0_read_wave_vgprs(struct amdgpu_device *adev, uint32_t simd,
start + SQIND_WAVE_VGPRS_OFFSET, size, dst);
}
+static void gfx_v10_0_select_me_pipe_q(struct amdgpu_device *adev,
+ u32 me, u32 pipe, u32 q, u32 vm)
+ {
+ nv_grbm_select(adev, me, pipe, q, vm);
+ }
+
static const struct amdgpu_gfx_funcs gfx_v10_0_gfx_funcs = {
.get_gpu_clock_counter = &gfx_v10_0_get_gpu_clock_counter,
@@ -1068,6 +1074,7 @@ static const struct amdgpu_gfx_funcs gfx_v10_0_gfx_funcs = {
.read_wave_data = &gfx_v10_0_read_wave_data,
.read_wave_sgprs = &gfx_v10_0_read_wave_sgprs,
.read_wave_vgprs = &gfx_v10_0_read_wave_vgprs,
+ .select_me_pipe_q = &gfx_v10_0_select_me_pipe_q,
};
static void gfx_v10_0_gpu_early_init(struct amdgpu_device *adev)
--
2.21.0
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] drm/amd/amdgpu: Add missing select_me_pipe_q() for gfx10
[not found] ` <20190712134406.30374-2-tom.stdenis-5C7GfCeVMHo@public.gmane.org>
@ 2019-07-12 18:37 ` Alex Deucher
2019-07-16 9:20 ` Christian König
1 sibling, 0 replies; 5+ messages in thread
From: Alex Deucher @ 2019-07-12 18:37 UTC (permalink / raw)
To: StDenis, Tom; +Cc: amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
On Fri, Jul 12, 2019 at 9:44 AM StDenis, Tom <Tom.StDenis@amd.com> wrote:
>
> Signed-off-by: Tom St Denis <tom.stdenis@amd.com>
Please add a patch description. With that fixed, the series is:
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> index 0d94c812df1b..7d5207bbe382 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> @@ -1061,6 +1061,12 @@ static void gfx_v10_0_read_wave_vgprs(struct amdgpu_device *adev, uint32_t simd,
> start + SQIND_WAVE_VGPRS_OFFSET, size, dst);
> }
>
> +static void gfx_v10_0_select_me_pipe_q(struct amdgpu_device *adev,
> + u32 me, u32 pipe, u32 q, u32 vm)
> + {
> + nv_grbm_select(adev, me, pipe, q, vm);
> + }
> +
>
> static const struct amdgpu_gfx_funcs gfx_v10_0_gfx_funcs = {
> .get_gpu_clock_counter = &gfx_v10_0_get_gpu_clock_counter,
> @@ -1068,6 +1074,7 @@ static const struct amdgpu_gfx_funcs gfx_v10_0_gfx_funcs = {
> .read_wave_data = &gfx_v10_0_read_wave_data,
> .read_wave_sgprs = &gfx_v10_0_read_wave_sgprs,
> .read_wave_vgprs = &gfx_v10_0_read_wave_vgprs,
> + .select_me_pipe_q = &gfx_v10_0_select_me_pipe_q,
> };
>
> static void gfx_v10_0_gpu_early_init(struct amdgpu_device *adev)
> --
> 2.21.0
>
> _______________________________________________
> 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
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] drm/amd/amdgpu: Add VMID to SRBM debugfs bank selection
[not found] ` <20190712134406.30374-1-tom.stdenis-5C7GfCeVMHo@public.gmane.org>
2019-07-12 13:44 ` [PATCH 2/2] drm/amd/amdgpu: Add missing select_me_pipe_q() for gfx10 StDenis, Tom
@ 2019-07-16 9:18 ` Christian König
1 sibling, 0 replies; 5+ messages in thread
From: Christian König @ 2019-07-16 9:18 UTC (permalink / raw)
To: StDenis, Tom,
amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Am 12.07.19 um 15:44 schrieb StDenis, Tom:
> Add 5 bits to the offset for SRBM selection to handle VMIDs. Also
> update the select_me_pipe_q() callback to also select VMID.
Maybe split in two patches? Either way Reviewed-by: Christian König
<christian.koenig@amd.com>.
>
> Signed-off-by: Tom St Denis <tom.stdenis@amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c | 9 +++++----
> drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h | 4 ++--
> drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c | 2 +-
> drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c | 4 ++--
> drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 4 ++--
> drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c | 4 ++--
> 6 files changed, 14 insertions(+), 13 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
> index 4997efa09ce7..87b32873046f 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_debugfs.c
> @@ -103,10 +103,10 @@ static int amdgpu_debugfs_process_reg_op(bool read, struct file *f,
> ssize_t result = 0;
> int r;
> bool pm_pg_lock, use_bank, use_ring;
> - unsigned instance_bank, sh_bank, se_bank, me, pipe, queue;
> + unsigned instance_bank, sh_bank, se_bank, me, pipe, queue, vmid;
>
> pm_pg_lock = use_bank = use_ring = false;
> - instance_bank = sh_bank = se_bank = me = pipe = queue = 0;
> + instance_bank = sh_bank = se_bank = me = pipe = queue = vmid = 0;
>
> if (size & 0x3 || *pos & 0x3 ||
> ((*pos & (1ULL << 62)) && (*pos & (1ULL << 61))))
> @@ -132,6 +132,7 @@ static int amdgpu_debugfs_process_reg_op(bool read, struct file *f,
> me = (*pos & GENMASK_ULL(33, 24)) >> 24;
> pipe = (*pos & GENMASK_ULL(43, 34)) >> 34;
> queue = (*pos & GENMASK_ULL(53, 44)) >> 44;
> + vmid = (*pos & GENMASK_ULL(48, 45)) >> 54;
>
> use_ring = 1;
> } else {
> @@ -149,7 +150,7 @@ static int amdgpu_debugfs_process_reg_op(bool read, struct file *f,
> sh_bank, instance_bank);
> } else if (use_ring) {
> mutex_lock(&adev->srbm_mutex);
> - amdgpu_gfx_select_me_pipe_q(adev, me, pipe, queue);
> + amdgpu_gfx_select_me_pipe_q(adev, me, pipe, queue, vmid);
> }
>
> if (pm_pg_lock)
> @@ -182,7 +183,7 @@ static int amdgpu_debugfs_process_reg_op(bool read, struct file *f,
> amdgpu_gfx_select_se_sh(adev, 0xffffffff, 0xffffffff, 0xffffffff);
> mutex_unlock(&adev->grbm_idx_mutex);
> } else if (use_ring) {
> - amdgpu_gfx_select_me_pipe_q(adev, 0, 0, 0);
> + amdgpu_gfx_select_me_pipe_q(adev, 0, 0, 0, 0);
> mutex_unlock(&adev->srbm_mutex);
> }
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
> index f96407ba9770..1199b5828b90 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_gfx.h
> @@ -195,7 +195,7 @@ struct amdgpu_gfx_funcs {
> uint32_t wave, uint32_t start, uint32_t size,
> uint32_t *dst);
> void (*select_me_pipe_q)(struct amdgpu_device *adev, u32 me, u32 pipe,
> - u32 queue);
> + u32 queue, u32 vmid);
> };
>
> struct amdgpu_ngg_buf {
> @@ -327,7 +327,7 @@ struct amdgpu_gfx {
>
> #define amdgpu_gfx_get_gpu_clock_counter(adev) (adev)->gfx.funcs->get_gpu_clock_counter((adev))
> #define amdgpu_gfx_select_se_sh(adev, se, sh, instance) (adev)->gfx.funcs->select_se_sh((adev), (se), (sh), (instance))
> -#define amdgpu_gfx_select_me_pipe_q(adev, me, pipe, q) (adev)->gfx.funcs->select_me_pipe_q((adev), (me), (pipe), (q))
> +#define amdgpu_gfx_select_me_pipe_q(adev, me, pipe, q, vmid) (adev)->gfx.funcs->select_me_pipe_q((adev), (me), (pipe), (q), (vmid))
>
> /**
> * amdgpu_gfx_create_bitmask - create a bitmask
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
> index 0745370493f3..8c27c305e692 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
> @@ -3041,7 +3041,7 @@ static void gfx_v6_0_read_wave_sgprs(struct amdgpu_device *adev, uint32_t simd,
> }
>
> static void gfx_v6_0_select_me_pipe_q(struct amdgpu_device *adev,
> - u32 me, u32 pipe, u32 q)
> + u32 me, u32 pipe, u32 q, u32 vm)
> {
> DRM_INFO("Not implemented\n");
> }
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
> index bc5ff82565d2..e1e2a44ee13c 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
> @@ -4167,9 +4167,9 @@ static void gfx_v7_0_read_wave_sgprs(struct amdgpu_device *adev, uint32_t simd,
> }
>
> static void gfx_v7_0_select_me_pipe_q(struct amdgpu_device *adev,
> - u32 me, u32 pipe, u32 q)
> + u32 me, u32 pipe, u32 q, u32 vm)
> {
> - cik_srbm_select(adev, me, pipe, q, 0);
> + cik_srbm_select(adev, me, pipe, q, vm);
> }
>
> static const struct amdgpu_gfx_funcs gfx_v7_0_gfx_funcs = {
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> index 13432a9210a3..8c590a554675 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
> @@ -3432,9 +3432,9 @@ static void gfx_v8_0_select_se_sh(struct amdgpu_device *adev,
> }
>
> static void gfx_v8_0_select_me_pipe_q(struct amdgpu_device *adev,
> - u32 me, u32 pipe, u32 q)
> + u32 me, u32 pipe, u32 q, u32 vm)
> {
> - vi_srbm_select(adev, me, pipe, q, 0);
> + vi_srbm_select(adev, me, pipe, q, vm);
> }
>
> static u32 gfx_v8_0_get_rb_active_bitmap(struct amdgpu_device *adev)
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
> index ed48def12db1..d20d55199a1c 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v9_0.c
> @@ -1309,9 +1309,9 @@ static void gfx_v9_0_read_wave_vgprs(struct amdgpu_device *adev, uint32_t simd,
> }
>
> static void gfx_v9_0_select_me_pipe_q(struct amdgpu_device *adev,
> - u32 me, u32 pipe, u32 q)
> + u32 me, u32 pipe, u32 q, u32 vm)
> {
> - soc15_grbm_select(adev, me, pipe, q, 0);
> + soc15_grbm_select(adev, me, pipe, q, vm);
> }
>
> static const struct amdgpu_gfx_funcs gfx_v9_0_gfx_funcs = {
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 2/2] drm/amd/amdgpu: Add missing select_me_pipe_q() for gfx10
[not found] ` <20190712134406.30374-2-tom.stdenis-5C7GfCeVMHo@public.gmane.org>
2019-07-12 18:37 ` Alex Deucher
@ 2019-07-16 9:20 ` Christian König
1 sibling, 0 replies; 5+ messages in thread
From: Christian König @ 2019-07-16 9:20 UTC (permalink / raw)
To: StDenis, Tom,
amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Am 12.07.19 um 15:44 schrieb StDenis, Tom:
> Signed-off-by: Tom St Denis <tom.stdenis@amd.com>
> ---
> drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> index 0d94c812df1b..7d5207bbe382 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v10_0.c
> @@ -1061,6 +1061,12 @@ static void gfx_v10_0_read_wave_vgprs(struct amdgpu_device *adev, uint32_t simd,
> start + SQIND_WAVE_VGPRS_OFFSET, size, dst);
> }
>
> +static void gfx_v10_0_select_me_pipe_q(struct amdgpu_device *adev,
> + u32 me, u32 pipe, u32 q, u32 vm)
Looks oddly indented, but could as well be just my mail client.
Acked-by: Christian König <christian.koenig@amd.com> when you double
checked that.
Christian.
> + {
> + nv_grbm_select(adev, me, pipe, q, vm);
> + }
> +
>
> static const struct amdgpu_gfx_funcs gfx_v10_0_gfx_funcs = {
> .get_gpu_clock_counter = &gfx_v10_0_get_gpu_clock_counter,
> @@ -1068,6 +1074,7 @@ static const struct amdgpu_gfx_funcs gfx_v10_0_gfx_funcs = {
> .read_wave_data = &gfx_v10_0_read_wave_data,
> .read_wave_sgprs = &gfx_v10_0_read_wave_sgprs,
> .read_wave_vgprs = &gfx_v10_0_read_wave_vgprs,
> + .select_me_pipe_q = &gfx_v10_0_select_me_pipe_q,
> };
>
> static void gfx_v10_0_gpu_early_init(struct amdgpu_device *adev)
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2019-07-16 9:20 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-07-12 13:44 [PATCH 1/2] drm/amd/amdgpu: Add VMID to SRBM debugfs bank selection StDenis, Tom
[not found] ` <20190712134406.30374-1-tom.stdenis-5C7GfCeVMHo@public.gmane.org>
2019-07-12 13:44 ` [PATCH 2/2] drm/amd/amdgpu: Add missing select_me_pipe_q() for gfx10 StDenis, Tom
[not found] ` <20190712134406.30374-2-tom.stdenis-5C7GfCeVMHo@public.gmane.org>
2019-07-12 18:37 ` Alex Deucher
2019-07-16 9:20 ` Christian König
2019-07-16 9:18 ` [PATCH 1/2] drm/amd/amdgpu: Add VMID to SRBM debugfs bank selection Christian König
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox