linux-kernel - Re: [PATCH] pxa: remove incorrect __init annotation on pxa27x_set

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: <87r3kz6hq9.fsf@belgarion.home>
Date:	Mon, 12 Oct 2015 20:53:50 +0200
From:	Robert Jarzmik <robert.jarzmik@...e.fr>
To:	Arnd Bergmann <arnd@...db.de>
Cc:	linux-arm-kernel@...ts.infradead.org,
	Daniel Mack <daniel@...que.org>,
	Haojian Zhuang <haojian.zhuang@...il.com>,
	linux-kernel@...r.kernel.org, Eric Miao <eric.y.miao@...il.com>
Subject: Re: [PATCH] pxa: remove incorrect __init annotation on pxa27x_set_pwrmode

Robert Jarzmik <robert.jarzmik@...e.fr> writes:

> Arnd Bergmann <arnd@...db.de> writes:
>
>> The z2 machine calls pxa27x_set_pwrmode() in order to power off
>> the machine, but this function gets discarded early at boot because
>> it is marked __init, as pointed out by kbuild:
>>
>> WARNING: vmlinux.o(.text+0x145c4): Section mismatch in reference from the function z2_power_off() to the function .init.text:pxa27x_set_pwrmode()
>> The function z2_power_off() references
>> the function __init pxa27x_set_pwrmode().
>> This is often because z2_power_off lacks a __init
>> annotation or the annotation of pxa27x_set_pwrmode is wrong.
>>
>> This removes the __init section modifier to fix rebooting and the
>> build error.
>>
>> Signed-off-by: Arnd Bergmann <arnd@...db.de>
>> Fixes: ba4a90a6d86a ("ARM: pxa/z2: fix building error of pxa27x_cpu_suspend() no longer available")
>> ---
>> This is a fix for an old bug, I'd just put it into 4.4
>>
>> It showed up now because section mismatches now produce errors instead
>> of warnings.
>
> Hi Arnd,
>
> I already have this queued for 4.4 from Thierry's patch in :
>   http://www.spinics.net/lists/arm-kernel/msg449414.html
>
> Yet your patch is more complete, as it deals also with the header in pxa27x.h
> and has the fixes tag.
>
> Now I'm wondering if I should enhance Thierry's patch with yours and keeping
> your signoff with your permission, or drop Thierry's one to replace by yours (I
> don't know if it's sane behavior to drop an already queued patch ...)

Actually, I've rethought it over and Thierry's patch approach looks somehow more
appealing to me. What Thierry did is that he modified the _use_ of
pxa27x_set_pwrmode() to be called from a __init annonated function. While your
patch makes pxa27x_set_pwrmode() non __init.

It looks to me the powermode should be initialized once and for all in the
machine init code. So unless I've overseen something, I'll keep Thierry's patch.

Cheers.

--
Robert
--
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/