<feed xmlns='http://www.w3.org/2005/Atom'>
<title>igt-gpu-tools.git/benchmarks, branch vm-bind</title>
<subtitle>DRM IGT GPU Tools</subtitle>
<id>https://git.etezian.org/cgit.cgi/igt-gpu-tools.git/atom?h=vm-bind</id>
<link rel='self' href='https://git.etezian.org/cgit.cgi/igt-gpu-tools.git/atom?h=vm-bind'/>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/igt-gpu-tools.git/'/>
<updated>2022-06-15T06:33:47+00:00</updated>
<entry>
<title>benchmarks/gem_blt: fix baseline estimation</title>
<updated>2022-06-15T06:33:47+00:00</updated>
<author>
<name>Mauro Carvalho Chehab</name>
<email>mchehab@kernel.org</email>
</author>
<published>2022-05-19T07:06:16+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/igt-gpu-tools.git/commit/?id=74f7970332cedbf57da24959fb287b451da9a945'/>
<id>urn:sha1:74f7970332cedbf57da24959fb287b451da9a945</id>
<content type='text'>
This test is expected to run under a certain time defined by a command
line parameter (by default, 2s).

In order to achive the specified time, the baseline() function tries to
estimate the value of a counter that will get the minimal amount of
interaction to wait for &gt; 0.1s.

However, currently, the baseline estimation is broken, returning a too big
number, as it is measuring the memcpy() time, instead of actually
gem_execbuf().

Due to that, a default test without passing any command line parameter
would take more than 1.5 years to output the first benchmark data!

Fix it by using the same logic that it is inside run() at the baseline
time estimation.

Before this patch, baseline could return 399242693. After
it, it now return 20. That means an interval of about 120 ms at the
code which runs a gem_execbuf() loop,  which should be good enough to
ensure that the tests won't take too long, and will approximately match
the time specified by the excecution parameter '-t'.

Reviewed-by: Kamil Konieczny &lt;kamil.konieczny@linux.intel.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@kernel.org&gt;
</content>
</entry>
<entry>
<title>lib/intel_reg: Add common MI_* macros to remove duplicates</title>
<updated>2022-06-13T15:28:24+00:00</updated>
<author>
<name>Zbigniew Kempczyński</name>
<email>zbigniew.kempczynski@intel.com</email>
</author>
<published>2022-06-10T07:07:48+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/igt-gpu-tools.git/commit/?id=899ff3663841eba92aa503d760b5d316b2e972df'/>
<id>urn:sha1:899ff3663841eba92aa503d760b5d316b2e972df</id>
<content type='text'>
In few tests we got some MI_* duplicates (MI_MATH for example).
Add common definitions in intel_reg.h and remove local definitions
in the tests.

v2: Definitions MI_LOAD_REGISTER_MEM_GEN8 was removed so from now
    on user will need to encode length on it own. :

Signed-off-by: Zbigniew Kempczyński &lt;zbigniew.kempczynski@intel.com&gt;
Reviewed-by: Petri Latvala &lt;petri.latvala@intel.com&gt;
</content>
</entry>
<entry>
<title>benchmarks/gem_exec_fault: Add softpin mode to support gens with ppgtt</title>
<updated>2021-10-15T03:54:36+00:00</updated>
<author>
<name>Zbigniew Kempczyński</name>
<email>zbigniew.kempczynski@intel.com</email>
</author>
<published>2021-10-08T05:53:00+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/igt-gpu-tools.git/commit/?id=3c2ac88757f0d0ac9450487d314fcaceebc8bc26'/>
<id>urn:sha1:3c2ac88757f0d0ac9450487d314fcaceebc8bc26</id>
<content type='text'>
Alignment trick doesn't work properly for ppgtt gens - kernel is able
to keep previous offset and doesn't call unbind/bind. With softpin
on ppgtt we're able to enforce rebind and benchmark should behave
correctly on such gens.

To avoid inaccurate results kernel CONFIG_PROVE_LOCKING should be set
to N, otherwise kernel can call unbind/bind for same offset more than
one (backoff is not visible from userspace).

v2: rename to gem_allows_obj_alignment()

Signed-off-by: Zbigniew Kempczyński &lt;zbigniew.kempczynski@intel.com&gt;
Cc: Petri Latvala &lt;petri.latvala@intel.com&gt;
Cc: Ashutosh Dixit &lt;ashutosh.dixit@intel.com&gt;
Reviewed-by: Ashutosh Dixit &lt;ashutosh.dixit@intel.com&gt;
</content>
</entry>
<entry>
<title>benchmarks/gem_exec_fault: Add timeout argument</title>
<updated>2021-10-15T03:54:36+00:00</updated>
<author>
<name>Zbigniew Kempczyński</name>
<email>zbigniew.kempczynski@intel.com</email>
</author>
<published>2021-10-07T07:09:30+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/igt-gpu-tools.git/commit/?id=576b0c5c73984a53cb775e31ab23f7b76dd2294e'/>
<id>urn:sha1:576b0c5c73984a53cb775e31ab23f7b76dd2294e</id>
<content type='text'>
Add timeout argument and change elapsed time to inner loop to
be more precise in timeout processing.

Signed-off-by: Zbigniew Kempczyński &lt;zbigniew.kempczynski@intel.com&gt;
Cc: Ashutosh Dixit &lt;ashutosh.dixit@intel.com&gt;
Cc: Petri Latvala &lt;petri.latvala@intel.com&gt;
Reviewed-by: Ashutosh Dixit &lt;ashutosh.dixit@intel.com&gt;
</content>
</entry>
<entry>
<title>benchmarks/gem_busy: nuke LOCAL_IOCTL_SYNCOBJ_WAIT</title>
<updated>2021-07-15T19:01:57+00:00</updated>
<author>
<name>Lucas De Marchi</name>
<email>lucas.demarchi@intel.com</email>
</author>
<published>2021-07-11T06:12:09+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/igt-gpu-tools.git/commit/?id=b3c99f5822f94ac50fdbca12c6bd231ae49e59af'/>
<id>urn:sha1:b3c99f5822f94ac50fdbca12c6bd231ae49e59af</id>
<content type='text'>
This is DRM_IOCTL_SYNCOBJ_WAIT.

Signed-off-by: Lucas De Marchi &lt;lucas.demarchi@intel.com&gt;
Acked-by: Daniel Vetter &lt;daniel.vetter@ffwll.ch&gt;
Reviewed-by: Ashutosh Dixit &lt;ashutosh.dixit@intel.com&gt;
</content>
</entry>
<entry>
<title>benchmarks/gem_busy: nuke LOCAL_IOCTL_SYNCOBJ_CREATE</title>
<updated>2021-07-15T19:01:57+00:00</updated>
<author>
<name>Lucas De Marchi</name>
<email>lucas.demarchi@intel.com</email>
</author>
<published>2021-07-11T06:12:08+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/igt-gpu-tools.git/commit/?id=de02d108ec73b2dd8ad459d58cee4e1d8e60877d'/>
<id>urn:sha1:de02d108ec73b2dd8ad459d58cee4e1d8e60877d</id>
<content type='text'>
This is DRM_IOCTL_SYNCOBJ_CREATE.

Signed-off-by: Lucas De Marchi &lt;lucas.demarchi@intel.com&gt;
Acked-by: Daniel Vetter &lt;daniel.vetter@ffwll.ch&gt;
Reviewed-by: Ashutosh Dixit &lt;ashutosh.dixit@intel.com&gt;
</content>
</entry>
<entry>
<title>benchmarks/gem_exec_tracer: nuke LOCAL_IOCTL_I915_GEM_EXECBUFFER2_WR</title>
<updated>2021-07-15T19:01:57+00:00</updated>
<author>
<name>Lucas De Marchi</name>
<email>lucas.demarchi@intel.com</email>
</author>
<published>2021-07-11T06:12:07+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/igt-gpu-tools.git/commit/?id=9dc452c14f015d74b765ede279afc9dc52db6e2c'/>
<id>urn:sha1:9dc452c14f015d74b765ede279afc9dc52db6e2c</id>
<content type='text'>
Use DRM_IOCTL_I915_GEM_EXECBUFFER2_WR from kernel header.

Signed-off-by: Lucas De Marchi &lt;lucas.demarchi@intel.com&gt;
Acked-by: Daniel Vetter &lt;daniel.vetter@ffwll.ch&gt;
Reviewed-by: Ashutosh Dixit &lt;ashutosh.dixit@intel.com&gt;
</content>
</entry>
<entry>
<title>lib/i915/gem_create: Add gem_create_ext</title>
<updated>2021-05-27T10:36:21+00:00</updated>
<author>
<name>Andrzej Turko</name>
<email>andrzej.turko@linux.intel.com</email>
</author>
<published>2021-05-26T13:00:18+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/igt-gpu-tools.git/commit/?id=b0789992cefc3b08fb5fd891522a116b367c3963'/>
<id>urn:sha1:b0789992cefc3b08fb5fd891522a116b367c3963</id>
<content type='text'>
Add a wrapper for gem_create_ext ioctl (a version of gem_create that
accepts extensions). In preparation for the driver change implementing it,
a local definition of its id and necessary structs have been added,
which are to be erased as soon as those definitions
appear in the i915_drm.h file.

The new ioctl wrapper is added to a separate file.
For consistency the wrapper of the old ioctl, gem_create
is moved from ioctl_wrappers to gem_create.

Signed-off-by: Andrzej Turko &lt;andrzej.turko@linux.intel.com&gt;
Cc: Zbigniew Kempczynski &lt;zbigniew.kempczynski@intel.com&gt;
Cc: Dominik Grzegorzek &lt;dominik.grzegorzek@intel.com&gt;
Cc: Petri Latvala &lt;petri.latvala@intel.com&gt;
Cc: Chris P Wilson &lt;chris.p.wilson@intel.com&gt;
Signed-off-by: Matthew Auld &lt;matthew.auld@intel.com&gt;
Acked-by: Petri Latvala &lt;petri.latvala@intel.com&gt;
</content>
</entry>
<entry>
<title>benchmarks/gem_userptr_benchmark: Remove tests with unsynchronized flag</title>
<updated>2021-05-26T09:10:16+00:00</updated>
<author>
<name>Zbigniew Kempczyński</name>
<email>zbigniew.kempczynski@intel.com</email>
</author>
<published>2021-05-21T16:42:35+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/igt-gpu-tools.git/commit/?id=f62d8953c0bc5ed68ea978662e62f9dbb46cf101'/>
<id>urn:sha1:f62d8953c0bc5ed68ea978662e62f9dbb46cf101</id>
<content type='text'>
As flag I915_USERPTR_UNSYNCHRONIZED is not supported by the i915 anymore
remove tests which tries to use it in the benchmark.

Remove benchmark from meson libdrm depending code as it is a little bit
confusing.

Signed-off-by: Zbigniew Kempczyński &lt;zbigniew.kempczynski@intel.com&gt;
Cc: Tvrtko Ursulin &lt;tvrtko.ursulin@intel.com&gt;
Cc: Petri Latvala &lt;petri.latvala@intel.com&gt;
Reviewed-by: Tvrtko Ursulin &lt;tvrtko.ursulin.com&gt;
</content>
</entry>
<entry>
<title>gitignore: Remove various .gitignore files</title>
<updated>2021-04-23T10:27:44+00:00</updated>
<author>
<name>Petri Latvala</name>
<email>petri.latvala@intel.com</email>
</author>
<published>2021-04-23T08:29:19+00:00</published>
<link rel='alternate' type='text/html' href='https://git.etezian.org/cgit.cgi/igt-gpu-tools.git/commit/?id=667ba26b476db3f2d2e6e1620ee0cf8e338be8e7'/>
<id>urn:sha1:667ba26b476db3f2d2e6e1620ee0cf8e338be8e7</id>
<content type='text'>
Now that autotools is gone and you always build to a dedicated build
directory with meson, we can remove .gitignore files that only had the
purpose of ignoring built binaries in the source directories.

Signed-off-by: Petri Latvala &lt;petri.latvala@intel.com&gt;
Cc: Arkadiusz Hiler &lt;arek@hiler.eu&gt;
Reviewed-by: Arkadiusz Hiler &lt;arek@hiler.eu&gt;
</content>
</entry>
</feed>
