-
Notifications
You must be signed in to change notification settings - Fork 33
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
use common func wd_find_msg_in_pool to replace wd_xxx_get_msg #646
Open
gaozhangfei
wants to merge
2
commits into
Linaro:develop
Choose a base branch
from
gaozhangfei:wd_find_msg_in_pool
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
2 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you think it is unnecessary for alg.so, you should remove it from alg_SOURCE.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
如果完全其他库Makefile删掉wd_sched.c wd_util.c,
那libwd.map 还需要export 蛮多函数的
这些函数都是其他库用到的,
/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_alg_get_env_param' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_parse_async_poll_en'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_get_msg_from_pool' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_check_ctx'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_alg_attrs_init' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_ctx_param_init'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_add_task_to_async_queue' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_clear_ctx_config'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_alg_try_init' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_check_datalist'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_check_src_dst' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_init_sched'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_alg_uninit_driver' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_alg_drv_bind'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_set_ctx_attr' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_alg_env_init'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_uninit_async_request_pool' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_init_ctx_config'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_clear_sched' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_alg_drv_unbind'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_dlopen_drv' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_alg_env_uninit'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_ctx_param_uninit' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_put_msg_to_pool'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_alg_attrs_uninit' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_init_param_check'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_handle_msg_sync' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_alg_init_driver'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_parse_ctx_num' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_init_async_request_pool'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_memset_zero' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_set_epoll_en'/usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_get_lib_file_path' /usr/bin/ld: /home/linaro/work/uadk/.libs/libwd_crypto.so: undefined reference to
wd_dlclose_drv'这几个函数是app 或者是驱动用到的。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
那这样的逻辑下你无法删除原有的引用,反而要在libwd中新增了wd_sched和wd_util,它的目的在于什么地方呢?如果说你把所有接口全部整改那这个逻辑我理解是成立的,所有公共部分全部放到libwd中,alg对公共的依赖由libwd提供就好。但是现在的修改方案既不能保证alg和公共文件的解耦,由让libwd和公共文件耦合,无法理解它的好处。