feat(ncm): 三期第一批 — 歌单真分页(取消256截断)+专辑封面ncm_cache+歌词后台加载补锁
[agent-239] 仅改 patch_v2/ncm_app/。 1. 歌单分页(REPORT §14.1): track/all 全 404(§13.1) 弃用, 回 detail(n=1)全量 trackIds + song_detail 分批(≤NCM_SONG_BATCH=100) 触底逐页加载。appstate_load_playlist_songs_page offset=0 重建 song_ids[]/track_ids_count 载第0页, offset>0 追加(失败保留已载); playlist.c g_pl_more=(song_count<track_ids_count && <NCM_SONG_MAX), 触底 g_ploffset=song_count 起下一页(保留滚动), 首屏失败整页空态+点按重试, 追加失败 snack+保留已载。NCM_SONG_MAX 256→1000(单页渲染上限, bss +~1MB), NCM_SONG_BATCH 64→100, 删 NCM_PLAYLIST_PAGE(track/all 遗留)。 2. 专辑封面(§14.2): cover.c 封面改走 ncm_cache(LRU, 键=djb2(url)→song_cov<h>.jpg 同池), 缓存命中免下载/未命中下载→open/write/commit 落盘, 仍写 cover_current.jpg 供 launcher; 顺带修真 bug: save_cover_raw 在已持 g_cm 临界区内再 lock(非递归死锁)→封面永不刷新, 改调用方持锁。 歌单列表/详情头块封面保持现状。 3. 歌词后台加载(§14.3): lyric_worker 已异步(§11)未改线程模型; 补 lyric.c g_lyric_mu 数据竞争锁, 写侧 lyric_parse 整段持锁(清空用 lyric_clear_nolock 免二次取锁), 读侧 lyric_count/line_at/ line_text/line_tl 逐次取锁, 消除 UI/IPC 与 worker 间数据竞争(原 §10.9 FM4 良性竞态)。 验证(0 真实 NCM 请求): bench/pagination_sim.c 离线 stub 驱动真实 appstate 分页函数全 PASS (130首读全程/5191首封顶1000/空/越界无操作); mipsel build_ncm.sh 0 warning(text 842820/bss 2052068); 宿主 build_ncm_host.sh+MEOW_UI_DUMP home/playlist/player/lands 渲染正常无崩溃; host SDK_SRC 补 meow_pinyin(拼音IME py_predict 漏编致链接断)。ncm md5 b34bc1ed, ncm_host md5 1a2327e0。 未动 sdk/(主会话改 meow_ui_ime.c)/meow_launcherd/meow_amsd/vendor/core; 未 commit 它们。
This commit is contained in:
MeowOS committed
1 parent
5fac5e380d
commit
ee104f1b18
10 files changed
+471
-87
No files matched your search
@@ -926,3 +926,127 @@ start → home: playlists rc=0 count=120 uid=408996274 → exit
|
||||
- **`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/刷机，未动设备系统/恢复区**；部署留用户后续验收。
|
||||
|
||||
|
||||
---
|
||||
|
||||
## 14. 三期第一批：歌单真分页 + 专辑封面 ncm_cache 化 + 歌词后台加载补锁（2026-08-31，agent-239）
|
||||
|
||||
> 范围：仅 `patch_v2/ncm_app/`（含 build 脚本与 REPORT 本文件）。未动 `sdk/`（主会话在改 `meow_ui_ime.c`）、
|
||||
> `meow_launcherd/`、`vendor/`、`core/`。风控纪律：本批真实 NCM 请求 **0 次**（全部离线 stub/宿主导出/已落盘样本）。
|
||||
> 部署/验收留用户（真机验收点见 §14.7）。
|
||||
|
||||
### 14.1 歌单分页（detail 全量 trackIds + song_detail 分批，取消 256 硬截断，分页驱动）
|
||||
|
||||
- **背景**：§13.1 真机确认 `/api/playlist/track/all`（eapi/weapi、v6/无v6）全 404，§12.3 的“服务端分页”不可行；
|
||||
§13 回退到 detail（`n=1`）+ song_detail 但 `NCM_SONG_MAX=256` 硬截断、一次拉满（首屏 34.7s）。本期把
|
||||
“256 截断 + 一次拉满”改为“**detail 全量 trackIds 驱动 + song_detail 分批 + 触底逐页加载**”。
|
||||
- `app/logic/appstate.c` `appstate_load_playlist_songs_page(idx,offset,limit)` 重写：
|
||||
- `offset==0`（重头）：detail(`n=1`) 取全量 trackIds 入 `song_ids[]`（≤ `NCM_SONG_MAX`），记 `track_ids_count`；
|
||||
`song_count`/`songs[]` 清空；只载**第 0 页**（song_detail 前 `limit` 首，其余待触底再拉，不冻 UI、不一次拉满）。
|
||||
- `offset>0`（触底追加）：从 `song_ids[offset..]` 载 song_detail，追加写 `songs[offset..]`，
|
||||
`song_count = offset + 本页实际数`；**失败返回不修改 song_count**（保留已载页）。`offset>=track_ids_count` 或
|
||||
`offset>=NCM_SONG_MAX` 时无操作返回。
|
||||
- `track_ids_count` 为“是否还有下一页”唯一判据（`song_count < track_ids_count`）。
|
||||
- `app/ui/playlist.c`：`playlist_worker` 按页（`NCM_SONG_BATCH`）调 `appstate_call`；`g_pl_more =
|
||||
(rc==NCM_OK) && song_count<track_ids_count && song_count<NCM_SONG_MAX`；`pl_maybe_load_more()` 视口触底 →
|
||||
`g_ploffset = g_st.song_count` 起下一页（append 成功保留滚动位置防跳顶）；**首屏失败**整页空态 + 点按重试；
|
||||
**追加失败** snack + 保留已载歌曲 + 重置 `g_pl_more`（可再触底/点击重试）。
|
||||
- 加载/失败/空态三态：`LOADING 正在加载曲目...` / `LOAD FAILED·SESSION EXPIRED·NETWORK ERROR（点按重试）` / `NO TRACKS`。
|
||||
点按重试：`playlist_on_tap` 在 `g_load_err!=0` 时命中内容区触发 `playlist_refresh()`。
|
||||
- **上限：`NCM_SONG_MAX` 256→1000**（单页渲染上限 = 分页驱动；每首 `ncm_song_t` ≈1.32KB → 歌曲数组 ~1.37MB，
|
||||
bss +~1MB，14MB 内安全）；`NCM_SONG_BATCH` 64→100（≤500 安全窗内取保守 100；`clist[4096]` 装 100 首 ≈2KB，
|
||||
如需 200 首需先扩 clist）；删除 `NCM_PLAYLIST_PAGE`（track/all 遗留，已无引用）。footer 显示 `已载/总数`（总数取
|
||||
`playlists[..].track_count`，如 5191 歌单读到 1000 时显示 `1000/5191`）。
|
||||
|
||||
### 14.2 专辑封面（黑胶用真实专辑图 + 缓存进 ncm_cache LRU + 修一个真死锁）
|
||||
|
||||
- `ncm_song_t.al_pic` + `parse_songs_array` 解析 `al.picUrl`（§12.1 已落地，本期未改 core）；player 黑胶盘封面取自
|
||||
`s->al_pic[0] ? s->al_pic : g_st.cover_url`（§12.1），**歌单列表/详情头块封面保持现状**（`cover_get()`，未改）。
|
||||
- `app/logic/cover.c` 封面图改走 **ncm_cache**（LRU 口径沿用 ncm_cache）：键 = djb2(封面 URL) → 文件
|
||||
`song_cov<hex>.jpg`（与 mp3 `song_<id>.mp3` 同池，`ncm_cache_lookup`/`commit` 刷新 mtime=LRU）。
|
||||
worker 流程：① 缓存命中直接解码（免下载、二次进入离线可看）；② 未命中 → 下载解码 → `ncm_cache_open/write/commit`
|
||||
落盘；仍写 `cover_current.jpg`（`save_cover_raw`）供 launcher 媒体卡片。
|
||||
- **顺带修真 bug**：旧 `save_cover_raw` 在已持 `g_cm` 的临界区内再 `pthread_mutex_lock(&g_cm)`（非递归锁自锁死）→
|
||||
cover worker 永久卡死、`g_gen` 永不 +1 → 黑胶封面**永不刷新**。改为“调用方持有 `g_cm`”（不再二次取锁）；
|
||||
封面写盘缓冲扩到 `NCM_CACHE_DIR_MAX+32`（顺带消宿主 glibc fortify `-Wformat-truncation` 告警，mipsel musl 本就无）。
|
||||
|
||||
### 14.3 歌词后台加载（确认已异步；补数据竞争锁）
|
||||
|
||||
- 歌词后台加载 §11（`player_manager.c` `lyric_worker`：轮询当前曲 → 异步 `ncm_api_lyric` → `lyric_parse`，
|
||||
UI 不阻塞；`lyric_state()` 判「加载中/无词」）**已落地**，本批未改后台线程模型（确实不阻塞 UI）。
|
||||
- 补 `app/logic/lyric.c` **数据竞争锁** `g_lyric_mu`：写侧 `lyric_parse` 整段持锁（清空用 `lyric_clear_nolock`
|
||||
避免二次取锁自锁死）；读侧 `lyric_count`/`lyric_line_at`/`lyric_line_text`/`lyric_line_tl` 每次取锁。
|
||||
消除 UI 线程（player.c 歌词区）与 IPC 线程（ipc_media.c `lyric_count`）同 lyric worker 间的数据竞争
|
||||
（早前为“良性竞态”§10.9 FM4；本期按“禁止数据竞争”要求收紧）。
|
||||
- 与现有 `g_play.mu` 同模式（函数级锁、无嵌套）。已知残余：worker 读 `g_st.songs[idx].id` 无锁——
|
||||
`songs[]` 写读竞态（§10.9 FM4 接受，与播放线程同源），本期未扩范围。
|
||||
|
||||
### 14.4 验证记录（0 真实 NCM 请求）
|
||||
|
||||
| 项 | 方式 | 结果 |
|
||||
|---|---|---|
|
||||
| 歌单分页状态机 | `bench/pagination_sim.c` 离线 stub（0 网络）驱动**真实** `appstate_load_playlist_songs_page` | **全 PASS**：130 首读全程（1 detail + 2 song_detail 页，`song_count=130/track_ids_count=130`，末首含 al_pic）；5191 首封顶 `NCM_SONG_MAX`（1000，10 页，越界无操作）；空歌单 0；越过尾部/上限再 load 均无副作用 |
|
||||
| mipsel 构建 | `bash build_ncm.sh`（自有代码 `-Wall -Wextra`） | **0 warning**；`text 842820 / data 856 / bss 2052068`（bss +~1MB 来自 NCM_SONG_MAX 1000） |
|
||||
| 宿主离屏渲染 | `build_ncm_host.sh` + `MEOW_UI_DUMP`（mock，0 网络） | home/playlist/player/lands 渲染正常（歌单 6 行 + footer `6/128`、黑胶 + 歌词当前行红色高亮），无崩溃 |
|
||||
| 宿主链接 | `build_ncm_host.sh` | 补 `meow_pinyin.o`（主会话 229 拼音 IME 引入 `py_predict`，旧 host SDK_SRC 漏编 → 链接断）；cover.c 截断告警已消 |
|
||||
|
||||
### 14.5 md5（本批改动文件 + 产物）
|
||||
|
||||
```
|
||||
b34bc1ed2c3539f1ca80a005884833cd ncm # mipsel 静态 App（新）
|
||||
1a2327e0ccd352d11bea88ebd35df549 ncm_host # 宿主离屏调试二进制
|
||||
0b0a4dd5e208d7b5eb504befb374a29e app/logic/appstate.c
|
||||
6d66eed99c556e301498c5823b43c294 app/logic/cover.c
|
||||
ace0def0e8c3ec991013b0675957f269 app/logic/lyric.c
|
||||
1a1871032c2ecae7a1cc80c7aa00381a app/ncm_app_internal.h
|
||||
ab87bd5f74b82fd3d62c49a16a4c9fe5 app/ui/playlist.c
|
||||
c73366373b3a372d96b2218e528c6381 build_ncm_host.sh
|
||||
141e9098a7cabeade9aa6b442ba20910 bench/pagination_sim.c
|
||||
```
|
||||
|
||||
### 14.6 出入清单（只改 `patch_v2/ncm_app/`）
|
||||
|
||||
```
|
||||
修改：
|
||||
app/logic/appstate.c # 歌单分页：offset=0 重建 trackIds 索引+载第0页；offset>0 追加；track_ids_count
|
||||
app/ui/playlist.c # 触底逐页加载 + g_pl_more 判据 + 加载/失败/空态 + 点按重试 + 追加失败保留已载
|
||||
app/ncm_app_internal.h # NCM_SONG_MAX 1000 / NCM_SONG_BATCH 100 / 删 NCM_PLAYLIST_PAGE + track_ids_count
|
||||
app/logic/cover.c # 封面走 ncm_cache(LRU) + 修 save_cover_raw 自锁死 + 缓冲扩容消截断告警
|
||||
app/logic/lyric.c # 加 g_lyric_mu（写 parse 整段持锁、读逐次持锁）消除数据竞争
|
||||
build_ncm_host.sh # host SDK_SRC 补 meow_pinyin（拼音 IME py_predict 漏编→链接断）
|
||||
ncm / ncm_host # 重编产物
|
||||
新增：
|
||||
bench/pagination_sim.c # 歌单分页离线状态机回归（stub 网络端点，0 请求）
|
||||
REPORT.md # 本批记录（§14）
|
||||
（未动 sdk/、meow_launcherd/、vendor/、core/；未 commit/push/刷机；未动设备系统/恢复区）
|
||||
```
|
||||
|
||||
### 14.7 诚实出入 / 已知限制
|
||||
|
||||
- **真机未部署/未验**（风控：0 新增真实请求）。部署链路同 §13.5（`scp ncm root@<ip>:/usr/data/mnt/sd_0/MeowAPPs/dev.player.ncm/ncm.new`
|
||||
→ `ssh root@<ip> '. /etc/profile; mv ...ncm.new ...ncm'`，先 `.new` 再 `mv` 防 ETXTBSY）。真机验收点：
|
||||
① 小歌单全程可读；大歌单（如 5191）下滑触底逐页加载到 1000 封顶（footer `已载/总数`），不冻 UI；
|
||||
② 黑胶显示真实专辑封面（首次下载→ncm_cache 落盘，二次进入免下载；第三方封面 `http://` 若握手失败回落歌单封面，
|
||||
为 §12.8 既有“http 封面走 TLS”旧疾，本批未改 core 规避）；③ 歌词后台加载不卡 UI、切歌刷新、下/上一首顺延。
|
||||
- **大歌单仍有内存上限（NCM_SONG_MAX=1000）**：这是 14MB 硬约束下的取舍（每 +100 首 ≈ +132KB）。
|
||||
若要全量 5000+ 首，需：a) 歌曲对象瘦身（`al_pic[512]` 每首占大头，可拆到按需加载），或 b) 列表滑动窗口
|
||||
（`meow_listview` 需 O(1) `src2(idx)` 重映射，复杂度高）。footer「已载/总数」已向用户标明进度。
|
||||
- **`ncm_cache_evict` 生产路径未显式触发超限淘汰**（§10.9 既有遗留：evict 仅 test 调用；song/cover commit 后不自动
|
||||
触发）。封面走 ncm_cache 同池后同样受该路径约束——**超限淘汰非本批引入**，本批也未扩大行为（不主动加 evict 调用，
|
||||
避免改已验证的缓存写入路径）。
|
||||
- **歌词锁残余**：读侧函数取锁后返回存量指针（`s_txt/s_tl` 静态数组），解锁后可能被下一次 `lyric_parse` 覆盖；
|
||||
写入仅发生在切歌瞬间（一次/歌），本地读在一两帧内完成，峰值可能显示一帧旧/空行（与 §10.9 FM4 同接受）。
|
||||
严格消除需调用方持锁拷贝（本批未做，接口需改，避免扩大 diff）。
|
||||
- **host 构建有 SDK `meow_ui_vol.c`「system 返回值忽略」告警**（SDK 源，非本 App；mipsel musl 无）。本 App 自有代码 0 warning。
|
||||
- **分页触底加载与误触/点按为逻辑推断**（无法经 SSH 驱动真实触摸/滚动）；误触修复（§13.3）未在分页下回归真机。
|
||||
建议真机快速下滑到底验证逐页加载、点按重试、追加失败保留已载。
|
||||
|
||||
### 14.8 数值权衡（供后续微调）
|
||||
|
||||
| 常量 | 旧值 | 新值 | 说明 |
|
||||
|---|---|---|---|
|
||||
| `NCM_SONG_MAX` | 256 | 1000 | 单页渲染上限（分页驱动），歌曲数组 ~1.32KB/首 → +100 ≈ +132KB bss |
|
||||
| `NCM_SONG_BATCH` | 64 | 100 | song_detail 每批（≤500 安全窗内保守 100；`clist[4096]` 装 100 首 ≈2KB） |
|
||||
| `NCM_PLAYLIST_PAGE` | 50 | 删除 | track/all 服务端分页遗留（该端点 404），改由 `NCM_SONG_BATCH` 承担页粒度 |
|
||||
| `g_st.track_ids_count` | — | 新增 | detail 取回 trackIds 总数，驱动「还有下一页」判据 |
|
||||
Reference in new issue
Block a user