Re: Latest bk kernel does not properly free PCI IO & MEM allocations

[Prarit Bhargava <prarit@sgi.com>]

Rajesh Shah wrote:

>Note that ia64 arch code calls pci_claim_resources() at bus
>scan time (in pcibios_fixup_bus()). This adds the resources
>  
>
I've noticed this as well ...

>in pci_dev structure to the resource list, even before the
>corresponding driver is loaded. It is fairly common for a
>device driver to then call pci_request_regions() when it is
>loaded. This causes a new resource descriptor to be allocated
>and added to the resource list.  For example, on my tiger4
>ia64 system, /proc/iomem shows:
>
><snip>
>f9000000-fbffffff : PCI Bus 0000:00
>  f9ff0000-f9ff03ff : 0000:00:1d.7
>    f9ff0000-f9ff03ff : ehci_hcd
>  fa000000-faffffff : 0000:01:01.0
>  fbfa0000-fbfbffff : 0000:01:01.0
>  fbfd0000-fbfd0fff : 0000:01:01.0
>  fbfe0000-fbffffff : 0000:01:00.0
>    fbfe0000-fbffffff : e1000
><snip>
>
>  
>
UGH.  This gets worse and worse, doesn't it :)

>Since we have 2 resource descriptors for the same resource in
>some cases, they both do need to be removed at device remove
>time. However, I saw only 1 resource in your /proc/iomem (which
>was presumably after the driver had loaded). I wonder if this is
>the problem in this case.
>
>In any case, claiming and freeing pci resources needs wider
>cleanup. See
>
>http://marc.theaimsgroup.com/?l=linux-kernel&m\x110841094228134&w=2
>http://marc.theaimsgroup.com/?l=linux-kernel&m\x110842019012889&w=2
>
>for related discussions.
>
>  
>
Thanks Rajesh -- I appreciate the pointers to earlier discussions.

P.



-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_ide95&alloc_id\x14396&op=click
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel