Skip to content

Conversation

@eriktamlin
Copy link
Contributor

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.

@eriktamlin
Copy link
Contributor Author

This is the trace recorder you get when installing Tracealyzer v4.3.11.

AniruddhaKanhere and others added 10 commits June 26, 2020 12:09
* 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
RichardBarry
RichardBarry previously approved these changes Jul 26, 2020
yuhui-zheng
yuhui-zheng previously approved these changes Jul 26, 2020
@yuhui-zheng
Copy link
Contributor

yuhui-zheng commented Jul 26, 2020

@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.

@eriktamlin eriktamlin dismissed stale reviews from yuhui-zheng and RichardBarry via 16018aa July 27, 2020 08:17
@eriktamlin
Copy link
Contributor Author

I rebased the PR to upstream/master.
It says a lot of files have changed...?

@yuhui-zheng
Copy link
Contributor

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.)

@eriktamlin
Copy link
Contributor Author

eriktamlin commented Aug 6, 2020

Should I close this one and we just go with pull/169 instead?

@yuhui-zheng
Copy link
Contributor

@eriktamlin No worries, we'll take care of it. This is in queue, we should be able to pick one and close the other.

@mingyue86010
Copy link
Contributor

The changes were accepted in this PR: #201
Also fixed the Tracealyzer demo build error.

@yuhui-zheng
Copy link
Contributor

Closing this PR. Thank you @mingyue86010.

If any pending discussion, let's move to #201.

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.

9 participants