Re: 2.6.37-rc1 kmemleak warnings when saving a file over cifs.

[Steve French]

On Mon, Nov 8, 2010 at 6:13 PM, Andrew Hendry <andrew.hendry@xxxxxxxxx> wrote:
> Have just noticed these in the past few days, now running 2.6.37-rc1+
> Seems to be 1 logged per file save when using a text file with gedit over cifs.

Can you tell the called function in cifs_setattr_unix vs. cifs_setattr_nounix
(the former will usually be called when the server is Samba, the latter
when the server is Windows or NetApp).  What is the server type?

> unreferenced object 0xffff88022ee08b40 (size 32):
>  comm "gedit", pid 2524, jiffies 4300160388 (age 2633.655s)
>  hex dump (first 32 bytes):
>    5c 2e 67 6f 75 74 70 75 74 73 74 72 65 61 6d 2d  \.goutputstream-
>    35 42 41 53 4c 56 00 de 09 00 00 00 2c 26 78 ee  5BASLV......,&x.
>  backtrace:
>    [<ffffffff81504a4d>] kmemleak_alloc+0x2d/0x60
>    [<ffffffff81136e13>] __kmalloc+0xe3/0x1d0
>    [<ffffffffa0313db0>] build_path_from_dentry+0xf0/0x230 [cifs]
>    [<ffffffffa031ae1e>] cifs_setattr+0x9e/0x770 [cifs]
>    [<ffffffff8115fe90>] notify_change+0x170/0x2e0
>    [<ffffffff81145ceb>] sys_fchmod+0x10b/0x140
>    [<ffffffff8100c172>] system_call_fastpath+0x16/0x1b
>    [<ffffffffffffffff>] 0xffffffffffffffff

Didn't see anything obvious in cifs_setattr_unix on the kmalloc/kfree
of full_path,
but it will help when we know which path we are in.  We did make some recent
changes in 2.6.37 in file allocation.

-- 
Thanks,

Steve
--
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/