Skip to content

Conversation

lslusarczyk
Copy link
Contributor

No description provided.

@lslusarczyk lslusarczyk requested a review from a team as a code owner September 22, 2025 07:48
@bratpiorka
Copy link
Contributor

could you show me what an example log would look like in dev mode after your changes?

@lslusarczyk
Copy link
Contributor Author

could you show me what an example log would look like in dev mode after your changes?

with pleasure

This is how now one can work with L0 logs inside IDE:
umf_logs_with_fileline
You click on a link and go directly to code where log is present.

#ifdef UMF_DEVELOPER_MODE
#define UMF_STRINGIFY(x) #x
#define UMF_TOSTRING(x) UMF_STRINGIFY(x)
#define UMF_FUNC_DESC() __FILE__ ":" UMF_TOSTRING(__LINE__)
Copy link
Contributor

Choose a reason for hiding this comment

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

could you change this to "file: line (func)" or sth similar that looks ok? I mean I would still like to know the name of the function from which the log was printed

Copy link
Contributor Author

Choose a reason for hiding this comment

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

applied

Copy link
Contributor

Choose a reason for hiding this comment

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

please show me what the current log looks like

Copy link
Contributor Author

Choose a reason for hiding this comment

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

please show me what the current log looks like

[DEBUG UMF] /home/lucasso/src/unified-memory-framework/src/libumf.c:82 umfInit: UMF tracker created
[DEBUG UMF] /home/lucasso/src/unified-memory-framework/src/libumf.c:92 umfInit: UMF IPC cache initialized
[DEBUG UMF] /home/lucasso/src/unified-memory-framework/src/libumf.c:99 umfInit: UMF library initialized

@bratpiorka bratpiorka requested a review from KFilipek September 23, 2025 10:03
Copy link
Contributor

@KFilipek KFilipek left a comment

Choose a reason for hiding this comment

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

Needs to be squashed

@lslusarczyk lslusarczyk merged commit 44f1c63 into oneapi-src:main Sep 23, 2025
75 checks passed
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.

3 participants