-
Notifications
You must be signed in to change notification settings - Fork 208
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
Reentrant version of decompress routine in CFE FS #36
Comments
Imported from trac issue 5. Created by jphickey on 2014-12-24T08:46:04, last modified: 2019-03-05T14:57:55 |
Trac comment by jphickey on 2014-12-24 10:29:36: Pushed [changeset:65a79276] to address this. Branch name is trac-5-cfe_fs_decompress_reentrant |
Trac comment by jphickey on 2015-04-06 11:21:58: This is ready for review/merge |
Trac comment by acudmore on 2015-04-06 14:01:05: Recommend accept |
Trac comment by glimes on 2015-04-07 12:45:58: Tested changeset [changeset:65a7927] as part of the ic-2015-03-10 merge. |
Trac comment by glimes on 2015-04-13 15:15:34: Part of integration candidate 2015-03-10, |
Trac comment by glimes on 2016-02-16 13:16:45: Susie confirmed these tickets have been approved for CFE 6.5 |
Trac comment by jhageman on 2019-03-05 14:57:55: Milestone renamed |
The FS application has a feature where it can decompress file content on the fly. However the decompression routine keeps its internal state in global variables which makes it non-reentrant. To protect against concurrent usage a mutex is used but this has a major performance impact.
The global variables should be replaced with a state structure so that it can be multithreaded like all other parts of CFS.
For the time being a global state object can be implemented in order to preserve API compatibility.
The text was updated successfully, but these errors were encountered: