On Fri, Mar 18, 2016 at 02:07:40PM +0530, akash.g...@intel.com wrote:
> +/* emit_store_qword
> + * populate batch buffer with MI_STORE_DWORD_IMM command
> + * @fd: drm file descriptor
> + * @cmd_buf: batch buffer
> + * @dw_offset: write offset in batch buffer
> + * @vaddr: destination Virtual address
> + * @data: u64 data to be stored at destination
> + */
> +static int emit_store_qword(int fd, uint32_t *cmd_buf, uint32_t dw_offset,
> + uint64_t vaddr, uint64_t data)
> +{
> + /* Check that softpin addresses are in the correct form */
> + igt_assert_eq_u64(vaddr, igt_canonical_addr(vaddr));
> +
> + /* SDI cannot write to unaligned addresses */
> + igt_assert((vaddr & 3) == 0);
If I remember correctly a qword write from SDI must be 8 byte aligned.
Right?
> +
> + cmd_buf[dw_offset++] = MI_STORE_DWORD_IMM | 0x3;
> + cmd_buf[dw_offset++] = (uint32_t)vaddr;
> + cmd_buf[dw_offset++] = (uint32_t)(vaddr >> 32);
> + cmd_buf[dw_offset++] = data;
> + cmd_buf[dw_offset++] = data >> 32;
> +
> + return dw_offset;
> +}
Hopefully final comments!
Missed EINTR handling during evict, If you repeat the busy/hang tests
within the igt_fork_signal_helper(); igt_stop_signal_helper() that
should cover catching an inopportune signal.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
