Skip to content

Shell kernel cmds adaptation #9363

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

Merged

Conversation

nordic-krch
Copy link
Collaborator

PR contains #9362 and should be merged together or after 'parent' PR.

Commands from kernel_service.c ported to new shell subsystem.

Fixes #8880

@nordic-krch nordic-krch added the DNM This PR should not be merged (Do Not Merge) label Aug 9, 2018
@codecov-io
Copy link

codecov-io commented Aug 9, 2018

Codecov Report

Merging #9363 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #9363   +/-   ##
=======================================
  Coverage   52.58%   52.58%           
=======================================
  Files         213      213           
  Lines       26148    26148           
  Branches     5636     5636           
=======================================
  Hits        13749    13749           
  Misses      10145    10145           
  Partials     2254     2254

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8feab48...624a021. Read the comment docs.

@nordic-krch nordic-krch mentioned this pull request Sep 12, 2018
@nordic-krch nordic-krch force-pushed the shell_kernel_cmds_adaptation branch 2 times, most recently from 1c950b6 to 25e71e7 Compare September 19, 2018 10:56
@nashif
Copy link
Member

nashif commented Sep 19, 2018

@nordic-krch please rebase

@nordic-krch nordic-krch force-pushed the shell_kernel_cmds_adaptation branch from 25e71e7 to a2954a2 Compare September 20, 2018 05:05
@nordic-krch nordic-krch removed the DNM This PR should not be merged (Do Not Merge) label Sep 20, 2018
@nordic-krch nordic-krch changed the title [DNM] Shell kernel cmds adaptation Shell kernel cmds adaptation Sep 20, 2018
@nordic-krch
Copy link
Collaborator Author

rebased, removed DNM label.

@nashif
Copy link
Member

nashif commented Sep 20, 2018

I am not getting the expected command when I run this for qemu_x86

@nordic-krch
Copy link
Collaborator Author

@nashif by default kernel commands are disabled in menuconfig in this example. I ran enable them by default.

@nashif
Copy link
Member

nashif commented Sep 20, 2018

@nashif by default kernel commands are disabled in menuconfig in this example. I ran enable them by default.

ah, no wonder. Not sure why it was disabled. Can we enable this again?

nordic-krch and others added 2 commits September 23, 2018 09:53
Kernel commands ported to new shell.

Signed-off-by: Krzysztof Chruscinski <[email protected]>
Enable kernel shell and adapt test data and remove limited whitelisting.

Signed-off-by: Anas Nashif <[email protected]>
@nashif nashif force-pushed the shell_kernel_cmds_adaptation branch from a2954a2 to 624a021 Compare September 23, 2018 15:06
@nashif nashif merged commit b3829ad into zephyrproject-rtos:master Sep 23, 2018
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.

3 participants