diff mbox

ARM: exynos_defconfig: Enable CONFIG_FHANDLE

Message ID 1417176414-21558-1-git-send-email-sjoerd.simons@collabora.co.uk (mailing list archive)
State New, archived
Headers show

Commit Message

Sjoerd Simons Nov. 28, 2014, 12:06 p.m. UTC
CONFIG_FHANDLE is required by systemd, which is the default init system
in more and more distributions. So lets enable it for Exynos as well
(it's already enabled in multi_v7_defconfig)

Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
---
 arch/arm/configs/exynos_defconfig | 1 +
 1 file changed, 1 insertion(+)

Comments

Andreas Färber Nov. 28, 2014, 12:08 p.m. UTC | #1
Am 28.11.2014 um 13:06 schrieb Sjoerd Simons:
> CONFIG_FHANDLE is required by systemd, which is the default init system
> in more and more distributions. So lets enable it for Exynos as well
> (it's already enabled in multi_v7_defconfig)
> 
> Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>

Reviewed-by: Andreas Färber <afaerber@suse.de>

Thanks,
Andreas
Javier Martinez Canillas Nov. 28, 2014, 12:35 p.m. UTC | #2
Hello Sjoerd,

On 11/28/2014 01:06 PM, Sjoerd Simons wrote:
> CONFIG_FHANDLE is required by systemd, which is the default init system
> in more and more distributions. So lets enable it for Exynos as well
> (it's already enabled in multi_v7_defconfig)
> 
> Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
> ---
>  arch/arm/configs/exynos_defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig
> index e419fac..d4751ef 100644
> --- a/arch/arm/configs/exynos_defconfig
> +++ b/arch/arm/configs/exynos_defconfig
> @@ -1,4 +1,5 @@
>  CONFIG_SYSVIPC=y
> +CONFIG_FHANDLE=y

This is certainly required for newer systemd and has to be enabled.

But it would also be great to have enabled all the config symbols
that are listed as requirements in the systemd README file:

http://cgit.freedesktop.org/systemd/systemd/tree/README.

Tony added that support to OMAP2+ defconfig in commit 673ce00c
("ARM: omap2plus_defconfig: Add support for distros with systemd")
so the same should be done in exynos_defconfig.

That can of course be a follow-up patch though, so for $subject:

Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>

Best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Joonyoung Shim Jan. 19, 2015, 5:43 a.m. UTC | #3
Hi Kukjin,

On 11/28/2014 09:35 PM, Javier Martinez Canillas wrote:
> Hello Sjoerd,
> 
> On 11/28/2014 01:06 PM, Sjoerd Simons wrote:
>> CONFIG_FHANDLE is required by systemd, which is the default init system
>> in more and more distributions. So lets enable it for Exynos as well
>> (it's already enabled in multi_v7_defconfig)
>>
>> Signed-off-by: Sjoerd Simons <sjoerd.simons@collabora.co.uk>
>> ---
>>  arch/arm/configs/exynos_defconfig | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig
>> index e419fac..d4751ef 100644
>> --- a/arch/arm/configs/exynos_defconfig
>> +++ b/arch/arm/configs/exynos_defconfig
>> @@ -1,4 +1,5 @@
>>  CONFIG_SYSVIPC=y
>> +CONFIG_FHANDLE=y
> 

I also hope merge this, already multi_v7_defconfig supports this.

Thanks.

> This is certainly required for newer systemd and has to be enabled.
> 
> But it would also be great to have enabled all the config symbols
> that are listed as requirements in the systemd README file:
> 
> http://cgit.freedesktop.org/systemd/systemd/tree/README.
> 
> Tony added that support to OMAP2+ defconfig in commit 673ce00c
> ("ARM: omap2plus_defconfig: Add support for distros with systemd")
> so the same should be done in exynos_defconfig.
> 
> That can of course be a follow-up patch though, so for $subject:
> 
> Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
> 
> Best regards,
> Javier
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Javier Martinez Canillas Jan. 23, 2015, 8:56 a.m. UTC | #4
Hello Kukjin,

On 01/19/2015 06:43 AM, Joonyoung Shim wrote:
>> 
> 
> I also hope merge this, already multi_v7_defconfig supports this.
> 

I would also hope this to be merged since it's required to boot
using systemd.

> Thanks.
> 
>> This is certainly required for newer systemd and has to be enabled.
>> 
>> But it would also be great to have enabled all the config symbols
>> that are listed as requirements in the systemd README file:
>> 
>> http://cgit.freedesktop.org/systemd/systemd/tree/README.
>> 
>> Tony added that support to OMAP2+ defconfig in commit 673ce00c
>> ("ARM: omap2plus_defconfig: Add support for distros with systemd")
>> so the same should be done in exynos_defconfig.
>> 
>> That can of course be a follow-up patch though, so for $subject:
>> 
>> Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
>>

Best regards,
Javier
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
kgene@kernel.org Jan. 24, 2015, 4:54 a.m. UTC | #5
On 01/23/15 17:56, Javier Martinez Canillas wrote:
> Hello Kukjin,
> 
Hi,

> On 01/19/2015 06:43 AM, Joonyoung Shim wrote:
>>>
>>
>> I also hope merge this, already multi_v7_defconfig supports this.
>>
> 
> I would also hope this to be merged since it's required to boot
> using systemd.
> 
>> Thanks.
>>
>>> This is certainly required for newer systemd and has to be enabled.
>>>
>>> But it would also be great to have enabled all the config symbols
>>> that are listed as requirements in the systemd README file:
>>>
>>> http://cgit.freedesktop.org/systemd/systemd/tree/README.
>>>
>>> Tony added that support to OMAP2+ defconfig in commit 673ce00c
>>> ("ARM: omap2plus_defconfig: Add support for distros with systemd")
>>> so the same should be done in exynos_defconfig.
>>>
>>> That can of course be a follow-up patch though, so for $subject:
>>>
>>> Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>
>>>
Thanks, applied.

- Kukjin
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig
index e419fac..d4751ef 100644
--- a/arch/arm/configs/exynos_defconfig
+++ b/arch/arm/configs/exynos_defconfig
@@ -1,4 +1,5 @@ 
 CONFIG_SYSVIPC=y
+CONFIG_FHANDLE=y
 CONFIG_NO_HZ=y
 CONFIG_HIGH_RES_TIMERS=y
 CONFIG_CGROUPS=y