Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 20 Jul 2026 22:50:19 +0000
From:      Adrian Chadd <adrian@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Cc:        Abdelkader Boudih <freebsd@seuros.com>
Subject:   git: 5b7449ad47be - main - firewire: drain pending xfers after callout stop in detach
Message-ID:  <6a5ea62b.30aff.5360d636@gitrepo.freebsd.org>

index | next in thread | raw e-mail

The branch main has been updated by adrian:

URL: https://cgit.FreeBSD.org/src/commit/?id=5b7449ad47be4582ff451ce4615753dab4a6f882

commit 5b7449ad47be4582ff451ce4615753dab4a6f882
Author:     Abdelkader Boudih <freebsd@seuros.com>
AuthorDate: 2026-07-20 22:47:02 +0000
Commit:     Adrian Chadd <adrian@FreeBSD.org>
CommitDate: 2026-07-20 22:47:22 +0000

    firewire: drain pending xfers after callout stop in detach
    
    Removes a TODO that predates the existing drain call.
    
    Reviewed by:    adrian
    Differential Revision:  https://reviews.freebsd.org/D58308
---
 sys/dev/firewire/firewire.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/sys/dev/firewire/firewire.c b/sys/dev/firewire/firewire.c
index d3808f5c8e62..cc0cf9e5b7ad 100644
--- a/sys/dev/firewire/firewire.c
+++ b/sys/dev/firewire/firewire.c
@@ -564,7 +564,9 @@ firewire_detach(device_t dev)
 	callout_stop(&fc->bmr_callout);
 	callout_stop(&fc->busprobe_callout);
 
-	/* TODO: cancel pending xfers in atq/ats queues */
+	/* Drain xfers queued by children during detach or pending callouts. */
+	fw_drain_txq(fc);
+
 	for (fwdev = STAILQ_FIRST(&fc->devices); fwdev != NULL;
 	     fwdev = fwdev_next) {
 		fwdev_next = STAILQ_NEXT(fwdev, link);


home | help

Want to link to this message? Use this
URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?6a5ea62b.30aff.5360d636>