Bugfixes in the kernel, changed sl to allow NON_BLOCKING sched_rcv

phanikishoreg commented 7 years ago

Major changes: KERNEL:

Fixed a problem where timeout was set when we change the tcap and switch to parent or another scheduler. This is a TODO for child scheds abiding by parent timeout rules, but for now, we don't have it.
A problem with restoring registers which I honestly complicated with confusing conditions before. Fixed that.

SL LIB:

Allow sl_sched_loop to use NON_BLOCKING in cos_sched_rcv. I feel like the naming in SL_SCHEDRCV_DEFAULT is not great.
Setting timeouts for cos_defswitch() when switching to the scheduler is not necessary. In fact, it was causing RK scheduler to switch to chronos even though RK had INF budget. Fixed that!

Choose one (Mandatory):

[ ] This PR is for a code-review and is intended to get feedback, but not to be pulled yet.
[x] This PR is mature, and ready to be integrated into the repo.

@gparmer

As part of this pull request, I've considered the following:

[x] Comments adhere to the Style Guide (SG)
[x] Spacing adhere's to the SG
[x] Naming adhere's to the SG
[x] All other aspects of the SG are adhered to, or exceptions are justified in this pull request
[ ] I have run the auto formatter on my code before submitting this PR (see doc/auto_formatter.md for instructions)

[x] I've made an attempt to remove all redundant code
[x] I've considered ways in which my changes might impact existing code, and cleaned it up
[x] I've formatted the code in an effort to make it easier to read (proper error handling, function use, etc...)
[x] I've commented appropriately where code is tricky
[x] I agree that there is no "throw-away" code, and that code in this PR is of high quality

I've tested the code using the following test programs (provide list here):

gparmer commented 7 years ago

I don't see a comment for the TODO you're referencing.

phanikishoreg commented 7 years ago

I'm sorry, I don't have a TODO in the code. I'll add it.

phanikishoreg commented 7 years ago

Closing this for now. Will do fresh PR after fixing for your comments and with some updates for Baremetal documentation soon.

gwsystems / composite