mcu-debug / rtos-views

RTOS views for microcontrollers
MIT License
24 stars 11 forks source link

Draft: Zephyr support #7

Closed PhilippHaefele closed 1 year ago

PhilippHaefele commented 1 year ago

Add Zephyr support

First work in progress state: grafik

For tests I'm currently using west build -p always -b nucleo_f429zi samples\basic\threads -DCONFIG_THREAD_MONITOR=y -DCONFIG_THREAD_NAME=y

west build -p always -b nucleo_f429zi samples\basic\threads -DCONFIG_DEBUG_THREAD_INFO=y should also work and get's the J-Link Plugin working too

Open points

haneefdm commented 1 year ago

I was about to merge and then I saw your open points. If I merge this PR will be closed and we will lose the open points. Maybe we should create an issue that we can attack later. I know nothing of Zephyr so...

I am sorry for not attending to this extension. I now have a 4-day holiday and see if I can make some progress in publishing the extension.

haneefdm commented 1 year ago

@PhilippHaefele Could I email you? My email address is public on Github. We have some admin issues to take care of.

I want to make these policies public on a CONTRIBUTIONS page or in general users as well. I don't have the bandwidth to handle every detail when I am not even aware of many of these RTOSes. It has to be a community effort.

Please contact me via email.

haneefdm commented 1 year ago

@PhilippHaefele Is there any value for launching a memory view for the stack address? What possible value is there? I did the Link type because I blindly copied MS Embedded Tools, but how does a user practically use it

PhilippHaefele commented 1 year ago

@PhilippHaefele Is there any value for launching a memory view for the stack address? What possible value is there? I did the Link type because I blindly copied MS Embedded Tools, but how does a user practically use it

You're right and i didn't thought about the usecase before. I can only think of a case where the stack end / size is not available and the user wants to have a look into the memory and have an educated guess on the current stack usage. Also it sometimes helps to see what maybe took a lot of stack space by having a look into the patters of the data in the stack.

So over all i do not really see a big need for that

haneefdm commented 1 year ago

I just release the extension. I had to do several things that involved code changes.

I did not update Zyphyr source code as you till have a TODO list you are working on. @PhilippHaefele, you may want to merge and fix any issues that might come up.

@vrepetenko you may be affected as well. At some point, I think we should split up the 2nd column as it would look better and more compact with separate columns. May also take less space.

haneefdm commented 1 year ago

FYI: https://marketplace.visualstudio.com/items?itemName=mcu-debug.rtos-views