Fails tests/drivers/display/display_read_write on SDL display driver

[soburi]

Describe the bug

Fails tests/drivers/display/display_read_write on SDL display driver.

At a glance, the #81184 change seems

display_read() behavior.

To Reproduce

 west twister -p native_sim -T tests/drivers/display/display_read_write/ 

Note: Currently, the test is disabled by the harness: display line.
To run the test, we need to remove this line from
tests/drivers/display/display_read_write/testcase.yaml

Expected behavior
Pass the test.

[aescolar]

CC @Finomnis @faxe1008 @pdgendt @kartben

[Finomnis]

What's the error?

[soburi]

@Finomnis

What's the error?

As mentioned above, to run the tests/drivers/display/display_read_write/ test, you must modify the testcase.yaml. However, two errors will occur if you do so and run the tests. One is an error where the thread priority is inappropriate and the test stops with an assert.

ASSERTION FAIL [((((prio)) == 15 && z_is_idle_thread_entry((entry))) || (((15 - 1) >= ((-16))) && ((prio)) >= ((-16)) && ((prio)) <= (15 - 1)))] @ WEST_TOPDIR/zephyr/kernel/thread.c:517
 invalid priority (15); allowed range: 14 to -16
@WEST_TOPDIR/zephyr/lib/os/assert.c:43
[00:00:00.000,000] <err> os: >>> ZEPHYR FATAL ERROR 4: Kernel panic on CPU 0
[00:00:00.000,000] <err> os: Current thread: 0x8140dc0 (main)
[00:00:00.000,000] <err> os: Halting system

This can be avoided by adjusting CONFIG_SDL_DISPLAY_THREAD_PRIORITY.
But I have no idea how to fix it properly at this point.

And if I work around it, the test will fail.
This test reads data written in the framebuffer memory and checks whether it is the expected content.
I think the reading behavior has probably changed.

The test passed in versions before 02d562e.
The fix in 02d562e
adds support for transparent colors, but it still affects settings other than ARGB8888. It seems strange.

*** Booting Zephyr OS build v4.1.0-2646-ge41909a32ce9 ***
Running TESTSUITE display_read_write
====================================================================
START - test_clear

 Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/display/display_read_write/src/main.c:88: verify_background_color: (buf32[i] not equal to color)
@0
 FAIL - test_clear in 0.000 seconds
====================================================================
START - test_read_does_not_clear_existing_buffer

 Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/display/display_read_write/src/main.c:64: verify_bytes_of_area: data not equal to disp_buffer

 FAIL - test_read_does_not_clear_existing_buffer in 0.000 seconds
====================================================================
START - test_write_to_buffer_head

 Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/display/display_read_write/src/main.c:64: verify_bytes_of_area: data not equal to disp_buffer

 FAIL - test_write_to_buffer_head in 0.000 seconds
====================================================================
START - test_write_to_buffer_tail

 Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/display/display_read_write/src/main.c:210: display_read_write_test_write_to_buffer_tail: data not equal to disp_buffer + (display_width * display_height * bpp - buf_size)

 FAIL - test_write_to_buffer_tail in 0.000 seconds
====================================================================
TESTSUITE display_read_write failed.

[Finomnis]

The thread priority issue is probably related to #87883.

@faxe1008

[faxe1008]

@soburi @Finomnis changing the priority to 8 I get:

*** Booting Zephyr OS build v4.1.0-2705-g76c266c0a70d ***
Running TESTSUITE display_read_write
===================================================================
START - test_clear

    Assertion failed at WEST_TOPDIR/zephyr/tests/drivers/display/display_read_write/src/main.c:106: verify_background_color: (!!(*tptr & BIT(i % 8)) not equal to !!(color))
@0
 FAIL - test_clear in 0.000 seconds
===================================================================
START - test_read_does_not_clear_existing_buffer
ASSERTION FAIL [(desc->height % 8) == 0U] @ WEST_TOPDIR/zephyr/drivers/display/display_sdl.c:273
	Input buffer height not aligned per 8 pixels
@ WEST_TOPDIR/zephyr/lib/os/assert.c:43
[00:00:00.000,000] <err> os: >>> ZEPHYR FATAL ERROR 4: Kernel panic on CPU 0
[00:00:00.000,000] <err> os: Current thread: 0x464500 (test_read_does_not_clear_existi)
[00:00:00.000,000] <err> os: Halting system

If I change the value to 0 I get the same output as you did.

[Finomnis]

I probably won't be about to contribute much, I have very little time right now