Skip to content

Feature: SMP #278

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

Merged
merged 14 commits into from
Mar 16, 2021
Merged

Feature: SMP #278

merged 14 commits into from
Mar 16, 2021

Conversation

n9wxu
Copy link
Member

@n9wxu n9wxu commented Mar 15, 2021

Feature branch for developing SMP

Description

This new branch holds the changes needed to support SMP operations for XMOS, TI, Espressif and Raspberry Pi. This feature comes from an XMOS private repo.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Michael Bruno and others added 12 commits January 19, 2021 18:56
Some minor modifications are also made to the kernel to support the
XCore compiler's automatic stack size calculation.
The XMOS XCore ports are also updated to support SMP.
The port set and clear interrupt mask from ISR macros were removed from
the ports so that the default macros found in FreeRTOS.h are used
instead. The default macros do not result in warnings when they are
used.
* feature/xcore:
  Fix compiler warnings in xcore ports
Inline function converted to macro. This should now build when
optimizations are off and inlining is disabled.
* feature/xcore:
  Remove inline function from timers.h
* main:
  Add some assertions and coverage exceptions to queue.c (#273)
  Moved git-secrets check to a different workflow (#271)
  Comment: fix code comment for xTaskAbortDelay (#272)
  Added git-secrets check to Github Actions (#270)
  fixed documentation for ulTaskNotifyTake() and ulTaskNotifyTakeIndexed() (#269)
  Add description for vTaskList (#206)
  Support allocating stack from separate heap (#267)
  Update python version to 3.7.10 (#265)
  Update URL in history.txt (#259)
  Remove unmatched braces in MessageBuffer pre tags (#256)
  Check: improve verbosity of url verifier (#260)
  Update issue templates
  incorporate updates from common (#255)
  Typos (#248)
  Delete gitattributes (#253)
  Fix typo in comment in task.h (#244)
  Fix Github checks after move from master to main (#246)
* feature/xcore:
  Add some assertions and coverage exceptions to queue.c (#273)
  Moved git-secrets check to a different workflow (#271)
  Comment: fix code comment for xTaskAbortDelay (#272)
  Added git-secrets check to Github Actions (#270)
  fixed documentation for ulTaskNotifyTake() and ulTaskNotifyTakeIndexed() (#269)
  Add description for vTaskList (#206)
  Support allocating stack from separate heap (#267)
  Update python version to 3.7.10 (#265)
  Update URL in history.txt (#259)
  Remove unmatched braces in MessageBuffer pre tags (#256)
  Check: improve verbosity of url verifier (#260)
  Update issue templates
  incorporate updates from common (#255)
  Typos (#248)
  Delete gitattributes (#253)
  Fix typo in comment in task.h (#244)
  Fix Github checks after move from master to main (#246)
@n9wxu n9wxu requested a review from a team as a code owner March 15, 2021 21:40
@n9wxu n9wxu changed the base branch from main to smp March 15, 2021 23:25
@n9wxu n9wxu merged commit 989bc33 into FreeRTOS:smp Mar 16, 2021
@cmdrf
Copy link
Contributor

cmdrf commented Jul 3, 2021

I'm very interested in SMP! But is there a reason why this touches almost every line in every file in the entire repository? #280 then touches everything again. This makes it very hard to work with this branch.

@aggarg
Copy link
Member

aggarg commented Jul 6, 2021

Those were probably (accidental) line ending changes. You can hide white-space changes to make it easier to look.

@alfred-ai
Copy link

For feature as big (and interesting) as this, irrelevant files shouldn't be touched so everyone else will understand the changes better.

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.

4 participants