| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <896514df-af33-6408-8b33-d8fd06e671ef@arinc9.com>
Date: Sat, 15 Apr 2023 01:41:07 +0300
From: Arınç ÜNAL <arinc.unal@...nc9.com>
To: Vladimir Oltean <olteanv@...il.com>,
Daniel Golle <daniel@...rotopia.org>,
Frank Wunderlich <frank-w@...lic-files.de>
Cc: netdev <netdev@...r.kernel.org>, erkin.bozoglu@...ont.com
Subject: mt7530: dsa_switch_parse_of() fails, causes probe code to run twice
Hey there,
I've been working on the MT7530 DSA subdriver. While doing some tests, I
realised mt7530_probe() runs twice. I moved enabling the regulators from
mt7530_setup() to mt7530_probe(). Enabling the regulators there ends up
with exception warnings on the first time. It works fine when
mt7530_probe() is run again.
This should not be an expected behaviour, right? Any ideas how we can make
it work the first time?
diff --git a/drivers/net/dsa/mt7530.c b/drivers/net/dsa/mt7530.c
index 02410ac439b7..57b262099791 100644
--- a/drivers/net/dsa/mt7530.c
+++ b/drivers/net/dsa/mt7530.c
@@ -3248,6 +3248,8 @@ mt7530_probe(struct mdio_device *mdiodev)
struct mt7530_priv *priv;
struct device_node *dn;
+ dev_info(&mdiodev->dev, "mt7530_probe() is running\n");
+
dn = mdiodev->dev.of_node;
priv = devm_kzalloc(&mdiodev->dev, sizeof(*priv), GFP_KERNEL);
diff --git a/net/dsa/dsa.c b/net/dsa/dsa.c
index e5f156940c67..738ca4420e85 100644
--- a/net/dsa/dsa.c
+++ b/net/dsa/dsa.c
@@ -1488,9 +1488,12 @@ static int dsa_switch_probe(struct dsa_switch *ds)
return -EINVAL;
if (np) {
+ dev_info(ds->dev, "np is true\n");
err = dsa_switch_parse_of(ds, np);
- if (err)
+ if (err) {
+ dev_info(ds->dev, "np dts parse failed\n");
dsa_switch_release_ports(ds);
+ }
} else if (pdata) {
err = dsa_switch_parse(ds, pdata);
if (err)
@@ -1502,6 +1505,8 @@ static int dsa_switch_probe(struct dsa_switch *ds)
if (err)
return err;
+ dev_info(ds->dev, "np is successful\n");
+
dst = ds->dst;
dsa_tree_get(dst);
err = dsa_tree_setup(dst);
[ 3.150567] mt7530 mdio-bus:1f: mt7530_probe() is running
[ 3.156403] mt7530 mdio-bus:1f: np is true
[ 3.160608] mt7530 mdio-bus:1f: np dts parse failed
[ 3.167094] mtk_soc_eth 1b100000.ethernet: generated random MAC address 96:70:de:9e:c0:88
[ 3.176535] mtk_soc_eth 1b100000.ethernet eth0: mediatek frame engine at 0xf09e0000, irq 213
[...]
[ 4.121791] mt7530 mdio-bus:1f: mt7530_probe() is running
[ 4.127678] mt7530 mdio-bus:1f: np is true
[ 4.138242] mt7530 mdio-bus:1f: np is successful
[ 4.154957] mt7530 mdio-bus:1f: no interrupt support
[ 4.189915] mt7530 mdio-bus:1f: configuring for fixed/trgmii link mode
[ 4.198619] mt7530 mdio-bus:1f: Link is Up - 1Gbps/Full - flow control rx/tx
[ 4.206437] mt7530 mdio-bus:1f wan (uninitialized): PHY [mt7530-0:00] driver [MediaTek MT7530 PHY] (irq=POLL)
[ 4.218450] mt7530 mdio-bus:1f lan0 (uninitialized): PHY [mt7530-0:01] driver [MediaTek MT7530 PHY] (irq=POLL)
[ 4.230201] mt7530 mdio-bus:1f lan1 (uninitialized): PHY [mt7530-0:02] driver [MediaTek MT7530 PHY] (irq=POLL)
[ 4.242000] mt7530 mdio-bus:1f lan2 (uninitialized): PHY [mt7530-0:03] driver [MediaTek MT7530 PHY] (irq=POLL)
[ 4.253755] mt7530 mdio-bus:1f lan3 (uninitialized): PHY [mt7530-0:04] driver [MediaTek MT7530 PHY] (irq=POLL)
[ 4.265271] mtk_soc_eth 1b100000.ethernet eth0: entered promiscuous mode
[ 4.272101] DSA: tree 0 setup
The exceptions:
[ 8.160099] ------------[ cut here ]------------
[ 8.164753] WARNING: CPU: 3 PID: 1 at drivers/regulator/core.c:2405 _regulator_put+0x170/0x178
[ 8.173450] Modules linked in:
[ 8.176519] CPU: 3 PID: 1 Comm: swapper/0 Not tainted 6.3.0-rc6-next-20230413+ #17
[ 8.184105] Hardware name: Mediatek Cortex-A7 (Device Tree)
[ 8.189693] unwind_backtrace from show_stack+0x18/0x1c
[ 8.194947] show_stack from dump_stack_lvl+0x40/0x4c
[ 8.200022] dump_stack_lvl from __warn+0x80/0x12c
[ 8.204839] __warn from warn_slowpath_fmt+0xc0/0x184
[ 8.209916] warn_slowpath_fmt from _regulator_put+0x170/0x178
[ 8.215775] _regulator_put from regulator_put+0x24/0x34
[ 8.221109] regulator_put from release_nodes+0x50/0xc4
[ 8.226356] release_nodes from devres_release_all+0x84/0xd0
[ 8.232034] devres_release_all from device_unbind_cleanup+0x14/0x68
[ 8.238411] device_unbind_cleanup from really_probe+0x268/0x400
[ 8.244441] really_probe from __driver_probe_device+0xa4/0x208
[ 8.250384] __driver_probe_device from driver_probe_device+0x38/0xc8
[ 8.256847] driver_probe_device from __device_attach_driver+0xb0/0x128
[ 8.263484] __device_attach_driver from bus_for_each_drv+0x98/0xec
[ 8.269773] bus_for_each_drv from __device_attach+0xb0/0x1dc
[ 8.275540] __device_attach from bus_probe_device+0x90/0x94
[ 8.281221] bus_probe_device from device_add+0x4d4/0x6c4
[ 8.286639] device_add from mdio_device_register+0x44/0x88
[ 8.292230] mdio_device_register from __of_mdiobus_register+0x1d8/0x3cc
[ 8.298949] __of_mdiobus_register from mtk_mdio_init+0x1c4/0x23c
[ 8.305065] mtk_mdio_init from mtk_probe+0x7cc/0x8ac
[ 8.310140] mtk_probe from platform_probe+0x64/0xb8
[ 8.315123] platform_probe from really_probe+0xe8/0x400
[ 8.320453] really_probe from __driver_probe_device+0xa4/0x208
[ 8.326396] __driver_probe_device from driver_probe_device+0x38/0xc8
[ 8.332859] driver_probe_device from __driver_attach+0x124/0x1d4
[ 8.338975] __driver_attach from bus_for_each_dev+0x84/0xd4
[ 8.344656] bus_for_each_dev from bus_add_driver+0xe8/0x208
[ 8.350335] bus_add_driver from driver_register+0x84/0x11c
[ 8.355930] driver_register from do_one_initcall+0x60/0x210
[ 8.361612] do_one_initcall from kernel_init_freeable+0x214/0x270
[ 8.367817] kernel_init_freeable from kernel_init+0x20/0x138
[ 8.373588] kernel_init from ret_from_fork+0x14/0x2c
[ 8.378658] Exception stack(0xf0825fb0 to 0xf0825ff8)
[ 8.383720] 5fa0: 00000000 00000000 00000000 00000000
[ 8.391910] 5fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000
[ 8.400099] 5fe0: 00000000 00000000 00000000 00000000 00000013 00000000
[ 8.406761] ---[ end trace 0000000000000000 ]---
[ 8.411588] ------------[ cut here ]------------
[ 8.416221] WARNING: CPU: 3 PID: 1 at drivers/regulator/core.c:2405 _regulator_put+0x170/0x178
[ 8.424906] Modules linked in:
[ 8.428001] CPU: 3 PID: 1 Comm: swapper/0 Tainted: G W 6.3.0-rc6-next-20230413+ #17
[ 8.437064] Hardware name: Mediatek Cortex-A7 (Device Tree)
[ 8.442646] unwind_backtrace from show_stack+0x18/0x1c
[ 8.447898] show_stack from dump_stack_lvl+0x40/0x4c
[ 8.452970] dump_stack_lvl from __warn+0x80/0x12c
[ 8.457787] __warn from warn_slowpath_fmt+0xc0/0x184
[ 8.462864] warn_slowpath_fmt from _regulator_put+0x170/0x178
[ 8.468722] _regulator_put from regulator_put+0x24/0x34
[ 8.474056] regulator_put from release_nodes+0x50/0xc4
[ 8.479302] release_nodes from devres_release_all+0x84/0xd0
[ 8.484980] devres_release_all from device_unbind_cleanup+0x14/0x68
[ 8.491355] device_unbind_cleanup from really_probe+0x268/0x400
[ 8.497385] really_probe from __driver_probe_device+0xa4/0x208
[ 8.503327] __driver_probe_device from driver_probe_device+0x38/0xc8
[ 8.509790] driver_probe_device from __device_attach_driver+0xb0/0x128
[ 8.516427] __device_attach_driver from bus_for_each_drv+0x98/0xec
[ 8.522716] bus_for_each_drv from __device_attach+0xb0/0x1dc
[ 8.528484] __device_attach from bus_probe_device+0x90/0x94
[ 8.534165] bus_probe_device from device_add+0x4d4/0x6c4
[ 8.539582] device_add from mdio_device_register+0x44/0x88
[ 8.545172] mdio_device_register from __of_mdiobus_register+0x1d8/0x3cc
[ 8.551889] __of_mdiobus_register from mtk_mdio_init+0x1c4/0x23c
[ 8.558004] mtk_mdio_init from mtk_probe+0x7cc/0x8ac
[ 8.563079] mtk_probe from platform_probe+0x64/0xb8
[ 8.568062] platform_probe from really_probe+0xe8/0x400
[ 8.573392] really_probe from __driver_probe_device+0xa4/0x208
[ 8.579335] __driver_probe_device from driver_probe_device+0x38/0xc8
[ 8.585799] driver_probe_device from __driver_attach+0x124/0x1d4
[ 8.591914] __driver_attach from bus_for_each_dev+0x84/0xd4
[ 8.597594] bus_for_each_dev from bus_add_driver+0xe8/0x208
[ 8.603274] bus_add_driver from driver_register+0x84/0x11c
[ 8.608868] driver_register from do_one_initcall+0x60/0x210
[ 8.614549] do_one_initcall from kernel_init_freeable+0x214/0x270
[ 8.620753] kernel_init_freeable from kernel_init+0x20/0x138
[ 8.626523] kernel_init from ret_from_fork+0x14/0x2c
[ 8.631594] Exception stack(0xf0825fb0 to 0xf0825ff8)
[ 8.636654] 5fa0: 00000000 00000000 00000000 00000000
[ 8.644844] 5fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000 00000000
[ 8.653033] 5fe0: 00000000 00000000 00000000 00000000 00000013 00000000
[ 8.659681] ---[ end trace 0000000000000000 ]---
Arınç
Powered by blists - more mailing lists