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

Limited memset with inline #5892

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Conversation

grom72
Copy link
Contributor

@grom72 grom72 commented Oct 24, 2023

Decrease the amount of stack memory usage.
Remove inline attribute for complex memset_* functions.

Requires:


This change is Reviewable

@grom72 grom72 added DAOS DAOS-related sprint goal This pull request is part of the ongoing sprint labels Oct 24, 2023
@grom72 grom72 requested review from janekmi and osalyk October 24, 2023 10:36
@grom72 grom72 force-pushed the limited-memset_ branch 2 times, most recently from badeed2 to 66c55fd Compare October 25, 2023 21:12
@codecov
Copy link

codecov bot commented Oct 25, 2023

Codecov Report

Merging #5892 (8963d0e) into master (a08e391) will increase coverage by 0.07%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

❗ Current head 8963d0e differs from pull request most recent head f601f55. Consider uploading reports for the commit f601f55 to get more accurate results

@@            Coverage Diff             @@
##           master    #5892      +/-   ##
==========================================
+ Coverage   71.00%   71.08%   +0.07%     
==========================================
  Files         131      131              
  Lines       19172    19233      +61     
  Branches     3192     3208      +16     
==========================================
+ Hits        13614    13671      +57     
- Misses       5558     5562       +4     

@janekmi janekmi removed the sprint goal This pull request is part of the ongoing sprint label Oct 31, 2023
@grom72 grom72 marked this pull request as draft November 8, 2023 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DAOS DAOS-related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants