Den 14.01.2019 03.15, skrev David Lechner:
> On 1/13/19 3:19 PM, David Lechner wrote:
>> On 1/11/19 2:12 PM, Noralf Trønnes wrote:
>>> This switches to drm_atomic_helper_dirtyfb() as the framebuffer dirty
>>> handler. All flushing will now happen in the pipe functions.
>>>
>>> Also enable the damage plane property for all except repaper which can
>>> only do full updates.
>>>
>>> ili9225:
>>> This change made ili9225_init() equal to mipi_dbi_init() so use it.
>>>
>>> v2: Remove fb check in mipi_dbi_enable_flush() it can't be NULL
>>> (kbuild test robot)
>>>
>>> Cc: David Lechner <david@xxxxxxxxxxxxxx>
>>> Cc: Eric Anholt <eric@xxxxxxxxxx>
>>> Signed-off-by: Noralf Trønnes <noralf@xxxxxxxxxxx>
>>> Acked-by: Sam Ravnborg <sam@xxxxxxxxxxxx>
>>> Acked-by: Daniel Vetter <daniel.vetter@xxxxxxxx>
>>> ---
>>
>> Tested-by: David Lechner <david@xxxxxxxxxxxxxx>
>> Reviewed-by: David Lechner <david@xxxxxxxxxxxxxx>
>>
>> Tested with ST7586 on LEGO MINDSTORMS EV3 and ILI9225 on
>> BeagleBone Green. (FWIW, I encounter other issues that had to
>> be fixed to make things work on both of these, but none were
>> actually related to this series.)
>>
>
> Actually, I have to take this back. It appears that the problems
> that I had on LEGO MINDSTORMS EV3 are actually caused by this
> series. I did a git bisect to be sure and ended up with this
> patch as the bad commit. I'm guessing that the damage helper
> must be causing some kind of memory corruption. Here is the
> crash I am getting:
>
> Unable to handle kernel NULL pointer dereference at virtual address
> 00000004
> pgd = (ptrval)
> [00000004] *pgd=00000000
> Internal error: Oops: 5 [#1] PREEMPT ARM
> Modules linked in:
> CPU: 0 PID: 1 Comm: swapper Not tainted 5.0.0-rc1-00125-ge9acadba5409 #1389
> Hardware name: Generic DA850/OMAP-L138/AM18x
> PC is at rb_insert_color+0x1c/0x1a4
> LR is at kernfs_link_sibling+0x94/0xcc
> pc : [<c04b95ec>] lr : [<c014bfdc>] psr: 60000013
> sp : c2831b38 ip : 00000000 fp : c06b762c
> r10: 00000000 r9 : c06b835c r8 : 00000000
> r7 : c2963f00 r6 : c066b028 r5 : c2016cc0 r4 : 00000000
> r3 : 00000000 r2 : c2983010 r1 : c2963f2c r0 : c2016cd0
> Flags: nZCv IRQs on FIQs on Mode SVC_32 ISA ARM Segment none
> Control: 0005317f Table: c0004000 DAC: 00000053
> Process swapper (pid: 1, stack limit = 0x(ptrval))
> Stack: (0xc2831b38 to 0xc2832000)
> 1b20: 00000000
> c2016cc0
> 1b40: c066b028 c014bfdc c2016cc0 c2963f00 c066b028 c014c768 c2963f00
> c014c448
> 1b60: 00000000 c2016cc0 c2963f00 c066b028 c2963f00 c014c860 00000000
> 00000001
> 1b80: c0589938 c2b4b408 00000000 c014ec70 00000000 c2b4b408 00000000
> c04c4cb0
> 1ba0: 0000070f 00000000 00000000 9fd04dd9 00000000 c2b4b408 c066b028
> 00000000
> 1bc0: c293ac98 c04b58f8 c059081c 00000000 c2b4b408 c066b028 00000000
> 00000000
> 1be0: 00000000 c04b5d64 c2831c08 9fd04dd9 c2b4b3c0 c293ac80 c2bd16c0
> c2b4b408
> 1c00: c00d650c c059081c c2bd16c0 c28e3a80 c2b4b3c0 00000000 c2b4b3c0
> 00000000
> 1c20: c06b7634 00020000 c06b835c c00d72f8 00000000 c00b0c24 00000000
> 00000074
> 1c40: c0590728 00000000 c0590728 c2b4b3c0 00000074 c0590728 00020000
> 00000000
> 1c60: c06b762c c00b0958 00000000 00380bc6 00000000 c06bbf1c c2bd9c00
> c2bfe000
> 1c80: c06bbf14 0000000c 00000000 00000000 c2831e0c c00b0a90 00000000
> 00000000
> 1ca0: 00000000 00000000 003b2580 c017d6e4 00000000 00000000 00000000
> c2bfe000
> 1cc0: c2bd9c00 00000000 003b2580 00000000 00000000 c01971a0 00001103
> 00000000
> 1ce0: c2bd8400 00000400 00000400 9fd04dd9 00000000 0000000a 00000002
> 00000000
> 1d00: ffffffff 00000000 ffffffff 00000000 00000000 00000001 00000a04
> 00000032
> 1d20: 00000000 0000000c 00000004 c00af550 c2bd9ecc 9fd04dd9 c2404480
> c2bd9eb4
> 1d40: c2bd8400 c2404480 00000002 00000001 00000000 00000000 00000001
> 00000000
> 1d60: 00000000 00000000 00000000 00000000 00000000 00000000 00000001
> 00000000
> 1d80: 00380bc6 00000000 003b257f 00000000 00000077 0000ffff 00000200
> 00000002
> 1da0: 00000001 0000ffff 00000000 00000401 c2bfe22c 00000000 00000000
> 00000000
> 1dc0: 00000000 00000000 c2012400 c24be100 00001000 00000000 c2404480
> 00000000
> 1de0: 00004003 9fd04dd9 00000000 c2bd9c00 c2404480 00000083 c24044fc
> 00008000
> 1e00: 00000000 00000020 00008000 c00e4d88 c2404480 00000000 00000000
> c0190afc
> 1e20: c2bd0be0 c0692898 c0692898 00000000 00000020 c0190b10 c0194f48
> c2bd0be0
> 1e40: c066b370 c00e568c c29f5380 c01002d4 c29f5380 c2bd0be0 00008000
> c0100488
> 1e60: c0692898 00000000 c066b028 c2bd0be0 c2bd0bc0 c01033ec 00000000
> ffffff00
> 1e80: ffff0a00 c0575ff4 c2bd0be0 c281a010 c2417098 c2bd0be0 0000000a
> 00000000
> 1ea0: 0000000a 9fd04dd9 0000000a c2bd0be0 c2bd0bc0 00000000 c0575ff8
> 00008000
> 1ec0: c066b028 00008000 c0575ff4 c0104178 00000000 00000000 c2014000
> c2014005
> 1ee0: c0575ff8 c3fb1280 c0652868 c062b1ec 00000000 c01013d8 c24c3558
> 00000000
> 1f00: 00000000 00006180 c24c3558 c00f17c4 e10c76ac 0b300002 c2858015
> c281a010
> 1f20: c2415da8 9fd04dd9 00000000 00000002 c06ad310 c066b028 c066da68
> 00000000
> 1f40: 00000000 00000000 00000000 c062b478 00000000 c0037200 00306b6c
> 9fd00000
> 1f60: c0576098 9fd04dd9 00000002 c06ad310 c0652878 00000000 00000000
> 00000000
> 1f80: 00000000 00000000 00000000 c062b5e4 00000000 00000000 00000000
> 00000000
> 1fa0: c04c7860 c04c7868 00000000 c00090e0 00000000 00000000 00000000
> 00000000
> 1fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000
> 00000000
> 1fe0: 00000000 00000000 00000000 00000000 00000013 00000000 00000000
> 00000000
> [<c04b95ec>] (rb_insert_color) from [<c014bfdc>]
> (kernfs_link_sibling+0x94/0xcc)
> [<c014bfdc>] (kernfs_link_sibling) from [<c014c768>]
> (kernfs_add_one+0x90/0x140)
> [<c014c768>] (kernfs_add_one) from [<c014c860>]
> (kernfs_create_dir_ns+0x48/0x74)
> [<c014c860>] (kernfs_create_dir_ns) from [<c014ec70>]
> (sysfs_create_dir_ns+0x68/0xd0)
> [<c014ec70>] (sysfs_create_dir_ns) from [<c04b58f8>]
> (kobject_add_internal+0x9c/0x2c4)
> [<c04b58f8>] (kobject_add_internal) from [<c04b5d64>]
> (kobject_init_and_add+0x54/0x94)
> [<c04b5d64>] (kobject_init_and_add) from [<c00d650c>]
> (sysfs_slab_add+0x10c/0x220)
> [<c00d650c>] (sysfs_slab_add) from [<c00d72f8>]
> (__kmem_cache_create+0x1d8/0x338)
> [<c00d72f8>] (__kmem_cache_create) from [<c00b0958>]
> (kmem_cache_create_usercopy+0x180/0x298)
> [<c00b0958>] (kmem_cache_create_usercopy) from [<c00b0a90>]
> (kmem_cache_create+0x20/0x28)
> [<c00b0a90>] (kmem_cache_create) from [<c017d6e4>]
> (ext4_mb_init+0x374/0x44c)
> [<c017d6e4>] (ext4_mb_init) from [<c01971a0>]
> (ext4_fill_super+0x2258/0x2ef0)
> [<c01971a0>] (ext4_fill_super) from [<c00e4d88>] (mount_bdev+0x154/0x18c)
> [<c00e4d88>] (mount_bdev) from [<c0190b10>] (ext4_mount+0x14/0x20)
> [<c0190b10>] (ext4_mount) from [<c00e568c>] (mount_fs+0x14/0xa8)
> [<c00e568c>] (mount_fs) from [<c0100488>] (vfs_kern_mount+0x48/0xf0)
> [<c0100488>] (vfs_kern_mount) from [<c01033ec>] (do_mount+0x180/0xb9c)
> [<c01033ec>] (do_mount) from [<c0104178>] (ksys_mount+0x8c/0xb4)
> [<c0104178>] (ksys_mount) from [<c062b1ec>] (mount_block_root+0x128/0x2a4)
> [<c062b1ec>] (mount_block_root) from [<c062b478>] (mount_root+0x110/0x154)
> [<c062b478>] (mount_root) from [<c062b5e4>] (prepare_namespace+0x128/0x188)
> [<c062b5e4>] (prepare_namespace) from [<c04c7868>] (kernel_init+0x8/0xf4)
> [<c04c7868>] (kernel_init) from [<c00090e0>] (ret_from_fork+0x14/0x34)
> Exception stack(0xc2831fb0 to 0xc2831ff8)
> 1fa0: 00000000 00000000 00000000
> 00000000
> 1fc0: 00000000 00000000 00000000 00000000 00000000 00000000 00000000
> 00000000
> 1fe0: 00000000 00000000 00000000 00000000 00000013 00000000
> Code: e5923000 e3130001 1a000054 e92d4070 (e593c004)
> ---[ end trace 2c8fc104914a532b ]---
> Kernel panic - not syncing: Attempted to kill init! exitcode=0x0000000b
> ---[ end Kernel panic - not syncing: Attempted to kill init!
> exitcode=0x0000000b ]---
>
First step is to test if it's the damage clip allocation/freeing that's
causing problems. Setting clips to NULL avoids allocation and results in
a full update each time:
diff --git a/drivers/gpu/drm/drm_damage_helper.c
b/drivers/gpu/drm/drm_damage_helper.c
index e16aa5ae00b4..943ae17b8b1c 100644
--- a/drivers/gpu/drm/drm_damage_helper.c
+++ b/drivers/gpu/drm/drm_damage_helper.c
@@ -182,6 +182,7 @@ int drm_atomic_helper_dirtyfb(struct drm_framebuffer
*fb,
}
state->acquire_ctx = &ctx;
+ clips = NULL;
if (clips) {
uint32_t inc = 1;
If that helps, try clearing/freeing the damage explicitly:
diff --git a/drivers/gpu/drm/drm_damage_helper.c
b/drivers/gpu/drm/drm_damage_helper.c
index e16aa5ae00b4..0a59263e4733 100644
--- a/drivers/gpu/drm/drm_damage_helper.c
+++ b/drivers/gpu/drm/drm_damage_helper.c
@@ -371,6 +371,8 @@ bool drm_atomic_helper_damage_merged(const struct
drm_plane_state *old_state,
valid = true;
}
+ drm_property_replace_blob(&state->fb_damage_clips, NULL);
+
return valid;
}
EXPORT_SYMBOL(drm_atomic_helper_damage_merged);
If that doesn't help, try disabling the call to st7586_fb_dirty() so we
can rule out the driver.
If it helps, verify that the rectangle is within bounds.
Noralf.
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel
