Messages in this thread | | | Date | Tue, 10 Jun 2008 14:14:30 -0700 (PDT) | From | Doug Thompson <> | Subject | Re: on static kobjects and double frees... |
| |
--- Arthur Jones <ajones@riverbed.com> wrote:
> Hi Greg, ... > > On Tue, Jun 10, 2008 at 09:23:41AM -0700, Greg KH wrote: > > On Tue, Jun 10, 2008 at 08:58:50AM -0700, Arthur Jones wrote: > > > Hi Greg, The edac pci sysfs generic layer uses a static > > > kobject as a placeholder parent where edac pci drivers > > > are inserted. > > > > Hm, stop right there. > > > > kobjects are not supposed to be static, bad things happen if you do that > > (including the kernel itself will warn you about them, unless you gave > > it an empty release function, and if so, then see > > Documentation/kobject.txt and prepare to be mocked...)
The edac_pci was my first coding with kobjects sometime ago and I have not reviewed them for quite awhile, since I have been working more on memory controllers. Thus it was my bad.
Arthur, thanks for tracking that down and reviewing it. All the memory controller kobjects are all dynamic. The edac PCI code needed to be refactored and it looks like you did it.
thanks again
doug t
For the patch:
Acked-by: doug thompson <dougthompson@xmission.com>
W1DUG
| |