All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] arc: Removed local initramfs source file from axs101 configuration.
@ 2016-01-04 14:55 CARLOS.PALMINHA
  2016-01-07 23:20 ` Alexey Brodkin
  0 siblings, 1 reply; 6+ messages in thread
From: CARLOS.PALMINHA @ 2016-01-04 14:55 UTC (permalink / raw)
  To: linux-snps-arc

From: Carlos Palminha <palminha@synopsys.com>

Cannot compile kernel with INITRAMFS SOURCE option.
Please move this option to a buildroot patch.

---
 arch/arc/configs/axs101_defconfig | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arc/configs/axs101_defconfig b/arch/arc/configs/axs101_defconfig
index f1ac981..3d14cf8 100644
--- a/arch/arc/configs/axs101_defconfig
+++ b/arch/arc/configs/axs101_defconfig
@@ -12,7 +12,6 @@ CONFIG_NAMESPACES=y
 # CONFIG_UTS_NS is not set
 # CONFIG_PID_NS is not set
 CONFIG_BLK_DEV_INITRD=y
-CONFIG_INITRAMFS_SOURCE="../arc_initramfs/"
 CONFIG_EMBEDDED=y
 CONFIG_PERF_EVENTS=y
 # CONFIG_VM_EVENT_COUNTERS is not set
-- 
2.5.0

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

* [PATCH] arc: Removed local initramfs source file from axs101 configuration.
  2016-01-04 14:55 [PATCH] arc: Removed local initramfs source file from axs101 configuration CARLOS.PALMINHA
@ 2016-01-07 23:20 ` Alexey Brodkin
  2016-01-09  0:25   ` Carlos Palminha
  0 siblings, 1 reply; 6+ messages in thread
From: Alexey Brodkin @ 2016-01-07 23:20 UTC (permalink / raw)
  To: linux-snps-arc

Hi Carlos,

On Mon, 2016-01-04@14:55 +0000, palminha@synopsys.com wrote:
> From: Carlos Palminha <palminha at synopsys.com>
> 
> Cannot compile kernel with INITRAMFS SOURCE option.
> Please move this option to a buildroot patch.

First of all if your problem is only related to Buildroot then
there's not much sense in adding linux-snps-arc at lists.infradead.org
which is used for pure Linux kernel discussions.

Then even in case of Buildroot CONFIG_INITRAMFS_SOURCE could be a problem
only if you don't set Buildroot's BR2_TARGET_ROOTFS_INITRAMFS=y, see
how it is done in snps_axs101_defconfig here
https://git.busybox.net/buildroot/tree/configs/snps_axs101_defconfig#n7

What happens if BR2_TARGET_ROOTFS_INITRAMFS=y then existing in kernel's
defconfig CONFIG_INITRAMFS_SOURCE gets overridden with new value, see
https://git.busybox.net/buildroot/tree/linux/linux.mk#n217

Now if you really want to not use built-in kernel initramfs then simplest
thing to do is to run "make linux-menuconfig" from your Buildroot's folder
and disable or set with nothing CONFIG_INITRAMFS_SOURCE.

-Alexey

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

* [PATCH] arc: Removed local initramfs source file from axs101 configuration.
  2016-01-07 23:20 ` Alexey Brodkin
@ 2016-01-09  0:25   ` Carlos Palminha
  2016-01-11  7:47     ` Vineet Gupta
  0 siblings, 1 reply; 6+ messages in thread
From: Carlos Palminha @ 2016-01-09  0:25 UTC (permalink / raw)
  To: linux-snps-arc

Hi Alexey,

Probably my e-mail comments before the patch mislead you...
If you download the kernel source and try to compile it, it won't work because of the INITRAMFS pointing to a non-existing directory. (check log below)

That's why that option should be removed from the standard kernel.

Regards,
C.Palminha

palminha at palminha-vm:~/src/kernels/linux-mainline-next$ make axs101_defconfig
  HOSTCC  scripts/kconfig/conf.o
  HOSTCC  scripts/kconfig/zconf.tab.o
  HOSTLD  scripts/kconfig/conf
#
# configuration written to .config
#
palminha at palminha-vm:~/src/kernels/linux-mainline-next$ make
scripts/kconfig/conf  --silentoldconfig Kconfig
  CHK     include/config/kernel.release
  CHK     include/generated/uapi/linux/version.h
  CHK     include/generated/utsrelease.h
  CC      kernel/bounds.s
  CHK     include/generated/bounds.h
  CHK     include/generated/timeconst.h
  CC      arch/arc/kernel/asm-offsets.s
  CHK     include/generated/asm-offsets.h
  CALL    scripts/checksyscalls.sh
  HOSTCC  scripts/dtc/dtc.o
  HOSTCC  scripts/dtc/flattree.o
  HOSTCC  scripts/dtc/fstree.o
  HOSTCC  scripts/dtc/data.o
  HOSTCC  scripts/dtc/livetree.o
  HOSTCC  scripts/dtc/treesource.o
  HOSTCC  scripts/dtc/srcpos.o
  HOSTCC  scripts/dtc/checks.o
  HOSTCC  scripts/dtc/util.o
  HOSTCC  scripts/dtc/dtc-lexer.lex.o
  HOSTCC  scripts/dtc/dtc-parser.tab.o
  HOSTLD  scripts/dtc/dtc
  CC      scripts/mod/empty.o
  HOSTCC  scripts/mod/mk_elfconfig
  MKELF   scripts/mod/elfconfig.h
  HOSTCC  scripts/mod/modpost.o
  CC      scripts/mod/devicetable-offsets.s
  GEN     scripts/mod/devicetable-offsets.h
  HOSTCC  scripts/mod/file2alias.o
  HOSTCC  scripts/mod/sumversion.o
  HOSTLD  scripts/mod/modpost
  HOSTCC  scripts/kallsyms
  HOSTCC  scripts/pnmtologo
  HOSTCC  scripts/conmakehash
  HOSTCC  scripts/sortextable
  CC      init/main.o
  CHK     include/generated/compile.h
  CC      init/version.o
  CC      init/do_mounts.o
  CC      init/do_mounts_initrd.o
  LD      init/mounts.o
  CC      init/initramfs.o
  CC      init/calibrate.o
  CC      init/init_task.o
  LD      init/built-in.o
  HOSTCC  usr/gen_init_cpio
  ./scripts/gen_initramfs_list.sh: Cannot open '../arc_initramfs/'
usr/Makefile:73: recipe for target 'usr/initramfs_data.cpio.gz' failed
make[1]: *** [usr/initramfs_data.cpio.gz] Error 1
Makefile:943: recipe for target 'usr' failed
make: *** [usr] Error 2
palminha at palminha-vm:~/src/kernels/linux-mainline-next$ 


On 07-01-2016 23:20, Alexey Brodkin wrote:
> Hi Carlos,
> 
> On Mon, 2016-01-04@14:55 +0000, palminha@synopsys.com wrote:
>> From: Carlos Palminha <palminha at synopsys.com>
>>
>> Cannot compile kernel with INITRAMFS SOURCE option.
>> Please move this option to a buildroot patch.
> 
> First of all if your problem is only related to Buildroot then
> there's not much sense in adding linux-snps-arc at lists.infradead.org
> which is used for pure Linux kernel discussions.
> 
> Then even in case of Buildroot CONFIG_INITRAMFS_SOURCE could be a problem
> only if you don't set Buildroot's BR2_TARGET_ROOTFS_INITRAMFS=y, see
> how it is done in snps_axs101_defconfig here
> https://git.busybox.net/buildroot/tree/configs/snps_axs101_defconfig#n7
> 
> What happens if BR2_TARGET_ROOTFS_INITRAMFS=y then existing in kernel's
> defconfig CONFIG_INITRAMFS_SOURCE gets overridden with new value, see
> https://git.busybox.net/buildroot/tree/linux/linux.mk#n217
> 
> Now if you really want to not use built-in kernel initramfs then simplest
> thing to do is to run "make linux-menuconfig" from your Buildroot's folder
> and disable or set with nothing CONFIG_INITRAMFS_SOURCE.
> 
> -Alexey
> 

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

* [PATCH] arc: Removed local initramfs source file from axs101 configuration.
  2016-01-09  0:25   ` Carlos Palminha
@ 2016-01-11  7:47     ` Vineet Gupta
  2016-01-11  7:59       ` Alexey Brodkin
  2016-01-13  1:13       ` Carlos Palminha
  0 siblings, 2 replies; 6+ messages in thread
From: Vineet Gupta @ 2016-01-11  7:47 UTC (permalink / raw)
  To: linux-snps-arc

On Saturday 09 January 2016 05:55 AM, Carlos Palminha wrote:
> Hi Alexey,
>
> Probably my e-mail comments before the patch mislead you...
> If you download the kernel source and try to compile it, it won't work because of the INITRAMFS pointing to a non-existing directory. (check log below)
>
> That's why that option should be removed from the standard kernel.

Since the axs101 DT cmdline lacks any reference to root device - how does this
unmodified kernel boot anyways w/o changing DT. Does uboot provide the cmdline for
rootfs ?

The reason we've had initramfs for ages (vs. relying on uboot) is because we can't
have uboot based workflow primarily because Alexey / I have 1 AXS MB and we
constantly keep switching between 700 and HS builds.

I think you can use KCONFIG_ALLCONFIG (grep in Documentation) to override the
INITRAMFS !

-Vineet

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

* [PATCH] arc: Removed local initramfs source file from axs101 configuration.
  2016-01-11  7:47     ` Vineet Gupta
@ 2016-01-11  7:59       ` Alexey Brodkin
  2016-01-13  1:13       ` Carlos Palminha
  1 sibling, 0 replies; 6+ messages in thread
From: Alexey Brodkin @ 2016-01-11  7:59 UTC (permalink / raw)
  To: linux-snps-arc

Hi Vineet,

On Mon, 2016-01-11@07:47 +0000, Vineet Gupta wrote:
> On Saturday 09 January 2016 05:55 AM, Carlos Palminha wrote:
> > Hi Alexey,
> > 
> > Probably my e-mail comments before the patch mislead you...
> > If you download the kernel source and try to compile it, it won't work because of the INITRAMFS pointing to a non
> > -existing directory. (check log below)
> > 
> > That's why that option should be removed from the standard kernel.
> 
> Since the axs101 DT cmdline lacks any reference to root device - how does this
> unmodified kernel boot anyways w/o changing DT. Does uboot provide the cmdline for
> rootfs ?

Indeed if initramfs is not built-in vmlinux/uImage then kernel command-line
must be adjusted to point to real root device.

This could be done in 2 ways:

 [1] Manually modify "bootargs" value in arch/arc/boot/dts/axs10{1|3}.dts
 [2] In case of U-Boot it's possible to load device tree blob together with uImage
     and in that case U-Boot's "bootargs" env variable will override the one in .dtb

-Alexey

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

* [PATCH] arc: Removed local initramfs source file from axs101 configuration.
  2016-01-11  7:47     ` Vineet Gupta
  2016-01-11  7:59       ` Alexey Brodkin
@ 2016-01-13  1:13       ` Carlos Palminha
  1 sibling, 0 replies; 6+ messages in thread
From: Carlos Palminha @ 2016-01-13  1:13 UTC (permalink / raw)
  To: linux-snps-arc

Hi guys,

I don't think maintaining that flag this in the kernel is correct... the kernel simply doesn't compile out of the box.
If you grep for  CONFIG_INITRAMFS_SOURCE, ARC is the only architecture that uses that kernel flag by default pointing to an external directory.
I don't think this is proper to have it like this by default from the kernel point of view.

Regards,
C.Palminha

palminha at PALMINHA-E7440 ~/Development/linus.git.linux ((v4.3))
$ grep -r INITRAMFS_SOURCE *|grep configs
arch/arc/configs/axs101_defconfig:CONFIG_INITRAMFS_SOURCE="../arc_initramfs/"
arch/arc/configs/axs103_defconfig:CONFIG_INITRAMFS_SOURCE="../../arc_initramfs_hs/"
arch/arc/configs/axs103_smp_defconfig:CONFIG_INITRAMFS_SOURCE="../../arc_initramfs_hs/"
arch/arc/configs/nsimosci_defconfig:CONFIG_INITRAMFS_SOURCE="../arc_initramfs/"
arch/arc/configs/nsimosci_hs_defconfig:CONFIG_INITRAMFS_SOURCE="../arc_initramfs_hs/"
arch/arc/configs/nsimosci_hs_smp_defconfig:CONFIG_INITRAMFS_SOURCE="../arc_initramfs_hs/"
arch/arc/configs/nsim_700_defconfig:CONFIG_INITRAMFS_SOURCE="../arc_initramfs/"
arch/arc/configs/nsim_hs_defconfig:CONFIG_INITRAMFS_SOURCE="../arc_initramfs_hs/"
arch/arc/configs/nsim_hs_smp_defconfig:CONFIG_INITRAMFS_SOURCE="../arc_initramfs_hs/"
arch/arc/configs/tb10x_defconfig:CONFIG_INITRAMFS_SOURCE="../tb10x-rootfs.cpio"
arch/mips/configs/nlm_xlr_defconfig:CONFIG_INITRAMFS_SOURCE=""
arch/mips/configs/rt305x_defconfig:CONFIG_INITRAMFS_SOURCE=""
arch/unicore32/configs/unicore32_defconfig:#CONFIG_INITRAMFS_SOURCE="arch/unicore/ramfs/ramfs_config"
arch/xtensa/configs/common_defconfig:CONFIG_INITRAMFS_SOURCE=""

On 11-01-2016 07:47, Vineet Gupta wrote:
> On Saturday 09 January 2016 05:55 AM, Carlos Palminha wrote:
>> Hi Alexey,
>>
>> Probably my e-mail comments before the patch mislead you...
>> If you download the kernel source and try to compile it, it won't work because of the INITRAMFS pointing to a non-existing directory. (check log below)
>>
>> That's why that option should be removed from the standard kernel.
> 
> Since the axs101 DT cmdline lacks any reference to root device - how does this
> unmodified kernel boot anyways w/o changing DT. Does uboot provide the cmdline for
> rootfs ?
> 
> The reason we've had initramfs for ages (vs. relying on uboot) is because we can't
> have uboot based workflow primarily because Alexey / I have 1 AXS MB and we
> constantly keep switching between 700 and HS builds.
> 
> I think you can use KCONFIG_ALLCONFIG (grep in Documentation) to override the
> INITRAMFS !
> 
> -Vineet
> 

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

end of thread, other threads:[~2016-01-13  1:13 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-01-04 14:55 [PATCH] arc: Removed local initramfs source file from axs101 configuration CARLOS.PALMINHA
2016-01-07 23:20 ` Alexey Brodkin
2016-01-09  0:25   ` Carlos Palminha
2016-01-11  7:47     ` Vineet Gupta
2016-01-11  7:59       ` Alexey Brodkin
2016-01-13  1:13       ` Carlos Palminha

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.