Skip to content

mi: Add Identify function for secondary controller list#480

Merged
igaw merged 1 commit intolinux-nvme:masterfrom
CodeConstruct:dev/mi-id-secondary
Sep 26, 2022
Merged

mi: Add Identify function for secondary controller list#480
igaw merged 1 commit intolinux-nvme:masterfrom
CodeConstruct:dev/mi-id-secondary

Conversation

@jk-ozlabs
Copy link
Copy Markdown
Collaborator

This change adds a helper for the Identify command, retrieving the secondary controller list. We add a simple test too.

Signed-off-by: Jeremy Kerr jk@codeconstruct.com.au

This change adds a helper for the Identify command, retrieving the
secondary controller list. We add a simple test too.

Signed-off-by: Jeremy Kerr <jk@codeconstruct.com.au>
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #480 (0b2c99d) into master (901bd0a) will increase coverage by 0.17%.
The diff coverage is 65.21%.

@@            Coverage Diff             @@
##           master     #480      +/-   ##
==========================================
+ Coverage   22.41%   22.58%   +0.17%     
==========================================
  Files          31       31              
  Lines        5760     5783      +23     
  Branches     1179     1187       +8     
==========================================
+ Hits         1291     1306      +15     
  Misses       4039     4039              
- Partials      430      438       +8     
Impacted Files Coverage Δ
test/mi.c 65.32% <60.00%> (-0.16%) ⬇️
src/nvme/mi.h 72.09% <100.00%> (+2.09%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@igaw igaw merged commit e4d3816 into linux-nvme:master Sep 26, 2022
@jk-ozlabs jk-ozlabs deleted the dev/mi-id-secondary branch September 26, 2022 09:22
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