Re: [Mesa-dev] [PATCH v2 1/1] clover: Wait for requested operation if blocking flag is set

2017-08-17 Thread Francisco Jerez
Jan Vesely writes: > On Tue, 2017-08-15 at 12:00 -0700, Francisco Jerez wrote: >> Jan Vesely writes: >> >> > On Sat, 2017-08-12 at 20:14 -0700, Francisco Jerez wrote: >> > > Jan Vesely writes: >> > > >> > > > On Sat,

Re: [Mesa-dev] [PATCH v2 1/1] clover: Wait for requested operation if blocking flag is set

2017-08-17 Thread Jan Vesely
On Tue, 2017-08-15 at 12:00 -0700, Francisco Jerez wrote: > Jan Vesely writes: > > > On Sat, 2017-08-12 at 20:14 -0700, Francisco Jerez wrote: > > > Jan Vesely writes: > > > > > > > On Sat, 2017-08-05 at 12:34 -0700, Francisco Jerez wrote: > > >

Re: [Mesa-dev] [PATCH v2 1/1] clover: Wait for requested operation if blocking flag is set

2017-08-15 Thread Francisco Jerez
Jan Vesely writes: > On Sat, 2017-08-12 at 20:14 -0700, Francisco Jerez wrote: >> Jan Vesely writes: >> >> > On Sat, 2017-08-05 at 12:34 -0700, Francisco Jerez wrote: >> > > Francisco Jerez writes: >> > > >> > > > Jan

Re: [Mesa-dev] [PATCH v2 1/1] clover: Wait for requested operation if blocking flag is set

2017-08-14 Thread Jan Vesely
On Sat, 2017-08-12 at 20:14 -0700, Francisco Jerez wrote: > Jan Vesely writes: > > > On Sat, 2017-08-05 at 12:34 -0700, Francisco Jerez wrote: > > > Francisco Jerez writes: > > > > > > > Jan Vesely writes: > > > > > > > >

Re: [Mesa-dev] [PATCH v2 1/1] clover: Wait for requested operation if blocking flag is set

2017-08-14 Thread Aaron Watry
On Mon, Aug 14, 2017 at 4:29 PM, Aaron Watry wrote: > On Mon, Aug 14, 2017 at 1:53 PM, Francisco Jerez > wrote: >> Aaron Watry writes: >> >>> On Sat, Aug 12, 2017 at 10:14 PM, Francisco Jerez >>> wrote:

Re: [Mesa-dev] [PATCH v2 1/1] clover: Wait for requested operation if blocking flag is set

2017-08-14 Thread Aaron Watry
On Mon, Aug 14, 2017 at 1:53 PM, Francisco Jerez wrote: > Aaron Watry writes: > >> On Sat, Aug 12, 2017 at 10:14 PM, Francisco Jerez >> wrote: >>> Jan Vesely writes: >>> On Sat, 2017-08-05 at 12:34

Re: [Mesa-dev] [PATCH v2 1/1] clover: Wait for requested operation if blocking flag is set

2017-08-14 Thread Francisco Jerez
Aaron Watry writes: > On Sat, Aug 12, 2017 at 10:14 PM, Francisco Jerez > wrote: >> Jan Vesely writes: >> >>> On Sat, 2017-08-05 at 12:34 -0700, Francisco Jerez wrote: Francisco Jerez writes:

Re: [Mesa-dev] [PATCH v2 1/1] clover: Wait for requested operation if blocking flag is set

2017-08-12 Thread Francisco Jerez
Jan Vesely writes: > On Sat, 2017-08-05 at 12:34 -0700, Francisco Jerez wrote: >> Francisco Jerez writes: >> >> > Jan Vesely writes: >> > >> > > Hi, >> > > >> > > thanks for detailed explanation. I indeed missed the

Re: [Mesa-dev] [PATCH v2 1/1] clover: Wait for requested operation if blocking flag is set

2017-08-12 Thread Jan Vesely
On Sat, 2017-08-05 at 12:34 -0700, Francisco Jerez wrote: > Francisco Jerez writes: > > > Jan Vesely writes: > > > > > Hi, > > > > > > thanks for detailed explanation. I indeed missed the writeBuffer part > > > in specs. > > > > > > On Wed,

Re: [Mesa-dev] [PATCH v2 1/1] clover: Wait for requested operation if blocking flag is set

2017-08-05 Thread Francisco Jerez
Francisco Jerez writes: > Jan Vesely writes: > >> Hi, >> >> thanks for detailed explanation. I indeed missed the writeBuffer part >> in specs. >> >> On Wed, 2017-08-02 at 15:05 -0700, Francisco Jerez wrote: >>> These changes are somewhat redundant

Re: [Mesa-dev] [PATCH v2 1/1] clover: Wait for requested operation if blocking flag is set

2017-08-05 Thread Francisco Jerez
Jan Vesely writes: > Hi, > > thanks for detailed explanation. I indeed missed the writeBuffer part > in specs. > > On Wed, 2017-08-02 at 15:05 -0700, Francisco Jerez wrote: >> These changes are somewhat redundant and potentially >> performance-impacting, the reason is

Re: [Mesa-dev] [PATCH v2 1/1] clover: Wait for requested operation if blocking flag is set

2017-08-03 Thread Jan Vesely
Hi, thanks for detailed explanation. I indeed missed the writeBuffer part in specs. On Wed, 2017-08-02 at 15:05 -0700, Francisco Jerez wrote: > These changes are somewhat redundant and potentially > performance-impacting, the reason is that in the OpenCL API, > clEnqueueWrite* commands are

Re: [Mesa-dev] [PATCH v2 1/1] clover: Wait for requested operation if blocking flag is set

2017-08-02 Thread Francisco Jerez
These changes are somewhat redundant and potentially performance-impacting, the reason is that in the OpenCL API, clEnqueueWrite* commands are specified to block until the memory provided by the application as origin can be reused safely (i.e. until soft_copy_op()() runs), not necessarily until