Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

Tomasz, > -Original Message- > From: Tomasz Figa [mailto:tf...@chromium.org] > Sent: Thursday, August 10, 2017 5:29 PM > To: Marathe, Yogesh > Hi Yogesh, > > On Thu, Aug 10, 2017 at 5:25 PM, Marathe, Yogesh > wrote: > > Hi Tomasz, > >

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

Hi Yogesh, On Thu, Aug 10, 2017 at 5:25 PM, Marathe, Yogesh wrote: > Hi Tomasz, > >> -Original Message- >> From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf >> Of Tomasz Figa >> Sent: Tuesday, August 8, 2017 7:45 AM >> To: Marathe, Yogesh

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

Hi Tomasz, > -Original Message- > From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf > Of Tomasz Figa > Sent: Tuesday, August 8, 2017 7:45 AM > To: Marathe, Yogesh > >> > >> Changing the topic, the patch doesn't seem to change the > >> >

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

On Wed, Aug 9, 2017 at 3:17 PM, Marathe, Yogesh wrote: > Tomasz, > >> -Original Message- >> From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf >> Of Tomasz Figa >> Sent: Tuesday, August 8, 2017 7:43 AM >> To: Marathe, Yogesh

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

Tomasz, > -Original Message- > From: mesa-dev [mailto:mesa-dev-boun...@lists.freedesktop.org] On Behalf > Of Tomasz Figa > Sent: Tuesday, August 8, 2017 7:43 AM > To: Marathe, Yogesh > On Mon, Aug 7, 2017 at 2:19 PM, Marathe, Yogesh >

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

On Mon, Aug 7, 2017 at 2:19 PM, Marathe, Yogesh wrote: > Tomasz, > >> -Original Message- >> From: Tomasz Figa [mailto:tf...@chromium.org] >> Sent: Saturday, August 5, 2017 8:47 AM >> >> Hi Yogesh, >> >> On Sat, Aug 5, 2017 at 1:22 AM, Marathe, Yogesh

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

On Mon, Aug 7, 2017 at 3:44 PM, Marathe, Yogesh wrote: >> -Original Message- >> >> Tomasz, >> >> > -Original Message- >> > From: Tomasz Figa [mailto:tf...@chromium.org] >> > Sent: Saturday, August 5, 2017 8:47 AM >> > >> > Hi Yogesh, >> > >> > On Sat, Aug

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

> -Original Message- > > Tomasz, > > > -Original Message- > > From: Tomasz Figa [mailto:tf...@chromium.org] > > Sent: Saturday, August 5, 2017 8:47 AM > > > > Hi Yogesh, > > > > On Sat, Aug 5, 2017 at 1:22 AM, Marathe, Yogesh > > > > wrote: > > >>

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

Tomasz, > -Original Message- > From: Tomasz Figa [mailto:tf...@chromium.org] > Sent: Saturday, August 5, 2017 8:47 AM > > Hi Yogesh, > > On Sat, Aug 5, 2017 at 1:22 AM, Marathe, Yogesh > wrote: > >> -Original Message- > >> From: Tomasz Figa

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

Hi Yogesh, On Sat, Aug 5, 2017 at 1:22 AM, Marathe, Yogesh wrote: >> -Original Message- >> From: Tomasz Figa [mailto:tf...@chromium.org] >> Sent: Friday, August 4, 2017 9:39 PM >> On Sat, Aug 5, 2017 at 12:53 AM, Marathe, Yogesh >>

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

On 4 August 2017 at 14:23, Tomasz Figa wrote: > > If this needs so complicated series of checks, maybe it would make > more sense to just set enable_out_fence based on availability of the > capability at initialization time? > Either way is fine with me. >> Did you drop it

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

> -Original Message- > From: Tomasz Figa [mailto:tf...@chromium.org] > Sent: Friday, August 4, 2017 9:39 PM > On Sat, Aug 5, 2017 at 12:53 AM, Marathe, Yogesh > wrote: > > Tomasz, Emil, > > > >> -Original Message- > >> From: Tomasz Figa

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

On Sat, Aug 5, 2017 at 12:53 AM, Marathe, Yogesh wrote: > Tomasz, Emil, > >> -Original Message- >> From: Tomasz Figa [mailto:tf...@chromium.org] >> Sent: Friday, August 4, 2017 6:54 PM >> To: Emil Velikov >> Cc: Marathe, Yogesh

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

Tomasz, Emil, > -Original Message- > From: Tomasz Figa [mailto:tf...@chromium.org] > Sent: Friday, August 4, 2017 6:54 PM > To: Emil Velikov > Cc: Marathe, Yogesh ; Antognolli, Rafael > ; ML mesa-dev

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

On Fri, Aug 4, 2017 at 9:55 PM, Emil Velikov wrote: >>> >> - version check (2+) the fence extension, calling .create_fence_fd() >>> >> only when >>> >> .get_capabilities() advertises __DRI_FENCE_CAP_NATIVE_FD >> >> The check looks like below now, this is in

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

On Fri, Aug 4, 2017 at 12:21 PM, Marathe, Yogesh wrote: >> >> >> - version check (2+) the fence extension, calling >> >> >> .create_fence_fd() only when >> >> >> .get_capabilities() advertises __DRI_FENCE_CAP_NATIVE_FD >> > >> > The check looks like below now, this is

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

On 4 August 2017 at 04:21, Marathe, Yogesh wrote: >> > The check looks like below now, this is in dri2_surf_update_fence_fd() >> > before >> create_fence_fd is called. >> > >> > if (dri2_surf->enable_out_fence && dri2_dpy->fence) { >> >

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

On 3 August 2017 at 17:55, Marathe, Yogesh wrote: > Adding folks who were CCed for earlier versions. > > Hi Emil, few doubts and comments below. > >> -Original Message- >> From: Tomasz Figa [mailto:tf...@chromium.org] >> Sent: Thursday, August 3, 2017 7:19 PM >>

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

Tomasz, > -Original Message- > From: Tomasz Figa [mailto:tf...@chromium.org] > Sent: Friday, August 4, 2017 6:52 AM > To: Marathe, Yogesh > Cc: Emil Velikov ; Gao, Shuo > ; Liu, Zhiquan ;

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

Hi Yogesh, On Fri, Aug 4, 2017 at 1:55 AM, Marathe, Yogesh wrote: > Adding folks who were CCed for earlier versions. > > Hi Emil, few doubts and comments below. > >> -Original Message- >> From: Tomasz Figa [mailto:tf...@chromium.org] >> Sent: Thursday, August 3,

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

Adding folks who were CCed for earlier versions. Hi Emil, few doubts and comments below. > -Original Message- > From: Tomasz Figa [mailto:tf...@chromium.org] > Sent: Thursday, August 3, 2017 7:19 PM > To: Marathe, Yogesh > Cc: Emil Velikov

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

On Thu, Aug 3, 2017 at 10:11 PM, Marathe, Yogesh wrote: > Emil, > >> -Original Message- >> From: Emil Velikov [mailto:emil.l.veli...@gmail.com] >> Sent: Thursday, August 3, 2017 4:06 PM >> To: Marathe, Yogesh >> Cc: ML mesa-dev

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

Emil, > -Original Message- > From: Emil Velikov [mailto:emil.l.veli...@gmail.com] > Sent: Thursday, August 3, 2017 4:06 PM > To: Marathe, Yogesh > Cc: ML mesa-dev ; Wu, Zhongmin > > Subject: Re: [Mesa-dev]

Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

On 2 August 2017 at 20:01, wrote: > From: Zhongmin Wu > > Before we queued the buffer with a invalid fence (-1), it causes > benchmarks such as flatland to fail. This patch enables explicit > sync feature on android. > > Now we get the out fence

[Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync fence for Android OS

From: Zhongmin Wu Before we queued the buffer with a invalid fence (-1), it causes benchmarks such as flatland to fail. This patch enables explicit sync feature on android. Now we get the out fence during the flushing buffer and then pass it to SurfaceFlinger in