Re: [Mesa-dev] [PATCH] mesa: Add missing _mm_mfence() before streaming loads.

2015-10-06 Thread Nuzman, Joseph
, October 06, 2015 12:13 To: Justen, Jordan L Cc: Matt Turner; mesa-dev@lists.freedesktop.org; Nuzman, Joseph Subject: Re: [Mesa-dev] [PATCH] mesa: Add missing _mm_mfence() before streaming loads. On Mon, Oct 05, 2015 at 12:01:25PM -0700, Jordan Justen wrote: > On 2015-10-05 11:55:57, Matt Tur

Re: [Mesa-dev] [PATCH] mesa: Add missing _mm_mfence() before streaming loads.

2015-10-06 Thread Chris Wilson
On Mon, Oct 05, 2015 at 12:01:25PM -0700, Jordan Justen wrote: > On 2015-10-05 11:55:57, Matt Turner wrote: > > On Thu, Oct 1, 2015 at 11:20 AM, Jordan Justen > > wrote: > > > On 2015-10-01 10:11:33, Matt Turner wrote: > > >> According to the Intel Software Development Manual: > > > > > > How abou

Re: [Mesa-dev] [PATCH] mesa: Add missing _mm_mfence() before streaming loads.

2015-10-05 Thread Jordan Justen
On 2015-10-05 11:55:57, Matt Turner wrote: > On Thu, Oct 1, 2015 at 11:20 AM, Jordan Justen > wrote: > > On 2015-10-01 10:11:33, Matt Turner wrote: > >> According to the Intel Software Development Manual: > > > > How about a more specific doc location? > > > > According to the Intel Software Devel

Re: [Mesa-dev] [PATCH] mesa: Add missing _mm_mfence() before streaming loads.

2015-10-05 Thread Matt Turner
On Thu, Oct 1, 2015 at 11:20 AM, Jordan Justen wrote: > On 2015-10-01 10:11:33, Matt Turner wrote: >> According to the Intel Software Development Manual: > > How about a more specific doc location? > > According to the Intel Software Development Manual (Volume 1: Basic > Architecture, 12.10.3 Stre

Re: [Mesa-dev] [PATCH] mesa: Add missing _mm_mfence() before streaming loads.

2015-10-04 Thread Nuzman, Joseph
[Mesa-dev] [PATCH] mesa: Add missing _mm_mfence() before streaming loads. On 2015-10-01 10:11:33, Matt Turner wrote: > According to the Intel Software Development Manual: How about a more specific doc location? According to the Intel Software Development Manual (Volume 1: Basic Architecture,

Re: [Mesa-dev] [PATCH] mesa: Add missing _mm_mfence() before streaming loads.

2015-10-01 Thread Jordan Justen
On 2015-10-01 10:11:33, Matt Turner wrote: > According to the Intel Software Development Manual: How about a more specific doc location? According to the Intel Software Development Manual (Volume 1: Basic Architecture, 12.10.3 Streaming Load Hint Instruction): >Streaming loads may be weakly

[Mesa-dev] [PATCH] mesa: Add missing _mm_mfence() before streaming loads.

2015-10-01 Thread Matt Turner
According to the Intel Software Development Manual: Streaming loads may be weakly ordered and may appear to software to execute out of order with respect to other memory operations. Software must explicitly use fences (e.g. MFENCE) if it needs to preserve order among streaming loads or