Skip to content

Fix/Add Bit Definitions NVMe Commands#383

Merged
igaw merged 2 commits intomasterfrom
unknown repository
Jun 14, 2022
Merged

Fix/Add Bit Definitions NVMe Commands#383
igaw merged 2 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jun 13, 2022

2 Fixes were made with this Pull Request

  • Added Storage Tag Check IO Control Flag bit definition
  • Fixed the LSP and LSI mask values used in the git log command.

Signed-off-by: Jeff Lien <jeff.lien@wdc.com>
Signed-off-by: Jeff Lien <jeff.lien@wdc.com>
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #383 (4f83b6a) into master (2657a41) will increase coverage by 1.17%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #383      +/-   ##
=========================================
+ Coverage    3.67%   4.85%   +1.17%     
=========================================
  Files          23      27       +4     
  Lines        3645    4098     +453     
  Branches      703     737      +34     
=========================================
+ Hits          134     199      +65     
- Misses       3440    3801     +361     
- Partials       71      98      +27     
Impacted Files Coverage Δ
src/nvme/ioctl.c 0.00% <ø> (ø)
src/nvme/types.h 0.00% <ø> (ø)
src/nvme/tree.c 8.94% <0.00%> (-0.03%) ⬇️
examples/display-tree.c 0.00% <0.00%> (ø)
src/nvme/mi.c 0.00% <0.00%> (ø)
src/nvme/mi-mctp.c 0.00% <0.00%> (ø)
examples/mi-mctp.c 0.00% <0.00%> (ø)
test/mi.c 69.89% <0.00%> (ø)

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 2657a41...4f83b6a. Read the comment docs.

@igaw igaw merged commit e757ffe into linux-nvme:master Jun 14, 2022
@igaw
Copy link
Copy Markdown
Collaborator

igaw commented Jun 14, 2022

Thanks for the followup!

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