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.

Sign up for GitHub

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

Jump to bottom

add emergency fallback test, currently fails due to #463

Open
wants to merge 50 commits into
base: master
Choose a base branch
from
Open

add emergency fallback test, currently fails due to #463

wants to merge 50 commits into from

Conversation

jpbland1
Copy link
Contributor

the sector flags needing to be reset by update trigger

danielinux
danielinux requested changes Jun 30, 2024
View reviewed changes
Copy link
Member

@danielinux danielinux left a comment

Choose a reason for hiding this comment

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

let's modify hal_flash_write as discussed on slack, to ensure we simulate the same behavior as an actual FLASH memory (& operation on existing content). That will make the test fail upon invalid state changes

hal/sim.c Show resolved Hide resolved
@jpbland1 jpbland1 requested a review from danielinux July 1, 2024 04:56
@dgarske dgarske removed their assignment Jul 1, 2024
@jpbland1
add emergency fallback test, currently fails due to
62b1a84 
the sector flags needing to be reset by update trigger
@jpbland1
fix bad return args
1fe0f43
@jpbland1
fix emergency fallback bug by zeroing sector flags
edff5f9 
with update trigger
@jpbland1
update footprint check
54180b3
@jpbland1
update more size checks
871141c
@jpbland1
set emergency trigger to first sector of boot to work
d4f5c6f 
with flags home
@jpbland1
add missing clean calls
bb9f3b6
@jpbland1
remove emergency fallback tests for DELTA updates
f6a096d 
since an emergency failure that corrupts the backup will always result
DELTA fallback panic
@jpbland1
update simulator hal to emulate bit progressive writes
d77ef2a
@jpbland1
fix bit reversion from FLAGS_FINAL to UPDATING bug
014633b
@jpbland1
update size check
7951fe4
@jpbland1
update size check again
64fae73
@jpbland1
update size check yet again
f5fe4a7
@jpbland1
update size check :D
73019ff
@jpbland1
add another sector to partition size of stm32c0
6028961
@jpbland1
increase stm32c0 bootloader size
379a5c0
@jpbland1
simplify state machine with re-entrant final erase
7b11730
@jpbland1
fix unopened boot image issue for final erase
dd50f1c
@jpbland1
fix interrupted delta update direction detector
d0fcb88
@jpbland1
fix bad cast
f3f0f67
@jpbland1
update powerfail fallback to test double fallback
64d5d38
@jpbland1
remove testing variables
b152bd1
@jpbland1
refactor wolfBoot_swap_and_final_erase to use the
4f8a7ff 
second to last available sector of boot for the special flags and encryption key
storage since you can't store the encryption information in swap
@jpbland1
trim wolfBoot_swap_and_final_erase
2161190
@jpbland1
fix bad pointer offset
c0c8273
@jpbland1
fix bad pointer offset
556bfa0
@jpbland1
attempt to trim code
54788bd
@jpbland1
trim test 2
89bfc70
@jpbland1
trim attempt 3
f41897f
@jpbland1
trim attempt 4
f7fa7c8
@jpbland1
remove flag erasure on wolfBoot_update_trigger since
745240a 
wolfBoot_swap_and_final_erase now erases the last sector, making it redundant
@jpbland1
undo previous commit
2aa2311
@jpbland1
trim attempt 5
df53664
@jpbland1
revert bad trimming attempts
12e15f2
@jpbland1
trim attempt 6
e424e4f
@jpbland1
revert another bad trim attempt
1387e69
@jpbland1
trim attempt 7
daa75d7
@jpbland1
trim attempt 8
6650f9b
@jpbland1
fix swapped parts
8a8e978
@jpbland1
trim attempt 9
7d1b99f
@jpbland1
trim attempt 10
97837e0
@jpbland1
trim attempt 11
9b166b0
@jpbland1
remove bad redefine
f1dbac1
@jpbland1
trim attempt 12
cc78d31
@jpbland1
update size test
77fddf3
@jpbland1
re-adjust size test
903d654
@jpbland1
fix magic not set issue when checking direction
8c961ce 
of delta update
@jpbland1
fix misplaced test from rebase
61b1bed
@jpbland1
simplify the delta update direction check
6fb1230
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@danielinux danielinux danielinux requested changes

@dgarske dgarske Awaiting requested review from dgarske

Requested changes must be addressed to merge this pull request.

Assignees

@danielinux danielinux

@jpbland1 jpbland1

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jpbland1 @danielinux @dgarske