[Intel-xe] ✓ CI.checkpatch: success for drm/xe: Fix two bind races
Patchwork
patchwork at emeril.freedesktop.org
Thu Jun 29 20:54:50 UTC 2023
== Series Details ==
Series: drm/xe: Fix two bind races
URL : https://patchwork.freedesktop.org/series/120031/
State : success
== Summary ==
+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
c7d32770e3cd31d9fc134ce41f329b10aa33ee15
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 520ea8e88572877fe3e0d50578fdd4c8d1cd91e0
Author: Thomas Hellström <thomas.hellstrom at linux.intel.com>
Date: Thu Jun 29 22:51:34 2023 +0200
drm/xe: Fix the separate bind-engine race using coarse-granularity dependencies
Separate bind-engines operating on the same VM range might race updating
page-tables. To make sure that doesn't happen, each page-table update
operation needs to collect internal dependencies to await before the
job is executed.
Provide an infrastructure to do that. Initially we save a single dma-fence
for the entire VM, which thus removes the benefit of separate bind-engines
in favour of fixing the race, but a more fine-grained dependency
tracking can be achieved by using, for example, the same method as the
i915 vma_resources (an interval tree storing unsignaled fences). That
of course comes with increasing code complexity.
This patch will break the xe_vm at bind-engines-independent igt test, but that
would need an update anyway to avoid the independent binds using the
same address range. In any case, such a test would not work with the
initial xe implementation unless the binds were using different vms.
Signed-off-by: Thomas Hellström <thomas.hellstrom at linux.intel.com>
+ /mt/dim checkpatch 2ec46ad7578ebba3048d6031c1a75c21920f0e19 drm-intel
245d25479 drm/xe: Make page-table updates using the default engine happen in order
520ea8e88 drm/xe: Fix the separate bind-engine race using coarse-granularity dependencies
More information about the Intel-xe
mailing list