Communication tests and fixes - [merged]

[eggroll-bot]

In GitLab by @MelonShooter on Feb 20, 2023, 15:28

Merges communication-fixes-and-tests -> main

This merge requests adds communications tests for Runtime's publicly facing API with Uart0Controller and Uart1Controller. These tests will likely involve manual intervention, maybe through a script to send and receive through UART0/1.

[eggroll-bot]

In GitLab by @MelonShooter on Feb 20, 2023, 15:28

requested review from @MelonShooter

[eggroll-bot]

In GitLab by @MelonShooter on Feb 20, 2023, 15:40

added 2 commits

• 77812590 - Fix clippy warnings
• e3b14151 - Fix hex encode panic, closes #4

Compare with previous version

[eggroll-bot]

In GitLab by @group_21660_bot2 on Feb 21, 2023, 24:00

added 1 commit

• b83be08f - Add basic UART 0 test

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 21, 2023, 24:38

added 1 commit

• 1fd9fdf6 - Change baud rate to match that of the serial to socket bridge

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 21, 2023, 14:10

added 1 commit

• 784f4064 - Change UART0 controller to have pins in type

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 22, 2023, 15:54

added 1 commit

• 87565f99 - Add comm_tests workspace with two board crates

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 22, 2023, 17:09

added 1 commit

• 7593b39e - Add skeleton for multi-UART testing

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 22, 2023, 17:38

added 1 commit

• b3c971ff - Add basic test

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 22, 2023, 17:46

added 1 commit

• e4d16c67 - Add receive error tests

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 22, 2023, 18:06

added 1 commit

• e2d5340c - Add large text test

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 22, 2023, 18:38

added 1 commit

• ef1fc15c - Add binary tests

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 22, 2023, 18:48

added 1 commit

• 3131ad44 - Change tests to be UART1 only for now

Compare with previous version

[eggroll-bot]

In GitLab by @group_21660_bot2 on Feb 22, 2023, 18:54

added 1 commit

• 69b9fd47 - Rename test crate to be more descriptive

Compare with previous version

[eggroll-bot]

In GitLab by @group_21660_bot2 on Feb 22, 2023, 21:10

added 1 commit

• 39d05e99 - Increase flash memory size for comm tests

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 22, 2023, 21:58

added 1 commit

• c230e48e - Add timeout on first real recv test

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 22, 2023, 21:58

added 1 commit

• 6ed38791 - Add timeout on first real recv test

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 22, 2023, 23:39

added 1 commit

• 3d43be33 - Add check to prevent sending 1 byte message, clarifying docs, and add pub const for metadata size

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 22, 2023, 23:58

added 1 commit

• dbcd7e37 - Fix crate names and add .gdbinit's

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 23, 2023, 24:09

added 1 commit

• f5f3b4c5 - Move .gdbinit and add optization flags

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 23, 2023, 01:15

added 1 commit

• 909ba0da - Make large_test.txt smaller

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 23, 2023, 01:27

added 1 commit

• 6ef2e995 - Remove large text test

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 23, 2023, 01:39

added 1 commit

• 0e089a7c - Add finish message

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 23, 2023, 02:48

added 1 commit

• ff7f1e4a - Increase send size of large binary test

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 23, 2023, 15:17

added 1 commit

• c385ee61 - Add crypto tests

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 23, 2023, 18:07

added 1 commit

• 58d8030f - Make crypto tests use lower level Rx

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 23, 2023, 20:07

added 1 commit

• 7bef43a9 - Remove large crypto test b/c of stack overflow

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 23, 2023, 23:50

added 1 commit

• 75a1bd75 - Fix crypto tests

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 23, 2023, 23:52

added 1 commit

• 1782adfc - Update clarifying doc

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 24, 2023, 24:46

As of right now on plantmachine, this is what happens when running these tests

~~stable toolchain: panics due to supposed reetrant init of once cell in SECONDARY_CSPRNG on plantmachine nightly toolchain: stack corruption, presumably in init_rng~~

However, when I run these locally on boards I have here, it kind of works on both stable and nightly.

The above is solved and is as a result of a bad stack pointer because of the flash offset being set to 0x8000 without a bootloader.

In the current state of tests, the receive tests sometimes fail for unknown reasons though it is likely timing related due to a shallow FIFO queue and probably not a cause for concern.

[eggroll-bot]

In GitLab by @MelonShooter on Feb 24, 2023, 18:56

added 37 commits

• 1782adfc...6fba3cb6 - 9 commits from branch main
• 042feeca - Write comm test skeleton
• 66c6b4e5 - Fix clippy warnings
• 13da6bd7 - Fix hex encode panic, closes #4
• 65e929fa - Add basic UART 0 test
• 8177e44d - Change baud rate to match that of the serial to socket bridge
• 34f5185b - Change UART0 controller to have pins in type
• 0641ff80 - Add comm_tests workspace with two board crates
• 82f2a31f - Add skeleton for multi-UART testing
• 51898040 - Add basic test
• 8d0d3e14 - Add receive error tests
• 28f18602 - Add large text test
• 04d0aa03 - Add binary tests
• f4f0263f - Change tests to be UART1 only for now
• e0917b6a - Rename test crate to be more descriptive
• 499c4c66 - Increase flash memory size for comm tests
• 722066d0 - Add timeout on first real recv test
• ab6cc7e4 - Add check to prevent sending 1 byte message, clarifying docs, and add pub const for metadata size
• d56c99f3 - Fix crate names and add .gdbinit's
• 442de999 - Move .gdbinit and add optization flags
• f898a01d - Make large_test.txt smaller
• 490b2be7 - Remove large text test
• 34c33d96 - Add finish message
• 19c78698 - Increase send size of large binary test
• 97bb98e6 - Add crypto tests
• 13c5cb84 - Make crypto tests use lower level Rx
• e4b2400a - Remove large crypto test b/c of stack overflow
• d779092f - Fix crypto tests
• b35d514f - Update clarifying doc

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 24, 2023, 19:31

added 1 commit

• 8c850771 - Add nonce uniqueness test

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 24, 2023, 23:18

added 1 commit

• ece1b0d3 - Rename test fn and remove extraneous comment

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 25, 2023, 01:08

added 1 commit

• 62f0db33 - Split common code into other crate

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 25, 2023, 12:57

added 1 commit

• 471f83f8 - Add uart0 send/recv board crates and rename old ones to uart1

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 25, 2023, 12:58

added 1 commit

• f6e60095 - Add .gdbinit file into new uart0 crates

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 25, 2023, 13:21

added 1 commit

• 111a4332 - Add host test crates

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 25, 2023, 14:13

added 1 commit

• 77d4e244 - Add host tool tests

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 25, 2023, 14:14

added 1 commit

• 01a1b95a - Fix print typo

Compare with previous version

[eggroll-bot]

In GitLab by @MelonShooter on Feb 25, 2023, 14:54

added 1 commit

• 957b006e - Skip first arg in host tests

Compare with previous version

[eggroll-bot]

In GitLab by @group_21660_bot2 on Feb 25, 2023, 17:59

added 1 commit

• 4bca93d5 - Change flash memory boundaries

Compare with previous version

[eggroll-bot]

In GitLab by @group_21660_bot2 on Feb 25, 2023, 18:03

added 1 commit

• d434bdcf - Add memory layout to other board crates

Compare with previous version

[eggroll-bot]

In GitLab by @group_21660_bot2 on Feb 25, 2023, 18:37

added 1 commit

• 4f977d9b - Add optimization flags to be the same as everything else

Compare with previous version

[eggroll-bot]

In GitLab by @group_21660_bot2 on Feb 25, 2023, 20:13

added 1 commit

• c399c96b - Add inline never annotations to optimize stack space

Compare with previous version

[eggroll-bot]

added 1 commit

• de1cabf8 - Fix clippy warning

Compare with previous version

[eggroll-bot]

@victorh12 @ext-ucsc, please resolve the panic on test 1 and code review after.

@MelonShooter, please code review the panic resolution when that is done.

[eggroll-bot]

Please also fix this:

Done.

[eggroll-bot]

In GitLab by @victorh12 on Mar 1, 2023, 19:59

added 1 commit

• a39804b4 - Fix panic in too_small_recv_error_test

Compare with previous version

[eggroll-bot]

added 50 commits

• a39804b4...0f18c2d4 - 7 commits from branch main
• f3853077 - Write comm test skeleton
• 2a38ab04 - Fix clippy warnings
• bf32d145 - Fix hex encode panic, closes #4
• aa68b69a - Add basic UART 0 test
• 3d7c9789 - Change baud rate to match that of the serial to socket bridge
• e7e67dfb - Change UART0 controller to have pins in type
• b181debb - Add comm_tests workspace with two board crates
• b787e041 - Add skeleton for multi-UART testing
• 2f4b8f6b - Add basic test
• c73c8e6d - Add receive error tests
• 7acc6707 - Add large text test
• 0b730ef7 - Add binary tests
• 3da66bbd - Change tests to be UART1 only for now
• dabe32cd - Rename test crate to be more descriptive
• e392f46f - Increase flash memory size for comm tests
• adc6085b - Add timeout on first real recv test
• a357f652 - Add check to prevent sending 1 byte message, clarifying docs, and add pub const for metadata size
• 7fb83733 - Fix crate names and add .gdbinit's
• c6d617fa - Move .gdbinit and add optization flags
• 7a68ca31 - Make large_test.txt smaller
• 17a388a8 - Remove large text test
• 47e16c2e - Add finish message
• 5ab21a5c - Increase send size of large binary test
• 86b20fc9 - Add crypto tests
• 94a48262 - Make crypto tests use lower level Rx
• 65cc07f6 - Remove large crypto test b/c of stack overflow
• 5dd537e4 - Fix crypto tests
• 92be0e5f - Update clarifying doc
• 3bbde036 - Add nonce uniqueness test
• 54666db7 - Rename test fn and remove extraneous comment
• 186fae3f - Split common code into other crate
• bbefd46d - Add uart0 send/recv board crates and rename old ones to uart1
• 06b54890 - Add .gdbinit file into new uart0 crates
• 7a15a0b5 - Add host test crates
• aa76521f - Add host tool tests
• b7838f6c - Fix print typo
• ae700dba - Skip first arg in host tests
• 94b96fda - Change flash memory boundaries
• 9da194df - Add memory layout to other board crates
• 6bb0a792 - Add optimization flags to be the same as everything else
• fae24766 - Add inline never annotations to optimize stack space
• 01e67f7a - Fix clippy warning
• 28c32625 - Fix panic in too_small_recv_error_test

Compare with previous version

[eggroll-bot]

In GitLab by @victorh12 on Mar 1, 2023, 22:19

added 1 commit

• 2d76537c - Add delays to avoid panics

Compare with previous version

[eggroll-bot]

In GitLab by @victorh12 on Mar 1, 2023, 22:28

added 1 commit

• 4faaef59 - Fix documentation links

Compare with previous version