public inbox for isar-users@googlegroups.com
 help / color / mirror / Atom feed
* [PATCH] wic: support passing extraargs for EFI partition creation
@ 2025-12-17 13:02 'Richa Bharti' via isar-users
  2025-12-18  7:28 ` Anton Mikanovich
  0 siblings, 1 reply; 10+ messages in thread
From: 'Richa Bharti' via isar-users @ 2025-12-17 13:02 UTC (permalink / raw)
  To: isar-users; +Cc: cedric.hombourger, srikanth.krishnakar, Richa Bharti

Some EFI firmwares require FAT filesystems to use specific geometry or
sector parameters.

Add support for an `extraargs` source parameter to allow passing
additional mkdosfs options when creating the EFI System Partition (ESP).
If no extra arguments are provided, mkdosfs defaults to "-S 512",
preserving existing behaviour.

Signed-off-by: Richa Bharti <Richa.Bharti@siemens.com>
---
 meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py b/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
index 446398d0..6862a288 100644
--- a/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
+++ b/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
@@ -506,8 +506,10 @@ class BootimgEFIPlugin(SourcePlugin):
 
         label = part.label if part.label else "ESP"
 
-        dosfs_cmd = "mkdosfs -n %s -i %s -C %s %d" % \
-                    (label, part.fsuuid, bootimg, blocks)
+        extraargs = source_params['extraargs'] or "-S 512"
+
+        dosfs_cmd = "mkdosfs -n %s -i %s %s -C %s %d" % \
+                    (label, part.fsuuid, extraargs, bootimg, blocks)
         exec_native_cmd(dosfs_cmd, native_sysroot)
 
         mcopy_cmd = "mcopy -i %s -s %s/* ::/" % (bootimg, hdddir)
-- 
2.39.5

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/20251217130253.947946-1-Richa.Bharti%40siemens.com.

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

* Re: [PATCH] wic: support passing extraargs for EFI partition creation
  2025-12-17 13:02 [PATCH] wic: support passing extraargs for EFI partition creation 'Richa Bharti' via isar-users
@ 2025-12-18  7:28 ` Anton Mikanovich
  2025-12-18  8:22   ` [PATCH v2] " 'Richa Bharti' via isar-users
  0 siblings, 1 reply; 10+ messages in thread
From: Anton Mikanovich @ 2025-12-18  7:28 UTC (permalink / raw)
  To: isar-users, Richa Bharti

Hello Richa,

17/12/2025 15:02, 'Richa Bharti' via isar-users wrote:
> Some EFI firmwares require FAT filesystems to use specific geometry or
> sector parameters.
>
> Add support for an `extraargs` source parameter to allow passing
> additional mkdosfs options when creating the EFI System Partition (ESP).
> If no extra arguments are provided, mkdosfs defaults to "-S 512",
> preserving existing behaviour.
>
> Signed-off-by: Richa Bharti <Richa.Bharti@siemens.com>
> ---
>   meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py b/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
> index 446398d0..6862a288 100644
> --- a/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
> +++ b/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
> @@ -506,8 +506,10 @@ class BootimgEFIPlugin(SourcePlugin):
>   
>           label = part.label if part.label else "ESP"
>   
> -        dosfs_cmd = "mkdosfs -n %s -i %s -C %s %d" % \
> -                    (label, part.fsuuid, bootimg, blocks)
> +        extraargs = source_params['extraargs'] or "-S 512"
This will fail with KeyError: 'extraargs' if no extraargs declared.
> +
> +        dosfs_cmd = "mkdosfs -n %s -i %s %s -C %s %d" % \
> +                    (label, part.fsuuid, extraargs, bootimg, blocks)
>           exec_native_cmd(dosfs_cmd, native_sysroot)
>   
>           mcopy_cmd = "mcopy -i %s -s %s/* ::/" % (bootimg, hdddir)

Moreover, features like this should be covered by some target or have a 
test\
case in CI.

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/03fe0f50-b239-4d2f-8825-4de8cd6e249d%40ilbers.de.

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

* [PATCH v2] wic: support passing extraargs for EFI partition creation
  2025-12-18  7:28 ` Anton Mikanovich
@ 2025-12-18  8:22   ` 'Richa Bharti' via isar-users
  2025-12-18  8:29     ` RICHA BHARTI
                       ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: 'Richa Bharti' via isar-users @ 2025-12-18  8:22 UTC (permalink / raw)
  To: isar-users, amikan; +Cc: cedric.hombourger, srikanth.krishnakar, Richa Bharti

Some EFI firmwares require FAT filesystems to use specific geometry or
sector parameters.

Add support for an `extraargs` source parameter to allow passing
additional mkdosfs options when creating the EFI System Partition (ESP).
If no extra arguments are provided, mkdosfs defaults to "-S 512",
preserving existing behaviour.

Signed-off-by: Richa Bharti <Richa.Bharti@siemens.com>
---
 meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py b/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
index 446398d0..6862a288 100644
--- a/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
+++ b/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
@@ -506,8 +506,10 @@ class BootimgEFIPlugin(SourcePlugin):
 
         label = part.label if part.label else "ESP"
 
-        dosfs_cmd = "mkdosfs -n %s -i %s -C %s %d" % \
-                    (label, part.fsuuid, bootimg, blocks)
+        extraargs = source_params.get('extraargs', "-S 512")
+
+        dosfs_cmd = "mkdosfs -n %s -i %s %s -C %s %d" % \
+                    (label, part.fsuuid, extraargs, bootimg, blocks)
         exec_native_cmd(dosfs_cmd, native_sysroot)
 
         mcopy_cmd = "mcopy -i %s -s %s/* ::/" % (bootimg, hdddir)
-- 
2.39.5

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/20251218082208.1031200-1-Richa.Bharti%40siemens.com.

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

* Re: [PATCH v2] wic: support passing extraargs for EFI partition creation
  2025-12-18  8:22   ` [PATCH v2] " 'Richa Bharti' via isar-users
@ 2025-12-18  8:29     ` RICHA BHARTI
  2025-12-18  8:39     ` 'Bharti, Richa' via isar-users
  2025-12-22  7:24     ` Anton Mikanovich
  2 siblings, 0 replies; 10+ messages in thread
From: RICHA BHARTI @ 2025-12-18  8:29 UTC (permalink / raw)
  To: isar-users


[-- Attachment #1.1: Type: text/plain, Size: 5410 bytes --]

*Without Extraargs:*
```
$ cat wic/generic-ufs-partitions.wks.in
part /boot --source bootimg-efi-isar --sourceparams "loader=systemd-boot" 
--label efi --part-type EF00 --align 1024 --use-uuid --size 512M
part --use-uuid --source rootfs --fstype=${BSP_ROOTFS_TYPE} --label system 
--active --align 1024 --exclude-path boot --fixed-size 
${UFS_PART_SYSTEM_SIZE}
bootloader --ptable gpt --timeout 0 --append "${BSP_KERNEL_CMDLINE}

$ file development-image-industrial-os-qcs615-adp-air.wic.p1
development-image-industrial-os-qcs615-adp-air.wic.p1: DOS/MBR boot sector, 
code offset 0x58+2, OEM-ID "mkfs.fat", sectors/cluster 8, Media descriptor 
0xf8, sectors/track 63, heads 32, sectors 1048572 (volumes > 32 MB), FAT 
(32 bit), sectors/FAT 1024, serial number 0xe9d3c0d4, label: "efi        "
```

*LOG:*
```
DEBUG: ['mkdosfs', '-n', 'efi', '-i', '0x1C1D0579', '-S', '512', '-C', 
'/tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.vlfgf_0b/boot.img', 
'524288']
DEBUG: exec_native_cmd: export 
PATH=//sbin://usr/sbin://usr/bin://usr/bin/None://bin:None:$PATH;mkdosfs -n 
efi -i 0x1C1D0579 -S 512 -C 
/tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.vlfgf_0b/boot.img 
524288
DEBUG: _exec_cmd: export 
PATH=//sbin://usr/sbin://usr/bin://usr/bin/None://bin:None:$PATH;mkdosfs -n 
efi -i 0x1C1D0579 -S 512 -C 
/tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.vlfgf_0b/boot.img 
524288
DEBUG: ['export', 
'PATH=//sbin://usr/sbin://usr/bin://usr/bin/None://bin:None:$PATH;mkdosfs', 
'-n', 'efi', '-i', '0x1C1D0579', '-S', '512', '-C', 
'/tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.vlfgf_0b/boot.img', 
'524288']
```

*With Extraargs:*

```
$ wic/generic-ufs-partitions.wks.in
part /boot --source bootimg-efi-isar --sourceparams "loader=systemd-boot,*extraargs=-F 
32 -S 4096 -r 2048*" --label efi --part-type EF00 --align 1024 --use-uuid 
--size 512M
part --use-uuid --source rootfs --fstype=${BSP_ROOTFS_TYPE} --label system 
--active --align 1024 --exclude-path boot --fixed-size 
${UFS_PART_SYSTEM_SIZE}
bootloader --ptable gpt --timeout 0 --append "${BSP_KERNEL_CMDLINE}"

$ file development-image-industrial-os-qcs615-adp-air.wic.p1
development-image-industrial-os-qcs615-adp-air.wic.p1: DOS/MBR boot sector, 
code offset 0x58+2, OEM-ID "mkfs.fat", Bytes/sector 4096, Media descriptor 
0xf8, sectors/track 32, heads 8, sectors 131072 (volumes > 32 MB), FAT (32 
bit), sectors/FAT 128, serial number 0xbf8798d0, label: "efi      "
```

*LOG: *
```
DEBUG: Added 469589 extra blocks to /boot to get to 524288 total blocks
DEBUG: ['mkdosfs', '-n', 'efi', '-i', '0xBF8798D0', '-F', '32', '-S', 
'4096', '-r', '2048', '-C', 
'/tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.qd9it81v/boot.img', 
'524288']
DEBUG: exec_native_cmd: export 
PATH=//sbin://usr/sbin://usr/bin://usr/bin/None://bin:None:$PATH;mkdosfs -n 
efi -i 0xBF8798D0 -F 32 -S 4096 -r 2048 -C 
/tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.qd9it81v/boot.img 
524288
DEBUG: _exec_cmd: export 
PATH=//sbin://usr/sbin://usr/bin://usr/bin/None://bin:None:$PATH;mkdosfs -n 
efi -i 0xBF8798D0 -F 32 -S 4096 -r 2048 -C 
/tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.qd9it81v/boot.img 
524288
DEBUG: ['export', 
'PATH=//sbin://usr/sbin://usr/bin://usr/bin/None://bin:None:$PATH;mkdosfs', 
'-n', 'efi', '-i', '0xBF8798D0', '-F', '32', '-S', '4096', '-r', '2048', 
'-C', 
'/tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.qd9it81v/boot.img', 
'524288']
```

On Thursday, December 18, 2025 at 1:52:32 PM UTC+5:30 Richa Bharti wrote:

> Some EFI firmwares require FAT filesystems to use specific geometry or
> sector parameters.
>
> Add support for an `extraargs` source parameter to allow passing
> additional mkdosfs options when creating the EFI System Partition (ESP).
> If no extra arguments are provided, mkdosfs defaults to "-S 512",
> preserving existing behaviour.
>
> Signed-off-by: Richa Bharti <Richa....@siemens.com>
> ---
> meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py 
> b/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
> index 446398d0..6862a288 100644
> --- a/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
> +++ b/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
> @@ -506,8 +506,10 @@ class BootimgEFIPlugin(SourcePlugin):
>
> label = part.label if part.label else "ESP"
>
> - dosfs_cmd = "mkdosfs -n %s -i %s -C %s %d" % \
> - (label, part.fsuuid, bootimg, blocks)
> + extraargs = source_params.get('extraargs', "-S 512")
> +
> + dosfs_cmd = "mkdosfs -n %s -i %s %s -C %s %d" % \
> + (label, part.fsuuid, extraargs, bootimg, blocks)
> exec_native_cmd(dosfs_cmd, native_sysroot)
>
> mcopy_cmd = "mcopy -i %s -s %s/* ::/" % (bootimg, hdddir)
> -- 
> 2.39.5
>
>

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/465454d2-d0e6-45a1-a5f9-891af0c84067n%40googlegroups.com.

[-- Attachment #1.2: Type: text/html, Size: 6332 bytes --]

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

* RE: [PATCH v2] wic: support passing extraargs for EFI partition creation
  2025-12-18  8:22   ` [PATCH v2] " 'Richa Bharti' via isar-users
  2025-12-18  8:29     ` RICHA BHARTI
@ 2025-12-18  8:39     ` 'Bharti, Richa' via isar-users
  2025-12-22  7:24     ` Anton Mikanovich
  2 siblings, 0 replies; 10+ messages in thread
From: 'Bharti, Richa' via isar-users @ 2025-12-18  8:39 UTC (permalink / raw)
  To: isar-users, amikan; +Cc: Hombourger, Cedric, Krishnakar, Srikanth

Without Extraargs:
```
$ cat wic/generic-ufs-partitions.wks.in
part /boot --source bootimg-efi-isar --sourceparams "loader=systemd-boot" --label efi --part-type EF00 --align 1024 --use-uuid --size 512M
part --use-uuid --source rootfs --fstype=${BSP_ROOTFS_TYPE} --label system --active --align 1024 --exclude-path boot --fixed-size ${UFS_PART_SYSTEM_SIZE}
bootloader --ptable gpt --timeout 0 --append "${BSP_KERNEL_CMDLINE}

$ file development-image-industrial-os-qcs615-adp-air.wic.p1
development-image-industrial-os-qcs615-adp-air.wic.p1: DOS/MBR boot sector, code offset 0x58+2, OEM-ID "mkfs.fat", sectors/cluster 8, Media descriptor 0xf8, sectors/track 63, heads 32, sectors 1048572 (volumes > 32 MB), FAT (32 bit), sectors/FAT 1024, serial number 0xe9d3c0d4, label: "efi        "
```
LOG:
```
DEBUG: ['mkdosfs', '-n', 'efi', '-i', '0x1C1D0579', '-S', '512', '-C', '/tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.vlfgf_0b/boot.img', '524288']
DEBUG: exec_native_cmd: export PATH=//sbin://usr/sbin://usr/bin://usr/bin/None://bin:None:$PATH;mkdosfs -n efi -i 0x1C1D0579 -S 512 -C /tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.vlfgf_0b/boot.img 524288
DEBUG: _exec_cmd: export PATH=//sbin://usr/sbin://usr/bin://usr/bin/None://bin:None:$PATH;mkdosfs -n efi -i 0x1C1D0579 -S 512 -C /tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.vlfgf_0b/boot.img 524288
DEBUG: ['export', 'PATH=//sbin://usr/sbin://usr/bin://usr/bin/None://bin:None:$PATH;mkdosfs', '-n', 'efi', '-i', '0x1C1D0579', '-S', '512', '-C', '/tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.vlfgf_0b/boot.img', '524288']
```

With Extraargs:

```
$ wic/generic-ufs-partitions.wks.in
part /boot --source bootimg-efi-isar --sourceparams "loader=systemd-boot,extraargs=-F 32 -S 4096 -r 2048" --label efi --part-type EF00 --align 1024 --use-uuid --size 512M
part --use-uuid --source rootfs --fstype=${BSP_ROOTFS_TYPE} --label system --active --align 1024 --exclude-path boot --fixed-size ${UFS_PART_SYSTEM_SIZE}
bootloader --ptable gpt --timeout 0 --append "${BSP_KERNEL_CMDLINE}"

$ file development-image-industrial-os-qcs615-adp-air.wic.p1
development-image-industrial-os-qcs615-adp-air.wic.p1: DOS/MBR boot sector, code offset 0x58+2, OEM-ID "mkfs.fat", Bytes/sector 4096, Media descriptor 0xf8, sectors/track 32, heads 8, sectors 131072 (volumes > 32 MB), FAT (32 bit), sectors/FAT 128, serial number 0xbf8798d0, label: "efi      "
```
LOG: 
```
DEBUG: Added 469589 extra blocks to /boot to get to 524288 total blocks
DEBUG: ['mkdosfs', '-n', 'efi', '-i', '0xBF8798D0', '-F', '32', '-S', '4096', '-r', '2048', '-C', '/tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.qd9it81v/boot.img', '524288']
DEBUG: exec_native_cmd: export PATH=//sbin://usr/sbin://usr/bin://usr/bin/None://bin:None:$PATH;mkdosfs -n efi -i 0xBF8798D0 -F 32 -S 4096 -r 2048 -C /tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.qd9it81v/boot.img 524288
DEBUG: _exec_cmd: export PATH=//sbin://usr/sbin://usr/bin://usr/bin/None://bin:None:$PATH;mkdosfs -n efi -i 0xBF8798D0 -F 32 -S 4096 -r 2048 -C /tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.qd9it81v/boot.img 524288
DEBUG: ['export', 'PATH=//sbin://usr/sbin://usr/bin://usr/bin/None://bin:None:$PATH;mkdosfs', '-n', 'efi', '-i', '0xBF8798D0', '-F', '32', '-S', '4096', '-r', '2048', '-C', '/tmp/development-image-industrial-os-qcs615-adp-air.wic/tmp.wic.qd9it81v/boot.img', '524288']
```

-----Original Message-----
From: Bharti, Richa (FT FDS CES LX PBU RSOL) <richa.bharti@siemens.com> 
Sent: 18 December 2025 13:52
To: isar-users@googlegroups.com; amikan@ilbers.de
Cc: Hombourger, Cedric (FT FDS CES LX) <cedric.hombourger@siemens.com>; Krishnakar, Srikanth (FT FDS CES LX PBU RSOL) <srikanth.krishnakar@siemens.com>; Bharti, Richa (FT FDS CES LX PBU RSOL) <richa.bharti@siemens.com>
Subject: [PATCH v2] wic: support passing extraargs for EFI partition creation

Some EFI firmwares require FAT filesystems to use specific geometry or sector parameters.

Add support for an `extraargs` source parameter to allow passing additional mkdosfs options when creating the EFI System Partition (ESP).
If no extra arguments are provided, mkdosfs defaults to "-S 512", preserving existing behaviour.

Signed-off-by: Richa Bharti <Richa.Bharti@siemens.com>
---
 meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py b/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
index 446398d0..6862a288 100644
--- a/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
+++ b/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
@@ -506,8 +506,10 @@ class BootimgEFIPlugin(SourcePlugin):
 
         label = part.label if part.label else "ESP"
 
-        dosfs_cmd = "mkdosfs -n %s -i %s -C %s %d" % \
-                    (label, part.fsuuid, bootimg, blocks)
+        extraargs = source_params.get('extraargs', "-S 512")
+
+        dosfs_cmd = "mkdosfs -n %s -i %s %s -C %s %d" % \
+                    (label, part.fsuuid, extraargs, bootimg, blocks)
         exec_native_cmd(dosfs_cmd, native_sysroot)
 
         mcopy_cmd = "mcopy -i %s -s %s/* ::/" % (bootimg, hdddir)
--
2.39.5

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/KUXPR06MB79667D0878307E4403FC16EF84A8A%40KUXPR06MB7966.apcprd06.prod.outlook.com.

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

* Re: [PATCH v2] wic: support passing extraargs for EFI partition creation
  2025-12-18  8:22   ` [PATCH v2] " 'Richa Bharti' via isar-users
  2025-12-18  8:29     ` RICHA BHARTI
  2025-12-18  8:39     ` 'Bharti, Richa' via isar-users
@ 2025-12-22  7:24     ` Anton Mikanovich
  2025-12-23 10:38       ` [PATCH v3] " 'Richa Bharti' via isar-users
  2025-12-23 10:44       ` [PATCH v2] " RICHA BHARTI
  2 siblings, 2 replies; 10+ messages in thread
From: Anton Mikanovich @ 2025-12-22  7:24 UTC (permalink / raw)
  To: Richa Bharti, isar-users
  Cc: cedric.hombourger, srikanth.krishnakar, Baurzhan Ismagulov

18/12/2025 10:22, Richa Bharti wrote:
> Some EFI firmwares require FAT filesystems to use specific geometry or
> sector parameters.
>
> Add support for an `extraargs` source parameter to allow passing
> additional mkdosfs options when creating the EFI System Partition (ESP).
> If no extra arguments are provided, mkdosfs defaults to "-S 512",
> preserving existing behaviour.
>
> Signed-off-by: Richa Bharti <Richa.Bharti@siemens.com>
Hello Richa,

v2 passed CI, but this functionality still not covered by any target.
It makes this feature unmaintainable, we'll never notice it will become 
broken.
Would you like to add some extraargs usage in any wks using fat?

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/c2005266-c883-4bec-8ff6-7c4733638673%40ilbers.de.

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

* [PATCH v3] wic: support passing extraargs for EFI partition creation
  2025-12-22  7:24     ` Anton Mikanovich
@ 2025-12-23 10:38       ` 'Richa Bharti' via isar-users
  2026-01-05  9:55         ` Anton Mikanovich
  2025-12-23 10:44       ` [PATCH v2] " RICHA BHARTI
  1 sibling, 1 reply; 10+ messages in thread
From: 'Richa Bharti' via isar-users @ 2025-12-23 10:38 UTC (permalink / raw)
  To: isar-users, amikan
  Cc: cedric.hombourger, srikanth.krishnakar, ibr, Richa Bharti

Some EFI firmwares require FAT filesystems to use specific geometry or
sector parameters.

Add support for an `extraargs` source parameter to allow passing
additional mkdosfs options when creating the EFI System Partition (ESP).
If no extra arguments are provided, mkdosfs defaults to "-S 512",
preserving existing behaviour.

Signed-off-by: Richa Bharti <Richa.Bharti@siemens.com>
---
 meta-isar/scripts/lib/wic/canned-wks/sdimage-efi-btrfs.wks | 2 +-
 meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py    | 6 ++++--
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/meta-isar/scripts/lib/wic/canned-wks/sdimage-efi-btrfs.wks b/meta-isar/scripts/lib/wic/canned-wks/sdimage-efi-btrfs.wks
index e22dba2f..219c5d06 100644
--- a/meta-isar/scripts/lib/wic/canned-wks/sdimage-efi-btrfs.wks
+++ b/meta-isar/scripts/lib/wic/canned-wks/sdimage-efi-btrfs.wks
@@ -2,7 +2,7 @@
 # long-description: Creates a partitioned EFI disk image without any swap that
 # the user can directly dd to boot media, where the last partition is btrfs.
 
-part /boot --source bootimg-efi-isar --sourceparams "loader=systemd-boot" --ondisk sda --label efi --part-type EF00 --align 1024
+part /boot --source bootimg-efi-isar --sourceparams "loader=systemd-boot,extraargs=-s 8" --ondisk sda --label efi --part-type EF00 --align 1024
 
 part / --source rootfs --ondisk sda --fstype btrfs --label platform --align 1024 --use-uuid --exclude-path boot/
 
diff --git a/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py b/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
index fd4d6017..b0edb8f5 100644
--- a/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
+++ b/meta/scripts/lib/wic/plugins/source/bootimg-efi-isar.py
@@ -537,8 +537,10 @@ class BootimgEFIPlugin(SourcePlugin):
 
         label = part.label if part.label else "ESP"
 
-        dosfs_cmd = "mkdosfs -n %s -i %s -C %s %d" % \
-                    (label, part.fsuuid, bootimg, blocks)
+        extraargs = source_params.get('extraargs', "-S 512")
+
+        dosfs_cmd = "mkdosfs -n %s -i %s %s -C %s %d" % \
+                    (label, part.fsuuid, extraargs, bootimg, blocks)
         exec_native_cmd(dosfs_cmd, native_sysroot)
 
         mcopy_cmd = "mcopy -i %s -s %s/* ::/" % (bootimg, hdddir)
-- 
2.30.2

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/20251223103814.16129-1-Richa.Bharti%40siemens.com.

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

* Re: [PATCH v2] wic: support passing extraargs for EFI partition creation
  2025-12-22  7:24     ` Anton Mikanovich
  2025-12-23 10:38       ` [PATCH v3] " 'Richa Bharti' via isar-users
@ 2025-12-23 10:44       ` RICHA BHARTI
  2025-12-23 15:22         ` Anton Mikanovich
  1 sibling, 1 reply; 10+ messages in thread
From: RICHA BHARTI @ 2025-12-23 10:44 UTC (permalink / raw)
  To: isar-users


[-- Attachment #1.1: Type: text/plain, Size: 1407 bytes --]

Added V3 for patch that will validate "extraargs" functionality with target 
"mc:qemuamd64-bullseye:isar-image-ci". I hope with this the feature will be 
trackable and any issue will be visible. Let me know if change in target is 
required.



On Monday, December 22, 2025 at 12:54:30 PM UTC+5:30 Anton Mikanovich wrote:

> 18/12/2025 10:22, Richa Bharti wrote:
> > Some EFI firmwares require FAT filesystems to use specific geometry or
> > sector parameters.
> >
> > Add support for an `extraargs` source parameter to allow passing
> > additional mkdosfs options when creating the EFI System Partition (ESP).
> > If no extra arguments are provided, mkdosfs defaults to "-S 512",
> > preserving existing behaviour.
> >
> > Signed-off-by: Richa Bharti <Richa....@siemens.com>
> Hello Richa,
>
> v2 passed CI, but this functionality still not covered by any target.
> It makes this feature unmaintainable, we'll never notice it will become 
> broken.
> Would you like to add some extraargs usage in any wks using fat?
>

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/b78709db-8d9c-4211-92c2-93bfc65abc02n%40googlegroups.com.

[-- Attachment #1.2: Type: text/html, Size: 2007 bytes --]

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

* Re: [PATCH v2] wic: support passing extraargs for EFI partition creation
  2025-12-23 10:44       ` [PATCH v2] " RICHA BHARTI
@ 2025-12-23 15:22         ` Anton Mikanovich
  0 siblings, 0 replies; 10+ messages in thread
From: Anton Mikanovich @ 2025-12-23 15:22 UTC (permalink / raw)
  To: isar-users, Richa Bharti

23/12/2025 12:44, RICHA BHARTI wrote:
> Added V3 for patch that will validate "extraargs" functionality with 
> target "mc:qemuamd64-bullseye:isar-image-ci". I hope with this the 
> feature will be trackable and any issue will be visible. Let me know 
> if change in target is required.
Hello Richa,

Thanks, v3 looks good enough to be used.
It is already passed fast CI, full one is ongoing.

P.S. I will be on holidays from 24.12.25 to 04.01.26, so only urgent 
fixes will
probably be merged to next on this period.

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/8d3f691e-287b-4fed-9664-023f805b445c%40ilbers.de.

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

* Re: [PATCH v3] wic: support passing extraargs for EFI partition creation
  2025-12-23 10:38       ` [PATCH v3] " 'Richa Bharti' via isar-users
@ 2026-01-05  9:55         ` Anton Mikanovich
  0 siblings, 0 replies; 10+ messages in thread
From: Anton Mikanovich @ 2026-01-05  9:55 UTC (permalink / raw)
  To: Richa Bharti, isar-users; +Cc: cedric.hombourger, srikanth.krishnakar, ibr

Applied to next, thanks.

23/12/2025 12:38, Richa Bharti wrote:
> Some EFI firmwares require FAT filesystems to use specific geometry or
> sector parameters.
>
> Add support for an `extraargs` source parameter to allow passing
> additional mkdosfs options when creating the EFI System Partition (ESP).
> If no extra arguments are provided, mkdosfs defaults to "-S 512",
> preserving existing behaviour.
>
> Signed-off-by: Richa Bharti <Richa.Bharti@siemens.com>

-- 
You received this message because you are subscribed to the Google Groups "isar-users" group.
To unsubscribe from this group and stop receiving emails from it, send an email to isar-users+unsubscribe@googlegroups.com.
To view this discussion visit https://groups.google.com/d/msgid/isar-users/14fddaaf-cb9e-48cd-9212-a2ae2810b7ce%40ilbers.de.

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

end of thread, other threads:[~2026-01-05  9:55 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-12-17 13:02 [PATCH] wic: support passing extraargs for EFI partition creation 'Richa Bharti' via isar-users
2025-12-18  7:28 ` Anton Mikanovich
2025-12-18  8:22   ` [PATCH v2] " 'Richa Bharti' via isar-users
2025-12-18  8:29     ` RICHA BHARTI
2025-12-18  8:39     ` 'Bharti, Richa' via isar-users
2025-12-22  7:24     ` Anton Mikanovich
2025-12-23 10:38       ` [PATCH v3] " 'Richa Bharti' via isar-users
2026-01-05  9:55         ` Anton Mikanovich
2025-12-23 10:44       ` [PATCH v2] " RICHA BHARTI
2025-12-23 15:22         ` Anton Mikanovich

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox