zephyrproject-rtos / zephyr

Primary Git Repository for the Zephyr Project. Zephyr is a new generation, scalable, optimized, secure RTOS for multiple hardware architectures.
https://docs.zephyrproject.org
Apache License 2.0
10.97k stars 6.68k forks source link

Deprecate qemu.cmake in favour of a qemu runner backend #5501

Open SebastianBoe opened 6 years ago

SebastianBoe commented 6 years ago

Hi,

I suspect we want to eventually rename/alias 'flash' to 'run'. And treat qemu emulation as just another runner backend. I believe this would make it easier for us to maintain the runner and qemu infrastructure, and would make it easier for us to keep the user-interface for flashing and running consistent.

A qemu backend would also allow us to move all the symlink/fifo/pipehandling out of qemu.cmake and into a python runner backend.

It is not clear how difficult it would be to do this.

SebastianBoe commented 6 years ago

@mbolivar : Did you already intend to move in this direction? I see you have added a dummy qemu.py backend already.

mbolivar commented 6 years ago

@SebastianBoe Happy new year!

That was a copy of a dummy qemu.sh that was already there when I started working on the runner backends. I created a dummy Python script just so people would have a place to start if they want to continue this work. I have no plans to do it.

SebastianBoe commented 6 years ago

Happy new year :)

Do you think it might make sense to incorporate running qemu into the runner infrastructure?

mbolivar commented 6 years ago

I definitely think it makes sense. It's just another way to run the code, after all. I haven't tried the new "native" port, but I think that would make sense too -- keeping interfaces uniform is good usability.

mbolivar commented 5 years ago

Upgrading the priority since I've received several questions about this recently.

nashif commented 3 years ago

see #35366

zephyrbot commented 9 months ago

Hi @nashif,

This issue, marked as an Enhancement, was opened a while ago and did not get any traction. Please confirm the issue is correctly assigned and re-assign it otherwise.

Please take a moment to review if the issue is still relevant to the project. If it is, please provide feedback and direction on how to move forward. If it is not, has already been addressed, is a duplicate, or is no longer relevant, please close it with a short comment explaining the reason.

@SebastianBoe you are also encouraged to help moving this issue forward by providing additional information and confirming this request/issue is still relevant to you.

Thanks!

nordicjm commented 1 week ago

Seems like this was implemented years ago?

mbolivar commented 1 week ago

Seems like this was implemented years ago?

no; https://github.com/zephyrproject-rtos/zephyr/blob/main/scripts/west_commands/runners/qemu.py has always been a stub waiting for a hero to step up and fill it in