svn commit: r198431 - head/sys/dev/pci
avg at freebsd.org
Mon Oct 26 16:50:35 UTC 2009
on 26/10/2009 18:32 Marcel Moolenaar said the following:
> 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.
Well, I think that maybe people who could fix this might not have the hardware
that is need to reproduce and test the problem. So they need help of people who
do have the hardware and can reproduce the problem.
Perhaps, just disabling the code _and_ stopping there is what should not be done.
> 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.
Well, PCI USB controller drivers would also have this bloat, but I am not sure if
hardware for the platforms you listed has PCI *HCI controllers.
Besides, only code execution is now disabled, the code itself is still present,
which also can be considered as a form of bloat.
Perhaps #if defined (arch1) ... would be more appropriate here? I am not sure.
More information about the svn-src-all