Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 26 Oct 2009 10:37:25 -0700
From:      Marcel Moolenaar <xcllnt@mac.com>
To:        John Baldwin <jhb@freebsd.org>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r198431 - head/sys/dev/pci
Message-ID:  <936EBD5F-31D2-4B6A-94E7-D9DB8FD236FE@mac.com>
In-Reply-To: <200910261254.49768.jhb@freebsd.org>
References:  <200910232253.n9NMr10R074584@svn.freebsd.org> <200910260837.23479.jhb@freebsd.org> <8082A638-C5BA-42F9-8C2E-DA31EF084840@mac.com> <200910261254.49768.jhb@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

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
xcllnt@mac.com






Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?936EBD5F-31D2-4B6A-94E7-D9DB8FD236FE>