John Garry
331d880b35
drm/hisilicon: hibmc: Do not carry error code in HiBMC framebuffer pointer
In hibmc_drm_fb_create(), when the call to hibmc_framebuffer_init() fails
with error, do not store the error code in the HiBMC device frame-buffer
pointer, as this will be later checked for non-zero value in
hibmc_fbdev_destroy() when our intention is to check for a valid function
pointer.
This fixes the following crash:
[ 9.699791] Unable to handle kernel NULL pointer dereference at virtual address 000000000000001a
[ 9.708672] Mem abort info:
[ 9.711489] ESR = 0x96000004
[ 9.714570] Exception class = DABT (current EL), IL = 32 bits
[ 9.720551] SET = 0, FnV = 0
[ 9.723631] EA = 0, S1PTW = 0
[ 9.726799] Data abort info:
[ 9.729702] ISV = 0, ISS = 0x00000004
[ 9.733573] CM = 0, WnR = 0
[ 9.736566] [000000000000001a] user address but active_mm is swapper
[ 9.742987] Internal error: Oops: 96000004 [#1] PREEMPT SMP
[ 9.748614] Modules linked in:
[ 9.751694] CPU: 16 PID: 293 Comm: kworker/16:1 Tainted: G W 4.19.0-rc4-next-20180920-00001-g9b0012c #322
[ 9.762681] Hardware name: Huawei Taishan 2280 /D05, BIOS Hisilicon D05 IT21 Nemo 2.0 RC0 04/18/2018
[ 9.771915] Workqueue: events work_for_cpu_fn
[ 9.776312] pstate: 60000005 (nZCv daif -PAN -UAO)
[ 9.781150] pc : drm_mode_object_put+0x0/0x20
[ 9.785547] lr : hibmc_fbdev_fini+0x40/0x58
[ 9.789767] sp : ffff00000af1bcf0
[ 9.793108] x29: ffff00000af1bcf0 x28: 0000000000000000
[ 9.798473] x27: 0000000000000000 x26: ffff000008f66630
[ 9.803838] x25: 0000000000000000 x24: ffff0000095abb98
[ 9.809203] x23: ffff8017db92fe00 x22: ffff8017d2b13000
[ 9.814568] x21: ffffffffffffffea x20: ffff8017d2f80018
[ 9.819933] x19: ffff8017d28a0018 x18: ffffffffffffffff
[ 9.825297] x17: 0000000000000000 x16: 0000000000000000
[ 9.830662] x15: ffff0000092296c8 x14: ffff00008939970f
[ 9.836026] x13: ffff00000939971d x12: ffff000009229940
[ 9.841391] x11: ffff0000085f8fc0 x10: ffff00000af1b9a0
[ 9.846756] x9 : 000000000000000d x8 : 6620657a696c6169
[ 9.852121] x7 : ffff8017d3340580 x6 : ffff8017d4168000
[ 9.857486] x5 : 0000000000000000 x4 : ffff8017db92fb20
[ 9.862850] x3 : 0000000000002690 x2 : ffff8017d3340480
[ 9.868214] x1 : 0000000000000028 x0 : 0000000000000002
[ 9.873580] Process kworker/16:1 (pid: 293, stack limit = 0x(____ptrval____))
[ 9.880788] Call trace:
[ 9.883252] drm_mode_object_put+0x0/0x20
[ 9.887297] hibmc_unload+0x1c/0x80
[ 9.890815] hibmc_pci_probe+0x170/0x3c8
[ 9.894773] local_pci_probe+0x3c/0xb0
[ 9.898555] work_for_cpu_fn+0x18/0x28
[ 9.902337] process_one_work+0x1e0/0x318
[ 9.906382] worker_thread+0x228/0x450
[ 9.910164] kthread+0x128/0x130
[ 9.913418] ret_from_fork+0x10/0x18
[ 9.917024] Code: a94153f3 a8c27bfd d65f03c0 d503201f (f9400c01)
[ 9.923180] ---[ end trace 2695ffa0af5be375 ]---
Fixes: d1667b8679 ("drm/hisilicon/hibmc: Add support for frame buffer")
Signed-off-by: John Garry <john.garry@huawei.com>
Reviewed-by: Xinliang Liu <z.liuxinliang@hisilicon.com>
Signed-off-by: Xinliang Liu <z.liuxinliang@hisilicon.com>
2018-09-26 11:38:52 +08:00
..
2018-08-25 18:13:10 -07:00
2018-08-08 11:05:47 +02:00
2018-08-27 14:25:54 -06:00
2018-08-18 11:04:51 -07:00
2018-08-24 11:52:34 +02:00
2018-08-30 13:39:04 -07:00
2018-08-21 16:56:20 +02:00
2018-08-28 09:58:03 -07:00
2018-08-29 08:09:20 -06:00
2018-08-20 16:30:27 -07:00
2018-08-30 14:47:41 -07:00
2018-08-19 09:56:38 -07:00
2018-08-23 13:52:46 -07:00
2018-08-25 13:16:08 +02:00
2018-08-29 13:38:39 -07:00
2018-08-25 18:43:59 -07:00
2018-08-18 11:04:51 -07:00
2018-08-26 11:48:42 -07:00
2018-09-14 11:44:59 +02:00
2018-08-17 15:13:34 +02:00
2018-08-17 16:20:27 -07:00
2018-08-23 15:44:58 -07:00
2018-08-08 15:44:47 +10:00
2018-08-20 15:38:44 -07:00
2018-09-26 11:38:52 +08:00
2018-08-20 15:59:01 -07:00
2018-08-02 10:20:59 +02:00
2018-08-26 17:45:25 -07:00
2018-07-30 20:54:51 -07:00
2018-08-23 18:48:43 -07:00
2018-08-30 23:13:59 +02:00
2018-08-22 07:40:33 -07:00
2018-08-23 18:48:43 -07:00
2018-08-23 15:34:48 -07:00
2018-08-23 13:44:43 -07:00
2018-08-25 14:12:36 -07:00
2018-08-26 09:55:28 -07:00
2018-08-16 12:26:24 -07:00
2018-08-06 23:03:12 +02:00
2018-08-24 14:42:42 +02:00
2018-08-15 09:53:07 +05:30
2018-08-25 18:13:10 -07:00
2018-08-26 11:48:42 -07:00
2018-08-23 13:52:46 -07:00
2018-07-30 23:17:53 -04:00
2018-08-21 17:40:46 -07:00
2018-08-26 11:48:42 -07:00
2018-08-23 10:00:49 +02:00
2018-08-27 20:41:46 +02:00
2018-08-02 10:23:02 +02:00
2018-08-27 11:59:39 -07:00
2018-08-25 18:43:59 -07:00
2018-08-28 08:40:44 +02:00
2018-08-30 13:53:05 -05:00
2018-08-18 11:04:51 -07:00
2018-08-22 12:34:08 -07:00
2018-08-18 12:30:42 -07:00
2018-08-18 11:04:51 -07:00
2018-07-30 10:04:58 +02:00
2018-08-20 15:38:44 -07:00
2018-08-22 14:14:15 -07:00
2018-08-23 18:48:43 -07:00
2018-08-18 11:04:51 -07:00
2018-08-20 11:36:07 +02:00
2018-08-22 10:52:51 -07:00
2018-08-20 15:38:44 -07:00
2018-08-16 17:39:55 -07:00
2018-08-25 14:12:36 -07:00
2018-07-30 23:40:23 -07:00
2018-08-20 16:30:27 -07:00
2018-08-25 18:13:10 -07:00
2018-08-26 11:48:42 -07:00
2018-08-02 13:54:06 +02:00
2018-08-23 14:02:22 -07:00
2018-08-18 16:45:27 -07:00
2018-08-09 18:47:47 +03:00
2018-09-11 14:11:01 +02:00
2018-08-26 11:48:42 -07:00
2018-08-23 13:52:46 -07:00
2018-08-28 16:11:34 -07:00
2018-07-25 10:55:29 +02:00
2018-09-11 14:11:51 +02:00
2018-08-19 09:30:44 -07:00
2018-08-23 13:52:46 -07:00
2018-08-17 11:32:50 -07:00
2018-08-27 11:59:39 -07:00
2018-09-11 14:11:54 +02:00
2018-08-24 08:45:19 -07:00
2018-08-21 18:06:27 -07:00
2018-08-22 17:31:34 -07:00
2018-08-31 08:45:16 -07:00
2018-08-18 11:04:51 -07:00