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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).