Quoting Matthew Auld (2020-12-24 10:29:05)
> The shadow batch needs to be in the user visible ppGTT, so make sure we
> are not leaking anything, if we can guess where the shadow will be
> placed.
>
> Signed-off-by: Matthew Auld <matthew.a...@intel.com>
> ---
> tests/i915/gen9_exec_parse.c | 129 +++++++++++++++++++++++++++++++++++
> 1 file changed, 129 insertions(+)
>
> diff --git a/tests/i915/gen9_exec_parse.c b/tests/i915/gen9_exec_parse.c
> index 087d6f35..6f54c4e1 100644
> --- a/tests/i915/gen9_exec_parse.c
> +++ b/tests/i915/gen9_exec_parse.c
> @@ -1051,6 +1051,132 @@ static void test_rejected(int i915, uint32_t handle,
> bool ctx_param)
> }
> }
>
> +#define PAGE_SHIFT 12
> +#define PAGE_SIZE (1ULL << 12)
> +
> +static inline uint32_t fill_and_copy_shadow(uint32_t *batch, uint32_t len,
> + uintptr_t src, uintptr_t dst)
> +{
> + unsigned int i = 0;
> +
> +#define XY_COLOR_BLT_CMD (2 << 29 | 0x50 << 22)
> +#define BLT_WRITE_ALPHA (1<<21)
> +#define BLT_WRITE_RGB (1<<20)
> + batch[i++] = XY_COLOR_BLT_CMD | BLT_WRITE_ALPHA | BLT_WRITE_RGB | (7
> - 2);
> + batch[i++] = 0xf0 << 16 | 1 << 25 | 1 << 24 | PAGE_SIZE;
> + batch[i++] = 0;
> + batch[i++] = len >> PAGE_SHIFT << 16 | PAGE_SIZE / 4;
> + batch[i++] = lower_32_bits(dst);
> + batch[i++] = upper_32_bits(dst);
> +
> + batch[i++] = 0xdeadbeaf;
> + batch[i++] = 0;
> +
> +#define COPY_BLT_CMD (2<<29|0x53<<22)
> + batch[i++] = COPY_BLT_CMD | BLT_WRITE_ALPHA | BLT_WRITE_RGB | 8;
> + batch[i++] = 0xcc << 16 | 1 << 25 | 1 << 24 | PAGE_SIZE;
> + batch[i++] = 0;
> + batch[i++] = len >> PAGE_SHIFT << 16 | PAGE_SIZE / 4;
> + batch[i++] = lower_32_bits(dst);
> + batch[i++] = upper_32_bits(dst);
> + batch[i++] = 0;
> + batch[i++] = PAGE_SIZE;
> + batch[i++] = lower_32_bits(src);
> + batch[i++] = upper_32_bits(src);
> +
> + batch[i++] = MI_BATCH_BUFFER_END;
> + batch[i++] = 0;
> +
> + return i * sizeof(uint32_t);
> +}
> +
> +static inline uint64_t sign_extend(uint64_t x, int index)
> +{
> + int shift = 63 - index;
> + return (int64_t)(x << shift) >> shift;
> +}
> +
> +static uint64_t gen8_canonical_address(uint64_t address)
> +{
> + return sign_extend(address, 47);
> +}
> +
> +static void test_shadow_peek(int fd)
> +{
> + uint64_t size = PAGE_SIZE;
> + struct drm_i915_gem_exec_object2 exec[2] = {};
> + struct drm_i915_gem_execbuffer2 execbuf = {
> + .buffers_ptr = to_user_pointer(exec),
> + .buffer_count = 2,
> + };
> + uint32_t *vaddr;
> + uint32_t len;
> + int i;
> +
> + exec[0].handle = gem_create(fd, size); /* scratch for shadow */
> + exec[0].flags = EXEC_OBJECT_PINNED |
> + EXEC_OBJECT_SUPPORTS_48B_ADDRESS |
> + EXEC_OBJECT_PAD_TO_SIZE;
> + exec[0].offset = 0;
> + /*
> + * Ensure the shadow has no place to hide, if say it were placed
> + * randomly within the address space. We leave enough space for our
> + * batch, which leaves exactly one perfect sized hole for the shadow
> to
> + * occupy later.
> + *
> + * Note that pad_to_size is just the node.size for the vma, which
> means
> + * we can easily occupy the entire 48b ppGTT, if we want, without
> + * needing an insane amount of physical memory.
> + */
> + exec[0].pad_to_size = gem_aperture_size(fd) - 2 * size;
Hmm. We do only allocate vma->size. Ok, I thought we did vma->node.size
there, so this won't consume as much RAM as I expected.
Given that, this should force the shadow exactly where you want it.
Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk>
There's a few more tricks we could try with poisoning the vma pool to
try and see if we can get info out of the second page, but this test
alone proves that we have a problem.
Hmm, speaking of second pages, this also highlights a problem where we
try to fit in a shadow vma that is larger than available space, a false
ENOSPC.
In fact, that is a test in of itself, a batch that occupies the whole
ppgtt should not generate ENOSPC. Back to gem_softpin...
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
