Re: [Mesa-dev] [PATCH] i965: remove second prototype of brw_batch_reloc

2019-12-05 Thread Ilia Mirkin
I am. Reading code too late at night, apparently. Sorry! On Thu, Dec 5, 2019 at 4:15 AM Lionel Landwerlin wrote: > > Hmm... I don't see it. > > Are you not confused by brw_batch_reloc/brw_state_reloc? > > -Lionel > > On 05/12/2019 06:56, Ilia Mirkin wrote: > > Signed-off-by: Ilia Mirkin > > ---

[Mesa-dev] [PATCH 1/4] anv: use os_get_total_physical_memory()

2019-12-05 Thread Jonathan Gray
Replace non-portable sysinfo() use with Mesa's os_get_total_physical_memory() Signed-off-by: Jonathan Gray --- src/intel/vulkan/anv_device.c | 9 - 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c index

[Mesa-dev] [PATCH 3/4] anv: use os_get_available_system_memory()

2019-12-05 Thread Jonathan Gray
Replace local get_available_system_memory() function with os_get_available_system_memory(). Signed-off-by: Jonathan Gray --- src/intel/vulkan/anv_device.c | 30 -- 1 file changed, 4 insertions(+), 26 deletions(-) diff --git a/src/intel/vulkan/anv_device.c

[Mesa-dev] [PATCH 4/4] util/os_misc: os_get_available_system_memory() for OpenBSD

2019-12-05 Thread Jonathan Gray
Return the smallest value of available non-kernel physical memory and the static per process data size limit as the amount of available system memory on OpenBSD. Signed-off-by: Jonathan Gray --- src/util/os_misc.c | 20 1 file changed, 20 insertions(+) diff --git

[Mesa-dev] [PATCH 2/4] util/os_misc: add os_get_available_system_memory()

2019-12-05 Thread Jonathan Gray
Add os_get_available_system_memory() derived from src/intel/vulkan/anv_device.c get_available_system_memory() Signed-off-by: Jonathan Gray --- src/util/os_misc.c | 31 +++ src/util/os_misc.h | 6 ++ 2 files changed, 37 insertions(+) diff --git

[Mesa-dev] [PATCH] nouveau: limit reported compute max memory and allocation size

2019-12-05 Thread Karol Herbst
Otherwise applications (like the OpenCL CTS) will try to allocate more memory than what the GPU is actually able to provide. Signed-off-by: Karol Herbst --- src/gallium/drivers/nouveau/nv50/nv50_screen.c | 7 +-- src/gallium/drivers/nouveau/nvc0/nvc0_screen.c | 7 +-- 2 files changed,

[Mesa-dev] [PATCH] util/atomic: Add p_atomic_add_return for the unlocked path

2019-12-05 Thread Jason Ekstrand
Fixes: 385d13f26d2 "util/atomic: Add a _return variant of p_atomic_add" --- src/util/u_atomic.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/util/u_atomic.h b/src/util/u_atomic.h index 45e8e2e0188..9cbc6dd1eaa 100644 --- a/src/util/u_atomic.h +++ b/src/util/u_atomic.h

Re: [Mesa-dev] [PATCH] util/atomic: Add p_atomic_add_return for the unlocked path

2019-12-05 Thread Ilia Mirkin
Reviewed-by: Ilia Mirkin On Thu, Dec 5, 2019 at 12:51 PM Jason Ekstrand wrote: > > Fixes: 385d13f26d2 "util/atomic: Add a _return variant of p_atomic_add" > --- > src/util/u_atomic.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/util/u_atomic.h

Re: [Mesa-dev] [PATCH] util/atomic: Add p_atomic_add_return for the unlocked path

2019-12-05 Thread Jason Ekstrand
Pushed. Thanks for catching this! On Thu, Dec 5, 2019 at 11:52 AM Ilia Mirkin wrote: > Reviewed-by: Ilia Mirkin > > On Thu, Dec 5, 2019 at 12:51 PM Jason Ekstrand > wrote: > > > > Fixes: 385d13f26d2 "util/atomic: Add a _return variant of p_atomic_add" > > --- > > src/util/u_atomic.h | 2 +-

Re: [Mesa-dev] Remove classic drivers or fork src/mesa for gallium?

2019-12-05 Thread Rob Clark
On Wed, Dec 4, 2019 at 11:58 PM Kenneth Graunke wrote: > > On Tuesday, December 3, 2019 4:39:15 PM PST Marek Olšák wrote: > > Hi, > > > > Here are 2 proposals to simplify and better optimize the GL->Gallium > > translation. > > > > 1) Move classic drivers to a fork of Mesa, and remove them from

Re: [Mesa-dev] [PATCH] nv50/ir: implement global atomics and handle it for nir

2019-12-05 Thread Karol Herbst
On Thu, Dec 5, 2019 at 11:57 AM Karol Herbst wrote: > > TGSI doesn't have any concept of global memory right now. > > Signed-off-by: Karol Herbst > --- > .../nouveau/codegen/nv50_ir_from_nir.cpp | 43 +-- > .../nouveau/codegen/nv50_ir_lowering_nvc0.cpp | 2 + > 2 files

[Mesa-dev] [PATCH] vc4: use intmax_t for formatted output of timespec members

2019-12-05 Thread Khem Raj
32bit architectures which have 64bit time_t does not fit the assumption of time_t being same as system long int Fixes error: format specifies type 'long' but the argument has type 'time_t' (aka 'long long') [-Werror,-Wformat] time.tv_sec);

Re: [Mesa-dev] [PATCH] i965: remove second prototype of brw_batch_reloc

2019-12-05 Thread Lionel Landwerlin
Hmm... I don't see it. Are you not confused by brw_batch_reloc/brw_state_reloc? -Lionel On 05/12/2019 06:56, Ilia Mirkin wrote: Signed-off-by: Ilia Mirkin --- src/mesa/drivers/dri/i965/intel_batchbuffer.h | 5 - 1 file changed, 5 deletions(-) diff --git

Re: [Mesa-dev] Remove classic drivers or fork src/mesa for gallium?

2019-12-05 Thread apinheiro
On 5/12/19 8:48, Kenneth Graunke wrote: On Tuesday, December 3, 2019 4:39:15 PM PST Marek Olšák wrote: Hi, Here are 2 proposals to simplify and better optimize the GL->Gallium translation. 1) Move classic drivers to a fork of Mesa, and remove them from master. Classic drivers won't share any

[Mesa-dev] [PATCH] nv50/ir: implement global atomics and handle it for nir

2019-12-05 Thread Karol Herbst
TGSI doesn't have any concept of global memory right now. Signed-off-by: Karol Herbst --- .../nouveau/codegen/nv50_ir_from_nir.cpp | 43 +-- .../nouveau/codegen/nv50_ir_lowering_nvc0.cpp | 2 + 2 files changed, 41 insertions(+), 4 deletions(-) diff --git