linux-kernel - Re: [PATCH RESEND 0/2] kbuild: dead code elimination: ftrace fixes

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <6ea6505b-be41-1cd6-2fb2-63a58f1ac727@imgtec.com>
Date:   Thu, 22 Dec 2016 10:15:13 +0100
From:   Marcin Nowakowski <marcin.nowakowski@...tec.com>
To:     Nicholas Piggin <npiggin@...il.com>
CC:     Arnd Bergmann <arnd@...db.de>, <linux-kernel@...r.kernel.org>,
        Michal Marek <mmarek@...e.com>
Subject: Re: [PATCH RESEND 0/2] kbuild: dead code elimination: ftrace fixes

Hi Nick,

On 22.12.2016 10:08, Nicholas Piggin wrote:
> On Thu, 22 Dec 2016 09:51:45 +0100
> Marcin Nowakowski <marcin.nowakowski@...tec.com> wrote:
>
>> Enabling dead code & data elimination currently breaks ftrace operation,
>> as the __mcount_loc section is removed (as it is not referenced directly
>> anywhere in the code).
>> Moreover, there are a lot of entries missing in the __mcount_loc section
>> as the recordmcount tool doesn't currently properly handle the section
>> names as created by the use of -ffunction-sections.
>
> Thanks for keeping on top of these. I didn't have any objections, so
> if you would send them to Michal Marek for merge and cc linux-kbuild
> that would be appreciated.

I had previously sent them to Michal and cc:kbuild, but Michal has not 
picked them up yet hence I've sent them to lkml now.
It's not clear to me if Michal is the person responsible for 
recordmcount changes, but I initially posted this to kbuild as this is 
where all of your dead code & data elimination patches were going through.

Michal - do you plan to pick up these changes or would they need to be 
acked/merged by somebody else?

thanks,
Marcin