Re: [PATCH] syscalls: Cleanup references to sys_lookup_dcookie()

Namhyung Kim <[email protected]> Thu, 29 Jun 2023 22:33:51 -0700

Newsgroups	gmane.linux.ports.m68k,gmane.linux.kernel.api,gmane.linux.kernel.cross-arch,gmane.linux.ports.alpha,gmane.linux.kernel,gmane.linux.ports.arm.kernel,gmane.linux.ports.ia64,gmane.linux.ports.mips,gmane.linux.ports.parisc,gmane.linux.ports.ppc64.devel,gmane.linux.ports.sh.devel,gmane.linux.ports.sparc,gmane.linux.kernel.perf.user
Message-ID	<CAM9d7ch0GtTUjhtbph5rmCDvRBAKjLCN+25mukn_QPv4bDsjGQ@mail.gmail.com>

Hello,

On Wed, Jun 28, 2023 at 4:44 PM Randy Dunlap <[email protected]> wrote:
>
>
>
> On 6/28/23 16:09, Sohil Mehta wrote:
> > commit 'be65de6b03aa ("fs: Remove dcookies support")' removed the
> > syscall definition for lookup_dcookie.  However, syscall tables still
> > point to the old sys_lookup_dcookie() definition. Update syscall tables
> > of all architectures to directly point to sys_ni_syscall() instead.
> >
> > Signed-off-by: Sohil Mehta <[email protected]>
>
> Reviewed-by: Randy Dunlap <[email protected]>

I was about to say that it'd be nice if you split the tools/perf part
since it can support old kernels.  But if the syscall is only used for
oprofile then probably perf doesn't need to care about it. :)

For the perf part,
Acked-by: Namhyung Kim <[email protected]>

Thanks,
Namhyung