Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Tegra, Jetpack 5: add Jetson as working target #394

Conversation

TheYoctoJester
Copy link
Contributor

This PR is a rebased replica of OE4T#19.

It carries the last state as of OE4T#19 (comment) comment, with the following additions:

Marius Westgaard and others added 18 commits July 27, 2024 21:37
Signed-off-by: Marius Westgaard <marius.westgaard@1x.tech>
Signed-off-by: Marius Westgaard <marius.westgaard@1x.tech>
Signed-off-by: Marius Westgaard <marius.westgaard@1x.tech>
…A partition will be flashed with the appropriate image

Signed-off-by: Iban Rodriguez <irodriguez@das-gate.com>
Signed-off-by: Iban Rodriguez <irodriguez@das-gate.com>
Signed-off-by: Marius Westgaard <marius.westgaard@1x.tech>
Signed-off-by: Marius Westgaard <marius.westgaard@1x.tech>
Signed-off-by: Iban Rodriguez <irodriguez@das-gate.com>
Signed-off-by: Carlo Kretzschmann <kretzschmann@3dvisionlabs.com>
When a rollback was started before rebooting the update capsule was still installed
on the next reboot and the boot chain was switched. We want to stay on the current chain
after a failed install though.

Signed-off-by: Carlo Kretzschmann <kretzschmann@3dvisionlabs.com>
Signed-off-by: Iban Rodriguez <irodriguez@das-gate.com>
Signed-off-by: Marius Westgaard <marius.westgaard@1x.tech>
Signed-off-by: Marius Westgaard <marius.westgaard@1x.tech>
Signed-off-by: Marius Westgaard <marius.westgaard@1x.tech>
…n tegra194 is target

Signed-off-by: Marius Westgaard <marius.westgaard@1x.tech>
Signed-off-by: Iban Rodriguez <irodriguez@veridas.com>
Signed-off-by: Iban Rodriguez <irodriguez@veridas.com>
…vier

Signed-off-by: Iban Rodriguez <irodriguez@veridas.com>
cakre and others added 9 commits July 29, 2024 10:51
Signed-off-by: Carlo Kretzschmann <kretzschmann@3dvisionlabs.com>
Signed-off-by: Carlo Kretzschmann <kretzschmann@3dvisionlabs.com>
Signed-off-by: Marius Westgaard <marius.westgaard@1x.tech>
Signed-off-by: Marius Westgaard <marius.westgaard@1x.tech>
Signed-off-by: Marius Westgaard <marius.westgaard@1x.tech>
Signed-off-by: Carlo Kretzschmann <kretzschmann@3dvisionlabs.com>
(cherry picked from commit 1e73876)
Signed-off-by: Marius Westgaard <marius.westgaard@1x.tech>
(cherry picked from commit 4d49299)
The mentioned known good boards in the README file do not have
corresponding links on the Mender Hub. To avoid confusion, remove
the sentence suggesting to visit those non-existing links.

Signed-off-by: Josef Holzmayr <jester@theyoctojester.info>
Signed-off-by: Josef Holzmayr <jester@theyoctojester.info>
@TheYoctoJester TheYoctoJester force-pushed the mwest90-working-jetson-orin branch from 5c78da8 to 34b9448 Compare July 29, 2024 08:52
@TheYoctoJester
Copy link
Contributor Author

@maxekman I have removed the commit fa468f5 which is missing the Signed-off-by: in this branch. Please let me know if you'd like to re-submit it or if I should just replicate the override removal and set you as Reported-by:?

Copy link

@mwest90 mwest90 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@TheYoctoJester looks good to me!

@maxekman
Copy link

Thanks @TheYoctoJester, yes please set me as reported by on fa468f5. I’m on vacation right now.

@TheYoctoJester
Copy link
Contributor Author

Thanks @TheYoctoJester, yes please set me as reported by on fa468f5. I’m on vacation right now.

Works for me, thanks!

Reported-by: Max Ekman <max.ekman@einride.tech>
Signed-off-by: Josef Holzmayr <jester@theyoctojester.info>
@TheYoctoJester TheYoctoJester merged commit 030df41 into mendersoftware:kirkstone Jul 30, 2024
1 check passed
@TheYoctoJester TheYoctoJester deleted the mwest90-working-jetson-orin branch October 23, 2024 14:42

FILESEXTRAPATHS:prepend := "${THISDIR}/files:"

SRC_URI += "file://0007-runtime-access-for-KernelCommandLine-efivar.patch;patchdir=.."
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for being late to the party. I hope, you don't mind me posting my question here.

I figured that mender uses the KernelCommandLine variable for passing the machine ID to the "new" RootFS if the feature "mender-persist-systemd-machine-id" is active.
Is this feature supposed to work despite removing this patch?

Upstream, the variable is still not available to runtime:
https://github.com/NVIDIA/edk2-nvidia/blob/r36.4.0/Silicon/NVIDIA/Drivers/NvidiaConfigDxe/NvidiaConfigHii.vfr#L63

Without runtime access to this variable, the script efi_systemd_machine_id.sh seems to fail persisting the machine ID with the message

efivar: Read-only file system

(I tested the script on a Jetpack 6.1 system, but I don't see a relevant difference to JP 5 code)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants