Messages in this thread | | | Date | Sat, 3 Jun 2006 23:30:11 +0200 | From | Pavel Machek <> | Subject | Re: [PATCH 2.6.17-rc5 7/8] Remove some of the kmemleak false positives |
| |
Hi!
> >>From: Catalin Marinas <catalin.marinas@arm.com> > >>+ memleak_debug_not_leak(res); > > > > I'd suggest some shorter/more generic name. > > > > not_freed(res); ? > > > > alloc_forever(res); ? > > It's true that the names are a bit long ("debug" is even superfluous). I > would, however, keep the memleak_ prefix as I think it makes the code > clearer (i.e. the function belongs to a certain part of the kernel).
Well, in future some other leak detector may want to use these hooks.
Anyway memleak_not_leak() is already quite good. Pavel
-- (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |