[Intel-gfx] [PATCH] drm/i915: execbuf2 support

Jesse Barnes jbarnes at virtuousgeek.org
Tue Dec 1 21:23:48 CET 2009


On Tue,  1 Dec 2009 20:13:51 +0000
Chris Wilson <chris at chris-wilson.co.uk> wrote:

> From: Jesse Barnes <jbarnes at virtuousgeek.org>
> 
> This patch adds a new execbuf ioctl, execbuf2, for use by clients that
> want to control fence register allocation more finely.  The buffer
> passed in to the new ioctl includes a new relocation type to indicate
> whether a given object needs a fence register assigned for the command
> buffer in question.
> 
> Compatibility with the existing execbuf ioctl is implemented in terms
> of the new code, preserving the assumption that fence registers are
> required for pre-965 rendering commands.
> 
> Signed-off-by: Jesse Barnes <jbarnes at virtuousgeek.org>
> [ickle: Remove pre-emptive clear_fence_reg()]
> Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> ---

I think this part of the patchset is fine.  IIRC I still had a bug in
the libdrm patch that caused us to not use fence regs even for old
clients.  Assuming Eric is ok with this patch I'll update the libdrm
side, test it, and commit it.

The mesa portion should be in fair shape as well, but needs to be split
up.  I think the DDX portion is ok too, though it probably needs to be
re-diffed.

-- 
Jesse Barnes, Intel Open Source Technology Center



More information about the Intel-gfx mailing list