On Oct 26, 2009, at 9:54 AM, John Baldwin wrote:
On Monday 26 October 2009 12:32:48 pm Marcel Moolenaar wrote:
On Oct 26, 2009, at 5:37 AM, John Baldwin wrote:
Log:
BIOSes, buggy or otherwise, are i386 or amd64 specific.
Have the early USB takeover enabled for i386 and amd64
by default.
This also avoids a panic on PowerPC where the resource
isn't released properly and we find a busy resource
when the USB host controller wants to allocate it...
Presumably such systems won't set the 'BIOS owned' bit in the their
legacy
support registers in which case these routines are NOPs (they just
read the
register, see the bit is clear, and exit). The resource bug sounds
like a
real one that should be fixed and would probably affect any x86
systems who
have USB disabled in the BIOS, so that should be fixed rather than
papered
over. Please revert.
*sigh*
The change was made because 1) doing this as part of the PCI code is
unnecessary for non-PC HW, and 2) it's entirely untested on non-PC
HW and the gratuitous change can therefore only do harm -- he, guess
what, it did do harm.
Unless people fix the resource stuff this change cannot be reverted.
After the resource fix has gone in, I still object to this being
reverted on grounds of gratuitous code bloat. I say this with ARM,
MIPS and PowerPC/Book-E in mind.
You didn't remove anything, you merely toggled the setting of a
variable.
Oops... Brainfart. I thought I did wrapped the whole logic...
/me resets brain
If someone is willing to fix the resource issue, I'm more than happy
to test patches. In the mean time we should keep the change to avoid
unnecessary productivity loss by people impacted by the kernel panic.
Once the resource issue is fixed, I don't care in particular what
the default is as long as the logic gets executed some time...
--
Marcel Moolenaar
xcl...@mac.com
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
