Re: [PATCH 01/16] initrd: Add generic code path for common initrd unloading logic.

From: Shea Levy
Date: Sun Mar 25 2018 - 18:20:30 EST


Hi Christophe,

LEROY Christophe <christophe.leroy@xxxxxx> writes:

> Shea Levy <shea@xxxxxxxxxxxx> a ÃcritÂ:
>
>> Signed-off-by: Shea Levy <shea@xxxxxxxxxxxx>
>> ---
>> init/initramfs.c | 7 +++++++
>> usr/Kconfig | 4 ++++
>> 2 files changed, 11 insertions(+)
>>
>> diff --git a/init/initramfs.c b/init/initramfs.c
>> index 7e99a0038942..de5ce873eb5a 100644
>> --- a/init/initramfs.c
>> +++ b/init/initramfs.c
>> @@ -526,6 +526,13 @@ extern unsigned long __initramfs_size;
>> #include <linux/initrd.h>
>> #include <linux/kexec.h>
>>
>> +#ifdef CONFIG_INITRAMFS_GENERIC_UNLOAD
>> +void free_initrd_mem(unsigned long start, unsigned long end)
>> +{
>> + free_reserved_area((void *)start, (void *)end, -1, "initrd");
>> +}
>> +#endif
>
> In powerpc this was an __init function. Why not also put the generic
> one in __init section ?
>

v2 series sent, thanks!

>
> Christophe
>
>
>> +
>> static void __init free_initrd(void)
>> {
>> #ifdef CONFIG_KEXEC_CORE
>> diff --git a/usr/Kconfig b/usr/Kconfig
>> index 43658b8a975e..fd79d4d6fa26 100644
>> --- a/usr/Kconfig
>> +++ b/usr/Kconfig
>> @@ -233,3 +233,7 @@ config INITRAMFS_COMPRESSION
>> default ".lzma" if RD_LZMA
>> default ".bz2" if RD_BZIP2
>> default ""
>> +
>> +# Arches can select this for a generic initrd unloading codepath
>> +config INITRAMFS_GENERIC_UNLOAD
>> + bool
>> --
>> 2.16.2

Attachment: signature.asc
Description: PGP signature