Re: [PATCH -tip] kcov: Make runtime functions noinstr-compatible
From: Peter Zijlstra
Date: Thu Jun 04 2020 - 12:04:24 EST
On Thu, Jun 04, 2020 at 04:02:54PM +0200, Andrey Konovalov wrote:
> On Thu, Jun 4, 2020 at 1:09 PM Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:
> > That whole kcov_remote stuff confuses me.
> > KCOV_ENABLE() has kcov_fault_in_area(), which supposedly takes the
> > vmalloc faults for the current task, but who does it for the remote?
> Hm, no one. This might be an issue, thanks for noticing!
> > Now, luckily Joerg went and ripped out the vmalloc faults, let me check
> > where those patches are... w00t, they're upstream in this merge window.
> Could you point me to those patches?
> Even though it might work fine now, we might get issues if we backport
> remote kcov to older kernels.
Thinking more about this; you can't actually pre-fault for kernel
threads, as kernel threads will run with the mm of whatever regular
thread ran before them, and who knows if they have that vmalloc region
So Joerg's patches are pretty much the only way to guarantee remotes
will not his the vmalloc fault.