On Thu, Sep 29, 2011 at 11:42 PM, Konrad Rzeszutek Wilk <konrad.wilk at oracle.com> wrote: > On Wed, Sep 28, 2011 at 04:10:02PM +0200, Thomas Hellstrom wrote: >> From: Jakob Bornecrantz <jakob at vmware.com> >> >> Signed-off-by: Jakob Bornecrantz <jakob at vmware.com> >> Reviewed-by: Thomas Hellstrom <thellstrom at vmware.com> >> --- >> ?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.
