Skip to content
This repository was archived by the owner on Nov 12, 2025. It is now read-only.

Limited memset with inline#5892

Draft
grom72 wants to merge 1 commit intopmem:masterfrom
grom72:limited-memset_
Draft

Limited memset with inline#5892
grom72 wants to merge 1 commit intopmem:masterfrom
grom72:limited-memset_

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
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@intel.com>
@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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

DAOS DAOS-related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants