Re: [PATCH] drm/amdgpu: Use the TGID for trace_amdgpu_vm_update_ptes
I'll add the stable CC. Thanks, Alex On Tue, Feb 7, 2023 at 2:34 AM Christian König wrote: > > That sounds like a good idea to me as well. > > If you think that a patch should be backported please add a "CC: > sta...@vger.kernel.org" tag to it before sending it out. > > We can always remove it if we don't think a backport is appropriated, > but maintainers seldom add it by themself. > > Thanks, > Christian. > > Am 07.02.23 um 00:09 schrieb Friedrich Vock: > > Hi, > > > > thanks for applying the patch! > > > > Do you think it'd also be possible to backport it to previous kernel > > versions or do you already plan to do that? > > Since it is a one-liner bugfix it shouldn't be too hard to backport. > > > > Thank you, > > Friedrich Vock > > > > On 06.02.23 21:26, Alex Deucher wrote: > >> Applied. Thanks! > >> > >> Alex > >> > >> On Mon, Feb 6, 2023 at 3:35 AM Christian König > >> wrote: > >>> > >>> > >>> Am 02.02.23 um 17:21 schrieb Friedrich Vock: > The pid field corresponds to the result of gettid() in userspace. > However, userspace cannot reliably attribute PTE events to processes > with just the thread id. This patch allows userspace to easily > attribute PTE update events to specific processes by comparing this > field with the result of getpid(). > > For attributing events to specific threads, the thread id is also > contained in the common fields of each trace event. > > Signed-off-by: Friedrich Vock > >>> Ah, yes that makes more sense. Reviewed-by: Christian König > >>> > >>> > >>> Alex do you pick this up or should I take care of it? > >>> > >>> Thanks, > >>> Christian. > >>> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c > b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c > index b5f3bba851db..01e42bdd8e4e 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c > @@ -974,7 +974,7 @@ int amdgpu_vm_ptes_update(struct > amdgpu_vm_update_params *params, > trace_amdgpu_vm_update_ptes(params, > frag_start, upd_end, > min(nptes, 32u), dst, incr, > upd_flags, > - vm->task_info.pid, > + vm->task_info.tgid, > vm->immediate.fence_context); > amdgpu_vm_pte_update_flags(params, > to_amdgpu_bo_vm(pt), > cursor.level, pe_start, dst, > -- > 2.39.1 > >
Re: [PATCH] drm/amdgpu: Use the TGID for trace_amdgpu_vm_update_ptes
That sounds like a good idea to me as well. If you think that a patch should be backported please add a "CC: sta...@vger.kernel.org" tag to it before sending it out. We can always remove it if we don't think a backport is appropriated, but maintainers seldom add it by themself. Thanks, Christian. Am 07.02.23 um 00:09 schrieb Friedrich Vock: Hi, thanks for applying the patch! Do you think it'd also be possible to backport it to previous kernel versions or do you already plan to do that? Since it is a one-liner bugfix it shouldn't be too hard to backport. Thank you, Friedrich Vock On 06.02.23 21:26, Alex Deucher wrote: Applied. Thanks! Alex On Mon, Feb 6, 2023 at 3:35 AM Christian König wrote: Am 02.02.23 um 17:21 schrieb Friedrich Vock: The pid field corresponds to the result of gettid() in userspace. However, userspace cannot reliably attribute PTE events to processes with just the thread id. This patch allows userspace to easily attribute PTE update events to specific processes by comparing this field with the result of getpid(). For attributing events to specific threads, the thread id is also contained in the common fields of each trace event. Signed-off-by: Friedrich Vock Ah, yes that makes more sense. Reviewed-by: Christian König Alex do you pick this up or should I take care of it? Thanks, Christian. --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c index b5f3bba851db..01e42bdd8e4e 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c @@ -974,7 +974,7 @@ int amdgpu_vm_ptes_update(struct amdgpu_vm_update_params *params, trace_amdgpu_vm_update_ptes(params, frag_start, upd_end, min(nptes, 32u), dst, incr, upd_flags, - vm->task_info.pid, + vm->task_info.tgid, vm->immediate.fence_context); amdgpu_vm_pte_update_flags(params, to_amdgpu_bo_vm(pt), cursor.level, pe_start, dst, -- 2.39.1
Re: [PATCH] drm/amdgpu: Use the TGID for trace_amdgpu_vm_update_ptes
Hi, thanks for applying the patch! Do you think it'd also be possible to backport it to previous kernel versions or do you already plan to do that? Since it is a one-liner bugfix it shouldn't be too hard to backport. Thank you, Friedrich Vock On 06.02.23 21:26, Alex Deucher wrote: Applied. Thanks! Alex On Mon, Feb 6, 2023 at 3:35 AM Christian König wrote: Am 02.02.23 um 17:21 schrieb Friedrich Vock: The pid field corresponds to the result of gettid() in userspace. However, userspace cannot reliably attribute PTE events to processes with just the thread id. This patch allows userspace to easily attribute PTE update events to specific processes by comparing this field with the result of getpid(). For attributing events to specific threads, the thread id is also contained in the common fields of each trace event. Signed-off-by: Friedrich Vock Ah, yes that makes more sense. Reviewed-by: Christian König Alex do you pick this up or should I take care of it? Thanks, Christian. --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c index b5f3bba851db..01e42bdd8e4e 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c @@ -974,7 +974,7 @@ int amdgpu_vm_ptes_update(struct amdgpu_vm_update_params *params, trace_amdgpu_vm_update_ptes(params, frag_start, upd_end, min(nptes, 32u), dst, incr, upd_flags, - vm->task_info.pid, + vm->task_info.tgid, vm->immediate.fence_context); amdgpu_vm_pte_update_flags(params, to_amdgpu_bo_vm(pt), cursor.level, pe_start, dst, -- 2.39.1
Re: [PATCH] drm/amdgpu: Use the TGID for trace_amdgpu_vm_update_ptes
Applied. Thanks! Alex On Mon, Feb 6, 2023 at 3:35 AM Christian König wrote: > > > > Am 02.02.23 um 17:21 schrieb Friedrich Vock: > > The pid field corresponds to the result of gettid() in userspace. > > However, userspace cannot reliably attribute PTE events to processes > > with just the thread id. This patch allows userspace to easily > > attribute PTE update events to specific processes by comparing this > > field with the result of getpid(). > > > > For attributing events to specific threads, the thread id is also > > contained in the common fields of each trace event. > > > > Signed-off-by: Friedrich Vock > > Ah, yes that makes more sense. Reviewed-by: Christian König > > > Alex do you pick this up or should I take care of it? > > Thanks, > Christian. > > > --- > > drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c > > b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c > > index b5f3bba851db..01e42bdd8e4e 100644 > > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c > > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c > > @@ -974,7 +974,7 @@ int amdgpu_vm_ptes_update(struct > > amdgpu_vm_update_params *params, > > trace_amdgpu_vm_update_ptes(params, frag_start, > > upd_end, > > min(nptes, 32u), dst, > > incr, > > upd_flags, > > - vm->task_info.pid, > > + vm->task_info.tgid, > > > > vm->immediate.fence_context); > > amdgpu_vm_pte_update_flags(params, > > to_amdgpu_bo_vm(pt), > > cursor.level, pe_start, > > dst, > > -- > > 2.39.1 > > >
Re: [PATCH] drm/amdgpu: Use the TGID for trace_amdgpu_vm_update_ptes
Am 02.02.23 um 17:21 schrieb Friedrich Vock: The pid field corresponds to the result of gettid() in userspace. However, userspace cannot reliably attribute PTE events to processes with just the thread id. This patch allows userspace to easily attribute PTE update events to specific processes by comparing this field with the result of getpid(). For attributing events to specific threads, the thread id is also contained in the common fields of each trace event. Signed-off-by: Friedrich Vock Ah, yes that makes more sense. Reviewed-by: Christian König Alex do you pick this up or should I take care of it? Thanks, Christian. --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c index b5f3bba851db..01e42bdd8e4e 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c @@ -974,7 +974,7 @@ int amdgpu_vm_ptes_update(struct amdgpu_vm_update_params *params, trace_amdgpu_vm_update_ptes(params, frag_start, upd_end, min(nptes, 32u), dst, incr, upd_flags, - vm->task_info.pid, + vm->task_info.tgid, vm->immediate.fence_context); amdgpu_vm_pte_update_flags(params, to_amdgpu_bo_vm(pt), cursor.level, pe_start, dst, -- 2.39.1
[PATCH] drm/amdgpu: Use the TGID for trace_amdgpu_vm_update_ptes
The pid field corresponds to the result of gettid() in userspace. However, userspace cannot reliably attribute PTE events to processes with just the thread id. This patch allows userspace to easily attribute PTE update events to specific processes by comparing this field with the result of getpid(). For attributing events to specific threads, the thread id is also contained in the common fields of each trace event. Signed-off-by: Friedrich Vock --- drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c index b5f3bba851db..01e42bdd8e4e 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vm_pt.c @@ -974,7 +974,7 @@ int amdgpu_vm_ptes_update(struct amdgpu_vm_update_params *params, trace_amdgpu_vm_update_ptes(params, frag_start, upd_end, min(nptes, 32u), dst, incr, upd_flags, - vm->task_info.pid, + vm->task_info.tgid, vm->immediate.fence_context); amdgpu_vm_pte_update_flags(params, to_amdgpu_bo_vm(pt), cursor.level, pe_start, dst, -- 2.39.1