From 6b4a3b6b411f22eb68272a9e56c7898d8f10b6c5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=D0=90=D0=BB=D0=B5=D0=BA=D1=81=D0=B0=D0=BD=D0=B4=D1=80=20?= =?UTF-8?q?=D0=9F=D0=B5=D1=82=D1=80=D0=BE=D0=B2?= Date: Sat, 1 Aug 2026 16:15:21 +0300 Subject: [PATCH] =?UTF-8?q?docs:=20=D0=B8=D1=82=D0=BE=D0=B3=20KBD-1=20?= =?UTF-8?q?=E2=80=94=20=D1=87=D1=82=D0=BE=20=D0=BB=D0=B5=D1=87=D0=B8=D1=82?= =?UTF-8?q?=20=D0=BF=D0=BB=D0=BE=D1=82=D0=BD=D1=8B=D0=B9=20=D0=BE=D0=BF?= =?UTF-8?q?=D1=80=D0=BE=D1=81=20=D0=B8=20=D1=87=D1=82=D0=BE=20=D0=BE=D1=81?= =?UTF-8?q?=D1=82=D0=B0=D0=BB=D0=BE=D1=81=D1=8C?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Ручная проверка пользователем: стало значительно лучше, но редкие пропуски стрелок при зажатом Shift всё же ощущаются. Счётчики на 35 нажатиях подряд потерь не показали, то есть остаточная частота заметно ниже прежних ~15 %. Задача отложена до финальной полировки программы (решение пользователя) — для работы клавиатура пригодна. Записано, где именно осталась дыра, чтобы не начинать с нуля: idle-хук покрывает простой (~2/3 кадра), а в занятой трети DI-окно одного accel-прохода доходит до ~650 мкс при допуске FIFO ~300 мкс — пачка байт, целиком попавшая в такое окно, ещё может потерять байт. Порядок действий на возврат: вызовы между блитами занятой фазы, замер тем же счётным методом от 50 нажатий, и только потом рычаги вне нашего кода (Scan Code Set 3 через BIOS $EA — в MAME непроверяемо; общий m_irq_off_timer в драйвере). Заодно сняты оговорки «плотный опрос ещё не подтверждён замером» в kbd_raw.h и libc-reference.md — теперь там штатный рецепт через gfx_set_idle_hook. Co-Authored-By: Claude Opus 5 --- applications/PoP/roomtest/TASKS.md | 26 ++++++++++++++++++++++++++ docs/TODO.md | 7 +++++++ docs/libc-reference.md | 19 +++++++++++++------ libc/include/kbd_raw.h | 20 ++++++++++++-------- 4 files changed, 58 insertions(+), 14 deletions(-) diff --git a/applications/PoP/roomtest/TASKS.md b/applications/PoP/roomtest/TASKS.md index aa2749a..a02e61e 100644 --- a/applications/PoP/roomtest/TASKS.md +++ b/applications/PoP/roomtest/TASKS.md @@ -13,6 +13,8 @@ ## P0 — делаем сейчас +*(KBD-1 закрыт до финальной полировки — см. ниже; следующая в работе — CLIP-1.)* + ### KBD-1. Shift + стрелки: нажатия теряются — **ПРИЧИНА НАЙДЕНА, ФИКС ПРОВЕРЕН** > **Итог (2026-08-01).** Причина — не наш код и не DI-окна графики: при @@ -27,6 +29,30 @@ > четыре Shift+→ подряд дали четыре осторожных шага, `Kid.x` 114 → 147. > Цена: `_CODE` +170 Б, кадровый бюджет не затронут (опрос стоит в > простое). Ниже — полный протокол, как к этому пришли. +> +> **ОСТАТОК (ручная проверка пользователем, 2026-08-01): «стало значительно +> лучше, но иногда при зажатом Shift стрелка всё-таки пропускается».** +> Ощущение, не замер — счётчики на 35 нажатиях подряд потерь не показали, +> значит остаточная частота заметно ниже прежних ~15 %. **Задача осознанно +> ОТЛОЖЕНА до финальной полировки всей программы** (решение пользователя); +> сейчас клавиатура пригодна для работы. +> +> **Где именно осталась дыра — чтобы на полировке не начинать с нуля.** +> Idle-хук покрывает простой, то есть ~2/3 кадра. Оставшаяся треть — это +> занятая фаза, и там DI-окно одного accel-прохода доходит до ~650 мкс при +> допуске FIFO ~300 мкс. Пачка байт, целиком попавшая в такое окно, всё +> ещё может потерять байт — ровно «иногда». Порядок действий, если +> вернёмся: +> 1. Вернуть вызовы `kbd_raw_poll()` между блитами занятой фазы (они +> бесплатны; сами по себе не помогали, но вместе с хуком закрывают +> именно этот зазор) и при необходимости внутрь тайловых циклов +> `pop_bg` — тогда слепым остаётся только тело одного блита. +> 2. Мерить тем же счётным методом (см. ниже), а не на ощупь: скриптовые +> нажатия ровнее человеческих, поэтому набирать выборку от 50 нажатий. +> 3. Если и это не добьёт — остаются два рычага вне нашего кода: Scan Code +> Set 3 через BIOS `$EA` (убирает «fake shift» в корне, но в MAME +> непроверяемо — обратный путь к клавиатуре не разведён) и общий +> `m_irq_off_timer` в драйвере MAME. **Симптом (пользователь, 2026-08-01).** Залипаний почти нет, но при УДЕРЖИВАЕМОМ Shift первые один-два нажатия ← дают осторожные шаги, дальше diff --git a/docs/TODO.md b/docs/TODO.md index ad01e12..6b694c9 100644 --- a/docs/TODO.md +++ b/docs/TODO.md @@ -301,6 +301,13 @@ Quick wins: подозреваемый — `m_irq_off_timer` в sprinter.cpp: он ОДИН на два источника (экран + клавиатура), `irq_off()` гасит обе линии, так что кадровое прерывание способно обрезать клавиатурный импульс. + **ЗАКРЫТО в рабочем объёме 2026-08-01:** лечится ПЛОТНЫМ опросом — + `kbd_raw_poll` повешен idle-хуком графики (`gfx_set_idle_hook`, + новый API libbgi) на ожидание кадра, где процессор всё равно + простаивает ~2/3 периода. 35 нажатий с зажатым Shift → 35 + дошедших make против 9 из 10 без хука. Пользователь на ручной + проверке отмечает, что редкие пропуски всё же ощущаются — остаток + отложен до финальной полировки, следующий шаг описан там же. Полный протокол — applications/PoP/roomtest/TASKS.md, KBD-1. Recovery-политика уже терпима к частым overrun'ам (селективный wipe, docs/kbd-games.md), но аккорды «держу →, тапнул ↑» и diff --git a/docs/libc-reference.md b/docs/libc-reference.md index f5743e3..67db660 100644 --- a/docs/libc-reference.md +++ b/docs/libc-reference.md @@ -351,16 +351,23 @@ FIFO SIO — 3 байта, и рассчитывать на «каждый ба (чтение порта 0x18 деструктивно, а read-modify-write карты гонится с трамплином) и БЕЗУСЛОВНО делает `EI` на выходе — из ISR звать нельзя. -**Замер эффективности (2026-08-01, PoP roomtest — читать до применения!).** +**Как её звать (замеры 2026-08-01, PoP roomtest — читать до применения!).** Расстановка «несколько вызовов за кадр, после тяжёлых фаз» **не даёт -ничего**: 9 дошедших make-байт из 10 нажатий и с ней, и без неё. Причина — -такие вызовы попадают ровно в участки с разрешёнными прерываниями и лишь +ничего**: 9 дошедших make-байт из 10 нажатий и с ней, и без неё — такие +вызовы попадают ровно в участки с разрешёнными прерываниями и лишь дублируют трамплин. Там же измерено, что длина DI-окон графики на потери НЕ влияет (в кадре вообще без блитов потерь больше), а теряется байт ДО чтения порта: примерно 44 % импульсов запроса прерывания не обслуживаются, -и трёхбайтовый FIFO переполняется. Полный протокол и куда копать — -`applications/PoP/roomtest/TASKS.md`, задача KBD-1. Смысл может быть -только у ПЛОТНОГО опроса (в цикле ожидания кадра), это ещё не проверено. +и трёхбайтовый FIFO переполняется. + +**Работает только ПЛОТНЫЙ опрос — порядка раза в 0.5 мс.** Штатный способ +взять эту частоту даром — повесить функцию idle-хуком графики: +`gfx_set_idle_hook()` (``) зовёт её, пока `gfx_wait_vsync` крутит +опрос луча, а это ~2/3 периода кадра. Проверено: 35 нажатий стрелки с +зажатым Shift → 35 дошедших make против 9 из 10 без хука. Остаточные +редкие потери возможны (пачка целиком внутри DI-окна одного accel-прохода); +полный протокол и что делать дальше — `applications/PoP/roomtest/TASKS.md`, +задача KBD-1. **ГЛАВНОЕ СЛЕДСТВИЕ:** пока `kbd_raw_open()` активен, `kbhit/getch/getkey/ kbd_mod_state` НЕ получают новых событий (в т.ч. CTRLKEY подряд отдаёт diff --git a/libc/include/kbd_raw.h b/libc/include/kbd_raw.h index e359769..e6bdf07 100644 --- a/libc/include/kbd_raw.h +++ b/libc/include/kbd_raw.h @@ -68,14 +68,18 @@ void kbd_raw_sync(void); * этом переполняют FIFO: теряется make («нажатие не сработало») или break * (залипание). * - * ЧЕСТНАЯ ОГОВОРКА ПО ЭФФЕКТИВНОСТИ (замер 2026-08-01, PoP roomtest, - * см. applications/PoP/roomtest/TASKS.md, KBD-1): расстановка «несколько - * вызовов за кадр, после тяжёлых фаз» НЕ ДАЁТ НИЧЕГО — потери те же, что - * без неё. Такие вызовы попадают в участки, где прерывания и так - * разрешены, и лишь дублируют трамплин. Смысл появляется только у - * ПЛОТНОГО опроса (в цикле ожидания, десятки-сотни раз за кадр), и это - * ещё не подтверждено замером. Не ставить эту функцию в игровой цикл - * «на всякий случай» — сначала померить. + * КАК ЕЁ ЗВАТЬ (замеры 2026-08-01, PoP roomtest — см. + * applications/PoP/roomtest/TASKS.md, KBD-1): + * - «несколько вызовов за кадр, после тяжёлых фаз» НЕ ДАЁТ НИЧЕГО — + * потери те же, что без них: такие вызовы попадают в участки, где + * прерывания и так разрешены, и лишь дублируют трамплин; + * - работает только ПЛОТНЫЙ опрос, порядка раза в 0.5 мс. Столько + * времени есть даром в ожидании кадра, поэтому штатный способ — + * повесить эту функцию idle-хуком графики: + * gfx_set_idle_hook(my_poll_wrapper); // + * Проверено: 35 нажатий стрелки с зажатым Shift → 35 дошедших make + * против 9 из 10 без хука. + * Не ставить в игровой цикл «на всякий случай» — сначала померить. * * Тело идёт под DI и БЕЗУСЛОВНО делает EI на выходе: рассчитано на вызов * из главного цикла, из ISR звать нельзя. */