Re: [PATCH V1 resend] dcookies: Make dcookies depend on CONFIG_OPROFILE

From: Viresh Kumar
Date: Mon Jan 11 2021 - 23:38:14 EST


On 11-01-21, 11:28, Linus Torvalds wrote:
> On Sun, Jan 10, 2021 at 10:02 PM Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> >
> > From: Arnd Bergmann <arnd@xxxxxxxx>
> >
> > The dcookies stuff is used only with OPROFILE and there is no need to
> > build it if CONFIG_OPROFILE isn't enabled. Build it depending on
> > CONFIG_OPROFILE instead of CONFIG_PROFILING.
>
> Umm. I think we should remove CONFIG_OPROFILE entirely, and then
> dcookies as part of it.
>
> We discussed removing CONFIG_OPROFILE for 5.9 (I htink) already - the
> oprofile user land tools don't use the kernel OPROFILE support any
> more, and haven't in a long time. User land has been converted to the
> perf interfaces.

Right, I followed that discussion but I wasn't sure if this patch will
go in first and then the later ones will follow. And since I couldn't
see a reply from Alexander for it, I thought it is best to resend it
to get merged first.

Okay, I will try to send some patches to get rid of OPROFILE and
dcookies now.

Thanks.

--
viresh