-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Latest Percepio Tracealyzer Recorder v4.3.11 #108
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
Latest Percepio Tracealyzer Recorder v4.3.11 #108
Conversation
|
This is the trace recorder you get when installing Tracealyzer v4.3.11. |
* Update folder structure * Correct project files * Move test folder * Some changes after Yuki's comments
* Add fail-safes to FreeRTOS_Socket.c * Use all 'pd' errors * Correction after Hein's comments * Correction after Hein's comments v2 * Changes after Hein's comments * Update after Gary's comments
This was creating problem with the onboard LPCLink debug probe. Signed-off-by: Gaurav Aggarwal <[email protected]>
* Add Full-TCP suite * delete unnecessary files * Change after Joshua's comments
…RTOS#78) * Adding a combined +TCP driver for SAM4E and SAME70 * Changes after review from Aniruddha Co-authored-by: Hein Tibosch <[email protected]> Co-authored-by: Aniruddha Kanhere <[email protected]>
* Prove buffer lemmas * Update queue proofs to latest kernel source All changes were syntactic due to uncrustify code-formatting * Strengthen prvCopyDataToQueue proof * Add extract script for diff comparison Co-authored-by: Yuhui Zheng <[email protected]>
* DNS.c commit * IP.c commit * Add various source & header files
|
@eriktamlin The branch is out of sync due to the uncrustify PRs, and I do not seem to have permission to push to your branch for rebasing purpose. Would you prefer us replacing this PR with a new one on your behalf, or rebasing your fork branch by yourself? And thank you for the contribution. |
….3.11' of https://github.com/percepio/FreeRTOS into upstream-pull-requests/Percepio-Tracealyzer-Recorder-v4.3.11
16018aa
|
I rebased the PR to upstream/master. |
|
I believe branch merge caused some fraction. (Please see #169 the attached branch history snapshots, that may better explain.) I'll put down "approved" for now for this PR, since I believe the contents are the same. (Note for other reviewers -- We'll need to do a squash merge for this PR, if accept as-is.) |
|
Should I close this one and we just go with pull/169 instead? |
|
@eriktamlin No worries, we'll take care of it. This is in queue, we should be able to pick one and close the other. |
|
The changes were accepted in this PR: #201 |
|
Closing this PR. Thank you @mingyue86010. If any pending discussion, let's move to #201. |
Latest Percepio Tracealyzer Recorder, v4.3.11
Description
The recorder in the repository is greatly outdated and doesn't compile with later FreeRTOS versions.
As part of Percepio's new release procedure we will be creating pull requests to keep the trace recorder in FreeRTOS-Plus up-to-date.
Test Steps
N/A
Related Issue
N/A
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice. Note that the terms of usage stated by Percepio is provided in the source files.