firewire: core: use long bus reset on gap count error
authorTakashi Sakamoto <o-takashi@sakamocchi.jp>
Thu, 29 Feb 2024 13:17:37 +0000 (22:17 +0900)
committerTakashi Sakamoto <o-takashi@sakamocchi.jp>
Thu, 29 Feb 2024 13:18:14 +0000 (22:18 +0900)
commitd0b06dc48fb15902d7da09c5c0861e7f042a9381
treebab7d066982ad66acbf0eaec38a1e48334c2f845
parentd206a76d7d2726f3b096037f2079ce0bd3ba329b
firewire: core: use long bus reset on gap count error

When resetting the bus after a gap count error, use a long rather than
short bus reset.

IEEE 1394-1995 uses only long bus resets. IEEE 1394a adds the option of
short bus resets. When video or audio transmission is in progress and a
device is hot-plugged elsewhere on the bus, the resulting bus reset can
cause video frame drops or audio dropouts. Short bus resets reduce or
eliminate this problem. Accordingly, short bus resets are almost always
preferred.

However, on a mixed 1394/1394a bus, a short bus reset can trigger an
immediate additional bus reset. This double bus reset can be interpreted
differently by different nodes on the bus, resulting in an inconsistent gap
count after the bus reset. An inconsistent gap count will cause another bus
reset, leading to a neverending bus reset loop. This only happens for some
bus topologies, not for all mixed 1394/1394a buses.

By instead sending a long bus reset after a gap count inconsistency, we
avoid the doubled bus reset, restoring the bus to normal operation.

Signed-off-by: Adam Goldman <adamg@pobox.com>
Link: https://sourceforge.net/p/linux1394/mailman/message/58741624/
Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
drivers/firewire/core-card.c