Skip to content

Conversation

@sharkwouter
Copy link
Member

Lets merge this after the new changes are released. Let me know if the wording is okay like this and if I made any mistakes.

debugging.md Outdated
To profile the application we run `psp-gprof` on the `gmon.out` file like so from a terminal in the directory where the application is:
```sh
psp-gprof file gmon.out
Copy link
Member

@fjtrujy fjtrujy Jan 12, 2026

Choose a reason for hiding this comment

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

shouldn't be something like psp-gprof {binary_file} gmon.out better?

Copy link
Member

Choose a reason for hiding this comment

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

So far is fine, you can also put a link to the existing examples in pspsdk/src/samples/gprof folder

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah, maybe that is better. Will change it. I copied some parts for the gdb example.

Copy link
Member Author

Choose a reason for hiding this comment

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

I've updated it now.

@sharkwouter sharkwouter merged commit fa1344b into master Jan 24, 2026
3 checks passed
@sharkwouter sharkwouter deleted the add-profiling-info branch January 24, 2026 09:46
@doniwah
Copy link

doniwah commented Jan 25, 2026

Application profiling is the process of analyzing how an application uses CPU, memory, disk, network, and GPU resources to identify performance bottlenecks and inefficiencies

@sharkwouter
Copy link
Member Author

I can update it, thanks. That is good wording. I'll make another PR and tag you.

@sharkwouter
Copy link
Member Author

On second thought, could you perhaps make a small PR with improved wording @doniwah? I feel like you'd be better suited to do so in this case.

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