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

esp_pm_dump_locks() Wrong report data format with CONFIG_PM_PROFILING enabled (IDFGH-10457) #11704

Closed
3 tasks done
MacWyznawca opened this issue Jun 19, 2023 · 9 comments
Closed
3 tasks done
Assignees
Labels
Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Type: Bug bugs in IDF

Comments

@MacWyznawca
Copy link

MacWyznawca commented Jun 19, 2023

Answers checklist.

  • I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
  • I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
  • I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.

v5.1-rc1-dirty

Operating System used.

macOS

How did you build your project?

Command line with idf.py

If you are using Windows, please specify command line type.

None

Development Kit.

Meaningless

Power Supply used.

External 5V

What is the expected behavior?

Well-formatted and readable text report from esp_pm_dump_locks() function with CONFIG_PM_PROFILING enabled.

What is the actual behavior?

Maybe no end-of-line characters in esp_pm_dump_locks() function in lpm_locks.c.

The effect in the attached image.
Zrzut ekranu 2023-06-19 o 18 14 39

Steps to reproduce.

Enable CONFIG_PM_PROFILING and use esp_pm_dump_locks()

Debug Logs.

No response

More Information.

No response

@MacWyznawca MacWyznawca added the Type: Bug bugs in IDF label Jun 19, 2023
@espressif-bot espressif-bot added the Status: Opened Issue is new label Jun 19, 2023
@github-actions github-actions bot changed the title esp_pm_dump_locks() Wrong report data format with CONFIG_PM_PROFILING enabled esp_pm_dump_locks() Wrong report data format with CONFIG_PM_PROFILING enabled (IDFGH-10457) Jun 19, 2023
@esp-wzh
Copy link
Collaborator

esp-wzh commented Jun 20, 2023

@MacWyznawca
This feature works well on my side,
image

Are you using idf.py monitor? Or can you try a different terminal or monitor tool?

@MacWyznawca
Copy link
Author

@esp-wzh As you can see I'm using a normal IDF (you can see in the window bar at the top).
As I manually typed the end-of-line characters in the mentioned function, it helps, although not always.

image

@esp-wzh
Copy link
Collaborator

esp-wzh commented Jun 20, 2023

OK, I reproduced it, seems it caused by Time field string too long, will give fix soon.

@MacWyznawca
Copy link
Author

@esp-wzh It is possible that the problem does not occur when you use a small terminal window (80 characters). However, I can't work that way.

@MacWyznawca
Copy link
Author

Great! Thanks @esp-wzh

@esp-wzh
Copy link
Collaborator

esp-wzh commented Jun 20, 2023

0001-bugfix-increase-pm_dump-time-field-width.patch.txt
This patch can fix the problem, thanks for report! @MacWyznawca

@espressif-bot espressif-bot added Status: In Progress Work is in progress Status: Reviewing Issue is being reviewed and removed Status: Opened Issue is new Status: In Progress Work is in progress labels Jun 20, 2023
@MacWyznawca
Copy link
Author

Thanks @esp-wzh Works! ;-)

@MacWyznawca
Copy link
Author

By the way @esp-wzh, could you suggest where to look for help in lowering the power consumption of the ESP32-C3? I'm making a light bulb, but the power consumption is 0.66, and the EU requires less than 0.5 W. Note: we are only using ESP-NOW.
I use combinations with esp_now_set_wake_window and esp_wifi_connectionless_module_set_wake_interval, which lowers power consumption, but unfortunately causes sometimes no response (transmission errors).

Please ;-)

@esp-wzh
Copy link
Collaborator

esp-wzh commented Jun 20, 2023

about ESP-NOW low power issues, please create a new issue to discuss and ask @Espressif-liuuuu.

@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: NA Issue resolution is unavailable and removed Status: Reviewing Issue is being reviewed labels Jun 21, 2023
espressif-bot pushed a commit that referenced this issue Jul 7, 2023
espressif-bot pushed a commit that referenced this issue Jul 12, 2023
espressif-bot pushed a commit that referenced this issue Jul 15, 2023
espressif-bot pushed a commit that referenced this issue Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: NA Issue resolution is unavailable Status: Done Issue is done internally Type: Bug bugs in IDF
Projects
None yet
Development

No branches or pull requests

3 participants