All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ARM: at91: sama5: Update defconfig
@ 2016-05-05 20:31 Marek Vasut
  2016-05-06 14:30 ` Nicolas Ferre
  0 siblings, 1 reply; 11+ messages in thread
From: Marek Vasut @ 2016-05-05 20:31 UTC (permalink / raw)
  To: linux-arm-kernel

Enable CGROUPS support, so we can boot userspace that uses systemd on
top of kernel configured using this defconfig. The systemd userspace
depends on cgroups and without them, it dies in mysterious ways.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Ludovic Desroches <ludovic.desroches@atmel.com>
Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
 arch/arm/configs/sama5_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/sama5_defconfig b/arch/arm/configs/sama5_defconfig
index afbda41..1b60be2 100644
--- a/arch/arm/configs/sama5_defconfig
+++ b/arch/arm/configs/sama5_defconfig
@@ -3,6 +3,7 @@
 CONFIG_SYSVIPC=y
 CONFIG_IRQ_DOMAIN_DEBUG=y
 CONFIG_LOG_BUF_SHIFT=14
+CONFIG_CGROUPS=y
 CONFIG_BLK_DEV_INITRD=y
 CONFIG_EMBEDDED=y
 CONFIG_SLAB=y
-- 
2.7.0

^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH] ARM: at91: sama5: Update defconfig
  2016-05-05 20:31 [PATCH] ARM: at91: sama5: Update defconfig Marek Vasut
@ 2016-05-06 14:30 ` Nicolas Ferre
  2016-05-06 16:27   ` Marek Vasut
  0 siblings, 1 reply; 11+ messages in thread
From: Nicolas Ferre @ 2016-05-06 14:30 UTC (permalink / raw)
  To: linux-arm-kernel

Le 05/05/2016 22:31, Marek Vasut a ?crit :
> Enable CGROUPS support, so we can boot userspace that uses systemd on
> top of kernel configured using this defconfig. The systemd userspace
> depends on cgroups and without them, it dies in mysterious ways.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Ludovic Desroches <ludovic.desroches@atmel.com>
> Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>

Hi Marek,

Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>

BTW, isn't it something else required to run a systemd enabled image?
I've seen that some of the option may be missing as well:

CONFIG_FHANDLE (libudev, mount and bind mount handling)
from: https://github.com/systemd/systemd/blob/master/README#L51

In the " Optional but strongly recommended:" I see interest in:
 CONFIG_AUTOFS4_FS
 CONFIG_TMPFS_XATTR
 CONFIG_{TMPFS,EXT4,XFS,BTRFS_FS,...}_POSIX_ACL

But it's maybe not so interesting for our little embedded systems.

I'm also puzzled about the need for more options about cgroups...

I add a couple of other people in Cc so we can figure out the proper
configuration set...

Thanks Marek for having started the discussion, bye,

> ---
>  arch/arm/configs/sama5_defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/configs/sama5_defconfig b/arch/arm/configs/sama5_defconfig
> index afbda41..1b60be2 100644
> --- a/arch/arm/configs/sama5_defconfig
> +++ b/arch/arm/configs/sama5_defconfig
> @@ -3,6 +3,7 @@
>  CONFIG_SYSVIPC=y
>  CONFIG_IRQ_DOMAIN_DEBUG=y
>  CONFIG_LOG_BUF_SHIFT=14
> +CONFIG_CGROUPS=y
>  CONFIG_BLK_DEV_INITRD=y
>  CONFIG_EMBEDDED=y
>  CONFIG_SLAB=y
> 


-- 
Nicolas Ferre

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [PATCH] ARM: at91: sama5: Update defconfig
  2016-05-06 14:30 ` Nicolas Ferre
@ 2016-05-06 16:27   ` Marek Vasut
  2016-05-10  8:03       ` Nicolas Ferre
  0 siblings, 1 reply; 11+ messages in thread
From: Marek Vasut @ 2016-05-06 16:27 UTC (permalink / raw)
  To: linux-arm-kernel

On 05/06/2016 04:30 PM, Nicolas Ferre wrote:
> Le 05/05/2016 22:31, Marek Vasut a ?crit :
>> Enable CGROUPS support, so we can boot userspace that uses systemd on
>> top of kernel configured using this defconfig. The systemd userspace
>> depends on cgroups and without them, it dies in mysterious ways.
>>
>> Signed-off-by: Marek Vasut <marex@denx.de>
>> Cc: Ludovic Desroches <ludovic.desroches@atmel.com>
>> Cc: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> 
> Hi Marek,

Hi!

> Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
> 
> BTW, isn't it something else required to run a systemd enabled image?
> I've seen that some of the option may be missing as well:
> 
> CONFIG_FHANDLE (libudev, mount and bind mount handling)
> from: https://github.com/systemd/systemd/blob/master/README#L51

This one is absolutely mandatory, but this is also enabled by default
already. Debian testing with systemd won't boot without FHANDLE too,
so this is easy to trap for me.

> In the " Optional but strongly recommended:" I see interest in:
>  CONFIG_AUTOFS4_FS
>  CONFIG_TMPFS_XATTR
>  CONFIG_{TMPFS,EXT4,XFS,BTRFS_FS,...}_POSIX_ACL
> 
> But it's maybe not so interesting for our little embedded systems.

Hm, these are disabled, but I cannot comment on their significance.

> I'm also puzzled about the need for more options about cgroups...

Enabling CONFIG_CGROUPS is enough to make systemd happy and booting.
I didn't probe further, sorry.

> I add a couple of other people in Cc so we can figure out the proper
> configuration set...

Thanks!

> Thanks Marek for having started the discussion, bye,
> 
>> ---
>>  arch/arm/configs/sama5_defconfig | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/configs/sama5_defconfig b/arch/arm/configs/sama5_defconfig
>> index afbda41..1b60be2 100644
>> --- a/arch/arm/configs/sama5_defconfig
>> +++ b/arch/arm/configs/sama5_defconfig
>> @@ -3,6 +3,7 @@
>>  CONFIG_SYSVIPC=y
>>  CONFIG_IRQ_DOMAIN_DEBUG=y
>>  CONFIG_LOG_BUF_SHIFT=14
>> +CONFIG_CGROUPS=y
>>  CONFIG_BLK_DEV_INITRD=y
>>  CONFIG_EMBEDDED=y
>>  CONFIG_SLAB=y
>>
> 
> 


-- 
Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [PATCH] ARM: at91/defconfig: sama5: add CONFIG_FHANDLE
  2016-05-06 16:27   ` Marek Vasut
@ 2016-05-10  8:03       ` Nicolas Ferre
  0 siblings, 0 replies; 11+ messages in thread
From: Nicolas Ferre @ 2016-05-10  8:03 UTC (permalink / raw)
  To: linux-arm-kernel

CONFIG_FHANDLE is listed as a mandatory kernel option for systemd. So
explicitly enable it to allow easy use of systemd userspace.

Reported-by: Marek Vasut <marex@denx.de>
Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com>
---
Hi Marek,

It seems that on my side, CONFIG_FHANDLE is not selected automatically...
So I add this patch: can you tell me your thoughts?

Bye,
   Nico.

 arch/arm/configs/sama5_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/sama5_defconfig b/arch/arm/configs/sama5_defconfig
index 307f92b31e57..9cb1a85bb166 100644
--- a/arch/arm/configs/sama5_defconfig
+++ b/arch/arm/configs/sama5_defconfig
@@ -1,6 +1,7 @@
 # CONFIG_LOCALVERSION_AUTO is not set
 # CONFIG_SWAP is not set
 CONFIG_SYSVIPC=y
+CONFIG_FHANDLE=y
 CONFIG_IRQ_DOMAIN_DEBUG=y
 CONFIG_LOG_BUF_SHIFT=14
 CONFIG_CGROUPS=y
-- 
2.1.3

^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH] ARM: at91/defconfig: sama5: add CONFIG_FHANDLE
@ 2016-05-10  8:03       ` Nicolas Ferre
  0 siblings, 0 replies; 11+ messages in thread
From: Nicolas Ferre @ 2016-05-10  8:03 UTC (permalink / raw)
  To: marex
  Cc: Alexandre Belloni, linux-arm-kernel, linux-kernel,
	Boris BREZILLON, Venkateshprabhu.Subramanian, m.olbrich,
	Ludovic Desroches, Nicolas Ferre

CONFIG_FHANDLE is listed as a mandatory kernel option for systemd. So
explicitly enable it to allow easy use of systemd userspace.

Reported-by: Marek Vasut <marex@denx.de>
Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com>
---
Hi Marek,

It seems that on my side, CONFIG_FHANDLE is not selected automatically...
So I add this patch: can you tell me your thoughts?

Bye,
   Nico.

 arch/arm/configs/sama5_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/sama5_defconfig b/arch/arm/configs/sama5_defconfig
index 307f92b31e57..9cb1a85bb166 100644
--- a/arch/arm/configs/sama5_defconfig
+++ b/arch/arm/configs/sama5_defconfig
@@ -1,6 +1,7 @@
 # CONFIG_LOCALVERSION_AUTO is not set
 # CONFIG_SWAP is not set
 CONFIG_SYSVIPC=y
+CONFIG_FHANDLE=y
 CONFIG_IRQ_DOMAIN_DEBUG=y
 CONFIG_LOG_BUF_SHIFT=14
 CONFIG_CGROUPS=y
-- 
2.1.3

^ permalink raw reply related	[flat|nested] 11+ messages in thread

* [PATCH] ARM: at91/defconfig: sama5: add CONFIG_FHANDLE
  2016-05-10  8:03       ` Nicolas Ferre
@ 2016-05-10 12:31         ` Marek Vasut
  -1 siblings, 0 replies; 11+ messages in thread
From: Marek Vasut @ 2016-05-10 12:31 UTC (permalink / raw)
  To: linux-arm-kernel

On 05/10/2016 10:03 AM, Nicolas Ferre wrote:
> CONFIG_FHANDLE is listed as a mandatory kernel option for systemd. So
> explicitly enable it to allow easy use of systemd userspace.
> 
> Reported-by: Marek Vasut <marex@denx.de>
> Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com>
> ---
> Hi Marek,

Hi!

> It seems that on my side, CONFIG_FHANDLE is not selected automatically...
> So I add this patch: can you tell me your thoughts?

Uh, weird, I tried this on next just now:

linux-2.6$ git describe HEAD
next-20160510

linux-2.6$ export ARCH=arm ; export CROSS_COMPILE=arm-linux-gnueabi-

linux-2.6$ make sama5_defconfig ; grep FHANDLE .config
  HOSTCC  scripts/basic/fixdep
  HOSTCC  scripts/kconfig/conf.o
  SHIPPED scripts/kconfig/zconf.tab.c
  SHIPPED scripts/kconfig/zconf.lex.c
  SHIPPED scripts/kconfig/zconf.hash.c
  HOSTCC  scripts/kconfig/zconf.tab.o
  HOSTLD  scripts/kconfig/conf
#
# configuration written to .config
#
CONFIG_FHANDLE=y

> Bye,
>    Nico.
> 
>  arch/arm/configs/sama5_defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/configs/sama5_defconfig b/arch/arm/configs/sama5_defconfig
> index 307f92b31e57..9cb1a85bb166 100644
> --- a/arch/arm/configs/sama5_defconfig
> +++ b/arch/arm/configs/sama5_defconfig
> @@ -1,6 +1,7 @@
>  # CONFIG_LOCALVERSION_AUTO is not set
>  # CONFIG_SWAP is not set
>  CONFIG_SYSVIPC=y
> +CONFIG_FHANDLE=y
>  CONFIG_IRQ_DOMAIN_DEBUG=y
>  CONFIG_LOG_BUF_SHIFT=14
>  CONFIG_CGROUPS=y
> 


-- 
Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] ARM: at91/defconfig: sama5: add CONFIG_FHANDLE
@ 2016-05-10 12:31         ` Marek Vasut
  0 siblings, 0 replies; 11+ messages in thread
From: Marek Vasut @ 2016-05-10 12:31 UTC (permalink / raw)
  To: Nicolas Ferre
  Cc: Alexandre Belloni, linux-arm-kernel, linux-kernel,
	Boris BREZILLON, Venkateshprabhu.Subramanian, m.olbrich,
	Ludovic Desroches

On 05/10/2016 10:03 AM, Nicolas Ferre wrote:
> CONFIG_FHANDLE is listed as a mandatory kernel option for systemd. So
> explicitly enable it to allow easy use of systemd userspace.
> 
> Reported-by: Marek Vasut <marex@denx.de>
> Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com>
> ---
> Hi Marek,

Hi!

> It seems that on my side, CONFIG_FHANDLE is not selected automatically...
> So I add this patch: can you tell me your thoughts?

Uh, weird, I tried this on next just now:

linux-2.6$ git describe HEAD
next-20160510

linux-2.6$ export ARCH=arm ; export CROSS_COMPILE=arm-linux-gnueabi-

linux-2.6$ make sama5_defconfig ; grep FHANDLE .config
  HOSTCC  scripts/basic/fixdep
  HOSTCC  scripts/kconfig/conf.o
  SHIPPED scripts/kconfig/zconf.tab.c
  SHIPPED scripts/kconfig/zconf.lex.c
  SHIPPED scripts/kconfig/zconf.hash.c
  HOSTCC  scripts/kconfig/zconf.tab.o
  HOSTLD  scripts/kconfig/conf
#
# configuration written to .config
#
CONFIG_FHANDLE=y

> Bye,
>    Nico.
> 
>  arch/arm/configs/sama5_defconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/configs/sama5_defconfig b/arch/arm/configs/sama5_defconfig
> index 307f92b31e57..9cb1a85bb166 100644
> --- a/arch/arm/configs/sama5_defconfig
> +++ b/arch/arm/configs/sama5_defconfig
> @@ -1,6 +1,7 @@
>  # CONFIG_LOCALVERSION_AUTO is not set
>  # CONFIG_SWAP is not set
>  CONFIG_SYSVIPC=y
> +CONFIG_FHANDLE=y
>  CONFIG_IRQ_DOMAIN_DEBUG=y
>  CONFIG_LOG_BUF_SHIFT=14
>  CONFIG_CGROUPS=y
> 


-- 
Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [PATCH] ARM: at91/defconfig: sama5: add CONFIG_FHANDLE
  2016-05-10 12:31         ` Marek Vasut
@ 2016-05-10 12:42           ` Nicolas Ferre
  -1 siblings, 0 replies; 11+ messages in thread
From: Nicolas Ferre @ 2016-05-10 12:42 UTC (permalink / raw)
  To: linux-arm-kernel

Le 10/05/2016 14:31, Marek Vasut a ?crit :
> On 05/10/2016 10:03 AM, Nicolas Ferre wrote:
>> CONFIG_FHANDLE is listed as a mandatory kernel option for systemd. So
>> explicitly enable it to allow easy use of systemd userspace.
>>
>> Reported-by: Marek Vasut <marex@denx.de>
>> Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com>
>> ---
>> Hi Marek,
> 
> Hi!
> 
>> It seems that on my side, CONFIG_FHANDLE is not selected automatically...
>> So I add this patch: can you tell me your thoughts?
> 
> Uh, weird, I tried this on next just now:
> 
> linux-2.6$ git describe HEAD
> next-20160510
> 
> linux-2.6$ export ARCH=arm ; export CROSS_COMPILE=arm-linux-gnueabi-
> 
> linux-2.6$ make sama5_defconfig ; grep FHANDLE .config
>   HOSTCC  scripts/basic/fixdep
>   HOSTCC  scripts/kconfig/conf.o
>   SHIPPED scripts/kconfig/zconf.tab.c
>   SHIPPED scripts/kconfig/zconf.lex.c
>   SHIPPED scripts/kconfig/zconf.hash.c
>   HOSTCC  scripts/kconfig/zconf.tab.o
>   HOSTLD  scripts/kconfig/conf
> #
> # configuration written to .config
> #
> CONFIG_FHANDLE=y

Absolutely. The patch f76be61755c5 ("Make CONFIG_FHANDLE default y") has
been added in v4.6-rc2... while I was doing my tests on at91-next that
is based on v4.6-rc1 :-\

Anyway. It's in now. Thanks,

Bye,
   Nicolas

>>  arch/arm/configs/sama5_defconfig | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/configs/sama5_defconfig b/arch/arm/configs/sama5_defconfig
>> index 307f92b31e57..9cb1a85bb166 100644
>> --- a/arch/arm/configs/sama5_defconfig
>> +++ b/arch/arm/configs/sama5_defconfig
>> @@ -1,6 +1,7 @@
>>  # CONFIG_LOCALVERSION_AUTO is not set
>>  # CONFIG_SWAP is not set
>>  CONFIG_SYSVIPC=y
>> +CONFIG_FHANDLE=y
>>  CONFIG_IRQ_DOMAIN_DEBUG=y
>>  CONFIG_LOG_BUF_SHIFT=14
>>  CONFIG_CGROUPS=y
>>
> 
> 


-- 
Nicolas Ferre

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] ARM: at91/defconfig: sama5: add CONFIG_FHANDLE
@ 2016-05-10 12:42           ` Nicolas Ferre
  0 siblings, 0 replies; 11+ messages in thread
From: Nicolas Ferre @ 2016-05-10 12:42 UTC (permalink / raw)
  To: Marek Vasut
  Cc: Alexandre Belloni, linux-arm-kernel, linux-kernel,
	Boris BREZILLON, Venkateshprabhu.Subramanian, m.olbrich,
	Ludovic Desroches

Le 10/05/2016 14:31, Marek Vasut a écrit :
> On 05/10/2016 10:03 AM, Nicolas Ferre wrote:
>> CONFIG_FHANDLE is listed as a mandatory kernel option for systemd. So
>> explicitly enable it to allow easy use of systemd userspace.
>>
>> Reported-by: Marek Vasut <marex@denx.de>
>> Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com>
>> ---
>> Hi Marek,
> 
> Hi!
> 
>> It seems that on my side, CONFIG_FHANDLE is not selected automatically...
>> So I add this patch: can you tell me your thoughts?
> 
> Uh, weird, I tried this on next just now:
> 
> linux-2.6$ git describe HEAD
> next-20160510
> 
> linux-2.6$ export ARCH=arm ; export CROSS_COMPILE=arm-linux-gnueabi-
> 
> linux-2.6$ make sama5_defconfig ; grep FHANDLE .config
>   HOSTCC  scripts/basic/fixdep
>   HOSTCC  scripts/kconfig/conf.o
>   SHIPPED scripts/kconfig/zconf.tab.c
>   SHIPPED scripts/kconfig/zconf.lex.c
>   SHIPPED scripts/kconfig/zconf.hash.c
>   HOSTCC  scripts/kconfig/zconf.tab.o
>   HOSTLD  scripts/kconfig/conf
> #
> # configuration written to .config
> #
> CONFIG_FHANDLE=y

Absolutely. The patch f76be61755c5 ("Make CONFIG_FHANDLE default y") has
been added in v4.6-rc2... while I was doing my tests on at91-next that
is based on v4.6-rc1 :-\

Anyway. It's in now. Thanks,

Bye,
   Nicolas

>>  arch/arm/configs/sama5_defconfig | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/configs/sama5_defconfig b/arch/arm/configs/sama5_defconfig
>> index 307f92b31e57..9cb1a85bb166 100644
>> --- a/arch/arm/configs/sama5_defconfig
>> +++ b/arch/arm/configs/sama5_defconfig
>> @@ -1,6 +1,7 @@
>>  # CONFIG_LOCALVERSION_AUTO is not set
>>  # CONFIG_SWAP is not set
>>  CONFIG_SYSVIPC=y
>> +CONFIG_FHANDLE=y
>>  CONFIG_IRQ_DOMAIN_DEBUG=y
>>  CONFIG_LOG_BUF_SHIFT=14
>>  CONFIG_CGROUPS=y
>>
> 
> 


-- 
Nicolas Ferre

^ permalink raw reply	[flat|nested] 11+ messages in thread

* [PATCH] ARM: at91/defconfig: sama5: add CONFIG_FHANDLE
  2016-05-10 12:42           ` Nicolas Ferre
@ 2016-05-10 12:46             ` Marek Vasut
  -1 siblings, 0 replies; 11+ messages in thread
From: Marek Vasut @ 2016-05-10 12:46 UTC (permalink / raw)
  To: linux-arm-kernel

On 05/10/2016 02:42 PM, Nicolas Ferre wrote:
> Le 10/05/2016 14:31, Marek Vasut a ?crit :
>> On 05/10/2016 10:03 AM, Nicolas Ferre wrote:
>>> CONFIG_FHANDLE is listed as a mandatory kernel option for systemd. So
>>> explicitly enable it to allow easy use of systemd userspace.
>>>
>>> Reported-by: Marek Vasut <marex@denx.de>
>>> Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com>
>>> ---
>>> Hi Marek,
>>
>> Hi!
>>
>>> It seems that on my side, CONFIG_FHANDLE is not selected automatically...
>>> So I add this patch: can you tell me your thoughts?
>>
>> Uh, weird, I tried this on next just now:
>>
>> linux-2.6$ git describe HEAD
>> next-20160510
>>
>> linux-2.6$ export ARCH=arm ; export CROSS_COMPILE=arm-linux-gnueabi-
>>
>> linux-2.6$ make sama5_defconfig ; grep FHANDLE .config
>>   HOSTCC  scripts/basic/fixdep
>>   HOSTCC  scripts/kconfig/conf.o
>>   SHIPPED scripts/kconfig/zconf.tab.c
>>   SHIPPED scripts/kconfig/zconf.lex.c
>>   SHIPPED scripts/kconfig/zconf.hash.c
>>   HOSTCC  scripts/kconfig/zconf.tab.o
>>   HOSTLD  scripts/kconfig/conf
>> #
>> # configuration written to .config
>> #
>> CONFIG_FHANDLE=y
> 
> Absolutely. The patch f76be61755c5 ("Make CONFIG_FHANDLE default y") has
> been added in v4.6-rc2... while I was doing my tests on at91-next that
> is based on v4.6-rc1 :-\
> 
> Anyway. It's in now. Thanks,

Cool, thanks :)

-- 
Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 11+ messages in thread

* Re: [PATCH] ARM: at91/defconfig: sama5: add CONFIG_FHANDLE
@ 2016-05-10 12:46             ` Marek Vasut
  0 siblings, 0 replies; 11+ messages in thread
From: Marek Vasut @ 2016-05-10 12:46 UTC (permalink / raw)
  To: Nicolas Ferre
  Cc: Alexandre Belloni, linux-arm-kernel, linux-kernel,
	Boris BREZILLON, Venkateshprabhu.Subramanian, m.olbrich,
	Ludovic Desroches

On 05/10/2016 02:42 PM, Nicolas Ferre wrote:
> Le 10/05/2016 14:31, Marek Vasut a écrit :
>> On 05/10/2016 10:03 AM, Nicolas Ferre wrote:
>>> CONFIG_FHANDLE is listed as a mandatory kernel option for systemd. So
>>> explicitly enable it to allow easy use of systemd userspace.
>>>
>>> Reported-by: Marek Vasut <marex@denx.de>
>>> Signed-off-by: Nicolas Ferre <nicolas.ferre@atmel.com>
>>> ---
>>> Hi Marek,
>>
>> Hi!
>>
>>> It seems that on my side, CONFIG_FHANDLE is not selected automatically...
>>> So I add this patch: can you tell me your thoughts?
>>
>> Uh, weird, I tried this on next just now:
>>
>> linux-2.6$ git describe HEAD
>> next-20160510
>>
>> linux-2.6$ export ARCH=arm ; export CROSS_COMPILE=arm-linux-gnueabi-
>>
>> linux-2.6$ make sama5_defconfig ; grep FHANDLE .config
>>   HOSTCC  scripts/basic/fixdep
>>   HOSTCC  scripts/kconfig/conf.o
>>   SHIPPED scripts/kconfig/zconf.tab.c
>>   SHIPPED scripts/kconfig/zconf.lex.c
>>   SHIPPED scripts/kconfig/zconf.hash.c
>>   HOSTCC  scripts/kconfig/zconf.tab.o
>>   HOSTLD  scripts/kconfig/conf
>> #
>> # configuration written to .config
>> #
>> CONFIG_FHANDLE=y
> 
> Absolutely. The patch f76be61755c5 ("Make CONFIG_FHANDLE default y") has
> been added in v4.6-rc2... while I was doing my tests on at91-next that
> is based on v4.6-rc1 :-\
> 
> Anyway. It's in now. Thanks,

Cool, thanks :)

-- 
Best regards,
Marek Vasut

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2016-05-10 12:47 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-05-05 20:31 [PATCH] ARM: at91: sama5: Update defconfig Marek Vasut
2016-05-06 14:30 ` Nicolas Ferre
2016-05-06 16:27   ` Marek Vasut
2016-05-10  8:03     ` [PATCH] ARM: at91/defconfig: sama5: add CONFIG_FHANDLE Nicolas Ferre
2016-05-10  8:03       ` Nicolas Ferre
2016-05-10 12:31       ` Marek Vasut
2016-05-10 12:31         ` Marek Vasut
2016-05-10 12:42         ` Nicolas Ferre
2016-05-10 12:42           ` Nicolas Ferre
2016-05-10 12:46           ` Marek Vasut
2016-05-10 12:46             ` Marek Vasut

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.