Messages in this thread | | | Date | Thu, 3 Sep 2009 13:43:19 -0700 | From | "Luis R. Rodriguez" <> | Subject | Re: [PATCH] cfg80211: clear cfg80211_inform_bss() from kmemleak reports |
| |
On Thu, Sep 03, 2009 at 11:17:17AM -0700, Johannes Berg wrote: > On Thu, 2009-09-03 at 11:13 -0700, Luis R. Rodriguez wrote: > > > What I meant is it gobbles it up and spits another thing out. When it > > gobbles it up the routine then uses kref_put(). > > > > > Why can it not track this? > > > > It probably can, just not sure if it follows kref_put(), I was under > > the impression here it doesn't and because of it we were getting false > > positives. Catalin, can you confirm? > > Ah I'd think that if it can't track it then that's because we use a > pointer to the middle of the struct to keep track of it much of the > time.
So you agree with the patch but not the commit log entry?
Luis
| |