* [PATCH] wic/rootfs-u-boot: Allow to set root with source_params
@ 2021-04-08 10:02 Q. Gylstorff
2021-04-08 10:51 ` Henning Schild
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Q. Gylstorff @ 2021-04-08 10:02 UTC (permalink / raw)
To: isar-users; +Cc: Quirin Gylstorff
From: Quirin Gylstorff <quirin.gylstorff@siemens.com>
If the root device needs to be modified during boot it needs
to be set as source parameter in the wic file similar to efi
boot.
Signed-off-by: Quirin Gylstorff <quirin.gylstorff@siemens.com>
---
.../lib/wic/plugins/source/rootfs-u-boot.py | 16 ++++++++++++----
1 file changed, 12 insertions(+), 4 deletions(-)
diff --git a/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py b/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py
index 3ac0aa0..1e10648 100644
--- a/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py
+++ b/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py
@@ -37,7 +37,7 @@ class RootfsUBootPlugin(RootfsPlugin):
# Prologue from RootfsPlugin.do_prepare_partition, retrieves the
# rootfs directory
if part.rootfs_dir is None:
- if not 'ROOTFS_DIR' in krootfs_dir:
+ if 'ROOTFS_DIR' not in krootfs_dir:
raise WicError("Couldn't find --rootfs-dir, exiting")
rootfs_dir = krootfs_dir['ROOTFS_DIR']
@@ -48,7 +48,8 @@ class RootfsUBootPlugin(RootfsPlugin):
rootfs_dir = part.rootfs_dir
else:
raise WicError("Couldn't find --rootfs-dir=%s connection or "
- "it is not a valid path, exiting" % part.rootfs_dir)
+ "it is not a valid path, exiting" %
+ part.rootfs_dir)
if os.path.isdir(rootfs_dir):
real_rootfs_dir = rootfs_dir
else:
@@ -59,6 +60,13 @@ class RootfsUBootPlugin(RootfsPlugin):
(rootfs_dir, image_rootfs_dir))
real_rootfs_dir = image_rootfs_dir
+ root_dev = cr.rootdev
+ if not root_dev:
+ root_dev = source_params.get("root", None)
+ if not root_dev:
+ raise WicError("root not defined, exiting.")
+ root_dev = root_dev.replace(":", "=")
+
u_boot_script = os.path.join(real_rootfs_dir,
"etc/default/u-boot-script")
if not os.path.exists(u_boot_script):
@@ -68,8 +76,8 @@ class RootfsUBootPlugin(RootfsPlugin):
with open(u_boot_script, 'w') as cfg:
cfg.write('# Generated by wic, rootfs-u-boot plugin\n')
cfg.write('ROOT_PARTITION="%d"\n' % part.realnum)
- cfg.write('KERNEL_ARGS="root=%s %s"\n' % \
- (cr.rootdev, cr.ks.bootloader.append or ""))
+ cfg.write('KERNEL_ARGS="root=%s %s"\n' %
+ (root_dev, cr.ks.bootloader.append or ""))
no_initrd = source_params.get('no_initrd') or ''
cfg.write('NO_INITRD="%s"\n' % no_initrd)
overlays = source_params.get('overlays') or ''
--
2.20.1
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] wic/rootfs-u-boot: Allow to set root with source_params
2021-04-08 10:02 [PATCH] wic/rootfs-u-boot: Allow to set root with source_params Q. Gylstorff
@ 2021-04-08 10:51 ` Henning Schild
2021-04-08 10:53 ` Henning Schild
2021-04-08 11:53 ` Jan Kiszka
2 siblings, 0 replies; 4+ messages in thread
From: Henning Schild @ 2021-04-08 10:51 UTC (permalink / raw)
To: [ext] Q. Gylstorff; +Cc: isar-users
This needs to be done in OE and will eventually arrive here. We can
shorten the time with a cherry pick, but this really needs to go into
OE first.
Alternative would be a new forked plugin, but also not nice.
Henning
Am Thu, 8 Apr 2021 12:02:41 +0200
schrieb "[ext] Q. Gylstorff" <Quirin.Gylstorff@siemens.com>:
> From: Quirin Gylstorff <quirin.gylstorff@siemens.com>
>
> If the root device needs to be modified during boot it needs
> to be set as source parameter in the wic file similar to efi
> boot.
>
> Signed-off-by: Quirin Gylstorff <quirin.gylstorff@siemens.com>
> ---
> .../lib/wic/plugins/source/rootfs-u-boot.py | 16
> ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-)
>
> diff --git a/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py
> b/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py index
> 3ac0aa0..1e10648 100644 ---
> a/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py +++
> b/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py @@ -37,7 +37,7
> @@ class RootfsUBootPlugin(RootfsPlugin): # Prologue from
> RootfsPlugin.do_prepare_partition, retrieves the # rootfs directory
> if part.rootfs_dir is None:
> - if not 'ROOTFS_DIR' in krootfs_dir:
> + if 'ROOTFS_DIR' not in krootfs_dir:
> raise WicError("Couldn't find --rootfs-dir, exiting")
>
> rootfs_dir = krootfs_dir['ROOTFS_DIR']
> @@ -48,7 +48,8 @@ class RootfsUBootPlugin(RootfsPlugin):
> rootfs_dir = part.rootfs_dir
> else:
> raise WicError("Couldn't find --rootfs-dir=%s
> connection or "
> - "it is not a valid path, exiting" %
> part.rootfs_dir)
> + "it is not a valid path, exiting" %
> + part.rootfs_dir)
> if os.path.isdir(rootfs_dir):
> real_rootfs_dir = rootfs_dir
> else:
> @@ -59,6 +60,13 @@ class RootfsUBootPlugin(RootfsPlugin):
> (rootfs_dir, image_rootfs_dir))
> real_rootfs_dir = image_rootfs_dir
>
> + root_dev = cr.rootdev
> + if not root_dev:
> + root_dev = source_params.get("root", None)
> + if not root_dev:
> + raise WicError("root not defined, exiting.")
> + root_dev = root_dev.replace(":", "=")
> +
> u_boot_script = os.path.join(real_rootfs_dir,
> "etc/default/u-boot-script")
> if not os.path.exists(u_boot_script):
> @@ -68,8 +76,8 @@ class RootfsUBootPlugin(RootfsPlugin):
> with open(u_boot_script, 'w') as cfg:
> cfg.write('# Generated by wic, rootfs-u-boot plugin\n')
> cfg.write('ROOT_PARTITION="%d"\n' % part.realnum)
> - cfg.write('KERNEL_ARGS="root=%s %s"\n' % \
> - (cr.rootdev, cr.ks.bootloader.append or ""))
> + cfg.write('KERNEL_ARGS="root=%s %s"\n' %
> + (root_dev, cr.ks.bootloader.append or ""))
> no_initrd = source_params.get('no_initrd') or ''
> cfg.write('NO_INITRD="%s"\n' % no_initrd)
> overlays = source_params.get('overlays') or ''
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] wic/rootfs-u-boot: Allow to set root with source_params
2021-04-08 10:02 [PATCH] wic/rootfs-u-boot: Allow to set root with source_params Q. Gylstorff
2021-04-08 10:51 ` Henning Schild
@ 2021-04-08 10:53 ` Henning Schild
2021-04-08 11:53 ` Jan Kiszka
2 siblings, 0 replies; 4+ messages in thread
From: Henning Schild @ 2021-04-08 10:53 UTC (permalink / raw)
To: [ext] Q. Gylstorff; +Cc: isar-users
Sorry, that is already a plugin of our own. Forget that other mail.
Henning
Am Thu, 8 Apr 2021 12:02:41 +0200
schrieb "[ext] Q. Gylstorff" <Quirin.Gylstorff@siemens.com>:
> From: Quirin Gylstorff <quirin.gylstorff@siemens.com>
>
> If the root device needs to be modified during boot it needs
> to be set as source parameter in the wic file similar to efi
> boot.
>
> Signed-off-by: Quirin Gylstorff <quirin.gylstorff@siemens.com>
> ---
> .../lib/wic/plugins/source/rootfs-u-boot.py | 16
> ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-)
>
> diff --git a/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py
> b/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py index
> 3ac0aa0..1e10648 100644 ---
> a/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py +++
> b/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py @@ -37,7 +37,7
> @@ class RootfsUBootPlugin(RootfsPlugin): # Prologue from
> RootfsPlugin.do_prepare_partition, retrieves the # rootfs directory
> if part.rootfs_dir is None:
> - if not 'ROOTFS_DIR' in krootfs_dir:
> + if 'ROOTFS_DIR' not in krootfs_dir:
> raise WicError("Couldn't find --rootfs-dir, exiting")
>
> rootfs_dir = krootfs_dir['ROOTFS_DIR']
> @@ -48,7 +48,8 @@ class RootfsUBootPlugin(RootfsPlugin):
> rootfs_dir = part.rootfs_dir
> else:
> raise WicError("Couldn't find --rootfs-dir=%s
> connection or "
> - "it is not a valid path, exiting" %
> part.rootfs_dir)
> + "it is not a valid path, exiting" %
> + part.rootfs_dir)
> if os.path.isdir(rootfs_dir):
> real_rootfs_dir = rootfs_dir
> else:
> @@ -59,6 +60,13 @@ class RootfsUBootPlugin(RootfsPlugin):
> (rootfs_dir, image_rootfs_dir))
> real_rootfs_dir = image_rootfs_dir
>
> + root_dev = cr.rootdev
> + if not root_dev:
> + root_dev = source_params.get("root", None)
> + if not root_dev:
> + raise WicError("root not defined, exiting.")
> + root_dev = root_dev.replace(":", "=")
> +
> u_boot_script = os.path.join(real_rootfs_dir,
> "etc/default/u-boot-script")
> if not os.path.exists(u_boot_script):
> @@ -68,8 +76,8 @@ class RootfsUBootPlugin(RootfsPlugin):
> with open(u_boot_script, 'w') as cfg:
> cfg.write('# Generated by wic, rootfs-u-boot plugin\n')
> cfg.write('ROOT_PARTITION="%d"\n' % part.realnum)
> - cfg.write('KERNEL_ARGS="root=%s %s"\n' % \
> - (cr.rootdev, cr.ks.bootloader.append or ""))
> + cfg.write('KERNEL_ARGS="root=%s %s"\n' %
> + (root_dev, cr.ks.bootloader.append or ""))
> no_initrd = source_params.get('no_initrd') or ''
> cfg.write('NO_INITRD="%s"\n' % no_initrd)
> overlays = source_params.get('overlays') or ''
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] wic/rootfs-u-boot: Allow to set root with source_params
2021-04-08 10:02 [PATCH] wic/rootfs-u-boot: Allow to set root with source_params Q. Gylstorff
2021-04-08 10:51 ` Henning Schild
2021-04-08 10:53 ` Henning Schild
@ 2021-04-08 11:53 ` Jan Kiszka
2 siblings, 0 replies; 4+ messages in thread
From: Jan Kiszka @ 2021-04-08 11:53 UTC (permalink / raw)
To: [ext] Q. Gylstorff, isar-users
On 08.04.21 12:02, [ext] Q. Gylstorff wrote:
> From: Quirin Gylstorff <quirin.gylstorff@siemens.com>
>
> If the root device needs to be modified during boot it needs
> to be set as source parameter in the wic file similar to efi
> boot.
>
> Signed-off-by: Quirin Gylstorff <quirin.gylstorff@siemens.com>
> ---
> .../lib/wic/plugins/source/rootfs-u-boot.py | 16 ++++++++++++----
> 1 file changed, 12 insertions(+), 4 deletions(-)
>
> diff --git a/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py b/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py
> index 3ac0aa0..1e10648 100644
> --- a/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py
> +++ b/meta/scripts/lib/wic/plugins/source/rootfs-u-boot.py
> @@ -37,7 +37,7 @@ class RootfsUBootPlugin(RootfsPlugin):
> # Prologue from RootfsPlugin.do_prepare_partition, retrieves the
> # rootfs directory
> if part.rootfs_dir is None:
> - if not 'ROOTFS_DIR' in krootfs_dir:
> + if 'ROOTFS_DIR' not in krootfs_dir:
> raise WicError("Couldn't find --rootfs-dir, exiting")
>
> rootfs_dir = krootfs_dir['ROOTFS_DIR']
> @@ -48,7 +48,8 @@ class RootfsUBootPlugin(RootfsPlugin):
> rootfs_dir = part.rootfs_dir
> else:
> raise WicError("Couldn't find --rootfs-dir=%s connection or "
> - "it is not a valid path, exiting" % part.rootfs_dir)
> + "it is not a valid path, exiting" %
> + part.rootfs_dir)
> if os.path.isdir(rootfs_dir):
> real_rootfs_dir = rootfs_dir
> else:
> @@ -59,6 +60,13 @@ class RootfsUBootPlugin(RootfsPlugin):
> (rootfs_dir, image_rootfs_dir))
> real_rootfs_dir = image_rootfs_dir
>
> + root_dev = cr.rootdev
> + if not root_dev:
> + root_dev = source_params.get("root", None)
> + if not root_dev:
> + raise WicError("root not defined, exiting.")
> + root_dev = root_dev.replace(":", "=")
> +
> u_boot_script = os.path.join(real_rootfs_dir,
> "etc/default/u-boot-script")
> if not os.path.exists(u_boot_script):
> @@ -68,8 +76,8 @@ class RootfsUBootPlugin(RootfsPlugin):
> with open(u_boot_script, 'w') as cfg:
> cfg.write('# Generated by wic, rootfs-u-boot plugin\n')
> cfg.write('ROOT_PARTITION="%d"\n' % part.realnum)
> - cfg.write('KERNEL_ARGS="root=%s %s"\n' % \
> - (cr.rootdev, cr.ks.bootloader.append or ""))
> + cfg.write('KERNEL_ARGS="root=%s %s"\n' %
> + (root_dev, cr.ks.bootloader.append or ""))
> no_initrd = source_params.get('no_initrd') or ''
> cfg.write('NO_INITRD="%s"\n' % no_initrd)
> overlays = source_params.get('overlays') or ''
>
Quite a bit of unrelated style changes in this patch - better split up.
Jan
--
Siemens AG, T RDA IOT
Corporate Competence Center Embedded Linux
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-04-08 11:57 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-04-08 10:02 [PATCH] wic/rootfs-u-boot: Allow to set root with source_params Q. Gylstorff
2021-04-08 10:51 ` Henning Schild
2021-04-08 10:53 ` Henning Schild
2021-04-08 11:53 ` Jan Kiszka
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox