On Tue, Apr 01, 2025 at 11:40:21AM -0700, Andrii Nakryiko wrote: > It is useful to be able to access current->mm to, say, record a bunch of > VMA information right before the task exits (e.g., for stack > symbolization reasons when dealing with short-lived processes that exit > in the middle of profiling session). We currently do have > trace_sched_process_exit() in the exit path, but it is called a bit too > late, after exit_mm() resets current->mm to NULL, which makes it > unsuitable for inspecting and recording task's mm_struct-related data > when tracing process lifetimes. > > There is a particularly suitable place, though, right after > taskstats_exit() is called, but before we do exit_mm(). taskstats > performs a similar kind of accounting that some applications do with > BPF, and so co-locating them seems like a good fit. > > Moving trace_sched_process_exit() a bit earlier would solve this problem > as well, and I'm open to that.
I don't see a problem with moving it.
