On 12/19/16 17:13, John Baldwin wrote:
On Monday, December 19, 2016 09:28:13 AM Hans Petter Selasky wrote:
Author: hselasky
Date: Mon Dec 19 09:28:12 2016
New Revision: 310242
URL: https://svnweb.freebsd.org/changeset/base/310242

Log:
  Defer USB enumeration until the SI_SUB_KICK_SCHEDULER is executed to avoid
  boot panics in conjunction with the recently added EARLY_AP_STARTUP feature.
  The panics happen due to using kernel facilities like callouts too early.

  Tested by:    jhb @
  MFC after:    1 week

As mentioned in the thread, the patches to usb_hub.c weren't needed and the 
block
in usb_process.c can just be removed unconditionally since it should never 
execute
in the non-EARLY_AP_STARTUP case.


Hi John,

The checks in usb_process.c are needed in case attaching a USB controller fails for some unknown reason. Then there is a problem that thread0 cannot wait/sleep for the process to exit, so that it doesn't access freed memory. Did I miss anything?

--HPS
_______________________________________________
svn-src-head@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-head
To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"

Reply via email to