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.

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

Add EBPF_LOG_MESSAGE_BINARY tracing macro with TraceLoggingBinary to print binary data. #3786

Open
shpalani opened this issue Aug 23, 2024 · 0 comments · May be fixed by #3781
Open

Add EBPF_LOG_MESSAGE_BINARY tracing macro with TraceLoggingBinary to print binary data. #3786

shpalani opened this issue Aug 23, 2024 · 0 comments · May be fixed by #3781
Assignees
Labels
enhancement New feature or request P2 triaged Discussed in a triage meeting
Milestone

Comments

@shpalani
Copy link
Collaborator

Describe the bug

Request to add EBPF_LOG_MESSAGE_BINARY tracing macro with TraceLoggingBinary to print binary data.

#define TraceLoggingBinary(pValue, cbValue, ...)

  • The pValue parameter is treated as a const void* so that any kind of data can be provided.
  • The cbValue parameter is the size of the data, in bytes.
    For eg,
    #define _EBPF_LOG_MESSAGE_BINARY(trace_level, keyword, message, type, name, pValue, cbValue)

OS information

Windows 10 and above

Steps taken to reproduce bug

New macro.

Expected behavior

Print the binary data.

Actual outcome

New macro.

Additional details

No response

@shpalani shpalani added bug Something isn't working enhancement New feature or request and removed bug Something isn't working labels Aug 23, 2024
@shankarseal shankarseal added this to the 2408 milestone Aug 26, 2024
@shankarseal shankarseal added P2 triaged Discussed in a triage meeting labels Aug 26, 2024
@shankarseal shankarseal modified the milestones: 2408, 2409 Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request P2 triaged Discussed in a triage meeting
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants