Hello Rich<br><br><div class="gmail_quote">2011/10/24 Rich Wrenn <span dir="ltr"><<a href="mailto:rwrenn@ddn.com">rwrenn@ddn.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div link="blue" vlink="purple" lang="EN-US"><div><p class="MsoNormal">Greetings,<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">The way that bcmxcp_usb reacts to and recovers from a USB cable unplug-plug appears to have changed significantly between 2.4.1 and 2.6.0.  While I can see some improvements, bcmxcp_usb 2.6.0 does not appear to handle a USB cable unplug-plug correctly.  Does anyone know if this has been fixed in subsequent releases?<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">Here are the details.<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">When the USB cable between my server and my PW5115 UPS is removed, bcmxcp_usb reports DATASTALE (as it should) and continues to respond to PING (as it should).  When the USB cable is reattached, bcmxcp_usb continues to report DATASTALE in response to DUMPALL even though lsusb indicates that the PW5115 is indeed connected and DATAOK is never reported in response to DUMPALL.  If I restart the bcmxcp_usb driver with upsdrvctl then it recovers the connection to the UPS.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">The bcmxcp_usb 2.4.1 would not respond to even a PING while the USB cable to the UPS was disconnected, but when the USB cable was reconnected then it would begin responding again without the need to restart bcmxcp_usb.  I am not suggesting that the driver be rolled back to this behavior, but I do need the ability to recover from a USB cable pull without restarting the driver.<u></u><u></u></p>
</div></div></blockquote></div><br>I've just fixed it in the trunk (r3303), and tested it for regression on both serial and USB XCP, with 5110 and 3105 (USB) and 9130 (serial).<br><br>The patch itself is (or will be) available here:<br>
<a href="http://trac.networkupstools.org/projects/nut/changeset/3303">trac.networkupstools.org/projects/nut/changeset/3303</a><br><a href="http://anonscm.debian.org/viewvc/nut/trunk/drivers/bcmxcp_usb.c?r1=3303&r2=3302&pathrev=3303">http://anonscm.debian.org/viewvc/nut/trunk/drivers/bcmxcp_usb.c?r1=3303&r2=3302&pathrev=3303</a><br>
<a href="http://anonscm.debian.org/viewvc/nut/trunk/drivers/bcmxcp_usb.c?view=patch&r1=3303&r2=3302&pathrev=3303">http://anonscm.debian.org/viewvc/nut/trunk/drivers/bcmxcp_usb.c?view=patch&r1=3303&r2=3302&pathrev=3303</a><br>
<br>thanks for your report,<br>Arnaud<br>-- <br>Linux / Unix Expert R&D - Eaton - <a href="http://powerquality.eaton.com" target="_blank">http://powerquality.eaton.com</a><br>Network UPS Tools (NUT) Project Leader - <a href="http://www.networkupstools.org/" target="_blank">http://www.networkupstools.org/</a><br>
Debian Developer - <a href="http://www.debian.org" target="_blank">http://www.debian.org</a><br>Free Software Developer - <a href="http://arnaud.quette.free.fr/" target="_blank">http://arnaud.quette.free.fr/</a><br><br>