fix(ncm): 歌单详情LoadFailed——track/all端点不存在(404)回退detail+分批song_detail已验证链路; 列表误触改按下锁定行+拖动抑制 (agent-227, 真机验证256首出曲) + perf(meowpanel): 屏幕帧整数定点DCT(无FPU软浮点3.96s→0.19s/帧,实测4-5fps)+前端链式轮询防堆积+实测fps显示 (agent-228)
ncm→73422166 已部署; meowpanel→f310a00e 已部署
This commit is contained in:
MeowOS committed
1 parent
d394c73a7c
commit
c041facd89
13 files changed
+447
-65
No files matched your search
+141
-1
@@ -782,7 +782,147 @@ c3a4aa5cd3da493d5cdb75965aaaa4fe app/ui/playlist.c 610a223232d250d2e04130
|
||||
- **topbar 图标字符潜在同病**：home/player topbar 动作标签 `⟳`(U+21BB)/`⚙`(U+2699)/`⌕`(U+2315) 走
|
||||
`meow_font_class_of`→OTHER 类，若 common 图集无对应字形同样会豆腐块；用户未报（可能在 topbar 用了符号字库槽或未留意），
|
||||
本期未动（避免改非报点区域），后续如需一并确认/替换（如改用 ASCII 或 in-atlas 字符）。
|
||||
- **`song_ids[]` 字段现未被歌单加载使用**（`track_all` 直返歌曲对象），字段保留在 `g_st`（无害，避免结构改动）。
|
||||
- **`song_ids[]` 字段曾在 §12 被闲置**（`track_all` 直返歌曲对象）；§13 已回退 detail+song_detail，重新用于
|
||||
暂存 `playlist/detail` 返回的 trackIds 以驱动 `song_detail` 分批（§13 为此更新）。
|
||||
- **歌词竞态**：UI 读 lyric 与 worker 写保留良性竞态（§10.9 FM4 同接受）；真机如需彻底消除可后续给 lyric 模块加锁。
|
||||
- **构建方差异**：宿主 `build_ncm_host.sh` 对 `cover.c` 有 glibc-fortify `-Wformat-truncation` 告警（**改动前即存在**，
|
||||
仅宿主 glibc 触发；mipsel musl 无此告警，`build_ncm.sh` 0 warning）。
|
||||
|
||||
---
|
||||
|
||||
## 13. 真机两修：歌单详情 Load Failed 根因（track/all 端点不存在）+ 歌单列表误触（2026-08-31，已真机闭环）
|
||||
|
||||
> 现象（用户真机验收）：① 登录/主页 64 歌单正常，但**进任意歌单都 Load Failed**——§12.3 新换的
|
||||
> `/api/playlist/track/all` 分页端点从未真机验过，真机响应字段存疑（§12.8 已自标风险）；
|
||||
> ② **偶发误触**：歌单列表点到非触摸坐标的行（列表触摸坐标有偏差/串行）。
|
||||
> 本批先真机用最小请求量抓 track/all 真实响应样本、对齐结论，再修复并用真机验证。仅改 `patch_v2/ncm_app/`，
|
||||
> 未动 sdk/、meow_launcherd/、vendor/；未 commit/push/刷机；未动设备系统/恢复区。
|
||||
|
||||
### 13.1 根因①：`/api/playlist/track/all` 端点根本不存在（404「接口未找到」）
|
||||
|
||||
按 §12.8 的“真机抓真实响应样本对齐”计划：设备侧链路（`bench/probe_track_all.c`，用与 App 相同的
|
||||
`ncm_req_build`+`ncm_http_request`）对小歌单 `18206253175`（Docu/1 首）抓**原始响应体**。为把“端点是否存在/响应字段”
|
||||
与“解析对错”解耦，探针**绕过解析器**直发请求，先打原始 body。三次（风控 ≤3）真实请求结果：
|
||||
|
||||
| 路由 | 加密 | URL | 原始响应 |
|
||||
|---|---|---|---|
|
||||
| `/api/playlist/track/all` | eapi | `https://interface.music.163.com/eapi/playlist/track/all` | `{"code":404,"message":"接口未找到！"}` |
|
||||
| `/api/v6/playlist/track/all` | weapi | `https://music.163.com/weapi/v6/playlist/track/all` | `{"code":404,"message":"接口未找到！"}` |
|
||||
| `/api/playlist/track/all` | weapi | `https://music.163.com/weapi/playlist/track/all` | `{"code":404,"message":"接口未找到！"}` |
|
||||
|
||||
→ **NCM 服务端没有 `track/all` 这条路由**（eapi/weapi、v6/无v6 全 404）。三次请求 body 均为
|
||||
`{"code":404,"message":"接口未找到！"}`（HTTP 200 但业务 code=404），故 `api_post` 判
|
||||
`NCM_ERR_HTTP(-10)` → `playlist_worker` 置 `g_load_err` → 任意歌单都 Load Failed。
|
||||
§12.3 把歌单详情从“验证过的 `/v6/playlist/detail` + `song_detail`（§10.11）”换成基于一条不存在的路由做“服务端分页”，
|
||||
是回归根因；`ncm_api_parse_track_all` 的“多路径容错”救不了**路由不存在**——收到的是 404 业务码，不是歌曲数组。
|
||||
|
||||
### 13.2 根因②：歌单列表误触 = 拖动滚动后 release 被 SDK 判 TAP，命中“释放点+新 scroll”而非按下行
|
||||
|
||||
先验证“触摸坐标与显示是否几何不一致”（任务提示的方向）。设备侧 `bench/probe_input.c`（本地 ioctl，零 NCM 请求）：
|
||||
```
|
||||
ABS_MT_POSITION_X min=0 max=480 ABS_MT_POSITION_Y min=0 max=720
|
||||
fb0 xres=480 yres=720 bpp=16
|
||||
```
|
||||
→ 触摸面板与帧缓冲 **1:1**（无缩放/无固定偏移），故“坐标有偏差”不是面板标定/缩放问题，App 侧命中公式
|
||||
（`w->y + w->scroll` 与绘同式）也一致——命中数学本身是对的。
|
||||
|
||||
真正根因是**手势时序**：SDK 抬手判 `adx/ady < 65px`（`SWIPE_THRESH`）为 TAP；而拖动过程中 `touch_move` 已
|
||||
`meow_listview_scroll_by(-dy)` **滚动列表**。于是“小拖动（0..64px）= 列表已滚 + release 仍触 TAP”，
|
||||
`playlist_on_tap`/`home_on_tap` 用 `meow_listview_hit(释放x, 释放y)` = **“释放点 + 更新后的 scroll”**命中，
|
||||
打到的是“已经滚动到手指下方”的那一行，与用户**按下**的行不一致 → “点到非触摸坐标的行”（偶发，手指微动即触发）。
|
||||
（§12.5 宿主离屏 `MEOW_UI_DUMP` 只验静态渲染、不发真实触摸，故未暴露。）
|
||||
|
||||
### 13.3 修法
|
||||
|
||||
- **Load Failed**（`app/logic/appstate.c`）：`appstate_load_playlist_songs_page` 回退 §10.11 已验证路径——
|
||||
`/api/v6/playlist/detail`（`n=1`，取前 `NCM_SONG_MAX` 首 trackIds）+ `/api/v3/song/detail` 分批取歌曲对象
|
||||
（`parse_songs_array` 仍带出 `al.picUrl`，保留 §12.1 专辑封面）。`offset==0` 重头整批载入（idn≤NCM_SONG_MAX）；
|
||||
`offset>0`（触底“加页”）只要 `NCM_OK` 无操作——detail 一次已取满至上限，无独立分页。
|
||||
- **误触**（`app/ui/home.c` 与 `app/ui/playlist.c`）：点行改用**按下行** `g_press_row`（touch_down 命中行，
|
||||
与“红条按压”同源）；手指移动量 `g_press_delta > 10`（与按压红条清行同阈值）判为**滚动收尾**，不触发点行。
|
||||
干净点击仍播放按下行（更符合“按哪行播哪行”）；拖动手势只滚动不再误播。`search.c` 同款模式，
|
||||
非本次报点，未改（避免扩范围，见 §13.8）。
|
||||
- 仅在 `patch_v2/ncm_app/`；`app/`/`core/` 均未动其它逻辑；`ncm_api_playlist_track_all`/`ncm_api_parse_track_all`
|
||||
保留在 core（现为不可达死代码；`offline_track_all`/`test_track_all_parse` 仍可作离线回归）。
|
||||
|
||||
### 13.4 真机验证记录
|
||||
|
||||
**触摸=fb**（probe_input，本地 ioctl）：`ABS_MT[0,480]×[0,720]`，`fb0 480×720 16bpp` → 1:1，排除标定。
|
||||
|
||||
**修复后数据链路**（`bench/probe_fix.c`，复刻修复后 `appstate_load_playlist_songs_page(offset=0)` 的
|
||||
detail→song_detail 完整路径；device_id=`BD6DE5…`, cookie_len=1842）：
|
||||
```
|
||||
=== small (Docu/1, 18206253175) ===
|
||||
detail rc=0 elapsed_ms=1175 idn=1 ids[0]=2048962657
|
||||
song off=0 batch=1 rc=0 elapsed_ms=717 bn=1
|
||||
RESULT song_count=1
|
||||
[0] Verve (feat. FJØRA) / Ryan Taubert / Verve (feat. FJØRA)
|
||||
FIX VERIFY OK
|
||||
=== big (我喜欢的音乐, 581857874, 5191) ===
|
||||
detail rc=0 elapsed_ms=34672 idn=256 ids[0]=36190432
|
||||
song off=0/64/128/192 batch=64 rc=0 bn=64 ×4
|
||||
RESULT song_count=256
|
||||
[0] Isla Blanca (Aerosoul Chillout Remix) / Aerosoul / Isla Blanca
|
||||
FIX VERIFY OK
|
||||
```
|
||||
→ 小歌单 1 首、大歌单 256 首（`NCM_SONG_MAX` 截断，§10.9 上限）**真机都出曲目，不再 Load Failed**。
|
||||
大歌单 detail 真机 ~34.7s（5191 首 trackIds ≈900KB），但由 `playlist_worker` 后台跑、UI 显示 LOADING 不冻结（§10.11 P1 已修）。
|
||||
|
||||
**部署二进制冒烟**（`ncm.new`→`mv`，起 8s 再 SIGTERM）：
|
||||
```
|
||||
start → home: playlists rc=0 count=120 uid=408996274 → exit
|
||||
```
|
||||
→ 新二进制启动正常、主页加载 120 歌单、干净退出，无崩溃。
|
||||
|
||||
**请求量**：track/all 诊断恰 3 次（风控 ≤3）；修复后验证走 detail+song_detail（验收必需）。
|
||||
真实请求总计数：诊断 3（track/all）+ 复验 7（small detail/song 2 + big detail/4×song 5）+ 冒烟 1（user/playlist）= **≈11 次**，
|
||||
均为必要开销。另有一次无效运行（data_dir 误传致 TLS 失败），未发服务器请求。
|
||||
|
||||
### 13.5 真机部署
|
||||
|
||||
`/usr/data/mnt/sd_0/MeowAPPs/dev.player.ncm/ncm`（先 `ncm.new` 再 `mv`；`ncm.bak2`=§12 旧回归二进制，
|
||||
`ncm.bak`=§10.11 前）。**新 md5 `734221661ce547393c03ae3568ca3f36`**（旧 §12 `9c4a79f669967d193cba9fdaa41f8442`）。
|
||||
|
||||
### 13.6 出入清单（只改 `patch_v2/ncm_app/`）
|
||||
|
||||
```
|
||||
修改：
|
||||
app/logic/appstate.c # track_all 404 回归：appstate_load_playlist_songs_page 回退 detail+song_detail
|
||||
app/ui/playlist.c # 误触：点行用按下行 g_press_row + 拖动(g_press_delta>10)抑制
|
||||
app/ui/home.c # 误触（同 playlist.c）：点行用按下行 + 拖动抑制
|
||||
新增：
|
||||
bench/probe_track_all.c # track_all 原始响应探针（诊断 404；抓样本用）
|
||||
bench/probe_input.c # 触摸 ABS 范围 + fb0 分辨率探针（本地 ioctl，零 NCM）
|
||||
bench/probe_fix.c # 修复后数据链路真机复验（detail→song_detail）
|
||||
REPORT.md # 本批记录（§13）
|
||||
（未动 sdk/、meow_launcherd/、vendor/；未 commit/push/刷机；未动设备系统/恢复区）
|
||||
```
|
||||
|
||||
### 13.7 md5（本批改动文件 + 产物）
|
||||
|
||||
```
|
||||
734221661ce547393c03ae3568ca3f36 ncm # mipsel 静态 App（新；已部署设备）
|
||||
453eeb5d29809854061d8e4e0300855a app/logic/appstate.c
|
||||
2ce92315d0e27492b36374f512fabaeb app/ui/playlist.c
|
||||
4217763fbf766c77917f86f63abb634c app/ui/home.c
|
||||
279ebbcfca65f9225b93b04213f8fde4 bench/probe_track_all.c 7dc88e448fb10a6da24ad48511023307 bench/probe_input.c
|
||||
5ad3107059bca1f8e14776afdf6abf82 bench/probe_fix.c
|
||||
```
|
||||
|
||||
### 13.8 诚实出入 / 已知限制
|
||||
|
||||
- **`track/all` 不存在使 §12.3 的整体翻页目标落空**：`/api/playlist/track/all`（eapi/weapi、v6/无v6）真机全 404，
|
||||
故“服务端分页 + 第 256 首之后继续加载”无法实现。本期回退为 §10.11 的 detail（全量 trackIds，浏览器首屏 ~34.7s，
|
||||
UI 后台 LOADING 不冻结）+ `NCM_SONG_MAX=256` 截断。若仍要超过 256 首滚动加载，只能改用 detail 已返回的全量
|
||||
trackIds 驱动 song_detail 分页（不依赖 `track/all`），列入后续；本期只恢复到“能出曲目”的可靠基线。
|
||||
- **`ncm_api_playlist_track_all`/`ncm_api_parse_track_all` 现为不可达死代码**（服务端无路由），保留以不破坏
|
||||
`offline_track_all.c`/`test_track_all_parse`（离线样本回归，0 网络）。后续可连同 §12.3 一并清理。
|
||||
- **误触修复未真机触摸复现**：无法经 SSH 驱动真实触摸；修复基于「按下行 + 拖动抑制」的明确逻辑 + 触摸=fb 1:1 的
|
||||
实测佐证（排除标定）。建议用户真机复验：快速横滑列表后抬手，应只滚动不误播；干净点行仍播按下行。
|
||||
- **`search.c` 列表点行同款误触模式**（未报点，本期未改，避免扩范围）：`search_on_tap` 仍用 `meow_listview_hit(释放点)`；
|
||||
如需一并修，套用 §13.3 同名改法即可。
|
||||
- **大歌单首屏 ~34.7s**：detail 全量 trackIds（≈900KB）在 MIPS32 上解析慢（§10.11 已记录，宿主 4.7s，~7×）；后台 worker
|
||||
避免冻 UI。已恢复到 §10.11 用户接受的基线。
|
||||
- **`al.picUrl` 封面保留**：song_detail 经 `parse_songs_array` 解析出 `al.picUrl`，§12.1 黑胶真专辑封面功能不退。
|
||||
- **构建**：`build_ncm.sh` mipsel 自有代码 `-Wall -Wextra` 0 warning（本批改动文件逐文件复核无告警）；vendor 豁免。
|
||||
- **未动 sdk/、meow_launcherd/、vendor/，未 commit/push/刷机，未动设备系统/恢复区**；部署留用户后续验收。
|
||||
Reference in new issue
Block a user