Add malloc stats #1298
Closed
Add malloc stats #1298
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This reverts commit e50f484. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit bf31d73. Signed-off-by: Robert Lubos <[email protected]>
…y gain/loss" This reverts commit 3b0ca1e. Signed-off-by: Robert Lubos <[email protected]>
…gr ignore_iface" This reverts commit 2eee4cb. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 734675e. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 160fd21. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 606e61f. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 56a8441. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit af4070a. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit dc47180. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 62ed73d. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 031eb6e. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 10dd355. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 2336257. Signed-off-by: Robert Lubos <[email protected]>
…ore array" This reverts commit 9ff8fc2. Signed-off-by: Robert Lubos <[email protected]>
… create connection v2." This reverts commit 41e70dd. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 2080584. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit d4c8639. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 3dee54c. Signed-off-by: Robert Lubos <[email protected]>
…r PAwR" This reverts commit 4359077. Signed-off-by: Robert Lubos <[email protected]>
…ction Procedure" This reverts commit 50570ca. Signed-off-by: Robert Lubos <[email protected]>
…nections while synced" This reverts commit 0e9435e. Signed-off-by: Robert Lubos <[email protected]>
…nnections from PAwR" This reverts commit 1bf676d. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 39cd3a2. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 6b53826. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 7660f3d. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 77991d3. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 946db88. Signed-off-by: Robert Lubos <[email protected]>
This reverts commit 35460d1. Signed-off-by: Robert Lubos <[email protected]>
… platform_allow" This reverts commit 6e436e0. Signed-off-by: Robert Lubos <[email protected]>
…list strtok_r is part of GNU99, but default Zephyr is built with C99, so, the GNU headers are not pulled in, even if we add "--std=gnu99" it won't override the C99 standard. As a workaround for build we redefine the prototype, but this causes a coding guideline warning, so, for now disable this warning for strtok_r. Upstream PR: zephyrproject-rtos/zephyr#60686 Signed-off-by: Sachin D Kulkarni <[email protected]>
Enhance wifi_mgmt group documentation by : - Hiding some internal entities i.e. the human-readable strings defined in the various static const char * const arrays, by marking them as @cond INTERNAL_HIDDEN blocks. - Moving the Wi-Fi management API page under the networking group and adding a human readable name. Upstream PR: zephyrproject-rtos/zephyr#60686 Signed-off-by: Sachin D Kulkarni <[email protected]>
Thingy91 CI tests dependencies added. Signed-off-by: Jørgen Kvalvaag <[email protected]>
MCUmgr client upload max data payload length was missing Transport layer CRC + 16bit lenght. Full net buf packet was possible to send but receiver side it was blocked because there was not space for calculate CRC and length. Signed-off-by: Juha Heiskanen <[email protected]> (cherry picked from commit ea84ef5)
Extended the github CI filter for crypto and TF-M tests to reduce the CI load and possibly speed up PRs. Signed-off-by: Magne Værnes <[email protected]>
fixup! [nrf noup] modules: mbedtls: Add symbols for PAKE Rename the SRP config: PSA_WANT_ALG_SRP -> PSA_WANT_ALG_SRP_6 since the Oberon PSA core uses this name. Noup since this symbol is not available upstream but only available in the Oberon PSA core. Signed-off-by: Georgios Vasilakis <[email protected]>
fixup! [nrf noup] modules: mbedtls: add PSA configurations Add missing PSA configurations for: CCM* (no tag) HKDF expand HKDF extract Signed-off-by: Joakim Andersson <[email protected]>
squash! [nrf noup] modules: mbedtls: Add symbols for PAKE Squash with new commit message: modules: mbedtls: Add PSA symbols for Oberon PSA extension Add PSA symbols for Oberon extension to the PSA specification. Noup commit until we can decide a better way to include PSA extension options, or these are accepted to be included in PSA specification with naming for the configurations that are compatible with Mbed TLS project. Signed-off-by: Joakim Andersson <[email protected]>
Add Sidewalk to be trigger on changes in this repo. Correct entry path for hci_rpmsg samples for Matter and Homekit. Signed-off-by: Tomasz Tyzenhauz <[email protected]>
Remove HAS_NO_PM option, in preparation for a new HAS_PM option (inverted logic). Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 26bf349)
Add a new Kconfig option that has to be selected by SoCs providing PM hooks. This option will be now required to enable CONFIG_PM. Before this change, CONFIG_PM could always be enabled, regardless of SoC providing any kind of low-power support. NOTE: commit adjusted to fix conflicts due to some missing previous commits. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 3d2194f)
Some tests provide their own PM hooks, adapted for testing purposes. Add a new option to select HAS_PM, so that CONFIG_PM can be enabled. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit bddf2d9)
Dummy inline functions are useful to avoid ifdeffery in code when a certain option is not available. pm_state_set/pm_state_exit_post_ops, are only called from the PM subsystem, so never called if CONFIG_PM=n, that is, never surrounded with ifdeffery. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit b3fa2ee)
Any system supporting PM must now implement pm_state_set/pm_exit_post_ops. Before this change any platform could enable CONFIG_PM=y, even though it did nothing, ie, no power savings at all. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit e1eedd1)
…t_post_ops Remove unnecessary __weak attribute from power management functions. These functions are now defined once, globally, and mandatory for systems that support CONFIG_PM. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 55f5a75)
The PM hooks were guarded with CONFIG_PM_POLICY_CUSTOM, however, they need to be guarded (if file is always compiled) with CONFIG_PM. In fact, CONFIG_PM_POLICY_CUSTOM requires to implement a custom policy hook, something this module did not provide. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit f603061)
Add a new API to perform an immediate system power off:
`sys_poweroff()`.
Until now, this functionality has been implemented via the system power
management module, but in a clunky fashion. The way system PM works is
by defining some idle states in devicetree, that, given some properties
(e.g. minimal residency, exit latency, etc.) are automatically selected
when system goes to idle based on the expected next wake-up. However,
system off is a power state that one typically wants to control manually
from the application because it implies state loss, and in most cases,
configuring some sort of wake-up source. So in general, it is not
desired to let the system enter this state automatically. This led to
the following stuff in-tree:
from `boards/arm/mimxrt595_evk/mimxrt595_evk_cm33.dts`:
```c
/*
* Deep power-down mode is supported in this SoC through
* 'PM_STATE_SOFT_OFF' state. There is no entry for this in device tree,
* user can call pm_state_force to enter this state.
*/
```
That is, state not being defined in devicetree so that PM subsystem
doesn't pick it automatically, but still implemented in in the PM hooks:
from `soc/arm/nxp_imx/rt5xx/power.c`, `pm_state_set()`:
```c
case PM_STATE_SOFT_OFF:
set_deepsleep_pin_config();
POWER_EnterDeepPowerDown(EXCLUDE_FROM_DEEP_POWERDOWN);
break;
```
And to actually make use of this state, users had to do this kind of
abominations:
```c
pm_state_force(0u, &(struct pm_state_info){ PM_STATE_SOFT_OFF, 0, 0 });
/* Now we need to go sleep. This will let the idle thread runs and
* the pm subsystem will use the forced state. To confirm that the
* forced state is used, lets set the same timeout used previously.
*/
k_sleep(K_SECONDS(SLEEP_S));
printk("ERROR: System off failed\n");
while (true) {
/* spin to avoid fall-off behavior */
}
```
Signed-off-by: Gerard Marull-Paretas <[email protected]>
(cherry picked from commit 90b247b)
Implement the z_sys_poweroff() hook and select HAS_POWEROFF. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 526a7bb)
Make use of the new sys_poweroff API to actually power off the system. This lets us remove many hacks present in the sample. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 37b26f4)
Nordic SoCs do all power management automagically when going to idle (ie k_cpu_idle()). The only extra state, system off, is now handled via sys_shutdown(), so there's no need to support the PM subsystem. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit 96b3827)
nRF53/91 require usage of nrf_regulators_system_off, so the API is not common with nRF51/52. This was an oversight during the conversion. Signed-off-by: Gerard Marull-Paretas <[email protected]> (cherry picked from commit b990082)
If malloc is enabled in common libc, add support for malloc stats using the mallinfo API. Signed-off-by: Chaitanya Tata <[email protected]>
This is useful as a placeholder for all commands common to applications. For now we start with malloc stats, useful in debugging HEAP issues. Signed-off-by: Chaitanya Tata <[email protected]>
This adds basic tests in a single testcase to verify the functionality of mallinfo2. Signed-off-by: Chaitanya Tata <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
40 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.