From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Widawsky Subject: Re: [Intel-gfx] [PATCH] intel: add a timed wait function Date: Wed, 30 May 2012 13:32:39 -0700 Message-ID: <20120530133239.4e155e38@bwidawsk.net> References: <20120527131654.4501e518@bwidawsk.net> <87vcjdk1mn.fsf@eliezer.anholt.net> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Daniel Vetter Cc: dri-devel@lists.freedesktop.org, mesa-dev@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On Wed, 30 May 2012 21:07:57 +0200 Daniel Vetter wrote: > On Wed, May 30, 2012 at 7:41 PM, Eric Anholt wrote: > > I guess GL_ALREADY_SIGNALED handling will be done using a check for > > bo_busy() before calling this. > > I've just read through the mesa code and gl_already_signalled seems to > be handled already by core mesa code in _mesa_ClientWaitSync (if the > driver sets syncObject->Status correctly). So I guess the current > kernel code should work as-is and only the libdrm interface needs some > colour adjustments around the timeout parameter. > -Daniel What have we all agreed on for the color? -- Ben Widawsky, Intel Open Source Technology Center