summaryrefslogtreecommitdiff
path: root/tests/intel-ci
AgeCommit message (Collapse)Author
2019-04-11intel-ci: Sanitycheck CRC readback on the first pipe onlyChris Wilson
For fast-feedback, we need to sanitycheck that CRC reads work, otherwise the display validation tests are flawed. However, we don't need to extensively test each pipe as the HW is fundamentally the same. Instead, do the full suite of tests on one pipe, and just verify that the interface works for the other pipes. This should speed BAT up by reducing NUM_PIPES suspend and resume cycles to just 1. The shards should cover the extended test suite on all pipes for complete validation. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Petri Latvala <petri.latvala@intel.com> Cc: Martin Peres <martin.peres@free.fr> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Acked-by: Martin Peres <martin.peres@free.fr> Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
2019-03-30i915_pm_rpm: remove gem-execbuf-stress-extra-wait because same as ↵Caz Yokoyama
gem-execbuf-stress Extra 5sec delay does not add any value more than gem-execbuf-stress. It waits until suspend state after a job is added by gem_execbuf(). There is no need to do more when GPU becomes suspended state. I confirm this by looking at pm_runtime_force_suspend() which exits on suspend state. Signed-off-by: Caz Yokoyama <caz.yokoyama@intel.com> Cc: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
2019-03-26i915/gem_exec_big: Add a single shot testChris Wilson
CI complains that the exhaustive test of trying every size up to the limit is too slow, so add a simple test that tries to submit one extreme batch buffer and check all the relocations land. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105555 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
2019-03-06i915/gem_exec_nop: Drop signal-all subtest from CIAshutosh Dixit
igt@gem-exec-nop@signal-all runs a single uninterrupted loop for 150 sec. This results in the test being one of the longest execution times in CI runs. According to mailing list discussions the test provides little of value. Therefore drop it from the CI. Signed-off-by: Ashutosh Dixit <ashutosh.dixit@intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Acked-by: Petri Latvala <petri.latvala@intel.com>
2019-03-01tests: s/core_prop_blob/kms_prop_blobDaniel Vetter
It's a kms test, name it accordingly. Also sort the build lists while at it, one test got misplaced. Reviewed-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2019-02-21intel-ci/fast-feedback: Prefix pm tests with i915_Arkadiusz Hiler
Cc: Petri Latvala <petri.latvala@intel.com> Reported-by: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com> Acked-by: Radoslaw Szwichtenberg <radoslaw.szwichtenberg@intel.com>
2019-02-20tests: Prefix pm_ tests with i915_ and move them to i915/Arkadiusz Hiler
They are i915-specific, so they belong to the directory. The (now) infix _pm_ is quite informative and worth keeping. v2: also prefix .c files Cc: Petri Latvala <petri.latvala@intel.com> Cc: Ewelina Musial <ewelina.musial@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Petri Latvala <petri.latvala@intel.com>
2019-02-15intel-ci: Disable i915_suspend@shrinkChris Wilson
This test produces an awful, awful lot of redundant output as it tries to find just the right amount of memory pressure to cause an out-of-memory event in the middle of suspend. That is always quite a slow process, taking 90s on a normal machine and 500+s on skl-y. Furthermore, even when we do achieve the perfect setup, the test frequently locks up and fails to resume with no indication that it is a bug in the driver. The shrinker and oomkiller (plus i915) do not make for a pleasant time! Enough of Martin's whinging, I see no way of easily making this test quieter, quicker and more efficacious, relegate it to the masochist only stable. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Martin Peres <martin.peres@free.fr> Cc: Petri Latvala <petri.latvala@intel.com> Reviewed-by: Martin Peres <martin.peres@free.fr>
2019-02-15intel-ci: Ignore performance-only pread/pwrite "tests"Chris Wilson
We don't need to waste time running perf-only test cases when we are not manually checking results. ezbench is that away! References: https://bugs.freedesktop.org/show_bug.cgi?id=109640 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Acked-by: Petri Latvala <petri.latvala@intel.com>
2019-01-30intel-ci: Drop gem_exec_nop from BATChris Wilson
This pair, gem_exec_nop/{series,parallel}, are very light stress tests of which we already perform the same sequence inside i915_selftests/live_requests. We keep basic uABI coverage (i.e. plain old gem_execbuf) via the likes of gem_exec_basic and gem_exec_reloc so all gem_exec_nop adds are nefarious HW details, much less likely to spot if a test is of sound quality for further testing (and again, we have alternate coverage of the same HW stress). Between the pair of subtests, gem_exec_nop takes about 20s of BAT. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Cc: Mika Kuoppala <mika.kuoppala@intel.com> Cc: Petri Latvala <petri.latvala@intel.com> Cc: Tomi Sarvela <tomi.p.sarvela@intel.com> Reviewed-by: Mika Kuoppala <mika.kuoppala@intel.com> Acked-by: Petri Latvala <petri.latvala@intel.com>
2019-01-15intel-ci: Drop gem_ctx_switch/heavy from BATChris Wilson
Th heavy variant of gem_ctx_switch does little more than provide an alternate timing for the basic gem_ctx_switch; the timing only effects the HW and does not stress the driver any differently. As such, including gem_ctx_switch/heavy provides no more basic coverage for BAT over and above the default gem_ctx_switch and i915_selftests/live_contexts. It takes around 45s, of a 600s total target time for BAT. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Mika Kuoppala <mika.kuoppala@linux.intel.com> Reviewed-by: Mika Kuoppala <mika.kuoppala@linux.intel.com> Acked-by: Tomi Sarvela <tomi.p.sarvela@intel.com> Acked-by: Petri Latvala <petri.latvala@intel.com>
2019-01-11intel-ci/blacklist: Exclude gem_exec_parse lriChris Wilson
These exercise a certain HW misfeature, no longer protected by the kernel cmdparser due to obsolete userspace requirements. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com> Acked-by: Petri Latvala <petri.latvala at intel.com>
2018-11-23igt/gem_exec_capture: Capture many, many objectsChris Wilson
Exercise O(N^2) behaviour in reading the error state, and push it to the extreme. Reported-by: Jason Ekstrand <jason@jlekstrand.net> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Katarzyna Dec <katarzyna.dec@intel.com>
2018-11-16tests: Move drv_ tests to i915 directoryArkadiusz Hiler
The drv_ tests are i915 specific, so let's move them to i915 directory and change the binary prefix to i915_. Additionally, while we at moving stuff around, let's rename module_reload to module_load and do the following cleanup with the subtests: basic-reload -> reload basic-no-display -> reload-no-display basic-reload-inject -> reload-with-fault-injection Cc: Martin Peres <martin.peres@linux.intel.com> Cc: Chris Wilson <chris@chris-wilson.co.uk> Cc: Petri Latvala <petri.latvala@intel.com> Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Acked-by: Martin Peres <martin.peres@linux.intel.com>
2018-08-16intel-ci: Exclude pm_rpm module-reload from full testingChris Wilson
The pm_rpm module-reload exists to exercise a rpm wakeref leak, and affects the random selection of tests run after it. Similar to the normal module-reload tests, care must be taken in its execution to avoid causing spurious failures elsewhere. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Acked-by: Petri Latvala <petri.latvala@intel.com>
2018-08-14igt/pm_rpm: Test reaquisition of runtime-pm after module reloadChris Wilson
It doesn't work right now and desperately needs to be fixed... Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Imre Deak <imre.deak@intel.com>
2018-08-10intel-ci: Skip module reloadsChris Wilson
Reloading the module may impact subsequent tests by destabilising the system. As we do for BAT, if we want to test reloads, it should be handled explicitly at the end of the run, rather than placed at random in the middle of the test list. v2: Commentary References: https://bugs.freedesktop.org/show_bug.cgi?id=106539 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Tomi Sarvela <tomi.p.sarvela@intel.com> Acked-by: Petri Latvala <petri.latvala@intel.com>
2018-08-10intel-ci: Comment on kernel selftest exclusionChris Wilson
The kernel selftests are excluded from the shard lists as those lists are compiled separately. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Acked-by: Petri Latvala <petri.latvala@intel.com>
2018-07-31intel-ci: Drop gem_cs_tlb from fast-feedbackChris Wilson
gem_cs_tlb tries to detect a HW issue and is not exercising a fundamental property of either the HW or uABI, it doesn't indicate general health of a driver with respect to testing a patchset and so need only be done later (after fast-feedback). Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Acked-by: Petri Latvala <petri.latvala@intel.com>
2018-07-31intel-ci: Move uABI hang testing to shardsChris Wilson
The foundational property of the driver to reset and recover hung hardware is covered by drv_selftests/live_hangcheck; that is all we need inside BAT. Beyond that checking the uABI behaves under hangs, and even wedging, remains covered by post-ff testing in the shards. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Tomi Sarvela <tomi.p.sarvela@intel.com> Cc: Petri Latvala <petri.latvala@intel.com> Cc: Martin Peres <martin.peres@linux.intel.com> Acked-by: Petri Latvala <petri.latvala@intel.com>
2018-07-31intel-ci: Remove gem_exec_flush from fast-feedbackChris Wilson
Basic gem_exec_flush coherency checking is performed by drv_selftests/live_coherency, so we do not need to duplicate coverage with BAT. However, we do want to keep the uABI checking for the shards. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Tomi Sarvela <tomi.p.sarvela@intel.com> Cc: Petri Latvala <petri.latvala@intel.com> Cc: Martin Peres <martin.peres@linux.intel.com> Acked-by: Petri Latvala <petri.latvala@intel.com>
2018-07-19tests/psr: Add a few PSR tests to fastfeedback test list for some coverageDhinakaran Pandiyan
PSR tests are primarily executed on shards machines that do not have the required panels, the only test in fastfeedback does not do much. So add these tests to provide some meaningful coverage. Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Cc: Martin Peres <martin.peres@linux.intel.com> Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com> Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com> Acked-by: Martin Peres <martin.peres@linux.intel.com>
2018-07-17tests/sink_crc: Remove kms_sink_crc_basicDhinakaran Pandiyan
"drm/i915: Kill sink_crc for good" removes the kernel support for sink crc. References: https://patchwork.freedesktop.org/series/46039/ Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2018-07-17igt: Remove gvt_basicChris Wilson
This was always a placeholder for GVT stakeholders to provide some better tests. 2 years later and none have been put forward so stop wasting CI's time running a placeholder. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=106989 Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Cc: Zhi Wang <zhi.a.wang@intel.com> Acked-by: Martin Peres <martin.peres@linux.intel.com>
2018-07-13tests/psr: Rename kms_psr_sink_crc.c to kms_psr.cDhinakaran Pandiyan
We don't use sink CRC anymore in this test. Cc: Rodrigo Vivi <rodrigo.vivi@intel.com> Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
2018-05-15tests/psr: Remove "psr_" prefix from basic and drrs subtestsDhinakaran Pandiyan
And rename psr_drrs to no_drrs. Makes the name consistent with other tests. Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@intel.com> Reviewed-by: Katarzyna Dec <katarzyna.dec@intel.com>
2018-04-05blacklist: Don't run tests on pipe-d, pipe-e or pipe-fLofstedt, Marta
There is currently no hardware in our labs that supports pipe-d, pipe-e nor pipe-f. Skipping takes time, so all tests on these pipes should be blacklisted. Signed-off-by: Marta Lofstedt <marta.lofstedt@intel.com> Acked-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-04-05blacklist: Don't run DRRS test on Intel CI systemLofstedt, Marta
Skipping takes time, specifically for the big amount of drrs related kms_frontbuffer_tracking tests. Since we currently don't have any system set up with DRRS panels, blacklisting all those test will save time, and we can avoid the need to increase the Jenkins timeout in order to solve the Bugzilla below. V2: Changed typo and added comment. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=105617 Signed-off-by: Marta Lofstedt <marta.lofstedt@intel.com> Acked-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2018-03-23intel-ci: Add full-suite test round blacklistPetri Latvala
The list of regular expressions for tests that are not to be run in sharded full test suite rounds, copied from https://intel-gfx-ci.01.org/blacklist.txt Signed-off-by: Petri Latvala <petri.latvala@intel.com> Reviewed-by: Marta Lofstedt <marta.lofstedt@intel.com>
2018-03-23intel-ci: Document purpose of fast-feedback.testlistPetri Latvala
Original text was written before shard runs existed. Removed the criteria for testlist contents; The list is still strictly controlled, but additions/removals will be considered case-by-case. Signed-off-by: Petri Latvala <petri.latvala@intel.com> Reviewed-by: Marta Lofstedt <marta.lofstedt@intel.com>
2018-03-23intel-ci: Document meta.testlistPetri Latvala
Signed-off-by: Petri Latvala <petri.latvala@intel.com> Cc: Marta Lofstedt <marta.lofstedt@intel.com> Reviewed-by: Marta Lofstedt <marta.lofstedt@intel.com>
2018-03-14tests/kms_pipe_crc_basic: Remove legacy crc testsMaarten Lankhorst
Commit 8c6b709d96cb9 ("drm/i915: Use new CRC debugfs API") has been in the kernel for more than a year. In commit 6d16875736b9f ("lib/igt_debugfs: Remove support for legacy CRC api.") we've removed support for the legacy CRC testing and we're about to remove the last remnants from the kernel, so it's time to remove the last tests for legacy CRC as well. Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2018-01-17meson: Refactor get_option() calls for directoriesPetri Latvala
Fetch the configuration values in the toplevel meson.build for all subdirs to share. v2: Also remember tests/intel-ci/meson.build Signed-off-by: Petri Latvala <petri.latvala@intel.com> Cc: Daniel Vetter <daniel.vetter@ffwll.ch> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2017-12-21meson: Install .testlist files and README from tests/intel-ciPetri Latvala
v2: README too, and a better variable name (Ville) Signed-off-by: Petri Latvala <petri.latvala@intel.com> Cc: Daniel Vetter <daniel.vetter@ffwll.ch> Cc: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2017-11-28igt: Remove gem_ctx_basicChris Wilson
This is just a very plain stress test that doesn't do any verification, and is entirely duplicated by the other context tests. The test currently leaks objects from every thread on every pass (which is triggering an oom on smaller machines), while fixing it would be trivial, it also is pointless as the test is of little merit. Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Reviewed-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
2017-11-21tests: Rename chamelium to kms_chamelium.Maarten Lankhorst
Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Reviewed-by: Lyude Paul <lyude@redhat.com> Acked-by: Martin Peres <martin.peres@linux.intel.com>
2017-10-18tests/intel-ci: Remove fast-feedback-simulation.testlistArkadiusz Hiler
Signed-off-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Acked-by: Radoslaw Szwichtenberg <radoslaw.szwichtenberg@intel.com> Acked-by: Petri Latvala <petri.latvala@intel.com>
2017-09-22intel-ci: Install testlist files to pkgdata dirPetri Latvala
Signed-off-by: Petri Latvala <petri.latvala@intel.com> Cc: Tomi Sarvela <tomi.p.sarvela@intel.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Tested-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2017-09-05Revert "intel-ci: Remove kms_psr_sink_crc@psr_basic from BAT"Daniel Vetter
This reverts commit effb78b382895c19d0946c217cee3602dcd94c50. cibuglog is for filtering unstable results, we can't simply opt to not test PSR. Acked-by: Petri Latvala <petri.latvala@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2017-08-24intel-ci: Remove generic.testlistPetri Latvala
The list has been unmaintained and unused. The set of tests is a subset of what CI runs in sharded runs so we are running all of them already. Signed-off-by: Petri Latvala <petri.latvala@intel.com> Acked-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Acked-by: Daniel Vetter <daniel.vetter@intel.com> Acked-by: Tomi Sarvela <tomi.p.sarvela@intel.com>
2017-08-23intel-ci: Add fast chamelium tests to the fast-feedback listPaul Kocialkowski
This adds the fastest chamelium tests to the Intel CI fast-feedback list, with the objective of running in under a minute. Signed-off-by: Paul Kocialkowski <paul.kocialkowski@linux.intel.com> Reviewed-by: Lyude Paul <lyude@redhat.com>
2017-08-16intel-ci: Add fast-feedback-simulation.testlistKelvin Gardiner
Added an initial list of fast feedback tests for simulation environments. v2 (Petri): Rebase README, Add the simulation list to EXTRA_DIST. Signed-off-by: Kelvin Gardiner <kelvin.gardiner@intel.com> Signed-off-by: Petri Latvala <petri.latvala@intel.com>
2017-08-04intel-ci: Also remove extended.testlist from EXTRA_DISTPetri Latvala
Fixes: 848fc49e22c6 ("tests: delete extended.testlist") Signed-off-by: Petri Latvala <petri.latvala@intel.com> Acked-by: Daniel Vetter <daniel.vetter@intel.com> Acked-by: Tomi Sarvela <tomi.p.sarvela@intel.com> Acked-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
2017-08-04tests/kms_busy: Only test against one engineDaniel Vetter
Back when we used cs flips it made sense to go through different engines, since a buffer busy on an engine that we couldnt' use for cs flipping ended up in different paths. But with atomic we use a worker for all flips, and going through the combinatorial growth of engines just wastes precious machine time. More so the more modern the platform is. Of course gem tests should still do some diagonal testing across all engines, but the kms side can afford to be a bit cheaper. v2: Make it compile. Oops. Also make sure we don't break fast-feedback.testlist. Adding a mode where we depend upon all engines isn't really possible, because there's only 1 exclusive fence and that's the only thing a flip waits for. Cc: Chris Wilson <chris@chris-wilson.co.uk> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Acked-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2017-08-04tests: delete extended.testlistDaniel Vetter
This approach died, we just run everything there is (and use that to apply some serious darwinian pressure onto testcase to prove themselves). v2 (Petri): Update README Cc: Martin Peres <martin.peres@linux.intel.com> Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Cc: Tomi Sarvela <tomi.p.sarvela@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com> Acked-by: Tomi Sarvela <tomi.p.sarvela@intel.com> Signed-off-by: Petri Latvala <petri.latvala@intel.com>
2017-08-03tests/drv_module_reload: Fix running single testsDaniel Vetter
We always need to make sure there's a working driver, hence need to move the -final test into the igt_fixture. This was broken in the conversion from the simple shell script (which didn't have any subtests) in commit 2fea8d26e589a9d256eca9f3d561750ecb3fb681 Author: Marius Vlad <marius.c.vlad@intel.com> Date: Thu Dec 1 14:23:57 2016 +0200 tests/drv_module_reload: Convert sh script to C version. Cc: tomi.p.sarvela@intel.com Tested-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
2017-07-24igt: Add debugfs_test.read_all_entries to the fast-feedback listMaarten Lankhorst
Every time we add something to debugfs, we test on the new platform but forget to test that old platforms still work. The test adds at most 200 ms extra time, which is worth it considering how often we break debugfs. Signed-off-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com> Reviewed by: Abdiel Janulgue <abdiel.janulgue@linux.intel.com>
2017-07-06Revert "igt: Remove default from the engine list"Chris Wilson
This reverts commit d7a0b61450797a3d6644c65aebf75c2a90da1a15.
2017-07-06Revert "gem_exec_basic: Exercise the default engine selection"Chris Wilson
This reverts commit f807953c4e90870df940ecb0e4b5c51caa50167e.
2017-07-06Revert "extended.testlist: Remove some test-subtest combinations"Chris Wilson
This reverts commit 537c4849d476661409425b0e2a5ad8bce4c561eb.