Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

2017-07-06 Thread Alex Deucher
On Thu, Jul 6, 2017 at 4:56 PM, Marek Olšák wrote: > On Thu, Jul 6, 2017 at 8:12 PM, Alex Deucher wrote: >> On Thu, Jul 6, 2017 at 1:13 PM, Jan Vesely wrote: >>> On Thu, 2017-07-06 at 12:09 +1000, Dave Airlie wrote: From: Dave Airlie Use family, but only set xnack+ for gfx9.

Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

2017-07-06 Thread Marek Olšák
On Thu, Jul 6, 2017 at 8:12 PM, Alex Deucher wrote: > On Thu, Jul 6, 2017 at 1:13 PM, Jan Vesely wrote: >> On Thu, 2017-07-06 at 12:09 +1000, Dave Airlie wrote: >>> From: Dave Airlie >>> >>> Use family, but only set xnack+ for gfx9. >>> >>> Signed-off-by: Dave Airlie >>> --- >>> src/amd/common

Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

2017-07-06 Thread Marek Olšák
On Thu, Jul 6, 2017 at 10:14 PM, Matt Arsenault wrote: > >> On Jul 6, 2017, at 13:08, Dave Airlie wrote: >> >> On 7 July 2017 at 05:07, Matt Arsenault wrote: >>> On Jul 5, 2017, at 19:09, Dave Airlie wrote: From: Dave Airlie Use family, but only set xnack+ for gfx9. >>

Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

2017-07-06 Thread Matt Arsenault
> On Jul 6, 2017, at 13:08, Dave Airlie wrote: > > On 7 July 2017 at 05:07, Matt Arsenault wrote: >> >>> On Jul 5, 2017, at 19:09, Dave Airlie wrote: >>> >>> From: Dave Airlie >>> >>> Use family, but only set xnack+ for gfx9. >>> >> >> The driver shouldn’t be explicitly setting this. Thi

Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

2017-07-06 Thread Dave Airlie
On 7 July 2017 at 05:07, Matt Arsenault wrote: > >> On Jul 5, 2017, at 19:09, Dave Airlie wrote: >> >> From: Dave Airlie >> >> Use family, but only set xnack+ for gfx9. >> > > The driver shouldn’t be explicitly setting this. This should be set as part > of the subtarget chosen Well I expect Ma

Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

2017-07-06 Thread Matt Arsenault
> On Jul 5, 2017, at 19:09, Dave Airlie wrote: > > From: Dave Airlie > > Use family, but only set xnack+ for gfx9. > The driver shouldn’t be explicitly setting this. This should be set as part of the subtarget chosen -Matt ___ mesa-dev mailing li

Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

2017-07-06 Thread Alex Deucher
On Thu, Jul 6, 2017 at 1:13 PM, Jan Vesely wrote: > On Thu, 2017-07-06 at 12:09 +1000, Dave Airlie wrote: >> From: Dave Airlie >> >> Use family, but only set xnack+ for gfx9. >> >> Signed-off-by: Dave Airlie >> --- >> src/amd/common/ac_llvm_util.c | 4 +++- >> 1 file changed, 3 insertions(+), 1

Re: [Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

2017-07-06 Thread Jan Vesely
On Thu, 2017-07-06 at 12:09 +1000, Dave Airlie wrote: > From: Dave Airlie > > Use family, but only set xnack+ for gfx9. > > Signed-off-by: Dave Airlie > --- > src/amd/common/ac_llvm_util.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/src/amd/common/ac_llvm_util.

[Mesa-dev] [PATCH 3/4] ac/llvm: set xnack like radeonsi does.

2017-07-05 Thread Dave Airlie
From: Dave Airlie Use family, but only set xnack+ for gfx9. Signed-off-by: Dave Airlie --- src/amd/common/ac_llvm_util.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/amd/common/ac_llvm_util.c b/src/amd/common/ac_llvm_util.c index 4f32d4b..84ed73f 100644 --- a/src/