Re: [Intel-gfx] [PATCH i-g-t] tests/prime_vgem: Give meaningful messages on SKIP

t; Fix it. > > Signed-off-by: Janusz Krzysztofik > Cc: Ewelina Musial > --- > tests/prime_vgem.c | 9 ++--- > 1 file changed, 6 insertions(+), 3 deletions(-) > > diff --git a/tests/prime_vgem.c b/tests/prime_vgem.c > index 6595818c..3bdb2300 100644 > --- a/test

Re: [Intel-gfx] [igt-dev] [PATCH i-g-t] igt/drv_suspend: Suspend under memory pressure

On Thu, Jun 07, 2018 at 09:50:54PM +0100, Chris Wilson wrote: > Recently we discovered that we have a race between swapping and > suspend in our resume path (we might be trying to page in an object > after disabling the block devices). Let's try to exercise that by > exhausting all of system

Re: [Intel-gfx] [igt-dev] [PATCH] selftests/i915_gem_gtt: Create igt_ggtt_scratch subtest

On Wed, Apr 11, 2018 at 11:43:34AM +0100, Chris Wilson wrote: > Quoting Ewelina Musial (2018-04-11 11:20:56) > > On Wed, Apr 11, 2018 at 09:48:21AM +0100, Chris Wilson wrote: > > > Quoting Ewelina Musial (2018-04-11 09:27:12) > > > > Test have similar functionality

Re: [Intel-gfx] [igt-dev] [PATCH] selftests/i915_gem_gtt: Create igt_ggtt_scratch subtest

On Wed, Apr 11, 2018 at 09:48:21AM +0100, Chris Wilson wrote: > Quoting Ewelina Musial (2018-04-11 09:27:12) > > Test have similar functionality that gem_cs_prefetch IGT test > > but gem_cs_prefetch is not up to date and there was an idea > > to move this test to kselftes

[Intel-gfx] [PATCH i-g-t] gem_cs_prefetch: Remove gem_cs_prefetch from IGT

gem_cs_prefetch is replaced by igt_ggtt_scratch subtest in i915_gem_gtt kselftest --- tests/Makefile.sources | 1 - tests/gem_cs_prefetch.c | 149 --- tests/intel-ci/blacklist.txt | 1 - tests/meson.build| 1 - 4 files changed,

[Intel-gfx] [PATCH] selftests/i915_gem_gtt: Create igt_ggtt_scratch subtest

Test have similar functionality that gem_cs_prefetch IGT test but gem_cs_prefetch is not up to date and there was an idea to move this test to kselftests so this is respond for this request. There is another patch on igt_dev which is removing gem_cs_prefetch test from IGT. ---

Re: [Intel-gfx] [PATCH igt 08/10] igt/pm_rc6_residency: Measure residency after checking for applicability

d > worse, that work is wasted if the subtest will be skipped. > > pm_rc6_residency is one such example that measured all the residencies > up front before skipping, each skip was therefore taking in excess of > 10s. > > Signed-off-by: Chris Wilson <ch...@chris-wilson.c

Re: [Intel-gfx] [PATCH igt 08/10] igt/pm_rc6_residency: Measure residency after checking for applicability

On Mon, Dec 04, 2017 at 09:27:29AM +, Chris Wilson wrote: > CI doesn't run in whole-test mode, but runs each subtest individually. > Tests that are designed to do a block of work to be shared between many > subtests end up running that work multiple times (once per subtest) and > worse, that

Re: [Intel-gfx] [QUERY] How many CI mails is too many?

On Tue, Nov 28, 2017 at 01:16:50PM +0200, Arkadiusz Hiler wrote: > On Mon, Nov 27, 2017 at 09:10:37PM +0530, Sagar Arun Kamble wrote: > > I feel we generally tend to ignore the results mails for series that > > we are not actively involved on (although we might be interested in > > series itself).

Re: [Intel-gfx] [PATCH 3/4] drm/i915: expose engine availability through sysfs

On Mon, Nov 20, 2017 at 03:57:49PM +, Tvrtko Ursulin wrote: > > On 20/11/2017 12:23, Lionel Landwerlin wrote: > > This enables userspace to discover the engines available on the GPU. > > Here is the layout on a Skylake GT4: > > > > /sys/devices/pci:00/:00:02.0/drm/card0/gt > > ├──

Re: [Intel-gfx] [PATCH v3 2/4] drm/i915: fix 64bit divide

Use div64_u64 (Matthew) > > v3: store frequency in kHz to avoid 64bit divs (Chris/Ville) > > Fixes: dab9178333 ("drm/i915: expose command stream timestamp frequency to > userspace") > Reported-by: Matthew Auld <matthew.a...@intel.com> > Signed-off-by: Li

Re: [Intel-gfx] [PATCH v3 1/4] drm/i915/perf: replace .reg accesses with i915_mmio_reg_offset

by: Lionel Landwerlin <lionel.g.landwer...@intel.com> Reviewed-by: Ewelina Musial <ewelina.mus...@intel.com> > --- > drivers/gpu/drm/i915/i915_perf.c | 39 --- > 1 file changed, 24 insertions(+), 15 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i

Re: [Intel-gfx] [PATCH 4/7] drm/i915: fix register naming

t; Hence why the name isn't used anywhere. > > v2: Fix register name again RPC->RCP (Matthew) > > Signed-off-by: Lionel Landwerlin <lionel.g.landwer...@intel.com> Reviewed-by: Ewelina Musial <ewelina.mus...@intel.com> > --- > drivers/gpu/drm/i915/i915_reg.h | 4 ++-- &

Re: [Intel-gfx] [PATCH 2/7] drm/i915/perf: add support for Coffeelake GT3

On Fri, Nov 10, 2017 at 07:08:40PM +, Lionel Landwerlin wrote: > We can enable GT3 as well as GT2. > > Signed-off-by: Lionel Landwerlin <lionel.g.landwer...@intel.com> > Reviewed-by: Matthew Auld <matthew.a...@intel.com> Reviewed-by: Ewelina Musial <ewelina.mus...@

Re: [Intel-gfx] [PATCH 1/7] drm/i915/perf: complete whitelisting for OA programming on HSW

On Fri, Nov 10, 2017 at 07:08:39PM +, Lionel Landwerlin wrote: > We were missing some registers and also can name one for which we only had > the offset. > > Signed-off-by: Lionel Landwerlin > Reviewed-by: Matthew Auld > --- >

Re: [Intel-gfx] [PATCH v2 6/9] drm/i915: expose command stream timestamp frequency to userspace

On Thu, Nov 02, 2017 at 04:29:46PM +, Lionel Landwerlin wrote: > We use to have this fixed per generation, but starting with CNL userspace > cannot tell just off the PCI ID. Let's make this information available. This > is particularly useful for performance monitoring where much of the >

Re: [Intel-gfx] [PATCH i-g-t v5 04/11] tests/perf: rc6: try to guess when rc6 is disabled

On Thu, Aug 31, 2017 at 11:35:08AM +0100, Lionel Landwerlin wrote: > Signed-off-by: Lionel Landwerlin > --- > tests/perf.c | 13 + > 1 file changed, 13 insertions(+) > > diff --git a/tests/perf.c b/tests/perf.c > index bd139bde..5fe0a332 100644 > ---

Re: [Intel-gfx] [PATCH 08/31] drm/i915: Rename intel_enable_rc6 to intel_rc6_enabled

k> > Cc: Imre Deak <imre.d...@intel.com> > Signed-off-by: Sagar Arun Kamble <sagar.a.kam...@intel.com> Reviewed-by: Ewelina Musial <ewelina.mus...@intel.com> - Ewelina > --- > drivers/gpu/drm/i915/i915_drv.c | 2 +- > drivers/gpu/drm/i915/i915_sysfs.c | 2 +- >

Re: [Intel-gfx] [PATCH i-g-t 7/9] tests/perf: Add testcase to verify mmio

id", 1); > + > + igt_fork(child, 1) { > + int prop_size = ARRAY_SIZE(properties); > + int num_reports = 10; > + int report_size = get_perf_report_size(properties, prop_size, > + test_oa_fo

Re: [Intel-gfx] [PATCH i-g-t 3/9] tests/perf: Add testcase to verify ctx id

ar.a.kam...@intel.com> Reviewed-by: Ewelina Musial <ewelina.mus...@intel.com> > --- > lib/ioctl_wrappers.h| 1 + > tests/intel_perf_dapc.c | 102 > +++- > 2 files changed, 101 insertions(+), 2 deletions(-) > > diff --git a/

Re: [Intel-gfx] [PATCH i-g-t 2/9] tests/perf: Test i915 assisted command stream based perf metrics capture

> > Cc: Lionel Landwerlin <lionel.g.landwer...@intel.com> > Signed-off-by: Sagar Arun Kamble <sagar.a.kam...@intel.com> Reviewed-by: Ewelina Musial <ewelina.mus...@intel.com> > --- > tests/Makefile.sources | 1 + > tests/intel_perf_dapc.c | 811 > +

Re: [Intel-gfx] [PATCH 1/2] dim: make dim tag-next generate annotated tags

On Mon, Sep 11, 2017 at 04:15:59PM +0300, Jani Nikula wrote: > Presumably an oversight while converting update-next-continue to use > annotated tags. > > Cc: Daniel Vetter <daniel.vet...@ffwll.ch> > Signed-off-by: Jani Nikula <jani.nik...@intel.com> Reviewed-by:

[Intel-gfx] [PATCH i-g-t 2/2] tests/drv_suspend: Add subtests to check that forcewake is kept after resume

zwichtenb...@intel.com> Cc: Michal Winiarski <michal.winiar...@intel.com> Signed-off-by: Ewelina Musial <ewelina.mus...@intel.com> --- tests/drv_suspend.c | 36 1 file changed, 32 insertions(+), 4 deletions(-) diff --git a/tests/drv_suspend.c b/t

[Intel-gfx] [PATCH i-g-t 1/2] igt: Move read_rc6_residency function to lib

Gem_mocs_settings and pm_rc6_residency tests are defining the same functionality to read residency from sysfs. Moving that function to lib/igt_aux and updating tests. Signed-off-by: Ewelina Musial <ewelina.mus...@intel.com> --- lib/igt_aux.c | 8 lib/igt

[Intel-gfx] [PATCH i-g-t] igt: Move read_rc6_residency function to lib

Gem_mocs_settings and pm_rc6_residency tests are defining the same functionality to read residency from sysfs. Moving that function to lib/igt_aux and updating tests. Signed-off-by: Ewelina Musial <ewelina.mus...@intel.com> --- lib/igt_aux.c | 8 lib/igt

[Intel-gfx] [PATCH i-g-t] tests/pm_rc6_residency: Add subtest to check RC6 suspend handling

com> Signed-off-by: Ewelina Musial <ewelina.mus...@intel.com> --- tests/pm_rc6_residency.c | 53 1 file changed, 53 insertions(+) diff --git a/tests/pm_rc6_residency.c b/tests/pm_rc6_residency.c index bdb9747..4e61326 100644 --- a/tests/pm_rc6