[PATCH] tests/intel/gem_ctx_param: Add GTT size test

Mikolaj Wasiak mikolaj.wasiak at intel.com
Fri Dec 13 15:01:31 UTC 2024


On 2024-12-09 at 23:19:19 +0100, Andi Shyti wrote:

Hi Andi,

I think this patch needs changes, but I wanted to clarify some things
regarding this test.

> is the power of two really needed?

The NEO UMD expects us to provide always full ppgtt range, which is
set in `ppgtt_init` to value that is a power of two. I see that ggtt total
size can have other values, but for ppgtt the total value doesn't change
outside of `ppgtt_init` and the reduction introduced in Commit 9bb66c179f50
("drm/i915: Reserve some kernel space per vm") that caused UMD problems.
I've also checked how the UMD handles the total size of ppgtt and before
introduction of aforementioned Commit they also expected the ppgtt size
to be a power of two.

Do you think that the test should check the GTT_SIZE in other way? I
just couldn't come up with different test.

Mikołaj


More information about the igt-dev mailing list