Skip to content

Conversation

@apop5
Copy link
Collaborator

@apop5 apop5 commented Nov 13, 2024

Description

MSVC AARCH64 /GS support is untested. When it is enabled, the StackCheckLibNull does not satisfy the required externs required by the linker.

Adding support to StackCheckibNull for MSVC.

  • Impacts functionality?
  • Impacts security?
  • Breaking change?
  • Includes tests?
  • Includes documentation?
  • Backport to release branch?

How This Was Tested

Compilation passes.

Integration Instructions

No changes required

@github-actions github-actions bot added the impact:non-functional Does not have a functional impact label Nov 13, 2024
@@ -0,0 +1,80 @@
;------------------------------------------------------------------------------
; AARCH64/StackCheckFunctionsMsvc.nasm
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
; AARCH64/StackCheckFunctionsMsvc.nasm
; AARCH64/StackCheckFunctionsMsvc.asm

@codecov-commenter
Copy link

codecov-commenter commented Nov 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 1.59%. Comparing base (ce81090) to head (79d1983).

Additional details and impacted files
@@             Coverage Diff             @@
##           dev/202405    #1221   +/-   ##
===========================================
  Coverage        1.59%    1.59%           
===========================================
  Files            1449     1449           
  Lines          362730   362730           
  Branches         5632     5632           
===========================================
  Hits             5776     5776           
  Misses         356847   356847           
  Partials          107      107           
Flag Coverage Δ
MdeModulePkg 0.68% <ø> (ø)
MdePkg 5.42% <ø> (ø)
NetworkPkg 0.55% <ø> (ø)
UefiCpuPkg 4.73% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions
Copy link

This PR has been automatically marked as stale because it has not had activity in 60 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions.

@github-actions github-actions bot added the state:stale Has not been updated in a long time label Jan 12, 2025
@apop5
Copy link
Collaborator Author

apop5 commented Jan 16, 2025

not stale

@apop5 apop5 removed the state:stale Has not been updated in a long time label Jan 16, 2025
@github-actions
Copy link

This PR has been automatically marked as stale because it has not had activity in 60 days. It will be closed if no further activity occurs within 7 days. Thank you for your contributions.

@github-actions github-actions bot added the state:stale Has not been updated in a long time label Mar 17, 2025
@github-actions
Copy link

This pull request has been automatically been closed because it did not have any activity in 60 days and no follow up within 7 days after being marked stale. Thank you for your contributions.

@github-actions github-actions bot closed this Mar 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

impact:non-functional Does not have a functional impact state:stale Has not been updated in a long time

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants