Skip to content

doc/ko: Add missing options in uftrace-record.md#1956

Open
KyeongTaek wants to merge 1 commit intonamhyung:masterfrom
KyeongTaek:master
Open

doc/ko: Add missing options in uftrace-record.md#1956
KyeongTaek wants to merge 1 commit intonamhyung:masterfrom
KyeongTaek:master

Conversation

@KyeongTaek
Copy link
Copy Markdown

The missing option '--trace=STATE' is translated and added to uftrace-record.md in Korean.

@MichelleJin12
Copy link
Copy Markdown
Contributor

Thank you for contributing @KyeongTaek

Could you add other missing options in uftrace-record.md?
like

-g, \--agent
: Spawn an agent thread in the target. At runtime, the agent receives
external commands and can change supported tracing options. See *AGENT*
section in the `uftrace-live`(1).

Comment thread doc/ko/uftrace-record.md Outdated
Comment thread doc/ko/uftrace-record.md Outdated
@ntw8651
Copy link
Copy Markdown

ntw8651 commented Sep 8, 2024

I think that It is a good contribution @KyeongTaek
Could you add other things in uftrace-record.md?

These filters can be applied either at record time or replay time. They can be
added and removed at runtime from the client. See `uftrace-live`(1) for details
about agent operations.

"They can -..."
and this

* "var": read value of a global variable given after ":".

This is how to use 'var' watchpoints:
$ uftrace -W var:mydata a.out
# DURATION TID FUNCTION
[239842] | __monstartup() {
[239842] | /* watch:var (mydata=7) */
3.534 us [239842] | } /* __monstartup */
0.191 us [239842] | __cxa_atexit();
[239842] | main() {
[239842] | foo() {
[239842] | /* watch:var (mydata=42) */
0.381 us [239842] | bar();
1.069 us [239842] | } /* foo */
2.698 us [239842] | } /* main */

thank you.

The missing option '--trace=STATE' is translated and added to
uftrace-record.md in Korean.

The missing option '-g, \--agent' is translated and added to
uftrace-record.md in Korean

corrected some translations

added missing description for filter

add 'var' watchpoint and its usage

Signed-off-by: KyeongTaek <tommyholic@naver.com>
Copy link
Copy Markdown
Owner

@namhyung namhyung left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants