linux-kernel - Re: [PATCH] mips: Fix build if PERF

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20150410140531.GD21107@linux-mips.org>
Date:	Fri, 10 Apr 2015 16:05:32 +0200
From:	Ralf Baechle <ralf@...ux-mips.org>
To:	Guenter Roeck <linux@...ck-us.net>
Cc:	linux-mips@...ux-mips.org, linux-kernel@...r.kernel.org,
	Andy Lutomirski <luto@...capital.net>,
	David Daney <david.daney@...ium.com>
Subject: Re: [PATCH] mips: Fix build if PERF_EVENTS is configured

On Wed, Apr 08, 2015 at 08:40:08PM -0700, Guenter Roeck wrote:

> >>kernel/events/core.c:4828: undefined reference to `perf_get_regs_user'
> >>
> >>The problem is caused by commit 3478e32c1545 ("MIPS: Add user stack and
> >>registers to perf.") in combination with commit 88a7c26af8da ("perf:
> >>Move task_pt_regs sampling into arch code"), which introduces
> >>perf_get_regs_user().
> >>
> >>Cc: Andy Lutomirski <luto@...capital.net>
> >>Cc: David Daney <david.daney@...ium.com>
> >>Signed-off-by: Guenter Roeck <linux@...ck-us.net>
> >
> >I've already applied the same change locally but due to patch ordering
> >constraints I'm considering to temporarily pull this patch until the
> >perf stuff and the remainder of the MIPS changes are in 4.1 or so.
> >
> Hi Ralf,
> 
> Does it hurt to have the function if 88a7c26af8da is not in the tree ?
> Sure, you'll get a smatch and/or sparse warning, but I would consider that
> to be less severe than build failures.

There are other dependencies between the pending MIPS perf patches, the
remainder of the pending MIPS patches and upstream perf patches for 4.1
and all in all I've decided it's too much headache and I should rather wait.

  Ralf
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/