Vladimir Oltean 9385973fe8 net: mscc: ocelot: unregister the PTP clock on deinit ... Currently a switch driver deinit frees the regmaps, but the PTP clock is still out there, available to user space via /dev/ptpN. Any PTP operation is a ticking time bomb, since it will attempt to use the freed regmaps and thus trigger kernel panics: [ 4.291746] fsl_enetc 0000:00:00.2 eth1: error -22 setting up slave phy [ 4.291871] mscc_felix 0000:00:00.5: Failed to register DSA switch: -22 [ 4.308666] mscc_felix: probe of 0000:00:00.5 failed with error -22 [ 6.358270] Unable to handle kernel NULL pointer dereference at virtual address 0000000000000088 [ 6.367090] Mem abort info: [ 6.369888] ESR = 0x96000046 [ 6.369891] EC = 0x25: DABT (current EL), IL = 32 bits [ 6.369892] SET = 0, FnV = 0 [ 6.369894] EA = 0, S1PTW = 0 [ 6.369895] Data abort info: [ 6.369897] ISV = 0, ISS = 0x00000046 [ 6.369899] CM = 0, WnR = 1 [ 6.369902] user pgtable: 4k pages, 48-bit VAs, pgdp=00000020d58c7000 [ 6.369904] [0000000000000088] pgd=00000020d5912003, pud=00000020d5915003, pmd=0000000000000000 [ 6.369914] Internal error: Oops: 96000046 [#1] PREEMPT SMP [ 6.420443] Modules linked in: [ 6.423506] CPU: 1 PID: 262 Comm: phc_ctl Not tainted 5.4.0-03625-gb7b2a5dadd7f #204 [ 6.431273] Hardware name: LS1028A RDB Board (DT) [ 6.435989] pstate: 40000085 (nZcv daIf -PAN -UAO) [ 6.440802] pc : css_release+0x24/0x58 [ 6.444561] lr : regmap_read+0x40/0x78 [ 6.448316] sp : ffff800010513cc0 [ 6.451636] x29: ffff800010513cc0 x28: ffff002055873040 [ 6.456963] x27: 0000000000000000 x26: 0000000000000000 [ 6.462289] x25: 0000000000000000 x24: 0000000000000000 [ 6.467617] x23: 0000000000000000 x22: 0000000000000080 [ 6.472944] x21: ffff800010513d44 x20: 0000000000000080 [ 6.478270] x19: 0000000000000000 x18: 0000000000000000 [ 6.483596] x17: 0000000000000000 x16: 0000000000000000 [ 6.488921] x15: 0000000000000000 x14: 0000000000000000 [ 6.494247] x13: 0000000000000000 x12: 0000000000000000 [ 6.499573] x11: 0000000000000000 x10: 0000000000000000 [ 6.504899] x9 : 0000000000000000 x8 : 0000000000000000 [ 6.510225] x7 : 0000000000000000 x6 : ffff800010513cf0 [ 6.515550] x5 : 0000000000000000 x4 : 0000000fffffffe0 [ 6.520876] x3 : 0000000000000088 x2 : ffff800010513d44 [ 6.526202] x1 : ffffcada668ea000 x0 : ffffcada64d8b0c0 [ 6.531528] Call trace: [ 6.533977] css_release+0x24/0x58 [ 6.537385] regmap_read+0x40/0x78 [ 6.540795] __ocelot_read_ix+0x6c/0xa0 [ 6.544641] ocelot_ptp_gettime64+0x4c/0x110 [ 6.548921] ptp_clock_gettime+0x4c/0x58 [ 6.552853] pc_clock_gettime+0x5c/0xa8 [ 6.556699] __arm64_sys_clock_gettime+0x68/0xc8 [ 6.561331] el0_svc_common.constprop.2+0x7c/0x178 [ 6.566133] el0_svc_handler+0x34/0xa0 [ 6.569891] el0_sync_handler+0x114/0x1d0 [ 6.573908] el0_sync+0x140/0x180 [ 6.577232] Code: d503201f b00119a1 91022263 b27b7be4 (f9004663) [ 6.583349] ---[ end trace d196b9b14cdae2da ]--- [ 6.587977] Kernel panic - not syncing: Fatal exception [ 6.593216] SMP: stopping secondary CPUs [ 6.597151] Kernel Offset: 0x4ada54400000 from 0xffff800010000000 [ 6.603261] PHYS_OFFSET: 0xffffd0a7c0000000 [ 6.607454] CPU features: 0x10002,21806008 [ 6.611558] Memory Limit: none And now that ocelot->ptp_clock is checked at exit, prevent a potential error where ptp_clock_register returned a pointer-encoded error, which we are keeping in the ocelot private data structure. So now, ocelot->ptp_clock is now either NULL or a valid pointer. Fixes: 4e3b0468e6 ("net: mscc: PTP Hardware Clock (PHC) support") Cc: Antoine Tenart <antoine.tenart@bootlin.com> Reviewed-by: Florian Fainelli <f.fainelli@gmail.com> Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com> Signed-off-by: David S. Miller <davem@davemloft.net>		2019-12-03 11:27:00 -08:00
..
appletalk
arcnet
bonding	bonding: symmetric ICMP transmit	2019-11-16 13:02:53 -08:00
caif	Driver core patches for 5.5-rc1	2019-11-27 11:06:20 -08:00
can	can: xilinx_can: Fix usage of skb memory	2019-12-03 11:15:08 +01:00
dsa	Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net	2019-11-25 14:57:26 -08:00
ethernet	net: mscc: ocelot: unregister the PTP clock on deinit	2019-12-03 11:27:00 -08:00
fddi
fjes
hamradio
hippi
hyperv	- Support for new VMBus protocols (Andrea Parri).	2019-11-30 14:50:51 -08:00
ieee802154	drivers: net: Fix Kconfig indentation, continued	2019-11-21 11:54:09 -08:00
ipvlan	Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net	2019-11-02 13:54:56 -07:00
netdevsim	netdevsim: Update dummy reporter's devlink binary interface	2019-11-12 11:25:44 -08:00
phy	net: phy: realtek: fix using paged operations with RTL8105e / RTL8208	2019-12-01 13:14:21 -08:00
plip
ppp	compat_ioctl: remove most of fs/compat_ioctl.c	2019-12-01 13:46:15 -08:00
slip	slip: Fix use-after-free Read in slip_open	2019-11-25 11:02:52 -08:00
team	Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net	2019-11-02 13:54:56 -07:00
usb	net: usb: aqc111: Use the correct style for SPDX License Identifier	2019-11-27 11:27:01 -08:00
vmxnet3
wan
wimax	Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net	2019-11-02 13:54:56 -07:00
wireless	Merge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net	2019-11-27 17:17:40 -08:00
xen-netback
dummy.c	net: dummy: use standard dev_lstats_add() and dev_lstats_read()	2019-11-07 20:03:08 -08:00
eql.c
geneve.c
gtp.c
ifb.c
Kconfig	drivers: net: Fix Kconfig indentation, continued	2019-11-21 11:54:09 -08:00
LICENSE.SRC
loopback.c	net: use u64_stats_t in struct pcpu_lstats	2019-11-07 20:03:08 -08:00
macsec.c
macvlan.c	macvlan: schedule bc_work even if error	2019-11-25 10:54:22 -08:00
macvtap.c
Makefile
mdio.c
mii.c
net_failover.c
netconsole.c
nlmon.c	net: nlmon: use standard dev_lstats_add() and dev_lstats_read()	2019-11-07 20:03:08 -08:00
ntb_netdev.c
rionet.c
sb1000.c
Space.c
sungem_phy.c
tap.c
thunderbolt.c
tun.c	tun: fix data-race in gro_normal_list()	2019-11-15 12:46:49 -08:00
veth.c	veth: use standard dev_lstats_add() and dev_lstats_read()	2019-11-07 20:03:08 -08:00
virtio_net.c	bpf: Convert bpf_prog refcnt to atomic64_t	2019-11-18 11:41:59 +01:00
vrf.c
vsockmon.c	vsockmon: use standard dev_lstats_add() and dev_lstats_read()	2019-11-07 20:03:08 -08:00
vxlan.c	vxlan: implement get_link_ksettings ethtool method	2019-11-12 19:52:15 -08:00
xen-netfront.c