Nut upsuser - Sep 2014 - Nut 2.7.2 on OpenBSD 5.6 with APC USB UPS

On 09/19/14 07:23, Charles Lepple wrote:
> On Sep 19, 2014, at 7:26 AM, Stan Gammons <sg063015 at gmail.com>
wrote:
>
>> --enable-debug-log was in the original makefile.  I added
--enable-debug.  At this point I guess trying to build from source or go back to
OpenBSD 5.5 are the only options left.
> Ah, my confusion was that --enable-debug-log was wrapped in ".ifdef
DEBUG", which I suspect is not enabled by default.
>
> On the other hand, if you add --enable-debug outside that .ifdef, it should
work: I forgot that autoconf scripts allow abbreviations of long option flags.
>
> If you do have to go back to 5.5, you may want to log something with the
OpenBSD folks to mention the regression.
>
Looks like this patch fixed the problem.  I see the status change now.

Index: usb_quirks.c
==================================================================RCS file:
/cvs/src/sys/dev/usb/usb_quirks.c,v
retrieving revision 1.73
diff -u -p -r1.73 usb_quirks.c
--- usb_quirks.c	20 Mar 2014 15:07:11 -0000	1.73
+++ usb_quirks.c	19 Sep 2014 13:29:43 -0000
@@ -109,6 +109,8 @@ const struct usbd_quirk_entry {
   { USB_VENDOR_NEC, USB_PRODUCT_NEC_PICTY920,	    ANY,   { UQ_BROKEN_BIDIR }},
   { USB_VENDOR_NEC, USB_PRODUCT_NEC_PICTY800,	    ANY,   { UQ_BROKEN_BIDIR }},
  
+ { USB_VENDOR_APC, USB_PRODUCT_APC_UPS,			ANY,	{ UQ_BAD_HID }},
+ { USB_VENDOR_APC, USB_PRODUCT_APC_UPS5G,		ANY,	{ UQ_BAD_HID }},
   { USB_VENDOR_APPLE, USB_PRODUCT_APPLE_IPHONE,		ANY,	{ UQ_BAD_HID }},
   { USB_VENDOR_APPLE, USB_PRODUCT_APPLE_IPHONE_3G,	ANY,	{ UQ_BAD_HID }},
   { USB_VENDOR_APPLE, USB_PRODUCT_APPLE_IPHONE_3GS,	ANY,	{ UQ_BAD_HID }},



Stan

On Sep 19, 2014, at 5:18 PM, Stan Gammons <sg063015 at gmail.com> wrote:
> Looks like this patch fixed the problem.  I see the status change now.
Good to hear.

I'm not familiar with the OpenBSD release process. Is that patch on track to
be a part of the final 5.6 release?

-- 
Charles Lepple
clepple at gmail

On 09/19/14 18:12, Charles Lepple wrote:
> On Sep 19, 2014, at 5:18 PM, Stan Gammons <sg063015 at gmail.com>
wrote:
>
>> Looks like this patch fixed the problem.  I see the status change now.
> Good to hear.
>
> I'm not familiar with the OpenBSD release process. Is that patch on
track to be a part of the final 5.6 release?
>
I don't know.  It's one that Stuart Henderson sent me.  I'll ask if
it
will be in the upcoming 5.6 release. One thing that should be noted, I 
disabled upd at boot before I started upsd and upsmon. Below is Stuart's 
reply from the message with the patch.

Thanks for your help.


Stan


Ah - you already sent the dmesg for that.. To get NUT to work properly
I think you will need to add this and build a new kernel. This knocks it
out from attaching as a HID device (as needed for upd) - attaching as a
HID device means that libusb has less control over it.

A better solution would be to allow the device to attach to upd, and write a
NUT driver that queries the hw.sensors sysctls rather than having NUT talk
USB to the device.


Index: usb_quirks.c
==================================================================RCS file:
/cvs/src/sys/dev/usb/usb_quirks.c,v
retrieving revision 1.73
diff -u -p -r1.73 usb_quirks.c
--- usb_quirks.c	20 Mar 2014 15:07:11 -0000	1.73
+++ usb_quirks.c	19 Sep 2014 13:29:43 -0000
@@ -109,6 +109,8 @@ const struct usbd_quirk_entry {
   { USB_VENDOR_NEC, USB_PRODUCT_NEC_PICTY920,	    ANY,   { UQ_BROKEN_BIDIR }},
   { USB_VENDOR_NEC, USB_PRODUCT_NEC_PICTY800,	    ANY,   { UQ_BROKEN_BIDIR }},
  
+ { USB_VENDOR_APC, USB_PRODUCT_APC_UPS,			ANY,	{ UQ_BAD_HID }},
+ { USB_VENDOR_APC, USB_PRODUCT_APC_UPS5G,		ANY,	{ UQ_BAD_HID }},
   { USB_VENDOR_APPLE, USB_PRODUCT_APPLE_IPHONE,		ANY,	{ UQ_BAD_HID }},
   { USB_VENDOR_APPLE, USB_PRODUCT_APPLE_IPHONE_3G,	ANY,	{ UQ_BAD_HID }},
   { USB_VENDOR_APPLE, USB_PRODUCT_APPLE_IPHONE_3GS,	ANY,	{ UQ_BAD_HID }},

* Charles Lepple <clepple at gmail.com> [2014-09-20
01:12]:
> On Sep 19, 2014, at 5:18 PM, Stan Gammons <sg063015 at gmail.com>
wrote:
> > Looks like this patch fixed the problem.  I see the status change now.
> Good to hear.
> I'm not familiar with the OpenBSD release process. Is that patch on
track to be a part of the final 5.6 release?
no, 5.6 is already cut. 5.7 will have it.

-- 
Henning Brauer, hb at bsws.de, henning at openbsd.org
BS Web Services GmbH, AG Hamburg HRB 128289, http://bsws.de
Full-Service ISP - Secure Hosting, Mail and DNS Services
Dedicated Servers, Rootservers, VMs/PVS, Application Hosting