-
Notifications
You must be signed in to change notification settings - Fork 7
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
zephyr-dom0-xt: Add vchan simple echo server #30
Conversation
36de73a
to
02889cb
Compare
Fixed comments |
There should readme update with full example as from dom0 side as from doimu side |
Updated README.rst |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you . I have few minor comments.
|
vchan_server_stop = 1; | ||
if (k_thread_join(pecho_thread, K_MSEC(1000))) { | ||
shell_error(sh, "Killing thread"); | ||
k_thread_abort(pecho_thread); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you please confirm that functions used in peacho_thread
does not use dynamic allocation? Otherwise, we will get resource leak here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pecho_thread is pointer to echo_thread static structure or NULL (line 74)
2a0234d
to
1c4c046
Compare
LGTM |
3d8638d
to
4b63643
Compare
Rebased to main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Acked-by: Volodymyr Babchuk <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Acked-by: Dmytro Firsov <[email protected]>
Also, @klogg can you please re-check?
|
Add vchan simple echo server. Server opens channel and all incoming data will be send back. Server will be enabled with option CONFIG_VCHAN_SHELL_CMDS. Server introduces 2 commands: vchan start [domid] Start echo server. domid - The peer domain that will be connecting (0 by default) vchan stop Stop server Updated README.rst with information about vchan echo server Signed-off-by: Dmytro Semenets <[email protected]> Reviewed-by: Grygorii Strashko <[email protected]> Acked-by: Oleksii Moisieiev <[email protected]> Acked-by: Volodymyr Babchuk <[email protected]> Acked-by: Dmytro Firsov <[email protected]> Reviewed-by: Artem Mygaiev <[email protected]>
Add vchan simple echo server. Server opens channel and all incoming data will be send back.