I want to consolidate 'struct drm_device' destruction into a single function in the next patch, and that function needs to be usable for both the primary device drm_backend::drm, and the secondary KMS devices drm_backend::kms_list. Sprites, a.k.a drm_planes, are objects fundamentally owned by their drm_device. To encapsulate drm_device destruction into a single function (that assumes that all outputs on that device are already destroyed), one cannot call destroy_sprites() from drm_shutdown() anymore - it logically belongs in drm_device_destroy(). The problem is that that will reverse the order of destroying drm_planes vs. output (GBM) surfaces. drm_plane_state holds a reference to drm_fb which points to a GBM BO that was received from a GBM surface. When the GBM surface is destroyed, all its GBM BOs are destroyed as well, which will cause all drm_fb pointing to them to be destroyed as well regardless of drm_fb reference count. Therefore, if any object was thought to hold a reference to a drm_fb, it now holds a dangling pointer. Why is drm_output_deinit() not clearing the drm_plane state it used, releasing all the drm_fb references? And do that before destroying the GBM surface? Because drm_output_deinit_planes() explicitly skips clearing the drm_plane state during compositor shutdown, because during shutdown the drm_planes have already been destroyed. If we change both, we get a much simpler tear-down logic: - drm_output_deinit_planes() will always clear drm_plane states, no special handling for shutdown (which is an elaborate way of saying "before outputs are destroyed"). - drm_output_deinit() calls drm_output_deinit_planes() before it destroys the GBM surface, which ensures no dangling drm_fb pointers are left. - destroy_sprites() call is moved *after* destruction of outputs where it logically belongs. This also means that the per-renderer fini functions do not need to clean up manually when the compositor is not shutting down. They can just assert the scanout_plane has already been cleaned up. Signed-off-by: Pekka Paalanen <pekka.paalanen@collabora.com> |
||
|---|---|---|
| .gitlab-ci | ||
| clients | ||
| data | ||
| desktop-shell | ||
| doc | ||
| frontend | ||
| fullscreen-shell | ||
| include | ||
| ivi-shell | ||
| kiosk-shell | ||
| libweston | ||
| lua-shell | ||
| man | ||
| pam | ||
| pipewire | ||
| protocol | ||
| remoting | ||
| shared | ||
| subprojects | ||
| tests | ||
| tools | ||
| wcap | ||
| westinyplus | ||
| xwayland | ||
| .editorconfig | ||
| .gitignore | ||
| .gitlab-ci.yml | ||
| .mailmap | ||
| CONTRIBUTING.md | ||
| COPYING | ||
| DCO-1.1.txt | ||
| meson.build | ||
| meson_options.txt | ||
| notes.txt | ||
| README.md | ||
| RELEASING.md | ||
| weston.ini.in | ||
Weston
Weston is a Wayland compositor designed for correctness, reliability, predictability, and performance.
Out of the box, Weston provides a very basic desktop, or a full-featured environment for non-desktop uses such as automotive, embedded, in-flight, industrial, kiosks, set-top boxes and TVs.
It also provides a library called libweston which allows users to build their own custom full-featured environments on top of Weston's core.
Building Weston
Weston is built using Meson. Weston often depends on the current release versions of Wayland and wayland-protocols.
If necessary, the latest Meson can be installed as a user with:
$ pip3 install --user meson
Weston's Meson build does not do autodetection and it defaults to all features enabled, which means you likely hit missing dependencies on the first try. If a dependency is avoidable through a build option, the error message should tell you what option can be used to avoid it. You may need to disable several features if you want to avoid certain dependencies.
$ git clone https://gitlab.freedesktop.org/wayland/weston.git
$ cd weston
$ meson build/ --prefix=...
$ ninja -C build/ install
$ cd ..
The meson command populates the build directory. This step can
fail due to missing dependencies. Any build options you want can be added on
that line, e.g. meson build/ --prefix=... -Ddemo-clients=false. All the build
options can be found in the file meson_options.txt.
Once the build directory has been successfully populated, you can inspect the
configuration with meson configure build/. If you need to change an
option, you can do e.g. meson configure build/ -Ddemo-clients=false.
Every push to the Weston master repository and its forks is built using GitLab CI. Reading the configuration may provide a useful example of how to build and install Weston.
More detailed documentation on building Weston is available on the Wayland site. There are also more details on how to run and write tests.
For building the documentation see documentation.
Running Weston
Once Weston is installed, most users can simply run it by typing weston. This
will launch Weston inside whatever environment you launch it from: when launched
from a text console, it will take over that console. When launched from inside
an existing Wayland or X11 session, it will start a 'nested' instance of Weston
inside a window in that session.
By default, Weston will start with a skeletal desktop-like environment called
desktop-shell. Other shells are available; for example, to load the kiosk
shell designed for single-application environments, you can start with:
$ weston --shell=kiosk
Help is available by running weston --help, or man weston, which will list
the available configuration options and display backends. It can also be
configured through a file on disk; more information on this can be found through
man weston.ini.
A small suite of example or demo clients are also provided: though they can be useful in themselves, their main purpose is to be an example or test case for others building compositors or clients.
Using libweston
libweston is designed to allow users to use Weston's core - its client support, backends and renderers - whilst implementing their own user interface, policy, configuration, and lifecycle. If you would like to implement your own window manager or desktop environment, we recommend building your project using the libweston API.
Building and installing Weston will also install libweston's shared library and development headers. libweston is both API-compatible and ABI-compatible within a single stable release. It is parallel-installable, so multiple stable releases can be installed and used side by side.
Documentation for libweston's API can be found within the source (see the documentation section), and also on Weston's online documentation for the current stable release.
Reporting issues and contributing
Weston's development is hosted on freedesktop.org GitLab. Please also see the contributing document, which details how to make code or non-technical contributions to Weston.
Weston and libweston are not suitable for severely memory-constrained environments
where the compositor is expected to continue running even in the face of
trivial memory allocations failing. If standard functions like malloc()
fail for small allocations,
you can expect libweston to abort.
This is only likely to occur if you have disabled your OS's 'overcommit'
functionality, and not in common cases.
Documentation
To read the Weston documentation online, head over to the Weston website.
For documenting weston we use sphinx together with breathe to process and augment code documentation from Doxygen. You should be able to install both sphinx and the breathe extension using pip3 command, or your package manager. Doxygen should be available using your distribution package manager.
Once those are set up, run meson with -Ddoc=true option in order to enable
building the documentation. Installation will place the documentation in the
prefix's path under datadir (i.e., share/doc).
Adding and improving documentation
For re-generating the documentation a special docs target has been added.
Although first time you build (and subsequently install) weston, you'll see the
documentation being built, updates to the spinx documentation files or to the
source files will only be updated when using docs target!
Example:
$ ninja install # generates and installs the documentation
# time passes, hack hack, add doc in sources or rST files
$ ninja install # not sufficient, docs will not be updated
$ ninja docs && ninja install # run 'docs' then install
Improving/adding documentation can be done by modifying rST files under
doc/sphinx/ directory or by modifying the source code using doxygen
directives.
