Re: [igt-dev] [PATCH i-g-t v2 3/4] lib/igt_kmod: add compatibility for KUnit

2022-11-03 Thread Petri Latvala
> mock: [drm] Incompatible option enable_guc=3 - HuC is not supported! > kmsg> mock: [drm] Incompatible option enable_guc=3 - GuC submission is N/A > kmsg> i915: Running intel_memory_region_mock_selftests/igt_mock_reserve > kmsg> i915: Running intel_memory_region_mock_selftests/igt_mock_fill > kmsg> i915: Running intel_memory_region_mock_selftests/igt_mock_contiguous > kmsg> i915: Running > intel_memory_region_mock_selftests/igt_mock_splintered_region > kmsg> i915: Running intel_memory_region_mock_selftests/igt_mock_max_segment > kmsg> i915: Running intel_memory_region_mock_selftests/igt_mock_io_size > kmsg> igt_mock_io_size with ps=1, io_size=1000, total=cf11 > kmsg> igt_mock_io_size mappable theft=(0MiB/256MiB), total=3313MiB > kmsg> ok 18 - mock_memory_region > kmsg> # i915 mock selftests: pass:18 fail:0 skip:0 total:18 > kmsg> # Totals: pass:18 fail:0 skip:0 total:18 > kmsg> ok 1 - i915 mock selftests > SUCCESS (29.259s) > > > Basically, the current output will break CI reports presented at: > > https://intel-gfx-ci.01.org/tree/drm-tip/bat-all.html?testfilter=selftest > > The IGT output after your change should be similar to what we had > before, e. g. each test should output SUCCESS/FAIL/SKIP (plus the > final result of all tests at the suite). That should not be done manually though. Only with igt_subtest / igt_dynamic blocks. -- Petri Latvala

Re: [igt-dev] [PATCH i-g-t v2 4/4] tests: DRM selftests: switch to KUnit

2022-11-01 Thread Petri Latvala
On Tue, Nov 01, 2022 at 01:54:25PM +0100, Mauro Carvalho Chehab wrote: > On Tue, 20 Sep 2022 11:18:25 +0300 > Petri Latvala wrote: > > > On Sun, Aug 28, 2022 at 09:09:20PM -0300, Isabella Basso wrote: > > > As the DRM selftests are now using KUnit [1], update IGT tests

Re: [igt-dev] [PATCH i-g-t v2 4/4] tests: DRM selftests: switch to KUnit

2022-09-20 Thread Petri Latvala
kunit but in LTS kernels they're not, and we'd like to be able to keep testing those. That means having both launchers here and just letting the "incorrect" one produce a 'skip'. -- Petri Latvala > diff --git a/tests/drm_mm.c b/tests/drm_mm.c > index 2052

Re: [Intel-gfx] [PATCH v2] drm/i915: fix i915_gem_object_wait_moving_fence

2022-04-08 Thread Petri Latvala
("drm/i915: drop bo->moving dependency") > Signed-off-by: Matthew Auld > Cc: Christian König > Cc: Lucas De Marchi > Cc: Daniel Vetter > Reviewed-by: Christian König #v1 For the record, patchwork is disabled at this time. Trybot is still up if you want CI to ve

Re: [Intel-gfx] [PATCH 2/3] drm/i915/dg2: Add initial gt/ctx/engine workarounds

2021-11-12 Thread Petri Latvala
gine_init_whitelist(struct intel_engine_cs > *engine) > > wa_init_start(w, "whitelist", engine->name); > > - if (IS_XEHPSDV(i915)) > + if (IS_DG2(i915)) > + dg2_whitelist_build(engine); > + else if (IS_XEHPSDV(i915)) >

Re: [igt-dev] [RFC PATCH 0/1] drm: selftest: Convert to KUnit

2021-11-11 Thread Petri Latvala
that I spotted but nothing that is unfixable when it's time to review in detail. -- Petri Latvala

Re: [PATCH] Revert "drm: add a locked version of drm_is_current_master"

2021-06-22 Thread Petri Latvala
e3090477fbde066ff314a7e9721ee0f > Fixes: 1815d9c86e30 ("drm: add a locked version of drm_is_current_master") > Cc: Desmond Cheong Zhi Xi > Cc: Emil Velikov > Cc: sta...@vger.kernel.org > Signed-off-by: Daniel Vetter > Cc: Maarten Lankhorst > Cc: Maxime Ripard >

Re: [PATCH i-g-t] MAINTAINERS: Change Arek's email address

2020-07-30 Thread Petri Latvala
On Thu, Jul 30, 2020 at 12:09:11PM +0300, Arkadiusz Hiler wrote: > The current one will become inactive after this Friday. > > Cc: Petri Latvala > Cc: Lukasz Fiedorowicz > Signed-off-by: Arkadiusz Hiler Acked-by: Petri Latvala > --- > MAINTAINERS | 2 +- > 1 f

Re: [igt-dev] [PATCH i-g-t] panfrost: Test labeling functionality

2020-05-29 Thread Petri Latvala
rg > --- > include/drm-uapi/drm.h| 23 ++- Please update drm.h in a separate commit, and state which kernel commit (and from which repo) it's copied from. Examples can be seen in the git log for the file. -- Petri Latvala

Re: [PATCH] drm/doc: Make igts for cross-driver stuff mandatory

2019-01-17 Thread Petri Latvala
pop up, we can > > go ahead with this. > > > > 1: https://patchwork.kernel.org/patch/10648851/ > > Cc: Petri Latvala > > Cc: Arkadiusz Hiler > > Cc: Liviu Dudau > > Cc: Sean Paul > > Cc: Eric Anholt > > Cc: Alex Deucher > > Cc:

[PATCH v4 1/3] drm/prime: Pass the right module owner through to dma_buf_export()

2016-10-05 Thread Petri Latvala
For the series: Reviewed-by: Petri Latvala On 10/05/2016 03:21 PM, Chris Wilson wrote: > dma_buf_export() adds a reference to the owning module to the dmabuf (to > prevent the driver from being unloaded whilst a third party still refers > to the dmabuf). However, drm_gem_prime_exp

[PATCH v2 1/2] drm/prime: Passing the right owner through to dma_buf_export()

2016-09-30 Thread Petri Latvala
The series is Tested-by: Petri Latvala On 09/30/2016 02:44 PM, Chris Wilson wrote: > dma_buf_export() adds a reference to the owning module to the dmabuf (to > prevent the driver from being unloaded whilst a third party still refers > to the dmabuf). However, drm_gem_prime_export() wa