I'll add the stable CC. Thanks, Alex On Tue, Feb 7, 2023 at 2:34 AM Christian König <christian.koenig@xxxxxxx> wrote: > > That sounds like a good idea to me as well. > > If you think that a patch should be backported please add a "CC: > stable@xxxxxxxxxxxxxxx" 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 > >> <christian.koenig@xxxxxxx> 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 <friedrich.vock@xxxxxx> > >>> Ah, yes that makes more sense. Reviewed-by: Christian König > >>> <christian.koenig@xxxxxxx> > >>> > >>> 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 > >>>> >
