Re: [PATCH 2.6.17-rc5 1/7] Base support for kmemleak

[Anne Thrax]

mm/memleak.c: In function `insert_pointer':
mm/memleak.c:216: warning: unused variable `i'
mm/memleak.c: At top level:
mm/memleak.c:586: warning: 'memleak_scan' defined but not used

> +/* Insert a pointer and its aliases into the pointer radix tree */
> +static inline void insert_pointer(unsigned long ptr, size_t size, int ref_count)
> +{
> +	struct memleak_alias *alias;
> +	struct hlist_node *elem;
> +	struct memleak_pointer *pointer;
> +	int err, i;
> +#ifdef CONFIG_FRAME_POINTER
> +	void *frame;
> +#endif

If I'm not mistaken, you don't use 'i' outside of #ifdef CONFIG_FRAME_POINTER code.

> +/* Scan the memory and print the orphan pointers */
> +static void memleak_scan(void)

I don't think this is used anywhere in memleak.c besides
#ifdef CONFIG_DEBUG_FS code.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/