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.

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

Feature.flow uavcan params #13

Open
wants to merge 6 commits into
base: feature.flow
Choose a base branch
from

Conversation

davids5
Copy link

@davids5 davids5 commented Sep 23, 2015

This adds the ability to set UAVCAN parameters from the command line. see PX4/PX4-Autopilot#2895

IMPORTANT!!

The last fix to nuttx is critical

@rmackay9
Copy link
Contributor

@davids5, is the nuttx fix critical in a general sense or just critical for getting uavcan working? I know @tridge is looking at updating ardupilot-master to the latest PX4 group's NuttX/PX4Firmware.

@davids5
Copy link
Author

davids5 commented Sep 24, 2015

@rmackay9 , @tridge

Yes it is critical in a general sense. See PX4/NuttX#66

In the sequence of the pthread_create function, prior to the group join, if a failure due to a lack of free memory (stack allocation) then the TCB has a group assigned but not joined. Because it has not been joined the ref count is N not N+1 to count this thread. On the error exit N is decremented to 1 and if this was the first thread to be added the group's resources are released.

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.

2 participants