fix(launcher): M47 r1 评审 P2 四项 — 未知态不写盘/不顶替 + deadline 从点击起算 + 开关弹窗后置清理
评审 r1 status=p2-4items / merge=fix-then-merge（reviewed e4eda3a）。逐条： P2-1 未知态被写成 settings=0，把失真推到别的入口 - bt_page.c / wifi_page.c：`meow_settings_set_bool(..., disp == 1)` 加 `disp >= 0` 门。 disp == -1（daemon 与直连都读不到）时**保留用户原意图不动**。该键被 notif_center / settings_page 当显示态读、被 app_launcher 用来决定自动拉起 WiFi，写 0 会把「不知道」 污染成「确定已关」。 - test_bt_page.c 5d 期望同步改：未知 → settings 保持原意图 1（不再期望「回滚为 0」）。 P2-2 收尾后「探测未知」仍显示成确定态，第三态丢失 - mtog_calc：显示态**只认 probe**，probe < 0 直接返回 -1；last_real 不再顶替确定态 （`(void)last_real`，仅由调用方用于弹窗文案）。超时/下发失败且探测未知 → 顶栏保留 「关+?」、行内保留「状态未知」，与文件头契约一致。 - 弹窗文案按结论区分：「无法确认当前状态（上次已知：开启/关闭/未知）」vs 「切换未生效：当前仍为开启/关闭」——不再出现「当前仍为开启」自相矛盾（该文案只在 probe 已知且 != 意图时出现）。日志加 elapsed。 P2-3 deadline 起点错，注释上限是假的 - bt_page/wifi_page：`deadline` 从**点击时刻**起算（含下发耗时）。 - 注释改成如实上限：快路径 1~2s / ≈16 次探测；慢路径（on-off 读超时 1.5s + hciconfig 0.3~2s，之后一次探测 status 1.5s + 直连 1.8s ≈3.3s）总阻塞最坏 ≈7s、预算内只够 1 次 探测，此时只能确认「探测超时」而非「设备无响应」——文案已按此区分。异步化留给后续单。 P2-4 开关路径弹窗后缺 after_modal / 按键基线复位 - subpage.c：顶开关命中分支补上与行命中分支相同的后置清理（排空触摸 + 40ms 二次排空、 触摸基线/cooldown、key_router_rebase、kd_*/kp_poll 基线、m_prev_*、after_modal=1）。 s_dd_modal 是全屏压暗层，否则关闭后状态栏(0..27)残留压暗到下一次时钟重绘，电源/播放键 也可能带进弹窗里的陈旧边沿。 顺带（评审观察，未改全局超时策略） - netmgr_ipc.c：拆出 `nm_cmd_t(..., int *read_timeout)`；`nm_bt_power_state()` 走它， **读超时 = 状态未知且不把可达性缓存打成 down**（netmgrd 单线程忙时 `bt status` 自己 跑 hciconfig 可能超 1.5s；旧行为会标 down 1s，令紧随的 bt_rows 闪出「服务未运行（直连 模式）」并把后续探测切到 1.8s 直连）。WiFi/DNS 等其它调用保持旧语义（传 NULL）。 验证 - `bash patch_v2/meow_launcherd/build_launcher.sh test` → exit 0，**65 PASS / 0 FAIL / 0 warning** （矩阵 13 组，新增「下发失败·探测未知 → 未知不顶替」）。 - 交叉编译 0 warning；产物（临时副本，不入库）901020 B / md5 e21450ff6621a67fd854165413c58be1。 - `test_time32_abi.sh` → 4263=3 / 4403=0 ALL PASS；`pkg_install` 与 7 条切换中/未知态文案 均在 strings 中（旧的两条「无法确认蓝牙/WiFi 状态」已合并为「无法确认当前状态（上次已知：%s）」）。 - 真机 --sysdump（只读，临时文件已删）：exit 0，bt_on=1/wifi_on=1 与设备现状一致； Music du 前后 199164544；未替换 /usr/bin/meow_launcherd、未重启 launcherd。 - 交付形态不变：本 commit 只动 5 个源码文件；二进制与 scripts/build/fw_content.conf 对 main 零 diff。
This commit is contained in:
AskaEth committed
1 parent
e4eda3a06f
commit
f49585dd62
5 files changed
+110
-37
No files matched your search
@@ -64,8 +64,13 @@ typedef struct {
|
||||
extern int nm_bt_power_state(void); /* netmgr_ipc.c：daemon 权威 → 直连探测，三态 */
|
||||
|
||||
/* 开关收敛参数（宿主测试可在 include 前改小，见 test_bt_page.c）。
|
||||
* 依据：本机慢——hciconfig 一次 0.3~2s，netmgrd 的 on/off 自身还带 ~1.5s 等待；
|
||||
* 250ms 间隔在 4s 内最多探 16 次，既不 20ms 级忙轮询，又能在 1s 内看到首次收敛。 */
|
||||
* 上限从**点击时刻**起算（评审 r1 P2-3），故它约束的是「从点击到出结论」的总时长：
|
||||
* - 快路径（daemon 正常，`bt status` 本地 IPC 几 ms）：4s 内约 16 次探测，通常 1~2s 收敛；
|
||||
* - 慢路径（netmgrd 卡住：on/off 读超时 1.5s → 直连 s_net_toggle_bt 的 hciconfig 0.3~2s；
|
||||
* 之后一次探测 = status 1.5s 超时 + hciconfig 兜底 1.8s ≈ 3.3s）：总阻塞最坏
|
||||
* ≈3.5s(下发) + 3.3s(最后一次探测) ≈ **7s**，预算内只够 1 次探测。此时能确认的只是
|
||||
* 「探测超时」，**不能**宣称「设备无响应」——弹窗文案按此区分（「无法确认当前状态」）。
|
||||
* 250ms 只是快路径的轮询间隔（不 20ms 级忙轮询）；慢路径由探测自身耗时决定。 */
|
||||
#ifndef BT_TOGGLE_SETTLE_MS
|
||||
#define BT_TOGGLE_SETTLE_MS 4000
|
||||
#endif
|
||||
@@ -123,20 +128,23 @@ static const char *bt_conn_name(const bt_ctx_t *c, const char *mac)
|
||||
* dispatch_ok(下发命令是否成功)、probe(有界等待后的探测结果，-1 未知)、
|
||||
* timed_out(是否已到等待上限)。
|
||||
* 输出（四个访问器共用同一套判定，拆开只为调用处可读）：
|
||||
* meow_toggle_display() 应显示的态（-1 未知 / 0 / 1）：有 probe 只认 probe；probe 未知才回退
|
||||
* last_real（上一次「已知」值），两者都未知 → -1。
|
||||
* meow_toggle_display() 应显示的态（-1 未知 / 0 / 1）：**只认 probe**。probe 未知 → -1，
|
||||
* 不许回退 last_real 冒充确定态（评审 r1 P2-2：从没被确认过的旧值
|
||||
* 一旦显示成确定态，就与本单「探测失败=未知」的契约自相矛盾）。
|
||||
* meow_toggle_pending() 1=还要继续等：下发成功、未超时、且 probe 未等于意图。
|
||||
* 注意「probe 未知」也算没收敛——绝不因为 last_real 恰好等于意图就收工。
|
||||
* meow_toggle_rollback() 1=意图未达成（显示态 != 意图）→ UI/settings 必须回滚到真实态。
|
||||
* meow_toggle_rollback() 1=意图未达成（显示态 != 意图）→ UI 必须回滚到真实态。
|
||||
* meow_toggle_error() 1=收尾后仍未确认 → 必须给用户可见的错误提示（不许静默失败）。
|
||||
* last_real 不参与状态判定：它只由调用方用于弹窗文案（「上次已知为开启，当前无法确认」），
|
||||
* 以及后续异步化时判断「回滚到哪」；保留在接口里是因为它是本单约定的输入维度。
|
||||
* 判定矩阵见 test_bt_page.c「收敛矩阵」；wifi_page.c 复用同一实现。 */
|
||||
static void mtog_calc(int intent, int last_real, int dispatch_ok, int probe, int timed_out,
|
||||
int *disp, int *pending, int *rollback, int *error)
|
||||
{
|
||||
int want = intent ? 1 : 0;
|
||||
int d = (probe >= 0) ? (probe ? 1 : 0)
|
||||
: ((last_real >= 0) ? (last_real ? 1 : 0) : -1);
|
||||
int d = (probe >= 0) ? (probe ? 1 : 0) : -1; /* 未确认 = 未知，绝不拿旧值顶替 */
|
||||
int confirmed = (probe >= 0) && ((probe ? 1 : 0) == want);
|
||||
(void)last_real; /* 仅弹窗文案用，见函数头注释 */
|
||||
*disp = d;
|
||||
*pending = (dispatch_ok && !timed_out && !confirmed) ? 1 : 0;
|
||||
*rollback = (d == want) ? 0 : 1;
|
||||
@@ -192,12 +200,13 @@ static int bt_top_sw(void *ctx, int *on)
|
||||
static void bt_top_sw_set(void *ctx, int on)
|
||||
{
|
||||
bt_ctx_t *c = ctx;
|
||||
unsigned long long t_click = now_ms(); /* 上限从点击起算（含下发耗时，见参数注释） */
|
||||
int want = on ? 1 : 0;
|
||||
int last_real = bt_state_now(c, 0);
|
||||
meow_settings_set_bool("network.bluetooth_enabled", want); /* 意图持久化 */
|
||||
meow_settings_save();
|
||||
int dispatch_ok = (nm_bt_onoff(want) == MEOW_OK);
|
||||
unsigned long long deadline = now_ms() + BT_TOGGLE_SETTLE_MS;
|
||||
unsigned long long deadline = t_click + BT_TOGGLE_SETTLE_MS;
|
||||
int probe = bt_state_now(c, 1), timed_out = 0;
|
||||
while (meow_toggle_pending(want, last_real, dispatch_ok, probe, timed_out)) {
|
||||
if (now_ms() >= deadline) { timed_out = 1; break; }
|
||||
@@ -207,22 +216,32 @@ static void bt_top_sw_set(void *ctx, int on)
|
||||
int disp = meow_toggle_display(want, last_real, dispatch_ok, probe, timed_out);
|
||||
int rb = meow_toggle_rollback(want, last_real, dispatch_ok, probe, timed_out);
|
||||
int err = meow_toggle_error(want, last_real, dispatch_ok, probe, timed_out);
|
||||
unsigned long long elapsed = now_ms() - t_click;
|
||||
c->bt_on = disp; c->bt_on_ts = now_ms();
|
||||
/* 回滚语义：settings 只留「已确认的真实态」——失败/超时不留一个骗人的意图值 */
|
||||
meow_settings_set_bool("network.bluetooth_enabled", disp == 1);
|
||||
meow_settings_save();
|
||||
/* 回滚语义：只在拿到**真实态**时才改写 settings（确认态或确认的相反态）。
|
||||
* disp == -1（探测未知）时保留用户原意图不动——本键被 notif_center / settings_page 当
|
||||
* 显示态读、被 app_launcher 用来决定自动拉起 WiFi，写 0 会把「不知道」污染成「确定已关」
|
||||
* （评审 r1 P2-1）。 */
|
||||
if (disp >= 0) {
|
||||
meow_settings_set_bool("network.bluetooth_enabled", disp == 1);
|
||||
meow_settings_save();
|
||||
}
|
||||
bt_role_refresh(1); /* 先刷新角色（它会写 g_sys.bt_on），再用真实态覆盖 */
|
||||
if (disp >= 0) g_sys.bt_on = disp;
|
||||
paint_sb_right();
|
||||
if (disp == 1) nm_bt_scan_on();
|
||||
logmsg("bt_page: toggle want=%d ok=%d probe=%d to=%d disp=%d rollback=%d err=%d",
|
||||
want, dispatch_ok, probe, timed_out, disp, rb, err);
|
||||
logmsg("bt_page: toggle want=%d ok=%d probe=%d to=%d disp=%d rollback=%d err=%d elapsed=%llums",
|
||||
want, dispatch_ok, probe, timed_out, disp, rb, err, elapsed);
|
||||
if (err) {
|
||||
char line[96];
|
||||
char line[128];
|
||||
const char *txt[1]; txt[0] = line;
|
||||
if (disp < 0) snprintf(line, sizeof line, "无法确认蓝牙状态（服务/控制器无响应）");
|
||||
else snprintf(line, sizeof line, "切换未生效：当前仍为%s（%dms 内未确认）",
|
||||
disp ? "开启" : "关闭", BT_TOGGLE_SETTLE_MS);
|
||||
/* 文案必须与结论同义：probe 未知 = 「无法确认」（不冒充未生效/未响应）；
|
||||
* probe 已知且 != 意图 = 确认未生效。last_real 只在这里出现。 */
|
||||
if (disp < 0)
|
||||
snprintf(line, sizeof line, "无法确认当前状态（上次已知：%s）",
|
||||
last_real < 0 ? "未知" : (last_real ? "开启" : "关闭"));
|
||||
else
|
||||
snprintf(line, sizeof line, "切换未生效：当前仍为%s", disp ? "开启" : "关闭");
|
||||
s_dd_modal("蓝牙", txt, 1, -1);
|
||||
}
|
||||
}
|
||||
|
||||
Reference in new issue
Block a user