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

2017-08-10 Thread Marathe, Yogesh
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, > > > >> -Original Message- > >> From: mesa-dev

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

2017-08-10 Thread Tomasz Figa
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 >> >> > >> Changing the to

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

2017-08-10 Thread Marathe, Yogesh
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 > >> > >> implementation of swapBuf

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

2017-08-08 Thread Tomasz Figa
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 >> On Mon, Aug 7, 2017 at 2:19 PM, Marathe

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

2017-08-08 Thread Marathe, Yogesh
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 > wrote: > > Tomasz, > > > >> -Original Message

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

2017-08-07 Thread Tomasz Figa
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 >> wrote: >> >> -Original Message-

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

2017-08-07 Thread Tomasz Figa
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 5, 2017 at 1:22 AM, Marath

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

2017-08-06 Thread Marathe, Yogesh
> -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: > > >> -Original Message- > > >

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

2017-08-06 Thread Marathe, Yogesh
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 [mailto:tf...@chromium.org] > >> Sent:

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

2017-08-04 Thread Tomasz Figa
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 >> wrote: >> > Tomasz, Emil, >> > >> >> -Original Messa

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

2017-08-04 Thread Emil Velikov
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 all together or change

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

2017-08-04 Thread Marathe, Yogesh
> -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 [mailto:tf...@chromium.org] > >> On Fri, Aug

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

2017-08-04 Thread Tomasz Figa
ognolli, Rafael >> ; ML mesa-dev > d...@lists.freedesktop.org>; Wu, Zhongmin ; Gao, >> Shuo ; Liu, Zhiquan ; Daniel >> Stone ; Timothy Arceri ; Eric >> Engestrom ; Kenneth Graunke ; >> Kondapally, Kalyan ; Varad Gautam >> ; Rainer Hochecker ; >> Nicolai Hähnle >>

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

2017-08-04 Thread Marathe, Yogesh
Zhiquan ; Daniel > Stone ; Timothy Arceri ; Eric > Engestrom ; Kenneth Graunke ; > Kondapally, Kalyan ; Varad Gautam > ; Rainer Hochecker ; > Nicolai Hähnle > Subject: Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync > fence > for Android OS > &g

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

2017-08-04 Thread Tomasz Figa
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 dri2_surf_update_fence_fd() >> before

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

2017-08-04 Thread Tomasz Figa
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 in dri2_surf_update_fence_fd

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

2017-08-04 Thread Emil Velikov
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) { >> >if(__DRI_FENCE_CAP_NATIVE_FD | >> > dri2_dpy->fence-

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

2017-08-04 Thread Emil Velikov
>> To: Marathe, Yogesh >> Cc: Emil Velikov ; Antognolli, Rafael >> ; ML mesa-dev > d...@lists.freedesktop.org>; Wu, Zhongmin >> Subject: Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync >> fence >> for Android OS >

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

2017-08-03 Thread Marathe, Yogesh
t; ; Wu, Zhongmin ; Kenneth > Graunke ; Kondapally, Kalyan > ; Rainer Hochecker ; > ML mesa-dev ; Timothy Arceri > ; Varad Gautam > Subject: Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync > fence > for Android OS > > Hi Yogesh, > > On Fri, Aug 4,

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

2017-08-03 Thread Tomasz Figa
, August 3, 2017 7:19 PM >> To: Marathe, Yogesh >> Cc: Emil Velikov ; Antognolli, Rafael >> ; ML mesa-dev > d...@lists.freedesktop.org>; Wu, Zhongmin >> Subject: Re: [Mesa-dev] [PATCH v5 2/2] i965: Queue the buffer with a sync >> fence >> for Android OS >

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

2017-08-03 Thread Marathe, Yogesh
; ; ML mesa-dev d...@lists.freedesktop.org>; Wu, Zhongmin > Subject: 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, > > > >> -Origi

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

2017-08-03 Thread Tomasz Figa
>> Subject: 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), i

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

2017-08-03 Thread Marathe, Yogesh
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] [PATCH v5 2/2] i965: Queue the buffer with a sync >

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

2017-08-03 Thread Emil Velikov
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 during the flushing buffer and then pass > it to S

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

2017-08-02 Thread yogesh . marathe
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 eglSwapbuffer function. v2: a) A