Re: [Buildroot] [PATCH v9 7/7] support/testing/tests/package/test_flutter.py: new runtime test

["Yann E. MORIN" <yann.morin.1998@free.fr>]

Adam, All,

+Julien for his expertise in graphics in runtime tests

On 2023-09-19 14:42 -0600, Adam Duskett spake thusly:
> This is a simple test that builds and runs the futter-gallery application and
> checks if the service is active.

This runtime test does not demonstrate a worlking flutter stack, because
the application in fact does not work:

    $ journalctl -u flutter-gallery
    [...]
    journalctl -u flutter-gallery --boot |cat
    Sep 23 08:18:24 buildroot systemd[1]: Started flutter-gallery daemon.
    Sep 23 08:18:24 buildroot flutter-pi[129]: [locales] Warning: The system has no configured locale. The default "C" locale may or may not be supported by the app.
    Sep 23 08:18:24 buildroot flutter-pi[129]: MESA-LOADER: failed to open vkms: /usr/lib/dri/vkms_dri.so: cannot open shared object file: No such file or directory (search paths /usr/lib/dri, suffix _dri)
    Sep 23 08:18:24 buildroot flutter-pi[129]: MESA-LOADER: failed to open zink: /usr/lib/dri/zink_dri.so: cannot open shared object file: No such file or directory (search paths /usr/lib/dri, suffix _dri)
    Sep 23 08:18:28 buildroot flutter-pi[129]: xkbcommon: ERROR: failed to add default include path /usr/share/X11/xkb
    Sep 23 08:18:28 buildroot flutter-pi[129]: keyboard.c: Could not create XKB context.
    Sep 23 08:18:28 buildroot flutter-pi[129]: user_input.c: Could not initialize keyboard configuration. Flutter-pi will run without text/raw keyboard input.
    Sep 23 08:18:28 buildroot flutter-pi[129]: egl_gbm_render_surface.c: Couldn't create GBM surface for rendering.  gbm_surface_create_with_modifiers: Invalid argument
    Sep 23 08:18:28 buildroot flutter-pi[129]: window.c: Couldn't create EGL GBM rendering surface.
    Sep 23 08:18:28 buildroot systemd[1]: flutter-gallery.service: Main process exited, code=dumped, status=11/SEGV
    Sep 23 08:18:28 buildroot systemd[1]: flutter-gallery.service: Failed with result 'core-dump'.
    Sep 23 08:18:28 buildroot systemd[1]: flutter-gallery.service: Consumed 4.144s CPU time.
    Sep 23 08:18:29 buildroot systemd[1]: flutter-gallery.service: Scheduled restart job, restart counter is at 1.
    Sep 23 08:18:29 buildroot systemd[1]: Started flutter-gallery daemon.
    Sep 23 08:18:29 buildroot flutter-pi[148]: [locales] Warning: The system has no configured locale. The default "C" locale may or may not be supported by the app.
    Sep 23 08:18:29 buildroot flutter-pi[148]: MESA-LOADER: failed to open vkms: /usr/lib/dri/vkms_dri.so: cannot open shared object file: No such file or directory (search paths /usr/lib/dri, suffix _dri)
    Sep 23 08:18:29 buildroot flutter-pi[148]: MESA-LOADER: failed to open zink: /usr/lib/dri/zink_dri.so: cannot open shared object file: No such file or directory (search paths /usr/lib/dri, suffix _dri)
    Sep 23 08:18:31 buildroot flutter-pi[148]: xkbcommon: ERROR: failed to add default include path /usr/share/X11/xkb
    Sep 23 08:18:31 buildroot flutter-pi[148]: keyboard.c: Could not create XKB context.
    Sep 23 08:18:31 buildroot flutter-pi[148]: user_input.c: Could not initialize keyboard configuration. Flutter-pi will run without text/raw keyboard input.
    Sep 23 08:18:31 buildroot flutter-pi[148]: egl_gbm_render_surface.c: Couldn't create GBM surface for rendering.  gbm_surface_create_with_modifiers: Invalid argument
    Sep 23 08:18:31 buildroot flutter-pi[148]: window.c: Couldn't create EGL GBM rendering surface.
    Sep 23 08:18:31 buildroot systemd[1]: flutter-gallery.service: Main process exited, code=dumped, status=11/SEGV
    Sep 23 08:18:31 buildroot systemd[1]: flutter-gallery.service: Failed with result 'core-dump'.
    Sep 23 08:18:31 buildroot systemd[1]: flutter-gallery.service: Consumed 2.401s CPU time.
    Sep 23 08:18:31 buildroot systemd[1]: flutter-gallery.service: Scheduled restart job, restart counter is at 2.
    [...ad nauseam......]

So, a few things about this:

  - "-vga std -vnc none" are not needed; whether specified or not has
    no impact on the test. And indeed, this is now n aarch64 machine,
    so not sure there's vga on that;

  - the locales issues is not major, we don't care much for a test; the
    keybioard issue should not be major either ("Flutter-pi will run
    without text/raw keyboard input");

  - the real issues I'm most afraid of, are the one about mesa failing
    to load the proper drivers, and/or that it can't create an EGL GBM
    surface; I think this is the real problem;

  - in the console, I notice "vkms_vblank_simulate: vblank timer
    overrun" bwing displayed quite a few times;

  - you're pointing the gallery app to /usr/share/flutter/gallery/release/
    but therat does not exist (what should it contain, btw?).

Can you look at that test, please?

Regards,
Yann E. MORIN.

> Signed-off-by: Adam Duskett <adam.duskett@amarulasolutions.com>
> ---
> v8 -> v9:
>   - Changed to aarch64 to avoid problems with older versions of qemu.
>   - Added BR2_LINUX_KERNEL_NEEDS_HOST_LIBELF
> 
>  DEVELOPERS                                    |  1 +
>  support/testing/tests/package/test_flutter.py | 56 +++++++++++++++++++
>  .../package/test_flutter/linux-vkms.fragment  |  2 +
>  .../flutter-gallery.service                   |  1 +
>  .../systemd/system/flutter-gallery.service    | 11 ++++
>  5 files changed, 71 insertions(+)
>  create mode 100644 support/testing/tests/package/test_flutter.py
>  create mode 100644 support/testing/tests/package/test_flutter/linux-vkms.fragment
>  create mode 120000 support/testing/tests/package/test_flutter/overlay/etc/systemd/system/multi-user.target.wants/flutter-gallery.service
>  create mode 100644 support/testing/tests/package/test_flutter/overlay/usr/lib/systemd/system/flutter-gallery.service
> 
> diff --git a/DEVELOPERS b/DEVELOPERS
> index 48a4ec776a..f46d83679a 100644
> --- a/DEVELOPERS
> +++ b/DEVELOPERS
> @@ -37,6 +37,7 @@ F:	package/flutter-engine/
>  F:	package/flutter-gallery/
>  F:	package/flutter-pi/
>  F:	package/flutter-sdk-bin/
> +F:	support/testing/tests/package/test_flutter.py
>  
>  N:	Adam Heinrich <adam@adamh.cz>
>  F:	package/jack1/
> diff --git a/support/testing/tests/package/test_flutter.py b/support/testing/tests/package/test_flutter.py
> new file mode 100644
> index 0000000000..e17ed9f659
> --- /dev/null
> +++ b/support/testing/tests/package/test_flutter.py
> @@ -0,0 +1,56 @@
> +import os
> +import time
> +import infra.basetest
> +
> +
> +class TestFlutter(infra.basetest.BRTest):
> +    config = f"""
> +        BR2_aarch64=y
> +        BR2_TOOLCHAIN_EXTERNAL=y
> +        BR2_TOOLCHAIN_EXTERNAL_BOOTLIN=y
> +        BR2_ROOTFS_OVERLAY="{infra.filepath("tests/package/test_flutter/overlay")}"
> +        BR2_PER_PACKAGE_DIRECTORIES=y
> +        BR2_INIT_SYSTEMD=y
> +        BR2_LINUX_KERNEL=y
> +        BR2_LINUX_KERNEL_CUSTOM_VERSION=y
> +        BR2_LINUX_KERNEL_CUSTOM_VERSION_VALUE="6.1.54"
> +        BR2_LINUX_KERNEL_USE_CUSTOM_CONFIG=y
> +        BR2_LINUX_KERNEL_NEEDS_HOST_LIBELF=y
> +        BR2_LINUX_KERNEL_CUSTOM_CONFIG_FILE="board/qemu/aarch64-virt/linux.config"
> +        BR2_LINUX_KERNEL_CONFIG_FRAGMENT_FILES="{infra.filepath("tests/package/test_flutter/linux-vkms.fragment")}"
> +        BR2_PACKAGE_LIBDRM=y
> +        BR2_PACKAGE_MESA3D=y
> +        BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_SWRAST=y
> +        BR2_PACKAGE_MESA3D_GALLIUM_DRIVER_VIRGL=y
> +        BR2_PACKAGE_MESA3D_OPENGL_ES=y
> +        BR2_PACKAGE_FLUTTER_PI=y
> +        BR2_PACKAGE_FLUTTER_PI_RAW_KEYBOARD_PLUGIN=y
> +        BR2_PACKAGE_FLUTTER_PI_TEXT_INPUT_PLUGIN=y
> +        BR2_PACKAGE_FLUTTER_GALLERY=y
> +        BR2_PACKAGE_FLUTTER_ENGINE=y
> +        BR2_TARGET_ROOTFS_EXT2=y
> +        BR2_TARGET_ROOTFS_EXT2_4=y
> +        BR2_TARGET_ROOTFS_EXT2_SIZE="512M"
> +        # BR2_TARGET_ROOTFS_TAR is not set
> +        """
> +
> +    def test_run(self):
> +        timeout = 35 * self.emulator.timeout_multiplier
> +        img = os.path.join(self.builddir, "images", "rootfs.ext2")
> +        kern = os.path.join(self.builddir, "images", "Image")
> +        self.emulator.boot(
> +            arch="aarch64",
> +            kernel=kern,
> +            kernel_cmdline=["root=/dev/vda console=ttyAMA0"],
> +            options=["-M", "virt",
> +                     "-cpu", "cortex-a57",
> +                     "-m", "512M",
> +                     "-smp", "4",
> +                     "-vga", "std",
> +                     "-vnc", "none",
> +                     "-drive", f"file={img},if=virtio,format=raw"])
> +        self.emulator.login()
> +        cmd = "systemctl is-active flutter-gallery"
> +        output, exit_code = self.emulator.run(cmd, 10)
> +        self.assertEqual(exit_code, 0)
> +        self.assertEqual(output[0], "active")
> diff --git a/support/testing/tests/package/test_flutter/linux-vkms.fragment b/support/testing/tests/package/test_flutter/linux-vkms.fragment
> new file mode 100644
> index 0000000000..3fc7a5dded
> --- /dev/null
> +++ b/support/testing/tests/package/test_flutter/linux-vkms.fragment
> @@ -0,0 +1,2 @@
> +CONFIG_DEBUG_FS=y
> +CONFIG_DRM_VKMS=y
> diff --git a/support/testing/tests/package/test_flutter/overlay/etc/systemd/system/multi-user.target.wants/flutter-gallery.service b/support/testing/tests/package/test_flutter/overlay/etc/systemd/system/multi-user.target.wants/flutter-gallery.service
> new file mode 120000
> index 0000000000..40993fb16c
> --- /dev/null
> +++ b/support/testing/tests/package/test_flutter/overlay/etc/systemd/system/multi-user.target.wants/flutter-gallery.service
> @@ -0,0 +1 @@
> +../../../../usr/lib/systemd/system/flutter-gallery.service
> \ No newline at end of file
> diff --git a/support/testing/tests/package/test_flutter/overlay/usr/lib/systemd/system/flutter-gallery.service b/support/testing/tests/package/test_flutter/overlay/usr/lib/systemd/system/flutter-gallery.service
> new file mode 100644
> index 0000000000..88a2bcbf0b
> --- /dev/null
> +++ b/support/testing/tests/package/test_flutter/overlay/usr/lib/systemd/system/flutter-gallery.service
> @@ -0,0 +1,11 @@
> +[Unit]
> +Description=flutter-gallery daemon
> +After=dbus.service systemd-udevd.service
> +
> +[Service]
> +ExecStart=/usr/bin/flutter-pi --release /usr/share/flutter/gallery/release/
> +Restart=always
> +KillMode=process
> +
> +[Install]
> +WantedBy=multi-user.target
> -- 
> 2.39.2
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot