Re: kern/48494

[Patrick Welche <prlw1%cam.ac.uk@localhost>]

The following reply was made to PR kern/48494; it has been noted by GNATS.

From: Patrick Welche <prlw1%cam.ac.uk@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: prlw1%cam.ac.uk@localhost
Subject: Re: kern/48494
Date: Tue, 14 Jul 2015 15:53:43 +0100

 Does the numbering of the USB hubs really matter? I'm just wondering if
 that is a red herring (about to open a keyboard not detected with acpi PR).
 
 During experiments booting with -2 to disable ACPI, my problematic keyboard
 is always detected, but diffing 2 runs shows:
 
 
 --- withoutacpi.dmesg	2015-07-14 13:37:10.000000000 +0100
 +++ withoutacpi2.dmesg	2015-07-14 14:46:20.000000000 +0100
 @@ -4,7 +4,7 @@
  Copyright (c) 1982, 1986, 1989, 1991, 1993
      The Regents of the University of California.  All rights reserved.
  
 -NetBSD 7.99.20 (IBM) #44: Tue Jul 14 13:06:08 BST 2015
 +NetBSD 7.99.20 (IBM) #48: Tue Jul 14 14:44:10 BST 2015
  	prlw1@quark:/usr/obj/obj/sys/arch/amd64/compile/IBM
  total memory = 16135 MB
  avail memory = 15643 MB
 @@ -191,30 +191,30 @@
  pci22: i/o space, memory space enabled, rd/line, rd/mult, wr/inv ok
  coretemp0 at cpu0: thermal sensor, 1 C resolution, Tjmax=85
  timecounter: Timecounter "clockinterrupt" frequency 100 Hz quality 0
 -timecounter: Timecounter "TSC" frequency 2400122560 Hz quality 3000
 +timecounter: Timecounter "TSC" frequency 2400180160 Hz quality 3000
  IPsec: Initialized Security Association Processing.
  uhub0 at usb0: Intel EHCI root hub, class 9/0, rev 2.00/1.00, addr 1
  uhub0: 2 ports with 2 removable, self powered
  uhub1 at usb1: Intel EHCI root hub, class 9/0, rev 2.00/1.00, addr 1
  uhub1: 2 ports with 2 removable, self powered
 -uhub2 at uhub1 port 1: Intel product 8002, class 9/0, rev 2.00/0.05, addr 2
 +uhub2 at uhub0 port 1: Intel product 800a, class 9/0, rev 2.00/0.05, addr 2
  uhub2: single transaction translator
 -uhub3 at uhub0 port 1: Intel product 800a, class 9/0, rev 2.00/0.05, addr 2
 +uhub3 at uhub1 port 1: Intel product 8002, class 9/0, rev 2.00/0.05, addr 2
  uhub3: single transaction translator
 -uhub3: 6 ports with 6 removable, self powered
 -uhub2: 8 ports with 8 removable, self powered
 -uhidev0 at uhub2 port 1 configuration 1 interface 0
 +uhub2: 6 ports with 6 removable, self powered
 +uhub3: 8 ports with 8 removable, self powered
 +uhidev0 at uhub3 port 1 configuration 1 interface 0
  uhidev0: NOVATEK USB Keyboard, rev 1.10/1.12, addr 3, iclass 3/1
  ukbd0 at uhidev0: 8 modifier keys, 6 key codes
  wskbd1 at ukbd0 mux 1
  wskbd1: connecting to wsdisplay0
 -uhidev1 at uhub2 port 1 configuration 1 interface 1
 +uhidev1 at uhub3 port 1 configuration 1 interface 1
  uhidev1: NOVATEK USB Keyboard, rev 1.10/1.12, addr 3, iclass 3/0
  uhidev1: 4 report ids
  uhid0 at uhidev1 reportid 2: input=1, output=0, feature=0
  uhid1 at uhidev1 reportid 3: input=3, output=0, feature=0
  uhid2 at uhidev1 reportid 4: input=2, output=0, feature=0
 -uhub4 at uhub2 port 7: no manufacturer Gadget USB HUB, class 9/0, rev 2.00/0.00, addr 4
 +uhub4 at uhub3 port 7: no manufacturer Gadget USB HUB, class 9/0, rev 2.00/0.00, addr 4
  uhub4: 6 ports with 6 removable, self powered
  uhub4: port 5, device not enabled
  pad0: outputs: 44100Hz, 16-bit, stereo
 @@ -236,7 +236,7 @@
  nfs_boot: my_mask=255.255.255.0
  nfs_boot: gateway=0.0.0.0
  root on 10.168.204.2:/usr/obj/destdir
 -root time: 0x55a50249
 +root time: 0x55a5126c
  root file system type: nfs
  kern.module.path=/stand/amd64/7.99.20/modules
  init: copying out path `/sbin/init' 11
 
 So, the only difference is the numbering uhub2 <=> uhub3, the keyboard
 works either way (it was plugged into the same port throughout), and no
 ACPI is involved.  (oops, Oxford comma)
 
 (Also tried with ACPI, with and without the patch involving #if 0
 of the zeroing out, and uhub numbering didn't change.)