Re: [PATCH v1 4/8] arch, ftrace: For KASAN put hard/soft IRQ entries into separate sections

From: Alexander Potapenko
Date: Tue Feb 16 2016 - 10:32:37 EST


Ok, interrupt.h sounds good.

On Fri, Jan 29, 2016 at 3:45 PM, Steven Rostedt <rostedt@xxxxxxxxxxx> wrote:
> On Fri, 29 Jan 2016 12:59:13 +0100
> Alexander Potapenko <glider@xxxxxxxxxx> wrote:
>
>> On the other hand, this will require including <linux/irq.h> into
>> various files that currently use __irq_section.
>> But that header has a comment saying:
>>
>> /*
>> * Please do not include this file in generic code. There is currently
>> * no requirement for any architecture to implement anything held
>> * within this file.
>> *
>> * Thanks. --rmk
>> */
>>
>> Do we really want to put anything into that header?
>>
>
> What about interrupt.h?
>
> It's just weird to have KSAN needing to pull in ftrace.h for irq work.
>
> -- Steve



--
Alexander Potapenko
Software Engineer

Google Germany GmbH
Erika-Mann-StraÃe, 33
80636 MÃnchen

GeschÃftsfÃhrer: Matthew Scott Sucherman, Paul Terence Manicle
Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Diese E-Mail ist vertraulich. Wenn Sie nicht der richtige Adressat sind,
leiten Sie diese bitte nicht weiter, informieren Sie den
Absender und lÃschen Sie die E-Mail und alle AnhÃnge. Vielen Dank.
This e-mail is confidential. If you are not the right addressee please
do not forward it, please inform the sender, and please erase this
e-mail including any attachments. Thanks.