[igt-dev] [PATCH i-g-t] tests/i915/gem_exec_capture: Add support for local memory
Zbigniew Kempczyński
zbigniew.kempczynski at intel.com
Mon Dec 13 13:12:50 UTC 2021
On Mon, Dec 13, 2021 at 02:18:06PM +0530, sai.gowtham.ch at intel.com wrote:
> From: Ch Sai Gowtham <sai.gowtham.ch at intel.com>
>
> Add support for local memory region (Device memory)
>
> Signed-off-by: Ch Sai Gowtham <sai.gowtham.ch at intel.com>
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
> ---
> tests/i915/gem_exec_capture.c | 26 ++++++++++++++++++++++----
> 1 file changed, 22 insertions(+), 4 deletions(-)
>
> diff --git a/tests/i915/gem_exec_capture.c b/tests/i915/gem_exec_capture.c
> index b80f597f..27fbf008 100644
> --- a/tests/i915/gem_exec_capture.c
> +++ b/tests/i915/gem_exec_capture.c
> @@ -374,13 +374,13 @@ static void __capture1(int fd, int dir, uint64_t ahnd, const intel_ctx_t *ctx,
> }
>
> static void capture(int fd, int dir, const intel_ctx_t *ctx,
> - const struct intel_execution_engine2 *e)
> + const struct intel_execution_engine2 *e, uint32_t region)
> {
> uint32_t handle;
> uint64_t ahnd;
> int obj_size = 4096;
>
> - handle = gem_create(fd, obj_size);
> + handle = gem_create_in_memory_regions(fd, obj_size, region);
Ok, capture buffer is within different regions, but not bb.
> ahnd = get_reloc_ahnd(fd, ctx->id);
>
> __capture1(fd, dir, ahnd, ctx, e, handle, obj_size);
Please pass region to __capture1() and use it for creating
bb there.
For userptr() example just pass smem during __capture1() call.
--
Zbigniew
> @@ -740,6 +740,10 @@ igt_main
> igt_hang_t hang;
> int fd = -1;
> int dir = -1;
> + struct drm_i915_query_memory_regions *query_info;
> + struct igt_collection *regions, *set;
> + char *sub_name;
> + uint32_t region;
>
> igt_fixture {
> int gen;
> @@ -759,10 +763,24 @@ igt_main
> dir = igt_sysfs_open(fd);
> igt_require(igt_sysfs_set(dir, "error", "Begone!"));
> igt_require(safer_strlen(igt_sysfs_get(dir, "error")) > 0);
> + query_info = gem_get_query_memory_regions(fd);
> + igt_assert(query_info);
> + set = get_memory_region_set(query_info,
> + I915_SYSTEM_MEMORY,
> + I915_DEVICE_MEMORY);
> }
>
> - test_each_engine("capture", fd, ctx, e)
> - capture(fd, dir, ctx, e);
> + test_each_engine("capture", fd, ctx, e) {
> + igt_subtest_with_dynamic("capture") {
> + for_each_combination(regions, 1, set) {
> + sub_name = memregion_dynamic_subtest_name(regions);
> + region = igt_collection_get_value(regions, 0);
> + igt_dynamic_f("%s", sub_name)
> + capture(fd, dir, ctx, e, region);
> + free(sub_name);
> + }
> + }
> + }
>
> igt_subtest_f("many-4K-zero") {
> igt_require(gem_can_store_dword(fd, 0));
> --
> 2.32.0
>
More information about the igt-dev
mailing list