On Thu, 30 Jun 2011 19:04:55 +0200 Oliver Hartkopp <socketcan@xxxxxxxxxxxx> wrote: > On 30.06.2011 10:09, Ram Pai wrote: > > Multiple attempts to dynamically reallocate pci resources have unfortunately > > lead to regressions. Though we continue to fix the regressions and fine tune the > > dynamic-reallocation behavior, we have not reached a acceptable state yet. > > > > This patch provides a interim solution. It disables dynamic-reallocation; by > > default, with the ability to enable it through pci=realloc kernel command line > > parameter. > > What is the advantage of creating an 'interim' kernel parameter instead of > reverting the problematic commit and queue up a proper solution for 3.1 ? > > A kernel parameter needs to be observed, documented and set appropriately. > > I would prefer to have an automatic solution - if not in 3.0 then in 3.1 ... Yeah, we all want an automatic solution, but we still haven't been able to achieve one. My hope is that a parameter will let us keep the code upstream for Ram and others to keep fixing, then we can move to using it by default in some future release. Keeping the code upstream but behind a param should make development easier; at least that's the goal. -- Jesse Barnes, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html