Re: [PATCH 4/4] i2o: Use pci_bus_alloc_resource(), not allocate_resource() directly

[One Thousand Gnomes <gnomes@lxorguk.ukuu.org.uk>]

On Wed, 26 Feb 2014 12:09:27 -0700
Bjorn Helgaas <bhelgaas@google.com> wrote:

> Convert i2o_res_alloc() to use pci_bus_alloc_resource() rather than
> pci_find_parent_resource() and allocate_resource().  We don't have a
> resource to start with, so pci_find_parent_resource() can't do anything
> useful: a bus may have several memory resources available, so there might
> be several possible parents.  This is more likely on root buses because
> host bridges may have any number of apertures.
> 
> I'm pretty sure this didn't work in the first place because it passed
> size == min == max to allocate_resource().  The min and max parameters are
> constraints on the *addresses* of the resource, not on its size, so I think
> it was impossible for allocate_resource() to succeed.

I don't think many i2o controllers ever used that path, and I doubt any
in normal use did as the vision of offloading for devices on the host bus
basically never happened (it happened even less than i2o)

A rather more sensible question might be "If i2o went away is there
anyone who would even notice". About the only devices that ever used i2o
in the real world (AMI MegaRAID and some FC stuff) had native firmware or
modes that worked better anyway.

Alan