From: John Garry <john.garry@huawei.com>
To: Frank Rowand <frowand.list@gmail.com>, Rob Herring <robh+dt@kernel.org>
Cc: Anders Roxell <anders.roxell@linaro.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: Re: of_node_release() warn
Date: Fri, 6 Dec 2019 17:11:16 +0000 [thread overview]
Message-ID: <6021ac63-b5e0-ed3d-f964-7c6ef579cd68@huawei.com> (raw)
In-Reply-To: <0a6b7d4d-6d0f-8717-f3d3-26b0452f36e5@huawei.com>
On 06/12/2019 09:40, John Garry wrote:
> Hi Frank,
>
>>> [ 292.117483][ T406] driver: 'unittest-i2c-bus': driver_release
>>> [ 292.125954][ T487] kobject: 'unittest-i2c-dev': free name
>>> [ 292.134162][ T406] kobject: 'unittest-i2c-bus': free name
>>> [ 292.171965][ T504] kobject: 'test-unittest111'
>>> ((____ptrval____)): kobject_cleanup, parent (____ptrval____)
>>> [ 292.181866][ T504] kobject: 'test-unittest111'
>>> ((____ptrval____)): calling ktype release
>>
>> The following "OF: ERROR: memory leak ..." message is not an expected
>> error from the unittest:
Even just using a vanilla arm64 defconfig with CONFIG_OF_UNITTEST=y
gives many of these warnings for me:
18.022956] rtc-efi rtc-efi: setting system clock to
2019-12-06T17:04:30 UTC (1575651870)
[ 18.031448] Duplicate name in testcase-data, renamed to
"duplicate-name#1"
[ 18.038657] ### dt-test ### start of unittest - you will see error
messages
[ 18.045707] OF: /testcase-data/phandle-tests/consumer-a: could not
get #phandle-cells-missing for /testcase-data/phandle-tests/provider1
[ 18.057957] OF: /testcase-data/phandle-tests/consumer-a: could not
get #phandle-cells-missing for /testcase-data/phandle-tests/provider1
[ 18.070205] OF: /testcase-data/phandle-tests/consumer-a: could not
find phandle
[ 18.077505] OF: /testcase-data/phandle-tests/consumer-a: could not
find phandle
[ 18.084804] OF: /testcase-data/phandle-tests/consumer-a:
#phandle-cells = 3 found -1
[ 18.092538] OF: /testcase-data/phandle-tests/consumer-a:
#phandle-cells = 3 found -1
[ 18.093302] usb 1-2: new high-speed USB device number 3 using
ehci-platform
[ 18.100313] OF: /testcase-data/phandle-tests/consumer-b: could not
get #phandle-missing-cells for /testcase-data/phandle-tests/provider1
[ 18.119468] OF: /testcase-data/phandle-tests/consumer-b: could not
find phandle
[ 18.126769] OF: /testcase-data/phandle-tests/consumer-b:
#phandle-cells = 2 found -1
[ 18.135374] platform testcase-data:testcase-device2: IRQ index 0 not
found
[ 18.144546] OF: overlay: WARNING: memory leak will occur if overlay
removed, property:
/testcase-data/overlay-node/test-bus/test-unittest0/status
[ 18.157908] OF: overlay: WARNING: memory leak will occur if overlay
removed, property:
/testcase-data/overlay-node/test-bus/test-unittest1/status
[ 18.171028] OF: overlay: WARNING: memory leak will occur if overlay
removed, property:
/testcase-data/overlay-node/test-bus/test-unittest2/status
[ 18.184381] OF: overlay: WARNING: memory leak will occur if overlay
removed, property:
/testcase-data/overlay-node/test-bus/test-unittest3/status
[ 18.197890] OF: overlay: WARNING: memory leak will occur if overlay
removed, property:
/testcase-data/overlay-node/test-bus/test-unittest5/status
[ 18.211312] OF: overlay: WARNING: memory leak will occur if overlay
removed, property:
/testcase-data/overlay-node/test-bus/test-unittest6/status
[ 18.224649] OF: overlay: WARNING: memory leak will occur if overlay
removed, property:
/testcase-data/overlay-node/test-bus/test-unittest7/status
[ 18.238114] OF: overlay: WARNING: memory leak will occur if overlay
removed, property:
/testcase-data/overlay-node/test-bus/test-unittest8/status
[ 18.251444] OF: overlay: WARNING: memory leak will occur if overlay
removed, property:
/testcase-data/overlay-node/test-bus/test-unittest8/property-foo
[ 18.261857] hub 1-2:1.0: USB hub found
[ 18.265002] OF: overlay: node_overlaps_later_cs: #6 overlaps with #7
@/testcase-data/overlay-node/test-bus/test-unittest8
[ 18.268835] hub 1-2:1.0: 4 ports detected
[ 18.279670] OF: overlay: overlay #6 is not topmost
[ 18.281898] i2c i2c-1: Added multiplexed i2c bus 2
[ 18.293275] OF: overlay: WARNING: memory leak will occur if overlay
removed, property:
/testcase-data/overlay-node/test-bus/i2c-test-bus/test-unittest12/status
[ 18.307619] OF: overlay: WARNING: memory leak will occur if overlay
removed, property:
/testcase-data/overlay-node/test-bus/i2c-test-bus/test-unittest13/status
[ 18.322611] i2c i2c-1: Added multiplexed i2c bus 3
[ 18.328552] ### dt-test ### FAIL
of_unittest_overlay_high_level():2475 overlay_base_root not initialized
[ 18.338024] ### dt-test ### end of unittest - 237 passed, 1 failed
[ 18.344208] ------------[ cut here ]------------
Thanks,
John
>>
>>> [ 292.191376][ T504] OF: ERROR: memory leak before free overlay
>>> changeset,
>>> /testcase-data/overlay-node/test-bus/test-unittest11/test-unittest111
>>
>>
>>> [ 292.191386][ C47] ------------[ cut here ]------------
>>> [ 292.191391][ C47] refcount_t: addition on 0; use-after-free.
>>> [ 292.191397][ C47] WARNING: CPU: 47 PID: 504 at lib/refcount.c:25
>>> refcount_warn_saturate+0xf4/0x1b0
>>> [ 292.191399][ C47] Modules linked in:
>>> [ 292.191406][ C47] CPU: 47 PID: 504 Comm: kworker/47:2 Tainted: G
>>> W 5.4.0+ #1424
>>> [ 292.191411][ C47] Hardware name: Huawei Taishan 2280 /D05, BIOS
>>> Hisilicon D05 IT21 Nemo 2.0 RC0 04/18/2018
>>> [ 292.191414][ C47] Workqueue: events kobject_delayed_cleanup
>>> [ 292.191420][ C47] pstate: 80400085 (Nzcv daIf +PAN -UAO)
>>> [ 292.191423][ C47] pc : refcount_warn_saturate+0xf4/0x1b0
>>> [ 292.191427][ C47] lr : refcount_warn_saturate+0xf4/0x1b0
>>> [ 292.191429][ C47] sp : ffffff9da32574c0
>>> [ 292.191432][ C47] x29: ffffff9da32574c0 x28: 0000000000000001
>>> [ 292.191438][ C47] x27: ffffffd01231fc40 x26: ffffffd015cea179
>>> [ 292.191445][ C47] x25: ffffff9d29846818 x24: ffffffd012320160
>>> [ 292.191452][ C47] x23: ffffffd015cea510 x22: ffffffd015cea179
>>> [ 292.191458][ C47] x21: ffffff9d29844490 x20: 0000000000000002
>>> [ 292.191465][ C47] x19: ffffffd0151b186b x18: 0000000000000000
>>> [ 292.191471][ C47] x17: 0000000000000000 x16: 0000000000002760
>>> [ 292.191478][ C47] x15: 0000000000000000 x14: 726170202c70756e
>>> [ 292.191485][ C47] x13: 61656c635f746365 x12: 1ffffff3b5c94703
>>> [ 292.191491][ C47] x11: ffffffc3b5c94703 x10: dfffffd000000000
>>> [ 292.191498][ C47] x9 : ffffffc3b5c94704 x8 : 0000000000000001
>>> [ 292.191504][ C47] x7 : ffffff9dae4a381b x6 : ffffffc3b5c94704
>>> [ 292.191511][ C47] x5 : ffffffc3b5c94704 x4 : ffffffc3b5c94704
>>> [ 292.191517][ C47] x3 : ffffffd010000000 x2 : ffffffc3b464ae60
>>> [ 292.191524][ C47] x1 : 8bd48ccdcee9fb00 x0 : 0000000000000000
>>> [ 292.191530][ C47] Call trace:
>>> [ 292.191533][ C47] refcount_warn_saturate+0xf4/0x1b0
>>> [ 292.191536][ C47] kobject_get+0x9c/0x100
>>> [ 292.191539][ C47] of_node_get+0x2c/0x40
>>> [ 292.191541][ C47] of_get_parent+0x50/0x88
>>> [ 292.191544][ C47] of_fwnode_get_parent+0x4c/0xa8
>>> [ 292.191547][ C47] fwnode_count_parents+0x54/0x98
>>> [ 292.191551][ C47] fwnode_full_name_string+0x28/0xd8
>>> [ 292.191554][ C47] device_node_string+0x28c/0x5d0
>>> [ 292.191556][ C47] pointer+0x338/0x488
>>> [ 292.191559][ C47] vsnprintf+0x53c/0xad8
>>> [ 292.191562][ C47] vscnprintf+0x48/0x80
>>> [ 292.191565][ C47] vprintk_store+0x78/0x308
>>> [ 292.191567][ C47] vprintk_emit+0x258/0x600
>>> [ 292.191570][ C47] vprintk_default+0xac/0xc8
>>> [ 292.191573][ C47] vprintk_func+0x248/0x2d8
>>> [ 292.191575][ C47] printk+0xc4/0xf4
>>> [ 292.191578][ C47] of_node_release+0xd0/0x180
>>> [ 292.191581][ C47] kobject_delayed_cleanup+0x1f4/0x250
>>> [ 292.191584][ C47] process_one_work+0x800/0xd90
>>> [ 292.191587][ C47] worker_thread+0x41c/0x6c0
>>> [ 292.191590][ C47] kthread+0x1dc/0x1f8
>>> [ 292.191592][ C47] ret_from_fork+0x10/0x18
>>> [ 292.191595][ C47] irq event stamp: 2118
>>> [ 292.191600][ C47] hardirqs last enabled at (2117):
>>> [<ffffffd0115faaa0>] _raw_spin_unlock_irq+0x38/0xc0
>>> [ 292.191604][ C47] hardirqs last disabled at (2118):
>>> [<ffffffd0115ef040>] __schedule+0x310/0xd70
>>> [ 292.191609][ C47] softirqs last enabled at (1408):
>>> [<ffffffd0100e2fac>] __do_softirq+0x804/0x870
>>> [ 292.191613][ C47] softirqs last disabled at (1399):
>>> [<ffffffd0101b9d4c>] irq_exit+0x11c/0x1a8
>>> [ 292.191616][ C47] ---[ end trace f80ccb1560e43eff ]---
>>> [ 292.191620][ C47] ------------[ cut here ]------------
>>> [ 292.191623][ C47] refcount_t: underflow; use-after-free.
>>> [ 292.191628][ C47] WARNING: CPU: 47 PID: 504 at lib/refcount.c:28
>>> refcount_warn_saturate+0x12c/0x1b0
>>> [ 292.191630][ C47] Modules linked in:
>>> [ 292.191637][ C47] CPU: 47 PID: 504 Comm: kworker/47:2 Tainted: G
>>> W 5.4.0+ #1424
>>> [ 292.191641][ C47] Hardware name: Huawei Taishan 2280 /D05, BIOS
>>> Hisilicon D05 IT21 Nemo 2.0 RC0 04/18/2018
>>> [ 292.191645][ C47] Workqueue: events kobject_delayed_cleanup
>>> [ 292.191650][ C47] pstate: 80400085 (Nzcv daIf +PAN -UAO)
>>> [ 292.191653][ C47] pc : refcount_warn_saturate+0x12c/0x1b0
>>> [ 292.191656][ C47] lr : refcount_warn_saturate+0x12c/0x1b0
>>> [ 292.191659][ C47] sp : ffffff9da32574a0
>>> [ 292.191661][ C47] x29: ffffff9da32574a0 x28: 0000000000000001
>>> [ 292.191668][ C47] x27: ffffffd01231fc40 x26: ffffffd015cea179
>>> [ 292.191674][ C47] x25: ffffff9d29846818 x24: ffffffd012320160
>>> [ 292.191681][ C47] x23: ffffffd015cea510 x22: ffffff9d25c29668
>>> [ 292.191687][ C47] x21: ffffff9d29844490 x20: 0000000000000003
>>> [ 292.191694][ C47] x19: ffffffd0151b186b x18: 0000000000000000
>>> [ 292.191701][ C47] x17: 0000000000000000 x16: 000fffffc3b5c94704
>>> x8 : 0000000000000001
>>> [ 292.191733][ C47] x7 : ffffff9dae4a381b x6 : ffffffc3b5c94704
>>> [ 292.191740][ C47] x5 : ffffffc3b5c94704 x4 : ffffffc3b5c94704
>>> [ 292.191746][ C47] x3 : ffffffd010000000 x2 : ffffffc3b464ae5c
>>> [ 292.191753][ C47] x1 : 8bd48ccdcee9fb00 x0 : 0000000000000000
>>> [ 292.191759][ C47] Call trace:
>>> [ 292.191762][ C47] refcount_warn_saturate+0x12c/0x1b0
>>> [ 292.191765][ C47] kobject_put+0x1bc/0x1f8
>>> [ 292.191768][ C47] of_node_put+0x2c/0x40
>>> [ 292.191770][ C47] of_fwnode_put+0x4c/0x80
>>> [ 292.191773][ C47] fwnode_get_next_parent+0xa0/0xe8
>>> [ 292.191776][ C47] fwnode_count_parents+0x70/0x98
>>> [ 292.191779][ C47] fwnode_full_name_string+0x28/0xd8
>>> [ 292.191782][ C47] device_node_string+0x28c/0x5d0
>>> [ 292.191785][ C47] pointer+0x338/0x488
>>> [ 292.191788][ C47] vsnprintf+0x53c/0xad8
>>> [ 292.191790][ C47] vscnprintf+0x48/0x80
>>> [ 292.191793][ C47] vprintk_store+0x78/0x308
>>> [ 292.191796][ C47] vprintk_emit+0x258/0x600
>>> [ 292.191798][ C47] vprintk_default+0xac/0xc8
>>> [ 292.191801][ C47] vprintk_func+0x248/0x2d8
>>> [ 292.191804][ C47] printk+0xc4/0xf4
>>> [ 292.191806][ C47] of_node_release+0xd0/0x180
>>> [ 292.191809][ C47] kobject_delayed_cleanup+0x1f4/0x250
>>> [ 292.191812][ C47] process_one_work+0x800/0xd90
>>> [ 292.191815][ C47] worker_thread+0x41c/0x6c0
>>> [ 292.191818][ C47] kthread+0x1dc/0x1f8
>
> [...]
>
>>>
>>> Not sure if this is significant as it seems to originate from a test,
>>> but I thought I should report it anyway.
>>
>> Thanks for reporting it.
>>
>> Can I ask you to bisect to find where it first occurred?
>
> ok, but that may take a little time as it's a big'ish build.
>
>> As the first step of the bisect, I would recommend checking whether
>> the problem exists in version 5.4. As a further bound on the bisect
>> range, the furthest back the "OF: ERROR:" message could exist is
>> version 5.0, because that is when I added the message.
>>
>> If you can't bisect, I will try to reproduce the error.
>
> OK, let me get back to you early next week on this.
>
> Cheers,
> John
>
next prev parent reply other threads:[~2019-12-06 17:11 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-12-05 12:12 of_node_release() warn John Garry
2019-12-05 23:38 ` Frank Rowand
2019-12-06 9:40 ` John Garry
2019-12-06 17:11 ` John Garry [this message]
2019-12-10 7:10 ` Frank Rowand
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=6021ac63-b5e0-ed3d-f964-7c6ef579cd68@huawei.com \
--to=john.garry@huawei.com \
--cc=anders.roxell@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=frowand.list@gmail.com \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.