mu_basecore icon indicating copy to clipboard operation
mu_basecore copied to clipboard

[Rebase & FF] 202405: MdeModulePkg/Bus/Pci/NvmExpressDxe: Nvm Express Media Sanitize Protocol

Open apop5 opened this issue 1 year ago • 1 comments

Description

Implementation of MEDIA_SANITIZE_PROTOCOL for NIST purge/clear actions with mapping to NVM Express native commands.

Cherry-Pick and squash the following commits:

3e9cf8cdbd

For each item, place an "x" in between [ and ] if true. Example: [x]. (you can also check items in the GitHub UI)

  • [x] Impacts functionality?
    • Functionality - Does the change ultimately impact how firmware functions?
    • Examples: Add a new library, publish a new PPI, update an algorithm, ...
  • [ ] Impacts security?
    • Security - Does the change have a direct security impact on an application, flow, or firmware?
    • Examples: Crypto algorithm change, buffer overflow fix, parameter validation improvement, ...
  • [ ] Breaking change?
    • Breaking change - Will anyone consuming this change experience a break in build or boot behavior?
    • Examples: Add a new library class, move a module to a different repo, call a function in a new library class in a pre-existing module, ...
  • [ ] Includes tests?
    • Tests - Does the change include any explicit test code?
    • Examples: Unit tests, integration tests, robot tests, ...
  • [ ] Includes documentation?
    • Documentation - Does the change contain explicit documentation additions outside direct code modifications (and comments)?
    • Examples: Update readme file, add feature readme file, link to documentation on an a separate Web page, ...

How This Was Tested

N/A

Integration Instructions

N/A

apop5 avatar Jun 11 '24 21:06 apop5

Codecov Report

Attention: Patch coverage is 43.45550% with 108 lines in your changes missing coverage. Please review.

Please upload report for BASE (release/202405@b0e9e3e). Learn more about missing BASE report.

Files Patch % Lines
...kg/Bus/Pci/NvmExpressDxe/NvmExpressMediaSanitize.c 49.40% 83 Missing and 2 partials :warning:
MdeModulePkg/Bus/Pci/NvmExpressDxe/NvmExpress.c 0.00% 23 Missing :warning:
Additional details and impacted files
@@                Coverage Diff                @@
##             release/202405     #904   +/-   ##
=================================================
  Coverage                  ?    1.25%           
=================================================
  Files                     ?     1422           
  Lines                     ?   358572           
  Branches                  ?     4391           
=================================================
  Hits                      ?     4495           
  Misses                    ?   354016           
  Partials                  ?       61           
Flag Coverage Δ
MdeModulePkg 0.24% <43.45%> (?)
MdePkg 5.41% <ø> (?)
NetworkPkg 0.55% <ø> (?)
UefiCpuPkg 4.80% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Jun 26 '24 16:06 codecov-commenter