From mboxrd@z Thu Jan 1 00:00:00 1970 X-GM-THRID: 6880180325712920576 Date: Wed, 14 Oct 2020 09:48:30 -0700 (PDT) From: "vijaikumar....@gmail.com" To: isar-users Message-Id: In-Reply-To: <1e2d3ac5-b976-1075-8fc9-aae9a2beccde@siemens.com> References: <20201014132729.18571-1-Vijaikumar_Kanagarajan@mentor.com> <1e2d3ac5-b976-1075-8fc9-aae9a2beccde@siemens.com> Subject: Re: [PATCH v6 0/8] WIC update MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="----=_Part_18_203121021.1602694110729" X-TUID: +/mRfNIWtFjD ------=_Part_18_203121021.1602694110729 Content-Type: multipart/alternative; boundary="----=_Part_19_219684950.1602694110729" ------=_Part_19_219684950.1602694110729 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit On Wednesday, October 14, 2020 at 8:58:24 PM UTC+5:30 Jan Kiszka wrote: > On 14.10.20 15:27, Vijai Kumar K wrote: > > Changes since v5: > > - There was an issue in rebase of v4 in P3. Fixed the conflict properly > now. > > > > Changes since v4: > > - Rebase against latest next > > > > Changes since v3: > > > > - Rebase against the latest next. Drop P4 & P5 from previous series. > > - P7 fixes the kernel update in legacy bios boot image > > - Minor documentation changes. > > > > Changes since v2: > > > > - Dropped P9 from previous series and updated oe-core to the latest > revision > > which includes that patch. > > - Some minor commit message & API changelog changes. > > > > Changes since RFC(v1): > > > > - P9 commit is upstreamed and replaced with the version from OE-core. > > - P10: Lot of downstream projects tend to use /boot mountpoints for > bootloader/EFI > > partitions. Added RECIPE-API-CHANGELOG entry to warn user about > potential issues > > that might arise when doing so with latest wic. > > > > > > > > Vijai Kumar K (8): > > wic: Update to the latest wic from openembedded core > > wic/plugins: Fix wic plugins to work with the latest wic > > wic-img: Satisfy the quirks of latest wic > > wic_fakeroot: Handle standalone pseudo invocations > > meta-isar/conf: Add provision to debug WIC > > debian-common: Add tar as a dependency for wic > > Fix legacy bios boot partition > > meta-isar/canned-wks: Remove unwanted /boot mountpoint > > > > RECIPE-API-CHANGELOG.md | 26 + > > meta-isar/conf/local.conf.sample | 3 + > > .../lib/wic/canned-wks/common-isar.wks.inc | 2 +- > > .../scripts/lib/wic/canned-wks/hikey.wks | 2 +- > > .../lib/wic/canned-wks/sdimage-efi.wks | 2 +- > > meta/classes/wic-img.bbclass | 18 +- > > meta/conf/distro/debian-common.conf | 3 +- > > .../wic/plugins/source/bootimg-efi-isar.py | 2 +- > > .../wic/plugins/source/bootimg-pcbios-isar.py | 40 +- > > .../lib/wic/plugins/source/rootfs-u-boot.py | 2 +- > > scripts/lib/scriptpath.py | 32 ++ > > scripts/lib/wic/__init__.py | 14 +- > > scripts/lib/wic/canned-wks/common.wks.inc | 2 +- > > .../directdisk-bootloader-config.cfg | 8 +- > > .../lib/wic/canned-wks/efi-bootdisk.wks.in | 3 + > > scripts/lib/wic/canned-wks/mkhybridiso.wks | 2 +- > > scripts/lib/wic/canned-wks/qemuriscv.wks | 3 + > > .../lib/wic/canned-wks/qemux86-directdisk.wks | 2 +- > > .../lib/wic/canned-wks/sdimage-bootpart.wks | 4 +- > > .../lib/wic/canned-wks/systemd-bootdisk.wks | 4 +- > > scripts/lib/wic/engine.py | 421 +++++++++++++++- > > scripts/lib/wic/filemap.py | 170 ++++--- > > scripts/lib/wic/help.py | 401 ++++++++++++++-- > > scripts/lib/wic/ksparser.py | 121 +++-- > > scripts/lib/wic/{utils => }/misc.py | 100 ++-- > > scripts/lib/wic/partition.py | 234 ++++----- > > scripts/lib/wic/pluginbase.py | 36 +- > > scripts/lib/wic/plugins/imager/direct.py | 175 ++++--- > > .../wic/plugins/source/bootimg-biosplusefi.py | 213 +++++++++ > > scripts/lib/wic/plugins/source/bootimg-efi.py | 171 +++++-- > > .../wic/plugins/source/bootimg-partition.py | 153 ++++-- > > .../lib/wic/plugins/source/bootimg-pcbios.py | 91 ++-- > > scripts/lib/wic/plugins/source/fsimage.py | 56 --- > > .../wic/plugins/source/isoimage-isohybrid.py | 185 +++---- > > scripts/lib/wic/plugins/source/rawcopy.py | 44 +- > > scripts/lib/wic/plugins/source/rootfs.py | 159 ++++-- > > scripts/lib/wic/utils/__init__.py | 0 > > scripts/lib/wic/utils/runner.py | 114 ----- > > scripts/wic | 452 +++++++++++++----- > > scripts/wic_fakeroot | 5 + > > 40 files changed, 2433 insertions(+), 1042 deletions(-) > > create mode 100644 scripts/lib/scriptpath.py > > create mode 100644 scripts/lib/wic/canned-wks/efi-bootdisk.wks.in > > create mode 100644 scripts/lib/wic/canned-wks/qemuriscv.wks > > rename scripts/lib/wic/{utils => }/misc.py (70%) > > create mode 100644 scripts/lib/wic/plugins/source/bootimg-biosplusefi.py > > delete mode 100644 scripts/lib/wic/plugins/source/fsimage.py > > delete mode 100644 scripts/lib/wic/utils/__init__.py > > delete mode 100644 scripts/lib/wic/utils/runner.py > > > > Issue still present here - did you check against the STM32 case? > STM32 is a different issue. From the log I found this issue too. I am going through wic codebase and debugging the STM32 case. I got the yocto setup and things work fine there. Using that as a reference. Thanks, Vijai Kumar K > > Jan > > -- > Siemens AG, T RDA IOT > Corporate Competence Center Embedded Linux > ------=_Part_19_219684950.1602694110729 Content-Type: text/html; charset=utf-8 Content-Transfer-Encoding: 7bit

On Wednesday, October 14, 2020 at 8:58:24 PM UTC+5:30 Jan Kiszka wrote:
On 14.10.20 15:27, Vijai Kumar K wrote:
> Changes since v5:
> - There was an issue in rebase of v4 in P3. Fixed the conflict properly now.
>
> Changes since v4:
> - Rebase against latest next
>
> Changes since v3:
>
> - Rebase against the latest next. Drop P4 & P5 from previous series.
> - P7 fixes the kernel update in legacy bios boot image
> - Minor documentation changes.
>
> Changes since v2:
>
> - Dropped P9 from previous series and updated oe-core to the latest revision
> which includes that patch.
> - Some minor commit message & API changelog changes.
>
> Changes since RFC(v1):
>
> - P9 commit is upstreamed and replaced with the version from OE-core.
> - P10: Lot of downstream projects tend to use /boot mountpoints for bootloader/EFI
> partitions. Added RECIPE-API-CHANGELOG entry to warn user about potential issues
> that might arise when doing so with latest wic.
>
>
>
> Vijai Kumar K (8):
> wic: Update to the latest wic from openembedded core
> wic/plugins: Fix wic plugins to work with the latest wic
> wic-img: Satisfy the quirks of latest wic
> wic_fakeroot: Handle standalone pseudo invocations
> meta-isar/conf: Add provision to debug WIC
> debian-common: Add tar as a dependency for wic
> Fix legacy bios boot partition
> meta-isar/canned-wks: Remove unwanted /boot mountpoint
>
> RECIPE-API-CHANGELOG.md | 26 +
> meta-isar/conf/local.conf.sample | 3 +
> .../lib/wic/canned-wks/common-isar.wks.inc | 2 +-
> .../scripts/lib/wic/canned-wks/hikey.wks | 2 +-
> .../lib/wic/canned-wks/sdimage-efi.wks | 2 +-
> meta/classes/wic-img.bbclass | 18 +-
> meta/conf/distro/debian-common.conf | 3 +-
> .../wic/plugins/source/bootimg-efi-isar.py | 2 +-
> .../wic/plugins/source/bootimg-pcbios-isar.py | 40 +-
> .../lib/wic/plugins/source/rootfs-u-boot.py | 2 +-
> scripts/lib/scriptpath.py | 32 ++
> scripts/lib/wic/__init__.py | 14 +-
> scripts/lib/wic/canned-wks/common.wks.inc | 2 +-
> .../directdisk-bootloader-config.cfg | 8 +-
> .../lib/wic/canned-wks/efi-bootdisk.wks.in | 3 +
> scripts/lib/wic/canned-wks/mkhybridiso.wks | 2 +-
> scripts/lib/wic/canned-wks/qemuriscv.wks | 3 +
> .../lib/wic/canned-wks/qemux86-directdisk.wks | 2 +-
> .../lib/wic/canned-wks/sdimage-bootpart.wks | 4 +-
> .../lib/wic/canned-wks/systemd-bootdisk.wks | 4 +-
> scripts/lib/wic/engine.py | 421 +++++++++++++++-
> scripts/lib/wic/filemap.py | 170 ++++---
> scripts/lib/wic/help.py | 401 ++++++++++++++--
> scripts/lib/wic/ksparser.py | 121 +++--
> scripts/lib/wic/{utils => }/misc.py | 100 ++--
> scripts/lib/wic/partition.py | 234 ++++-----
> scripts/lib/wic/pluginbase.py | 36 +-
> scripts/lib/wic/plugins/imager/direct.py | 175 ++++---
> .../wic/plugins/source/bootimg-biosplusefi.py | 213 +++++++++
> scripts/lib/wic/plugins/source/bootimg-efi.py | 171 +++++--
> .../wic/plugins/source/bootimg-partition.py | 153 ++++--
> .../lib/wic/plugins/source/bootimg-pcbios.py | 91 ++--
> scripts/lib/wic/plugins/source/fsimage.py | 56 ---
> .../wic/plugins/source/isoimage-isohybrid.py | 185 +++----
> scripts/lib/wic/plugins/source/rawcopy.py | 44 +-
> scripts/lib/wic/plugins/source/rootfs.py | 159 ++++--
> scripts/lib/wic/utils/__init__.py | 0
> scripts/lib/wic/utils/runner.py | 114 -----
> scripts/wic | 452 +++++++++++++-----
> scripts/wic_fakeroot | 5 +
> 40 files changed, 2433 insertions(+), 1042 deletions(-)
> create mode 100644 scripts/lib/scriptpath.py
> create mode 100644 scripts/lib/wic/canned-wks/efi-bootdisk.wks.in
> create mode 100644 scripts/lib/wic/canned-wks/qemuriscv.wks
> rename scripts/lib/wic/{utils => }/misc.py (70%)
> create mode 100644 scripts/lib/wic/plugins/source/bootimg-biosplusefi.py
> delete mode 100644 scripts/lib/wic/plugins/source/fsimage.py
> delete mode 100644 scripts/lib/wic/utils/__init__.py
> delete mode 100644 scripts/lib/wic/utils/runner.py
>

Issue still present here - did you check against the STM32 case?

STM32 is a different issue. From the log I found this issue too. I am going through wic codebase and debugging the STM32 case. I got the yocto setup and things work
fine there. Using that as a reference.

Thanks,
Vijai Kumar K
 

Jan

--
Siemens AG, T RDA IOT
Corporate Competence Center Embedded Linux
------=_Part_19_219684950.1602694110729-- ------=_Part_18_203121021.1602694110729--