[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20180914212021.GD3876@unicorn.suse.cz>
Date: Fri, 14 Sep 2018 23:20:21 +0200
From: Michal Kubecek <mkubecek@...e.cz>
To: netdev@...r.kernel.org
Cc: Saeed Mahameed <saeedm@...lanox.com>,
Leon Romanovsky <leon@...nel.org>
Subject: mlx5_core: null pointer dereference in mlx5_accel_tls_device_caps()
(net-next kernel)
I just encountered a null pointer dereference on mlx5_core module
initialization while booting net-next kernel (based on commit
ee4fccbee7d3) on an aarch64 machine:
[ 12.021971] iommu: Adding device 0000:01:00.0 to group 3
[ 12.022925] mlx5_core 0000:01:00.0: firmware version: 12.17.2020
[ 12.022954] mlx5_core 0000:01:00.0: 63.008 Gb/s available PCIe bandwidth (8 GT/s x8 link)
[ 12.068709] Adding 98830144k swap on /dev/sda4. Priority:-2 extents:1 across:98830144k FS
[ 12.347571] (0000:01:00.0): E-Switch: Total vports 9, per vport: max uc(1024) max mc(16384)
[ 12.351962] mlx5_core 0000:01:00.0: Port module event: module 0, Cable plugged
[ 12.366306] mlx5_core 0000:01:00.0: MLX5E: StrdRq(0) RqSz(1024) StrdSz(128) RxCqeCmprss(0)
[ 12.366741] Unable to handle kernel NULL pointer dereference at virtual address 0000000000000050
[ 12.374603] Mem abort info:
[ 12.377368] ESR = 0x96000004
[ 12.380406] Exception class = DABT (current EL), IL = 32 bits
[ 12.386357] SET = 0, FnV = 0
[ 12.389347] EA = 0, S1PTW = 0
[ 12.392471] Data abort info:
[ 12.395343] ISV = 0, ISS = 0x00000004
[ 12.399156] CM = 0, WnR = 0
[ 12.402108] user pgtable: 4k pages, 48-bit VAs, pgdp = (____ptrval____)
[ 12.408711] [0000000000000050] pgd=0000000000000000
[ 12.413567] Internal error: Oops: 96000004 [#1] SMP
[ 12.418427] Modules linked in: fat mlx5_core(+) ipmi_ssif(+) aes_ce_blk crypto_simd cryptd aes_ce_cipher crc32_ce crct10dif_ce ghash_ce aes_arm64 sha2_ce sha256_arm64 sha1_ce ipmi_devintf ipmi_msghandler sbsa_gwdt tls mlxfw devlink at803x qcom_emac btrfs libcrc32c xor zlib_deflate raid6_pq ahci_platform libahci_platform hdma hdma_mgmt i2c_qup sg dm_multipath dm_mod scsi_dh_rdac scsi_dh_emc scsi_dh_alua efivarfs
[ 12.454800] CPU: 40 PID: 742 Comm: systemd-udevd Not tainted 4.19.0-rc3-ethnl.15-default #1
[ 12.463131] Hardware name: To be filled by O.E.M. To be filled by O.E.M./To be filled by O.E.M., BIOS 5.13 12/12/2012
[ 12.473722] pstate: 60400005 (nZCv daif +PAN -UAO)
[ 12.478559] pc : mlx5_accel_tls_device_caps+0x28/0x38 [mlx5_core]
[ 12.484598] lr : mlx5e_tls_build_netdev+0x24/0x98 [mlx5_core]
[ 12.490301] sp : ffff000021873a30
[ 12.493599] x29: ffff000021873a30 x28: ffff2a72560a7940
[ 12.498895] x27: ffff2a7256df6000 x26: ffff2a71a0fed650
[ 12.504190] x25: 0000000000000000 x24: ffff92c7f2b988c0
[ 12.509485] x23: ffff92c7fe01c0c0 x22: ffff2a71a0fcfa70
[ 12.514780] x21: ffff92c7f2b808c0 x20: ffff92c7f741c110
[ 12.520075] x19: ffff92c7f2b988c0 x18: ffff0000218739b0
[ 12.525370] x17: 0000000000000000 x16: ffff2a725625ade0
[ 12.530665] x15: 0000000029818ed4 x14: 00000000d47aab07
[ 12.535961] x13: 8a24000000000000 x12: 0000000000000000
[ 12.541256] x11: 0000000000000000 x10: 0000000000000000
[ 12.546551] x9 : 0000000000000000 x8 : 0000000000000000
[ 12.551846] x7 : 0000000000000000 x6 : ffff92c8159dc910
[ 12.557141] x5 : 0000000000000400 x4 : ffff7e4b205a20c7
[ 12.562436] x3 : 0000000000000000 x2 : ffff2a725625ae1c
[ 12.567731] x1 : 00000000ab078a24 x0 : 0000000000000000
[ 12.573027] Process systemd-udevd (pid: 742, stack limit = 0x(____ptrval____))
[ 12.580232] Call trace:
[ 12.582688] mlx5_accel_tls_device_caps+0x28/0x38 [mlx5_core]
[ 12.588419] mlx5e_build_nic_netdev+0x27c/0x348 [mlx5_core]
[ 12.593974] mlx5e_nic_init+0x1a0/0x258 [mlx5_core]
[ 12.598835] mlx5e_create_netdev+0x74/0x118 [mlx5_core]
[ 12.604043] mlx5e_add+0xf0/0x2c0 [mlx5_core]
[ 12.608384] mlx5_add_device+0x88/0x1a8 [mlx5_core]
[ 12.613246] mlx5_register_interface+0x78/0xb0 [mlx5_core]
[ 12.618713] mlx5e_init+0x24/0x30 [mlx5_core]
[ 12.623052] init+0x88/0xa0 [mlx5_core]
[ 12.626850] do_one_initcall+0x54/0x200
[ 12.630667] do_init_module+0x64/0x1d8
[ 12.634401] load_module+0x1480/0x1510
[ 12.638132] __se_sys_finit_module+0xc8/0xd8
[ 12.642385] __arm64_sys_finit_module+0x24/0x30
[ 12.646901] el0_svc_common+0x7c/0x118
[ 12.650631] el0_svc_handler+0x38/0x78
[ 12.654364] el0_svc+0x8/0xc
[ 12.657229] Code: d503201f f97c7e60 f9400bf3 a8c27bfd (f9402800)
[ 12.663306] ---[ end trace 57e772dd3cf718f1 ]---
The function looks like this:
------------------------------------------------------------------------
drivers/net/ethernet/mellanox/mlx5/core/accel/tls.c:
68 {
0x0000000000058230 <+0>: stp x29, x30, [sp, #-32]!
0x0000000000058234 <+4>: mov x29, sp
0x0000000000058238 <+8>: str x19, [sp, #16]
0x000000000005823c <+12>: mov x19, x0
0x0000000000058240 <+16>: mov x0, x30
drivers/net/ethernet/mellanox/mlx5/core/fpga/tls.h:
68 return mdev->fpga->tls->caps;
0x0000000000058244 <+20>: add x19, x19, #0x38, lsl #12
drivers/net/ethernet/mellanox/mlx5/core/accel/tls.c:
68 {
0x0000000000058248 <+24>: bl 0x58248
<mlx5_accel_tls_device_caps+24>
drivers/net/ethernet/mellanox/mlx5/core/fpga/tls.h:
68 return mdev->fpga->tls->caps;
0x000000000005824c <+28>: ldr x0, [x19, #30968]
drivers/net/ethernet/mellanox/mlx5/core/accel/tls.c:
70 }
0x0000000000058250 <+32>: ldr x19, [sp, #16]
0x0000000000058254 <+36>: ldp x29, x30, [sp], #32
drivers/net/ethernet/mellanox/mlx5/core/fpga/tls.h:
68 return mdev->fpga->tls->caps;
0x0000000000058258 <+40>: ldr x0, [x0, #80]
drivers/net/ethernet/mellanox/mlx5/core/accel/tls.c:
70 }
0x000000000005825c <+44>: ldr w0, [x0, #20]
0x0000000000058260 <+48>: ret
------------------------------------------------------------------------
so IIUC mdev->fpga is null (offset of tls in struct mlx5_fpga_device is
indeed 80 = 0x50).
The NIC is
Model: "Mellanox MT27700 Family [ConnectX-4]"
Vendor: pci 0x15b3 "Mellanox Technologies"
Device: pci 0x1013 "MT27700 Family [ConnectX-4]"
SubVendor: pci 0x15b3 "Mellanox Technologies"
SubDevice: pci 0x0003
Michal Kubecek
Powered by blists - more mailing lists