* [PATCH] ARM: at91: sama5: Update defconfig @ 2016-05-05 20:31 Marek Vasut 2016-05-06 14:30 ` Nicolas Ferre 0 siblings, 1 reply; 7+ 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] 7+ 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; 7+ 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] 7+ 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 ` [PATCH] ARM: at91/defconfig: sama5: add CONFIG_FHANDLE Nicolas Ferre 0 siblings, 1 reply; 7+ 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] 7+ 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 2016-05-10 12:31 ` Marek Vasut 0 siblings, 1 reply; 7+ 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] 7+ messages in thread
* [PATCH] ARM: at91/defconfig: sama5: add CONFIG_FHANDLE 2016-05-10 8:03 ` [PATCH] ARM: at91/defconfig: sama5: add CONFIG_FHANDLE Nicolas Ferre @ 2016-05-10 12:31 ` Marek Vasut 2016-05-10 12:42 ` Nicolas Ferre 0 siblings, 1 reply; 7+ 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] 7+ 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 2016-05-10 12:46 ` Marek Vasut 0 siblings, 1 reply; 7+ 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] 7+ 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 0 siblings, 0 replies; 7+ 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] 7+ messages in thread
end of thread, other threads:[~2016-05-10 12:46 UTC | newest] Thread overview: 7+ 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 12:31 ` Marek Vasut 2016-05-10 12:42 ` Nicolas Ferre 2016-05-10 12:46 ` Marek Vasut
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).