Author: hselasky
Date: Mon Jan  6 09:16:41 2020
New Revision: 356396
URL: https://svnweb.freebsd.org/changeset/base/356396

Log:
  MFC r356135:
  Make USB statistics per device instead of per bus.
  
  Bump the FreeBSD version due to structure change to
  force recompilation of external USB modules.
  
  Sponsored by: Mellanox Technologies

Modified:
  stable/10/sys/dev/usb/usb_bus.h
  stable/10/sys/dev/usb/usb_device.h
  stable/10/sys/dev/usb/usb_generic.c
  stable/10/sys/dev/usb/usb_transfer.c
  stable/10/sys/sys/param.h
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/dev/usb/usb_bus.h
==============================================================================
--- stable/10/sys/dev/usb/usb_bus.h     Mon Jan  6 09:10:13 2020        
(r356395)
+++ stable/10/sys/dev/usb/usb_bus.h     Mon Jan  6 09:16:41 2020        
(r356396)
@@ -1,6 +1,6 @@
 /* $FreeBSD$ */
 /*-
- * Copyright (c) 2008 Hans Petter Selasky. All rights reserved.
+ * Copyright (c) 2008-2019 Hans Petter Selasky. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -40,19 +40,10 @@ struct usb_bus_msg {
 };
 
 /*
- * The following structure defines the USB statistics structure.
- */
-struct usb_bus_stat {
-       uint32_t uds_requests[4];
-};
-
-/*
  * The following structure defines an USB BUS. There is one USB BUS
  * for every Host or Device controller.
  */
 struct usb_bus {
-       struct usb_bus_stat stats_err;
-       struct usb_bus_stat stats_ok;
 #if USB_HAVE_ROOT_MOUNT_HOLD
        struct root_hold_token *bus_roothold;
 #endif

Modified: stable/10/sys/dev/usb/usb_device.h
==============================================================================
--- stable/10/sys/dev/usb/usb_device.h  Mon Jan  6 09:10:13 2020        
(r356395)
+++ stable/10/sys/dev/usb/usb_device.h  Mon Jan  6 09:16:41 2020        
(r356396)
@@ -1,6 +1,6 @@
 /* $FreeBSD$ */
 /*-
- * Copyright (c) 2008 Hans Petter Selasky. All rights reserved.
+ * Copyright (c) 2008-2019 Hans Petter Selasky. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
@@ -175,10 +175,21 @@ union usb_device_scratch {
 };
 
 /*
+ * Helper structure to keep track of USB device statistics.
+ */
+struct usb_device_statistics {
+       uint32_t uds_requests[4];
+};
+
+/*
  * The following structure defines an USB device. There exists one of
  * these structures for every USB device.
  */
 struct usb_device {
+       /* statistics */
+       struct usb_device_statistics stats_err;
+       struct usb_device_statistics stats_ok;
+
        /* generic clear stall message */
        struct usb_udev_msg cs_msg[2];
        struct sx enum_sx;

Modified: stable/10/sys/dev/usb/usb_generic.c
==============================================================================
--- stable/10/sys/dev/usb/usb_generic.c Mon Jan  6 09:10:13 2020        
(r356395)
+++ stable/10/sys/dev/usb/usb_generic.c Mon Jan  6 09:16:41 2020        
(r356396)
@@ -2222,10 +2222,9 @@ ugen_ioctl_post(struct usb_fifo *f, u_long cmd, void *
                for (n = 0; n != 4; n++) {
 
                        u.stat->uds_requests_fail[n] =
-                           f->udev->bus->stats_err.uds_requests[n];
-
+                           f->udev->stats_err.uds_requests[n];
                        u.stat->uds_requests_ok[n] =
-                           f->udev->bus->stats_ok.uds_requests[n];
+                           f->udev->stats_ok.uds_requests[n];
                }
                break;
 

Modified: stable/10/sys/dev/usb/usb_transfer.c
==============================================================================
--- stable/10/sys/dev/usb/usb_transfer.c        Mon Jan  6 09:10:13 2020        
(r356395)
+++ stable/10/sys/dev/usb/usb_transfer.c        Mon Jan  6 09:16:41 2020        
(r356396)
@@ -2593,10 +2593,10 @@ usbd_transfer_done(struct usb_xfer *xfer, usb_error_t 
 #endif
        /* keep some statistics */
        if (xfer->error) {
-               info->bus->stats_err.uds_requests
+               info->udev->stats_err.uds_requests
                    [xfer->endpoint->edesc->bmAttributes & UE_XFERTYPE]++;
        } else {
-               info->bus->stats_ok.uds_requests
+               info->udev->stats_ok.uds_requests
                    [xfer->endpoint->edesc->bmAttributes & UE_XFERTYPE]++;
        }
 

Modified: stable/10/sys/sys/param.h
==============================================================================
--- stable/10/sys/sys/param.h   Mon Jan  6 09:10:13 2020        (r356395)
+++ stable/10/sys/sys/param.h   Mon Jan  6 09:16:41 2020        (r356396)
@@ -58,7 +58,7 @@
  *             in the range 5 to 9.
  */
 #undef __FreeBSD_version
-#define __FreeBSD_version 1004501      /* Master, propagated to newvers */
+#define __FreeBSD_version 1004502      /* Master, propagated to newvers */
 
 /*
  * __FreeBSD_kernel__ indicates that this system uses the kernel of FreeBSD,
_______________________________________________
svn-src-stable-10@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-stable-10
To unsubscribe, send any mail to "svn-src-stable-10-unsubscr...@freebsd.org"

Reply via email to