Skip to content

Conversation

dakejahl
Copy link
Contributor

I often find myself creating debug binaries that increases the logged rate of sensor data to unlimited for various sensors. Generally this is for debugging data loss or sensor error problems from external sensors. A profile for "sensor debugging" to capture full rate data is very useful for me, perhaps for others too?

@dakejahl dakejahl requested a review from julianoes May 23, 2025 01:47
Copy link

🔎 FLASH Analysis

px4_fmu-v5x [Total VM Diff: 152 byte (0.01 %)]
    FILE SIZE        VM SIZE    
--------------  -------------- 
+0.0%    +152  +0.0%    +152    .text
  [NEW]    +132  [NEW]    +132    px4::logger::LoggedTopics::add_high_rate_sensors_topics()
  +8.0%     +16  +8.0%     +16    px4::logger::LoggedTopics::initialize_configured_topics()
  +0.0%      +8  +0.0%      +8    [section .text]
  +0.8%      +4  +0.8%      +4    ModuleBase<>::unlock_module()
  -4.5%      -4  -4.5%      -4    FlightTask
  -0.0%      -4  -0.0%      -4    g_cromfs_image
+0.0%     +59  [ = ]       0    .debug_abbrev
+0.0%      +8  [ = ]       0    .debug_aranges
+0.0%     +40  [ = ]       0    .debug_frame
+0.0%    +697  [ = ]       0    .debug_info
+0.0%    +300  [ = ]       0    .debug_line
  [NEW]      +1  [ = ]       0    [Unmapped]
  +0.0%    +299  [ = ]       0    [section .debug_line]
+0.0%    +280  [ = ]       0    .debug_loclists
+0.0%     +60  [ = ]       0    .debug_rnglists
  [DEL]      -3  [ = ]       0    [Unmapped]
  +0.0%     +63  [ = ]       0    [section .debug_rnglists]
+0.0%    +108  [ = ]       0    .debug_str
-0.4%      -1  [ = ]       0    .shstrtab
+0.0%     +61  [ = ]       0    .strtab
 -40.0%     -16  [ = ]       0    __nxsem_tickwait_veneer
  +100%     +16  [ = ]       0    __sq_rem_veneer
  [NEW]     +61  [ = ]       0    px4::logger::LoggedTopics::add_high_rate_sensors_topics()
+0.0%     +64  [ = ]       0    .symtab
 -40.0%     -32  [ = ]       0    __nxsem_tickwait_veneer
   +67%     +32  [ = ]       0    __sq_rem_veneer
   +50%     +16  [ = ]       0    __stm32_i2c_sendstart_veneer
 -33.3%     -16  [ = ]       0    __uart_recvchars_veneer
  [NEW]     +64  [ = ]       0    px4::logger::LoggedTopics::add_high_rate_sensors_topics()
-1.7%    -152  [ = ]       0    [Unmapped]
+0.0% +1.64Ki  +0.0%    +152    TOTAL

px4_fmu-v6x [Total VM Diff: 152 byte (0.01 %)]
    FILE SIZE        VM SIZE    
--------------  -------------- 
+0.0%    +152  +0.0%    +152    .text
  [NEW]    +132  [NEW]    +132    px4::logger::LoggedTopics::add_high_rate_sensors_topics()
  +8.0%     +16  +8.0%     +16    px4::logger::LoggedTopics::initialize_configured_topics()
  +0.0%      +8  +0.0%      +8    [section .text]
  +1.0%      +4  +1.0%      +4    ModuleBase<>::unlock_module()
  -0.0%      -8  -0.0%      -8    g_cromfs_image
+0.0%     +59  [ = ]       0    .debug_abbrev
+0.0%      +8  [ = ]       0    .debug_aranges
+0.0%     +40  [ = ]       0    .debug_frame
+0.0%    +697  [ = ]       0    .debug_info
+0.0%    +300  [ = ]       0    .debug_line
  [NEW]      +1  [ = ]       0    [Unmapped]
  +0.0%    +299  [ = ]       0    [section .debug_line]
+0.0%    +280  [ = ]       0    .debug_loclists
+0.0%     +64  [ = ]       0    .debug_rnglists
   +50%      +1  [ = ]       0    [Unmapped]
  +0.0%     +63  [ = ]       0    [section .debug_rnglists]
+0.0%    +108  [ = ]       0    .debug_str
-0.4%      -1  [ = ]       0    .shstrtab
+0.0%     +61  [ = ]       0    .strtab
  [NEW]     +61  [ = ]       0    px4::logger::LoggedTopics::add_high_rate_sensors_topics()
+0.0%     +64  [ = ]       0    .symtab
  [NEW]     +64  [ = ]       0    px4::logger::LoggedTopics::add_high_rate_sensors_topics()
-2.1%    -152  [ = ]       0    [Unmapped]
+0.0% +1.64Ki  +0.0%    +152    TOTAL

Updated: 2025-05-23T01:53:14

@julianoes
Copy link
Contributor

I often have that problem as well!

@julianoes julianoes requested a review from bkueng May 25, 2025 21:15
@bkueng bkueng merged commit 3bbe3e5 into main May 27, 2025
68 of 69 checks passed
@bkueng bkueng deleted the pr-logged_topics_sensors_all branch May 27, 2025 06:09
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