mount command outputs the rootfs partition is in ro mode but yet modifies the checksum. I have no clue where I should look to investigate this issue further
Oki, actually when looking more closely, the script does the following:
mount /dev/mmcblk0p${new_boot_part} ${new_mender_boot_part_mountpoint}
Meaning it mounts the inactive partition that you just wrote. As it mounts it rw, it probably changes the checksum. Try changing this to:
mount -o ro /dev/mmcblk0p${new_boot_part} ${new_mender_boot_part_mountpoint}
Then it re-mounts the active partition to rw,
mount -oremount,rw /
This should actually not be a problem, as the delta update has already been performed by the time ArtifactReboot_Enter runs. But this can become a problem if the update is interrupted and tried again, then the checksum will be wrong (future problem).
Then it does:
cp ${new_mender_boot_part_mountpoint}/opt/ota_package/bl_update_payload_current /opt/ota_package/bl_update_payload_new
This again modifies the active partition, which should not affect the update in progress, but again can become a problem if the update is interrupted and tried again, then the checksum will be wrong (future problem).
@mirzak, @anishmonachan I know there was some discussion about support for delta updates on tegra, see https://github.com/OE4T/meta-mender-community/pull/1 and https://github.com/mendersoftware/meta-mender-community/pull/165 for instance. I haven’t tried delta updates myself yet, but I expect Neils has on the warrior branch with the patch in https://github.com/mendersoftware/meta-mender-community/pull/165. It would be great if someone could help us test/fix this on the master branch of https://github.com/OE4T/meta-mender-community and if we could have a plan for testing this going forward.
@dwalkes it looks like this patch is not solving the issue. I still have the failure message due to the same cause.
@anishmonachan which yocto branch are you using?
The code you shared above doesn’t appear to include the patch at https://github.com/mendersoftware/meta-mender-community/pull/165/files#diff-7b4fc058c2071a2124c7d6c5b271a66dL15 but are you saying you have switched to using the warrior branch with this patch included and still have the same problem?
Since I’m guessing nobody has tested on any configuration other than warrior and the specific configuration where the patch at https://github.com/mendersoftware/meta-mender-community/pull/165 was applied, assuming that doesn’t work for you, I’d suggest:
- Testing on dunfell using the latest source and https://github.com/OE4T/tegra-demo-distro
- If you still see the issue, open an issue to track at https://github.com/OE4T/meta-mender-community/issues and, if possible, share your customizations for https://github.com/OE4T/tegra-demo-distro so it’s easier for others to see your exact setup.
@dwalkes I’ve been using warrior branch. I didn’t have to switch branches. I have used this patch in the warrior branch itself.
Can you confirm that you are saying that the patch is tested on the warrior branch?
@anishmonachan
I haven’t tested it myself, however you can see the comments from Niels at https://github.com/mendersoftware/meta-mender-community/pull/165 regarding the testing he did on warrior.
As mentioned, it doesn’t look like you’ve got this patch based on the code you shared above, so I think this is an area to investigate. What do you see in redundant-boot-reboot-state-script in your sources and does it match what was added in the patch? It should be be used for the state script source based on the build step here: https://github.com/mendersoftware/meta-mender-community/blob/warrior/meta-mender-tegra/recipes-mender/tegra-state-scripts/tegra-state-scripts_1.0.bb#L13
Also FYI I’ve put this topic on our meeting list for this coming Friday at https://teamup.com/event/show/id/JVhkcgPMnZ2u9UawP76n4jEiFLc1rX to see if others are using/interested in delta updates please feel free to join if you like.
@dwalkes Thanks for your invitation. Unfortunately, I couldn’t partocipate in the meeting.
@anishmonachan no worries. We actually didn’t get a chance to cover it, nobody who was on the call is using delta updates today. I know from the PR comments delta updates have been used successfully in the past but I think we need someone to volunteer to help test and validate it with the differences required for NVIDIA redundant boot support.
@dwalkes I tried the patch and verified that the patch was applied. And yet I am stuck with the same error while updating delta. I am ready to volunteer to help with testing and validation. Just tell me how it should be done.
@anishmonachan I suggest:
- Testing on dunfell using the latest source and https://github.com/OE4T/tegra-demo-distro
- If you still see the issue, open an issue to track at https://github.com/OE4T/meta-mender-community/issues and, if possible, share your customizations for https://github.com/OE4T/tegra-demo-distro so it’s easier for others to see your exact setup.
@dwalkes I’ll get as soon as I have some results to share