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.

Sign up for GitHub

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

Jump to bottom

Replace calls to OS_fsBlocksFree #1049

Closed
jphickey opened this issue Dec 29, 2020 · 0 comments · Fixed by #1050 or #1088
Closed

Replace calls to OS_fsBlocksFree #1049

jphickey opened this issue Dec 29, 2020 · 0 comments · Fixed by #1050 or #1088
Assignees
@jphickey
Labels
deprecate enhancement
Milestone
Caelum

Comments

@jphickey
Copy link
Contributor

Is your feature request related to a problem? Please describe.
This function is scheduled to be deprecated in nasa/osal#718.

Describe the solution you'd like
It should be replaced with a call to OS_FileSysStatVolume() instead.

Additional context
Old function has insufficient range as noted in nasa/osal#573.

Requester Info
Joseph Hickey, Vantage Systems, Inc.
@jphickey jphickey self-assigned this Dec 29, 2020
jphickey added a commit to jphickey/cFE that referenced this issue Dec 29, 2020
@jphickey
Fix nasa#1049, replace OS_fsBlocksFree
0014ee7 
Use OS_FileSysStatVolume() instead.  This also enables the check
to be more appropriate as this call reports the number of total
blocks, not just the free blocks, so no workaround for desktop
machines is needed.
jphickey added a commit to jphickey/cFE that referenced this issue Dec 29, 2020
@jphickey
Fix nasa#1049, replace OS_fsBlocksFree
2988903 
Use OS_FileSysStatVolume() instead.  This also enables the check
to be more appropriate as this call reports the number of total
blocks, not just the free blocks, so no workaround for desktop
machines is needed.
@jphickey jphickey mentioned this issue Dec 29, 2020
Merged
@skliper skliper added this to the 7.0.0 milestone Jan 4, 2021
astrogeco added a commit that referenced this issue Jan 12, 2021
@astrogeco
Merge pull request #1050 from jphickey/fix-1049-fsblocksfree
eaa850f 
Fix #1049, replace OS_fsBlocksFree
@astrogeco astrogeco mentioned this issue Jan 13, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jphickey jphickey

Labels
deprecate enhancement
Projects
None yet
Milestone
Caelum
Development

Successfully merging a pull request may close this issue.

Fix #1049, replace OS_fsBlocksFree jphickey/cFE
cFE Integration candidate: 2021-01-12 nasa/cFE
3 participants
@jphickey @skliper @astrogeco