Conversation
BaoshanPang
commented
Aug 12, 2019
BaoshanPang
commented
- support crt-static
- change armv7_wrs_vxworks to armv7_wrs_vxworks_eabihf.
- code cleanup
There was a problem hiding this comment.
"mcount" (or the name we specify here) is the function called by compiler generated instrumentation code when profiling is enabled.
https://www.kernel.org/doc/Documentation/trace/ftrace-design.txt
https://ftp.gnu.org/pub/old-gnu/Manuals/gprof-2.9.1/html_node/gprof_25.html
VxWorks would need to implement this function (we don't currently have it). Removing this code doesn't actually disable calls to mcount, it just reverts to the default name "mcount".
For clarity I suggest adding a line to vxworks_base.rs:
target_mcount: "_mcount".to_string(), // VxWorks needs to implement this to support profiling
There was a problem hiding this comment.
Move the following common settings from the target specific files to vxworks_base.rs:
- target_os
- target_vendor
- linker_flavor
There was a problem hiding this comment.
Done. I also moved target_env to vxworks_base as they are all "gnu".
There was a problem hiding this comment.
Actually I need to revert the change here. The vxworks_base.rs suppose to hold all "options", but
they doesn't belong to "options".
There was a problem hiding this comment.
Actually I need to revert the change here
OK that's unfortunate.
BaoshanPang
force-pushed
the
vxworks_bpang_static
branch
4 times, most recently
from
29bac87 to
1146ea8
Compare
2. change armv7_wrs_vxworks to armv7_wrs_vxworks_eabihf. 3. use wr-** instead of vx-** 4. set PIE to false 5. code cleanup
BaoshanPang
force-pushed
the
vxworks_bpang_static
branch
from
1146ea8 to
f161efa
Compare
|
cc @bpangWR |
| wr_run.args(&[&prog]).args(args).envs(env.clone()); | ||
| self.compose_and_run( | ||
| vx_run, | ||
| wr_run, |
There was a problem hiding this comment.
This is the script I used to run test case on board(Simics, Vxsim or real board).
3 participants
