Skip to content
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

Reduce code pages touched during ebpf_api_initiate #2678

Merged

Conversation

Alan-Jowett
Copy link
Member

Description

Eliminate 5 pages from ebpf_api_initiate.

Testing

CI/CD + WPR + WPA comparative analysis (shows pages no longer being touched).

Documentation

No.

Notes:
The API load_ebpf_provider_data is best effort, so removing the result.

dthaler
dthaler previously approved these changes Jul 17, 2023
libs/api_common/windows_platform_common.cpp Show resolved Hide resolved
dthaler
dthaler previously approved these changes Jul 17, 2023
@codecov
Copy link

codecov bot commented Jul 17, 2023

Codecov Report

Merging #2678 (84d0ad7) into main (5c322af) will decrease coverage by 4.08%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##             main    #2678      +/-   ##
==========================================
- Coverage   87.18%   83.10%   -4.08%     
==========================================
  Files         139      146       +7     
  Lines       26542    28040    +1498     
==========================================
+ Hits        23141    23304     +163     
- Misses       3401     4736    +1335     
Impacted Files Coverage Δ
libs/api/ebpf_api.cpp 85.88% <ø> (-0.01%) ⬇️
libs/api_common/windows_platform_common.cpp 87.72% <89.28%> (-0.11%) ⬇️
...ution_context/unit/execution_context_unit_test.cpp 98.89% <100.00%> (+<0.01%) ⬆️

... and 19 files with indirect coverage changes

dv-msft
dv-msft previously approved these changes Jul 17, 2023
@dv-msft dv-msft added this pull request to the merge queue Jul 17, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Jul 17, 2023
@Alan-Jowett Alan-Jowett added this pull request to the merge queue Jul 18, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 18, 2023
@Alan-Jowett Alan-Jowett dismissed stale reviews from dv-msft and dthaler via 51dab75 July 18, 2023 17:34
dthaler
dthaler previously approved these changes Jul 19, 2023
Alan Jowett added 2 commits July 24, 2023 10:15
@Alan-Jowett Alan-Jowett force-pushed the ebpf_api_initiate_minimization branch from 40b6b82 to 84d0ad7 Compare July 24, 2023 17:17
@dv-msft dv-msft added this pull request to the merge queue Jul 31, 2023
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Jul 31, 2023
@Alan-Jowett Alan-Jowett added this pull request to the merge queue Aug 1, 2023
Merged via the queue into microsoft:main with commit 7beb0ff Aug 1, 2023
67 of 68 checks passed
@Alan-Jowett Alan-Jowett deleted the ebpf_api_initiate_minimization branch August 1, 2023 15:13
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.

5 participants