On Thu, Sep 29, 2011 at 11:42 PM, Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx> wrote: > On Wed, Sep 28, 2011 at 04:10:02PM +0200, Thomas Hellstrom wrote: >> From: Jakob Bornecrantz <jakob@xxxxxxxxxx> >> >> Signed-off-by: Jakob Bornecrantz <jakob@xxxxxxxxxx> >> Reviewed-by: Thomas Hellstrom <thellstrom@xxxxxxxxxx> >> --- >> drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c | 5 +++++ >> 1 files changed, 5 insertions(+), 0 deletions(-) >> >> diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c b/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c >> index df1c7c5..207f595 100644 >> --- a/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c >> +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c >> @@ -680,6 +680,9 @@ static int vmw_resize_cmd_bounce(struct vmw_sw_context *sw_context, >> * Creates a fence object and submits a command stream marker. >> * If this fails for some reason, We sync the fifo and return NULL. >> * It is then safe to fence buffers with a NULL pointer. >> + * >> + * If @p_handle is not NULL @file_priv must also not be not NULL. Creates > > not be not? > I think you meant: > > "must also not be NULL" > > >> + * a userspace handle if @p_handle is not NULL, does not if otherwise. > > "does not if otherwise".. how about ", otherwise not." Thanks I'll fix that. Cheers Jakob. _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel