The branch main has been updated by adrian: URL: https://cgit.FreeBSD.org/src/commit/?id=4b3d035668211c19fc6fd5d4d207c2e8f96e7ad3
commit 4b3d035668211c19fc6fd5d4d207c2e8f96e7ad3 Author: Abdelkader Boudih <[email protected]> AuthorDate: 2026-07-08 05:30:44 +0000 Commit: Adrian Chadd <[email protected]> CommitDate: 2026-07-08 05:32:43 +0000 fwohci: fix LPS delay, PHY_INT storm, and SID timeout recovery Fixed the post-LPS delay from 500us to the IEEE 1394a-2000 s6.1 mandated 10ms ceiling. Handled PHY_INT by clearing W1C status bits in register 5 (masked ISBR to avoid spurious bus resets). Added a SID timeout callout that recovers the state machine when a remote device fails to complete self-ID. Fixed FW_PHY_SPD operator precedence and gated noisy messages behind bootverbose/firewire_debug. Reviewed by: adrian Differential Revision: https://reviews.freebsd.org/D58033 --- sys/dev/firewire/fwohci.c | 59 ++++++++++++++++++++++++++++++++++++++------ sys/dev/firewire/fwohcivar.h | 3 +++ 2 files changed, 54 insertions(+), 8 deletions(-) diff --git a/sys/dev/firewire/fwohci.c b/sys/dev/firewire/fwohci.c index eeb35bb94b42..245edb1f8fe4 100644 --- a/sys/dev/firewire/fwohci.c +++ b/sys/dev/firewire/fwohci.c @@ -155,6 +155,7 @@ void fwohci_txbufdb (struct fwohci_softc *, int , struct fw_bulkxfer *); static void fwohci_task_busreset(void *, int); static void fwohci_task_sid(void *, int); static void fwohci_task_dma(void *, int); +static void fwohci_sid_timeout(void *); /* * memory allocated for DMA programs @@ -427,17 +428,20 @@ fwohci_probe_phy(struct fwohci_softc *sc, device_t dev) * It is not actually available port on your PC . */ OWRITE(sc, OHCI_HCCCTL, OHCI_HCC_LPS); - DELAY(500); + /* IEEE 1394a-2000 s6.1: PHY reset completes within 10 ms of LPS. */ + DELAY(10000); reg = fwphy_rddata(sc, FW_PHY_SPD_REG); if ((reg >> 5) != 7) { sc->fc.mode &= ~FWPHYASYST; sc->fc.nport = reg & FW_PHY_NP; - sc->fc.speed = reg & FW_PHY_SPD >> 6; + sc->fc.speed = (reg & FW_PHY_SPD) >> 6; if (sc->fc.speed > MAX_SPEED) { - device_printf(dev, "invalid speed %d (fixed to %d).\n", - sc->fc.speed, MAX_SPEED); + if (bootverbose) + device_printf(dev, + "invalid speed %d (fixed to %d).\n", + sc->fc.speed, MAX_SPEED); sc->fc.speed = MAX_SPEED; } device_printf(dev, @@ -449,8 +453,10 @@ fwohci_probe_phy(struct fwohci_softc *sc, device_t dev) sc->fc.nport = reg & FW_PHY_NP; sc->fc.speed = (reg2 & FW_PHY_ESPD) >> 5; if (sc->fc.speed > MAX_SPEED) { - device_printf(dev, "invalid speed %d (fixed to %d).\n", - sc->fc.speed, MAX_SPEED); + if (bootverbose) + device_printf(dev, + "invalid speed %d (fixed to %d).\n", + sc->fc.speed, MAX_SPEED); sc->fc.speed = MAX_SPEED; } device_printf(dev, @@ -762,6 +768,7 @@ fwohci_init(struct fwohci_softc *sc, device_t dev) TASK_INIT(&sc->fwohci_task_busreset, 2, fwohci_task_busreset, sc); TASK_INIT(&sc->fwohci_task_sid, 1, fwohci_task_sid, sc); TASK_INIT(&sc->fwohci_task_dma, 0, fwohci_task_dma, sc); + callout_init_mtx(&sc->sid_timeout_callout, FW_GMTX(&sc->fc), 0); fw_init(&sc->fc); fwohci_reset(sc, dev); @@ -804,6 +811,7 @@ fwohci_detach(struct fwohci_softc *sc, device_t dev) fwohci_db_free(&sc->it[i]); fwohci_db_free(&sc->ir[i]); } + callout_drain(&sc->sid_timeout_callout); if (sc->fc.taskqueue != NULL) { taskqueue_drain(sc->fc.taskqueue, &sc->fwohci_task_busreset); taskqueue_drain(sc->fc.taskqueue, &sc->fwohci_task_sid); @@ -1824,7 +1832,8 @@ fwohci_intr_core(struct fwohci_softc *sc, uint32_t stat, int count) /* Disable bus reset interrupt until sid recv. */ OWRITE(sc, FWOHCI_INTMASKCLR, OHCI_INT_PHY_BUS_R); - device_printf(fc->dev, "%s: BUS reset\n", __func__); + if (sc->sid_timeout_count == 0 || firewire_debug) + device_printf(fc->dev, "%s: BUS reset\n", __func__); OWRITE(sc, FWOHCI_INTMASKCLR, OHCI_INT_CYC_LOST); OWRITE(sc, OHCI_LNKCTLCLR, OHCI_CNTL_CYCSRC); @@ -1835,6 +1844,10 @@ fwohci_intr_core(struct fwohci_softc *sc, uint32_t stat, int count) if (!kdb_active) taskqueue_enqueue(sc->fc.taskqueue, &sc->fwohci_task_busreset); + + /* SID watchdog: recover if self-ID never completes. */ + callout_reset(&sc->sid_timeout_callout, hz / 2, + fwohci_sid_timeout, sc); } if (stat & OHCI_INT_PHY_SID) { /* Enable bus reset interrupt */ @@ -1902,6 +1915,8 @@ fwohci_intr_core(struct fwohci_softc *sc, uint32_t stat, int count) } fc->status = FWBUSINIT; + callout_stop(&sc->sid_timeout_callout); + sc->sid_timeout_count = 0; if (!kdb_active) taskqueue_enqueue(sc->fc.taskqueue, &sc->fwohci_task_sid); @@ -1982,7 +1997,13 @@ fwohci_intr_dma(struct fwohci_softc *sc, uint32_t stat, int count) device_printf(fc->dev, "unrecoverable error\n"); } if (stat & OHCI_INT_PHY_INT) { - device_printf(fc->dev, "phy int\n"); + uint32_t r5; + + /* Clear W1C interrupt bits; mask ISBR(6) to avoid bus reset. */ + r5 = fwphy_rddata(sc, 5); + if (firewire_debug) + device_printf(fc->dev, "phy int: reg5=0x%02x\n", r5); + fwphy_wrdata(sc, 5, (r5 & ~(1 << 6)) | 0x3c); } } @@ -2039,6 +2060,28 @@ fwohci_task_sid(void *arg, int pending) fw_drain_txq(fc); fw_sidrcv(fc, buf, plen); free(buf, M_FW); + +} + +/* SelfID timeout: called with FW_GLOCK held (callout_init_mtx). */ +static void +fwohci_sid_timeout(void *arg) +{ + struct fwohci_softc *sc = arg; + struct firewire_comm *fc = &sc->fc; + + FW_GLOCK_ASSERT(fc); + if (fc->status != FWBUSRESET) + return; + + sc->sid_timeout_count++; + if (sc->sid_timeout_count == 1) + device_printf(fc->dev, "SelfID timeout, retrying\n"); + + /* Reset state machine so next bus reset is recognized. */ + fc->status = FWBUSINIT; + OWRITE(sc, FWOHCI_INTSTATCLR, OHCI_INT_PHY_BUS_R); + OWRITE(sc, FWOHCI_INTMASK, OHCI_INT_PHY_BUS_R); } static void diff --git a/sys/dev/firewire/fwohcivar.h b/sys/dev/firewire/fwohcivar.h index 754d61bb3513..a9e7e06b5590 100644 --- a/sys/dev/firewire/fwohcivar.h +++ b/sys/dev/firewire/fwohcivar.h @@ -36,6 +36,7 @@ * */ +#include <sys/callout.h> #include <sys/taskqueue.h> typedef struct fwohci_softc { @@ -71,6 +72,8 @@ typedef struct fwohci_softc { struct task fwohci_task_busreset; struct task fwohci_task_sid; struct task fwohci_task_dma; + struct callout sid_timeout_callout; /* fires if SID phase doesn't complete */ + int sid_timeout_count; /* consecutive SID timeouts */ int cycle_lost; } fwohci_softc_t;
