Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1698806 > unrolled thread
| Started by | Herbert Xu <herbert@gondor.apana.org.au> |
|---|---|
| First post | 2017-07-28 16:00 +0200 |
| Last post | 2017-07-28 16:00 +0200 |
| Articles | 1 — 1 participant |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH v2] crypto: caam - free qman_fq after kill_fq Herbert Xu <herbert@gondor.apana.org.au> - 2017-07-28 16:00 +0200
| From | Herbert Xu <herbert@gondor.apana.org.au> |
|---|---|
| Date | 2017-07-28 16:00 +0200 |
| Subject | Re: [PATCH v2] crypto: caam - free qman_fq after kill_fq |
| Message-ID | <u8dY6-4rR-21@gated-at.bofh.it> |
On Thu, Jul 13, 2017 at 05:21:01AM -0400, Xulin Sun wrote: > kill_fq removes a complete frame queue, it needs to free the qman_fq > in the last. Else kmemleak will report the below warning: > > unreferenced object 0xffff800073085c80 (size 128): > comm "cryptomgr_test", pid 199, jiffies 4294937850 (age 67.840s) > hex dump (first 32 bytes): > 00 00 00 00 00 00 00 00 a0 80 7e 00 00 80 ff ff > 00 00 00 00 00 00 00 00 04 00 04 00 5c 01 00 00 > backtrace: > [<ffff8000001e5760>] create_object+0xf8/0x258 > [<ffff800000994e38>] kmemleak_alloc+0x58/0xa0 > [<ffff8000001d5f18>] kmem_cache_alloc_trace+0x2c8/0x358 > [<ffff8000007e8410>] create_caam_req_fq+0x40/0x170 > [<ffff8000007e870c>] caam_drv_ctx_update+0x54/0x248 > [<ffff8000007fca54>] aead_setkey+0x154/0x300 > [<ffff800000452120>] setkey+0x50/0xf0 > [<ffff80000045b144>] __test_aead+0x5ec/0x1028 > [<ffff80000045c28c>] test_aead+0x44/0xc8 > [<ffff80000045c368>] alg_test_aead+0x58/0xd0 > [<ffff80000045bdb4>] alg_test+0x14c/0x308 > [<ffff8000004588e8>] cryptomgr_test+0x50/0x58 > [<ffff8000000c3b2c>] kthread+0xdc/0xf0 > [<ffff800000083c00>] ret_from_fork+0x10/0x50 > > And check where the function kill_fq() is called to remove > the additional kfree to qman_fq and avoid re-calling the released qman_fq. > > Signed-off-by: Xulin Sun <xulin.sun@windriver.com> Patch applied. Thanks. -- Email: Herbert Xu <herbert@gondor.apana.org.au> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
Back to top | Article view | linux.kernel
csiph-web