Re: [PATCH] PCI: conditional resource-reallocation through kernel parameter pci=realloc

[Jesse Barnes <jbarnes@virtuousgeek.org>]

On Thu, 30 Jun 2011 19:04:55 +0200
Oliver Hartkopp <socketcan@hartkopp.net> 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