Re: [PATCH 2/2] Fix irq_alloc_host() reference counting and callers

2008-05-13 Thread Michael Ellerman
On Tue, 2008-05-13 at 20:58 +1000, Paul Mackerras wrote: > Michael Ellerman writes: > > > When I changed irq_alloc_host() to take an of_node > > (52964f87c64e6c6ea671b5bf3030fb1494090a48: "Add an > > optional device_node pointer to the irq_host"), > > I botched the reference counting semantics. >

Re: [PATCH 2/2] Fix irq_alloc_host() reference counting and callers

2008-05-13 Thread Paul Mackerras
Michael Ellerman writes: > When I changed irq_alloc_host() to take an of_node > (52964f87c64e6c6ea671b5bf3030fb1494090a48: "Add an > optional device_node pointer to the irq_host"), > I botched the reference counting semantics. > > Stephen pointed out that it's irq_alloc_host()'s > business if it

[PATCH 2/2] Fix irq_alloc_host() reference counting and callers

2008-05-07 Thread Michael Ellerman
When I changed irq_alloc_host() to take an of_node (52964f87c64e6c6ea671b5bf3030fb1494090a48: "Add an optional device_node pointer to the irq_host"), I botched the reference counting semantics. Stephen pointed out that it's irq_alloc_host()'s business if it needs to take an additional reference to