add-memory-trace-tags

Original：🇨🇳 Chinese

Translated

Use when adding new resource tracking types to memory profiling in developtools_profiler and third_party_musl repositories. Triggered by requests to add trace tags, resource labels, or memory tracking types.

2installs

Sourceopenharmonyinsight/openharmony-skills

Added on2026-02-10

NPX Install

npx skill4agent add openharmonyinsight/openharmony-skills add-memory-trace-tags

SKILL.md Content (Chinese)

View Translation Comparison →

Adding Memory Tracking Tags

Automatically add new resource tracking type tags across the developtools_profiler and third_party_musl repositories.

Overview

This skill automatically adds memory tracking tags to multiple files across two repositories, ensuring consistent definitions, protocol updates, and test coverage.

Usage Scenarios

Applicable in the following cases:

Adding new resource tracking types (COMMON_* or specific types)
Updating memory profiling functionality
Extending the trace tag system

Quick Reference

Component	Number of Modified Files	Key Changes
third_party_musl	2 files	TAG and MASK definitions
developtools_profiler	9 files	Protocols, logic, tests

Usage Instructions

Call directly in the conversation:

Please add a new resource tracking tag <tag name>

Use add-res-trace-tag to add <tag name>

Parameter Description

This skill will prompt for the following inputs:

Tag Name (Required)
- Format: Uppercase letters and underscores
- Style:
```
COMMON_<type name>
```
  or descriptive name
- Examples:
```
COMMON_PIXELMAP
```
  ,
```
CREATE_NATIVE_WINDOW_FROM_SURFACE
```
Description (Optional)
- Usage explanation
Bypass Size Filter? (Optional, Default: Yes)
- Common resource types typically need to bypass size filtering

Modified Content

third_party_musl (2 files)

```
src/hook/linux/memory_trace.h
```
- TAG, MASK, bitfield layout

porting/linux/user/src/hook/memory_trace.h

- Duplicate definitions

developtools_profiler (9 files)

Protocols:

protos/types/plugins/native_hook/native_hook_result.proto

- TraceType and MemoryType enums

Constants:

device/plugins/native_daemon/include/hook_common.h

- Index constants

Business Logic:

device/plugins/native_hook/src/hook_guard.cpp

- TagId mapping

device/plugins/native_daemon/src/hook_manager.cpp

- Mask conversion

device/plugins/native_daemon/src/stack_preprocess.cpp

- Statistics mapping

device/plugins/native_daemon/src/hook_record.cpp

- TraceType setting

device/plugins/native_hook/src/hook_client.cpp

- Filter conditions (COMMON_* types only)

Tests:

device/plugins/native_daemon/test/unittest/common/native/hook_record_test.cpp

device/plugins/native_daemon/test/unittest/common/native/stack_preprocess_test.cpp

Common Errors

MASK Definition Rules

Each resource type must define two macros:

// ✅ Correct: Define both macros
#define RES_COMMON_WINDOW          (1ULL << 33)
#define RES_COMMON_WINDOW_MASK     (1ULL << 33)

// ❌ Error: Missing _MASK suffix
#define RES_COMMON_WINDOW          (1ULL << 33)
// Missing RES_COMMON_WINDOW_MASK!

Why two definitions are needed:

```
RES_<NAME>
```
- Base mask value
```
RES_<NAME>_MASK
```
- Alias for unified interface

Common Error Scenarios:

Error 1: Forgot to add _MASK suffix
#define RES_COMMON_SURFACE (1ULL << 34)     // ✅ Exists
// ❌ Missing: #define RES_COMMON_SURFACE_MASK (1ULL << 34)

Error 2: Only copied the first line
#define RES_COMMON_TEXTURE (1ULL << 35)     // ✅ Exists
#define RES_COMMON_TEXTURE    // ❌ This line is wrong! Should be (1ULL << 35)

Skill Automatic Verification: This skill will automatically check if both definitions exist, and add them automatically if missing.

Verification Checklist

Check the following after adding a new tag:

musl main file: TAG is defined
musl main file: MASK is defined (both required)
musl porting file: TAG is defined
musl porting file: MASK is defined (both required)
proto: TraceType enum value added
proto: MemoryType enum value added
hook_common.h: Index defined
hook_common.h: RESTRACE_TYPE_COUNT updated
hook_guard.cpp: Mapping added
hook_manager.cpp: Conversion logic added
stack_preprocess.cpp: SaveMemTag and AddAllocStatistics added
hook_record.cpp: SetTraceType branch added
hook_client.cpp: Filter conditions updated (COMMON_* types only)
Tests: Test cases added

Constraints

Tag Naming: Uppercase letters and underscores only
Bit Allocation: Start from bit 34 (bit 33 is currently in use)
Index Allocation: Start from 23 (current maximum index is 22)
Enum Values: Auto-increment, no conflicts with existing values
Repository Path: Assumed to be
```
d:\Code\tools_develop\
```
RESTRACE_TYPE_COUNT: Must equal maximum index + 1

Notes

Compilation Order: Recompile after modifying musl so the profiler can use the new macro definitions
Protocol Compatibility: Regenerate C++ code after modifying protobuf
Test Coverage: Corresponding test cases are added for all modifications
COMMON_ Types*: Only types with the COMMON_* prefix require updating the filter conditions in
```
hook_client.cpp
```

Error Handling

Error	Solution
Tag already exists	Use a different tag name
File not found	Verify the repository path
Permission denied	Check file write permissions
Invalid format	Tag name must consist of uppercase letters and underscores
Missing MASK	Skill automatically adds the RES_*_MASK definition
Mismatched definitions	RES_* and RES_*_MASK must have the same value