Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/sys/dev/usb use sc_lock not sc_access_lock to check UHIDEV_O...



details:   https://anonhg.NetBSD.org/src/rev/11a231b26b2a
branches:  trunk
changeset: 840095:11a231b26b2a
user:      mrg <mrg%NetBSD.org@localhost>
date:      Sat Mar 23 02:19:31 2019 +0000

description:
use sc_lock not sc_access_lock to check UHIDEV_OPEN.

fixes a soft hang when usbhidaction has a uhid open and bzflag
tries to open it as well.

XXX: pullup-7, pullup-8 (where i saw this originally.)

diffstat:

 sys/dev/usb/uhid.c   |  12 +++++++-----
 sys/dev/usb/uhidev.h |   4 ++--
 2 files changed, 9 insertions(+), 7 deletions(-)

diffs (64 lines):

diff -r bd58b826ccdd -r 11a231b26b2a sys/dev/usb/uhid.c
--- a/sys/dev/usb/uhid.c        Sat Mar 23 02:07:32 2019 +0000
+++ b/sys/dev/usb/uhid.c        Sat Mar 23 02:19:31 2019 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: uhid.c,v 1.106 2019/03/01 11:06:56 pgoyette Exp $      */
+/*     $NetBSD: uhid.c,v 1.107 2019/03/23 02:19:31 mrg Exp $   */
 
 /*
  * Copyright (c) 1998, 2004, 2008, 2012 The NetBSD Foundation, Inc.
@@ -35,7 +35,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: uhid.c,v 1.106 2019/03/01 11:06:56 pgoyette Exp $");
+__KERNEL_RCSID(0, "$NetBSD: uhid.c,v 1.107 2019/03/23 02:19:31 mrg Exp $");
 
 #ifdef _KERNEL_OPT
 #include "opt_compat_netbsd.h"
@@ -317,21 +317,23 @@
        if (sc->sc_dying)
                return ENXIO;
 
-       mutex_enter(&sc->sc_access_lock);
+       mutex_enter(&sc->sc_lock);
 
        /*
         * uhid interrupts aren't enabled yet, so setup sc_q now, as
         * long as they're not already allocated.
         */
        if (sc->sc_hdev.sc_state & UHIDEV_OPEN) {
-               mutex_exit(&sc->sc_access_lock);
+               mutex_exit(&sc->sc_lock);
                return EBUSY;
        }
+       mutex_exit(&sc->sc_lock);
+
        if (clalloc(&sc->sc_q, UHID_BSIZE, 0) == -1) {
-               mutex_exit(&sc->sc_access_lock);
                return ENOMEM;
        }
 
+       mutex_enter(&sc->sc_access_lock);
        error = uhidev_open(&sc->sc_hdev);
        if (error) {
                clfree(&sc->sc_q);
diff -r bd58b826ccdd -r 11a231b26b2a sys/dev/usb/uhidev.h
--- a/sys/dev/usb/uhidev.h      Sat Mar 23 02:07:32 2019 +0000
+++ b/sys/dev/usb/uhidev.h      Sat Mar 23 02:19:31 2019 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: uhidev.h,v 1.19 2016/04/23 10:15:32 skrll Exp $        */
+/*     $NetBSD: uhidev.h,v 1.20 2019/03/23 02:19:31 mrg Exp $  */
 
 /*
  * Copyright (c) 2001 The NetBSD Foundation, Inc.
@@ -67,8 +67,8 @@
        struct uhidev_softc *sc_parent;
        uByte sc_report_id;
        uint8_t sc_state;
+#define        UHIDEV_OPEN     0x01    /* device is open */
        int sc_in_rep_size;
-#define        UHIDEV_OPEN     0x01    /* device is open */
        void (*sc_intr)(struct uhidev *, void *, u_int);
        krndsource_t     rnd_source;
 };



Home | Main Index | Thread Index | Old Index