fix(ncm): 启动卡顿+大歌单+No tracks 三连修 (agent-209) — ①home/playlist 加载改后台 worker ②大歌单 detail n=1(tracks瘦身)+响应cap 2MB ③song_detail c数组引号转义 ④P0: MIPS32 long=32位歌曲id溢出→int64 ⑤eapi批缓冲2048→4096
ncm→f6bd765a 已部署; 真机验证(风控合规): 小歌单✓ 大歌单256首✓ 启动不卡✓; 已知: 大歌单首屏~30s(二期改分页端点)
This commit is contained in:
MeowOS committed
1 parent
a651df0e9b
commit
4cec476dd9
16 files changed
+344
-36
No files matched your search
@@ -455,3 +455,100 @@ ed2d9658bc8707a5fd689b386da472f9 app/logic/cover.c e7cef918d2be04f6d4c5a
|
||||
- **`appstate_ensure_uid` 仍同步跑在 UI 线程**，首次（uid 未落盘时）会占 UI 至多 ~6s 硬上限（正常 ~0.7s）；
|
||||
真正的“不阻塞歌单页”可由「uid 落盘后短路」+「6s 硬上限」保证，未改为后台线程（避免范围扩散）。
|
||||
|
||||
|
||||
---
|
||||
|
||||
### 10.11 真机三问题修复：启动卡顿 + 大歌单 Load Failed + 小歌单 No tracks（2026-08-31，已真机闭环）
|
||||
|
||||
> 现象（用户真机实测）：登录通、歌单列表 64 个。三个问题：
|
||||
> ① 有登录态点开 App 卡数秒才进；②「我喜欢的音乐」(5191 首) 点开 Load Failed；③ 其余歌单全部 No tracks。
|
||||
> 本批定位多处独立缺陷并修复，全部经宿主复现 + 真机验证。仅改动 `patch_v2/ncm_app/`。
|
||||
|
||||
#### 根因（按现象对应）
|
||||
|
||||
- **P1 启动卡数秒**：`app/ui/home.c` `home_page_enter → home_reload → appstate_load_playlists`（weapi user/playlist）在 **UI 线程同步执行**；真机该 weapi 往返 ~2.5s+（verify_home: `elapsed_ms=2562`），uid 未落盘时还叠 status_check（6s 硬上限）。期间 UI 冻死（`paint_full` 不执行）。
|
||||
- **P2 大歌单 Load Failed**：`core/ncm_api.c:ncm_api_playlist_detail` 发 `{"id":..,"n":100000,"s":8}` + 响应 cap `NCM_RESP_MAX(1MB)`。5191 曲歌单 detail 真机实测 `status=200 cl=3168819`（3.0MB：`playlist.tracks` 塞满歌曲对象），1MB cap 溢出 → `NCM_ERR_FULL(-11)` → Load Failed。关键：**`trackIds` 数组与 `n` 无关**（n=1 时仍完整返回 5191 条，实测），是 `tracks`（完整歌曲对象）撑大响应。
|
||||
- **P3 小歌单 No tracks**：`ncm_api_song_detail` 把曲目 id 数组塞进 `{"c":"[...]"}` 时**内部引号未转义**，实发 `{"c":"[{"id":2048962657}]"}`（坏 JSON），服务端回 `200 Content-Length:0` → jsmn 空体 → `NCM_ERR_JSON(-9)` → 0 首。宿主复刻：转义后 `{"c":"[{\"id\":2048962657}]"}` → 服务端回 `2146B` 正常 songs。
|
||||
- **P0（连带新发现）**：网易云歌曲 id 可 >2^31（如 `2746301825`），而 App 用 `long` 存 song_ids；**MIPS32 的 long=32 位**，`strtol` 溢出饱和到 `LONG_MAX(2147483647)` → 大歌单 trackIds 全错（`2746301825` 被解析成 `2147483647`），song_detail 拿错误 id 只回 30 首。宿主 aarch64（long=64 位）不暴露，真机 MIPS32 才炸。
|
||||
- **P3b（连带）**：批量 song_detail 转义后 json 变长 → eapi `params[2048]` 溢出 → `ncm_crypto_eapi` 返 `NCM_ERR_CRYPTO(-8)`（batch=64 实测 rc=-8）。
|
||||
|
||||
#### 修法（全部在 `patch_v2/ncm_app/`）
|
||||
|
||||
- **P1** `app/ui/home.c`：加载改后台 worker（`home_worker`，detached）；`home_reload` 立即返回并画 LOADING；`home_tick` 按 `g_home_gen` 完成代数回填。`app_on_tick` 路由 `home_tick`。ensure_uid（status_check）+ user_playlist 全在 worker，UI 不阻塞。（顺带消掉 §10.10 遗留的「ensure_uid 同步跑 UI 线程」。）
|
||||
- **P2** `core/ncm_api.c:ncm_api_playlist_detail`：`n` 100000→**1**（`tracks` 只带 1 首，3MB→~900KB）；响应 cap 1MB→**2MB**（trackIds 大的歌单留余量）。
|
||||
- **P3** `core/ncm_api.c:ncm_api_song_detail`：`c` 数组内部引号转义（`{\\\"id\\\":%lld}`）。
|
||||
- **P0** `core/ncm_common.{h,c}` 新增 `ncm_parse_i64`（strtoll）；`core/ncm_json.{h,c}` 新增 `ncm_json_get_i64_at`；`core/ncm_api.h` 两处 id 签名 `long*`→`int64_t*`；`app/ncm_app_internal.h` `song_ids[]` long→int64_t。
|
||||
- **P3b** `core/ncm_req.c` eapi 局部缓冲 `full2[3072]→[4096]`、`params[2048]→[4096]`。
|
||||
- **防大歌单冻 UI**：`app/ui/playlist.c` 歌单详情同样后台化（`playlist_worker` + `playlist_tick`）；`appstate_load_playlist_songs` 不再覆写 `g_st.active_playlist`（防后台 worker 覆盖用户已切换的歌单）。
|
||||
|
||||
#### 验证记录
|
||||
|
||||
**宿主（同 core 库 + 真实服务器）**
|
||||
- `bench/host_chain_probe`（detail→song_detail 背靠背，小歌单 `18206253175`）：修前 `song_detail rc=-9`；修后 `rc=0 bn=1 song_count=1`。
|
||||
- `bench/host_big`（大歌单 `581857874`）：修前 big detail 溢出；修后 `detail rc=0 idn=256` + 4×64 批全 `rc=0` → `song_count=256`，首曲 `Chrono Rewind / CVRCH`。
|
||||
- 宿主 eapi 复刻：未转义 c → 服务端 `200 CL:0`；转义后 `200 len=2146`。**P3 钉死**。
|
||||
- `test/test_ncm_core_host` 全 PASS；`build_ncm.sh` mipsel `-Wall -Wextra` 0 warning。
|
||||
|
||||
**真机（`bench/verify_fixes`，一次运行，小歌单 2 请求 + 大歌单 5 请求）**
|
||||
```
|
||||
device_id=BD6DE5CC2DBCB617253A00438897DC6C02FFC7CFDB99DB0FA08E cookie_len=1842
|
||||
[small(Docu/1)] detail rc=0 idn=1 ids[0]=2048962657
|
||||
song off=0 batch=1 rc=0 bn=1 RESULT song_count=1
|
||||
[0] Verve (feat. FJØRA) / Ryan Taubert
|
||||
[big(5191)] detail rc=0 idn=256 ids[0]=2746301825 ← 修 P0 后正确（修前=2147483647）
|
||||
song off=0/64/128/192 batch=64 全 rc=0 bn=64
|
||||
RESULT song_count=256 [0] Chrono Rewind (时空回溯) / CVRCH
|
||||
```
|
||||
→ **P2/P3/P0/P3b 真机全消**：小歌单 1 首、大歌单 256 首（不再 Load Failed / No tracks）。
|
||||
|
||||
**真机启动（部署后后台运行 5s）**
|
||||
```
|
||||
6213257 [os.meow.ncm] INFO start
|
||||
6217019 [os.meow.ncm] INFO home: playlists rc=0 count=120 uid=408996274 ← 后台 worker 完成
|
||||
6218319 [os.meow.ncm] INFO exit
|
||||
```
|
||||
→ 启动即进 home；`home: playlists` 由 worker 与 UI 并发产出，UI 不再被 weapi 阻塞（**P1**）。修前旧二进制日志 `start→home playlists` 同链路为 UI 线程同步（`home.c` 阻塞）。
|
||||
|
||||
**MIPS32 溢出佐证（P0）**：修 P0 前真机 `verify_fixes`：`big detail idn=256 ids[0]=2147483647`（strtol 溢出饱和）；修后 `ids[0]=2746301825`。
|
||||
|
||||
#### 部署 + 出入清单（本批，只改 `patch_v2/ncm_app/`）
|
||||
|
||||
- 部署：`/usr/data/mnt/sd_0/MeowAPPs/os.meow.ncm/ncm`（先 `ncm.new` 再 `mv`），**新 md5 `f6bd765a374effaf866784c3165e476e`**（旧 `bf6d2a79504d49cdc9b87e6f34cc6aec`）。
|
||||
|
||||
```
|
||||
修改：
|
||||
core/ncm_api.c # P2 n=1+2MB cap；P3 c 转义；P0 int64 签名
|
||||
core/ncm_api.h # int64_t ids 签名
|
||||
core/ncm_req.c # P3b eapi 局部缓冲提升
|
||||
core/ncm_common.h/.c # 新增 ncm_parse_i64
|
||||
core/ncm_json.h/.c # 新增 ncm_json_get_i64_at
|
||||
app/app.c # app_on_tick 路由 home_tick/playlist_tick
|
||||
app/ncm_app_internal.h # song_ids→int64_t；声明 home_tick/playlist_tick
|
||||
app/ui/home.c # P1 home 加载后台化
|
||||
app/ui/playlist.c # 歌单详情后台化（防大歌单冻 UI）
|
||||
app/logic/appstate.c # load_playlist_songs 不覆写 active_playlist
|
||||
新增：
|
||||
bench/verify_fixes.c # 真机复验小歌单+大歌单链路（回归工具）
|
||||
REPORT.md # 本批记录
|
||||
（未动 sdk/、meow_launcherd/、vendor/；未 commit/push/刷机；未动设备系统/恢复区）
|
||||
```
|
||||
|
||||
变更文件 md5：
|
||||
```
|
||||
ceb89ee9e83752208056e2eadb5330fd core/ncm_api.c 63d4af4307c5cdb3922e59dc70116407 core/ncm_api.h
|
||||
cc1ab202c8d603e296f02715718a8a25 core/ncm_req.c 45efe7071b8cbff552ac89fe8b2e9495 core/ncm_common.c
|
||||
8fc658f656a1004f80ab5bc1555c2048 core/ncm_common.h df46e0cd3215d359025519b4d41316d6 core/ncm_json.c
|
||||
6d473afe13176230268c68ddc655cb34 core/ncm_json.h 3d2daea41204c887915d8c078543afe0 app/app.c
|
||||
282c24806fe0e9c97adf464f6a5c2776 app/ncm_app_internal.h 10c3b4b1097f1b0d82550659d0a87c4f app/ui/home.c
|
||||
0efef753b32eef144f2c30e4592fbfea app/ui/playlist.c 94d10237c1aa73f7273f82ff319792f5 app/logic/appstate.c
|
||||
bcfd75e5eb872a751c166c456eea0184 bench/verify_fixes.c f6bd765a374effaf866784c3165e476e ncm
|
||||
```
|
||||
|
||||
#### 诚实出入 / 已知限制
|
||||
|
||||
- **大歌单 load 慢（真机 detail ≈30s）**：`/v6/playlist/detail` 无论 `n`/`s` 都返回**全量 trackIds**（5191 个 ≈900KB，实测 n=1/0/100000 均 5191 条），无法用该端点分页/限流。真机 MIPS32 下载 + jsmn 解析 ≈30s（宿主 4.7s，~7× 慢，符合 MIPS vs aarch64 预期）。已用「playlist 后台化」避免冻 UI（显示 LOADING），但大歌单首屏仍 30s 级等待；要根治需改走 `/api/playlist/track/all`（服务端分页）或流式 jsmn——**建议二期**。
|
||||
- **`n` 只限 `tracks` 不影响 trackIds**：`n=1` 对 ≤256 曲歌单仍完整返回全部 trackIds（idn=实际曲数，无回归）；仅把「多出的完整歌曲对象」从响应里去掉，从而把 3MB 压回 ~900KB。
|
||||
- **曲目截断仍 `NCM_SONG_MAX=256`**（现协议）：大歌单显示前 256 首，未做滚动加载（二期）。
|
||||
- **`g_st.songs[]/song_count` 由 playlist worker 写、play/IPC 线程读（§10.9 FM4 旧已知风险）**：入播放页仍在同歌单，竞态窗口极小；沿用未加锁（与项目现状一致）。
|
||||
- **真机真实请求计数**：本批新增 ~20 次（detail_probe 抓样 1 次（5 请求，样本落盘）+ song_probe 1 + app_path_probe 2 + verify_fixes 2 次 11 + 启动验证 2）。已尽量「一次请求存样本、离线重放」，但修复前后复验需真实响应，属必要开销。
|
||||
- **未动 sdk/、meow_launcherd/、vendor/，未 commit/push/刷机，未动设备系统/恢复区。**
|
||||
Reference in new issue
Block a user