Re: [Mesa-dev] [PATCH 1/3] st/vdpau: don't return a device if the screen doesn't support NPOT

2014-01-18 Thread Christian König
Am 18.01.2014 05:58, schrieb Ilia Mirkin: NV3x cards don't support NPOT textures. Technically this restriction could be worked around, but since it also doesn't expose any video decoding hw, just turn it off entirely. Signed-off-by: Ilia Mirkin Cc: 10.0 Actually we had a similar problem for

Re: [Mesa-dev] [PATCH 1/3] st/vdpau: don't return a device if the screen doesn't support NPOT

2014-01-18 Thread Ilia Mirkin
On Sat, Jan 18, 2014 at 5:27 AM, Christian König wrote: > Am 18.01.2014 05:58, schrieb Ilia Mirkin: > >> NV3x cards don't support NPOT textures. Technically this restriction >> could be worked around, but since it also doesn't expose any video >> decoding hw, just turn it off entirely. >> >> Signe

Re: [Mesa-dev] [PATCH 1/3] st/vdpau: don't return a device if the screen doesn't support NPOT

2014-01-18 Thread Christian König
Am 18.01.2014 11:34, schrieb Ilia Mirkin: On Sat, Jan 18, 2014 at 5:27 AM, Christian König wrote: Am 18.01.2014 05:58, schrieb Ilia Mirkin: NV3x cards don't support NPOT textures. Technically this restriction could be worked around, but since it also doesn't expose any video decoding hw, just

Re: [Mesa-dev] [PATCH 3/3] st/vdpau: check surface params before creating resources

2014-01-18 Thread Christian König
Am 18.01.2014 06:05, schrieb Ilia Mirkin: On Fri, Jan 17, 2014 at 11:58 PM, Ilia Mirkin wrote: Signed-off-by: Ilia Mirkin --- This seems like a function that should exist in gallium/util, but I didn't find it. I guess the mesa/st does these checks far enough away from actual resource creation

Re: [Mesa-dev] [PATCH 1/3] st/vdpau: don't return a device if the screen doesn't support NPOT

2014-01-18 Thread Ilia Mirkin
On Sat, Jan 18, 2014 at 5:40 AM, Christian König wrote: > Am 18.01.2014 11:34, schrieb Ilia Mirkin: > >> On Sat, Jan 18, 2014 at 5:27 AM, Christian König >> wrote: >>> >>> Am 18.01.2014 05:58, schrieb Ilia Mirkin: >>> NV3x cards don't support NPOT textures. Technically this restriction

Re: [Mesa-dev] Fwd: munmap_chunk(): invalid pointer in libgl1-mesa-dri_9.2.1-1

2014-01-18 Thread scarp
-BEGIN PGP SIGNED MESSAGE- Hash: SHA512 Ian Romanick: > On 01/16/2014 10:42 PM, scarp wrote: >> Hi, > >> I originally posted this to dri-users and got no response. I >> thought about it further and decided it's really a bug and >> deserves to be posted to dri-devel, after all it is a cra

[Mesa-dev] [Bug 73761] New: driCreateContextAttribs uses freed memory on context recreation

2014-01-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73761 Priority: medium Bug ID: 73761 Assignee: mesa-dev@lists.freedesktop.org Summary: driCreateContextAttribs uses freed memory on context recreation Severity: normal Classifi

Re: [Mesa-dev] [PATCH v3 1/1] clover: Don't crash on NULL global buffer objects.

2014-01-18 Thread Francisco Jerez
Jan Vesely writes: > Specs say "If the argument is a buffer object, the arg_value > pointer can be NULL or point to a NULL value in which case a NULL > value will be used as the value for the argument declared as a > pointer to __global or __constant memory in the kernel." > > So don't crash when

[Mesa-dev] [PATCH 2/2] gallium: remove PIPE_CAP_SCALED_RESOLVE

2014-01-18 Thread Marek Olšák
From: Marek Olšák If any driver doesn't support this, it can use a blit after resolving the samples. --- src/gallium/drivers/freedreno/freedreno_screen.c | 1 - src/gallium/drivers/i915/i915_screen.c | 1 - src/gallium/drivers/ilo/ilo_screen.c | 1 - src/gallium/drivers/llv

[Mesa-dev] [PATCH 1/2] gallium: remove PIPE_CAP_MAX_COMBINED_SAMPLERS

2014-01-18 Thread Marek Olšák
From: Marek Olšák This can be derived from the shader caps. --- src/gallium/docs/source/screen.rst | 2 -- src/gallium/drivers/freedreno/freedreno_screen.c | 2 -- src/gallium/drivers/i915/i915_screen.c | 7 --- src/gallium/drivers/ilo/ilo_screen.c | 2

[Mesa-dev] [Bug 73631] es2tri fragment shader did not compile AMD ARUBA

2014-01-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73631 --- Comment #3 from Aaron Paden --- Yep, that solves the problem on my machine as well. -- You are receiving this mail because: You are the assignee for the bug. ___ mesa-dev mailing list mesa-dev@lis

[Mesa-dev] [PATCH] i965: Add some informative debug when the X Server botches DRI2 GetBuffers.

2014-01-18 Thread Eric Anholt
We've had various bug reports over the years where miptrees are missing, and when I screwed it up while adding DRI2 to the modesetting driver, I figured I should put the info necessary for debug here. --- src/mesa/drivers/dri/i965/brw_context.c | 12 +++- 1 file changed, 11 insertions(+),

[Mesa-dev] [PATCH] i965: Remove redundant check in blitter-based glBlitFramebuffer().

2014-01-18 Thread Eric Anholt
The intel_miptree_blit() code checks the format for us now, plus it handles xrgb vs argb for us. --- src/mesa/drivers/dri/i965/intel_fbo.c | 10 -- 1 file changed, 10 deletions(-) diff --git a/src/mesa/drivers/dri/i965/intel_fbo.c b/src/mesa/drivers/dri/i965/intel_fbo.c index f094718..73

Re: [Mesa-dev] [PATCH 4/5] glcpp: Remove unused gl_api bits.

2014-01-18 Thread Kenneth Graunke
On 01/17/2014 09:09 PM, Matt Turner wrote: > --- > src/glsl/glcpp/glcpp-parse.y | 1 - > src/glsl/glcpp/glcpp.c | 1 - > 2 files changed, 2 deletions(-) > > diff --git a/src/glsl/glcpp/glcpp-parse.y b/src/glsl/glcpp/glcpp-parse.y > index c774662..427fdec 100644 > --- a/src/glsl/glcpp/glcpp-

Re: [Mesa-dev] code de-duplication and non-pci support v2

2014-01-18 Thread Emil Velikov
On 16/01/14 23:29, Kristian Høgsberg wrote: > On Sat, Jan 11, 2014 at 04:51:42PM -0500, Rob Clark wrote: >> > On Sat, Jan 11, 2014 at 11:54 AM, Emil Velikov >> > wrote: >>> > > This is an updated series of Rob's patches >>> > > >>> > > * The introduction of the util library is separated from the

Re: [Mesa-dev] [PATCH 4/5] glcpp: Remove unused gl_api bits.

2014-01-18 Thread Kenneth Graunke
On 01/18/2014 11:13 AM, Kenneth Graunke wrote: > On 01/17/2014 09:09 PM, Matt Turner wrote: >> --- >> src/glsl/glcpp/glcpp-parse.y | 1 - >> src/glsl/glcpp/glcpp.c | 1 - >> 2 files changed, 2 deletions(-) >> >> diff --git a/src/glsl/glcpp/glcpp-parse.y b/src/glsl/glcpp/glcpp-parse.y >> inde

Re: [Mesa-dev] [PATCH 00/10] nv50: out-of-bounds access of context specific attribs

2014-01-18 Thread Emil Velikov
On 16/01/14 18:54, Ilia Mirkin wrote: > On Thu, Jan 16, 2014 at 1:44 PM, Emil Velikov > wrote: >> Subject: [PATCH 00/10] nv50: out-of-bounds access of context specific attribs >> In-Reply-To: >> >> So here it goes, a few serious patches and some not as much >> >> Patches 1, 2 correct the range to

Re: [Mesa-dev] [RFC] Build testing, wine style

2014-01-18 Thread Emil Velikov
On 13/01/14 18:47, Tom Stellard wrote: > On Sat, Jan 11, 2014 at 03:53:58PM +, Emil Velikov wrote: >> Hello list, >> >> While going though mesa's build systems I was wondering what it would >> take to improve the overall experience of build testing. >> >> The only thing I can think of is a more

[Mesa-dev] [Bug 73761] driCreateContextAttribs uses freed memory on context recreation

2014-01-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73761 Ilia Mirkin changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---

[Mesa-dev] [Bug 73761] driCreateContextAttribs uses freed memory on context recreation

2014-01-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73761 --- Comment #2 from Török Edwin --- (In reply to comment #1) > Yes, I reported this very shortly before the 10.0 release, and it should be > fixed by 38366c0c6e715314367b15680702e382d5c46a4a in the main tree, and > db0dc5c008f3f4a475de1c6471f99f4

Re: [Mesa-dev] [RFC] Build testing, wine style

2014-01-18 Thread Dave Airlie
On Sun, Jan 19, 2014 at 5:45 AM, Emil Velikov wrote: > On 13/01/14 18:47, Tom Stellard wrote: >> On Sat, Jan 11, 2014 at 03:53:58PM +, Emil Velikov wrote: >>> Hello list, >>> >>> While going though mesa's build systems I was wondering what it would >>> take to improve the overall experience of

[Mesa-dev] [Bug 73776] New: NameError: name 'libloader' is not defined:

2014-01-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73776 Priority: medium Bug ID: 73776 Keywords: regression CC: emil.l.veli...@gmail.com, i...@freedesktop.org, k...@bitplanet.net, robcl...@freedesktop.org Assignee: mesa-dev@l

[Mesa-dev] [Bug 73777] New: xf86drm.h:40:17: error: drm.h: No such file or directory

2014-01-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73777 Priority: medium Bug ID: 73777 Keywords: regression CC: emil.l.veli...@gmail.com, i...@freedesktop.org, robcl...@freedesktop.org Assignee: mesa-dev@lists.freedesktop.org

Re: [Mesa-dev] [PATCH] i965: Only update renderbuffers on initial intelMakeCurrent

2014-01-18 Thread Kristian Høgsberg
On Fri, Jan 17, 2014 at 5:49 PM, Matt Turner wrote: > On Fri, Jan 17, 2014 at 5:21 PM, Kristian Høgsberg wrote: >> We call intel_prepare_render() in intelMakeCurrent() to make sure we have >> renderbuffers before calling _mesa_make_current(). The only reason we >> do this is so that we can have

[Mesa-dev] [PATCH] loader: ifdef libdrm specific code and include

2014-01-18 Thread Emil Velikov
Mesa provides the flexibility of building without the need to have libdrm present on the system. The situation has regressed with the recent commit This is a regression caused by commit commit 8c2e7fd8460750543367053b1be9368cc38e1d6a Author: Emil Velikov Date: Fri Jan 10 23:36:16 2014 +

[Mesa-dev] [Bug 73777] xf86drm.h:40:17: error: drm.h: No such file or directory

2014-01-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73777 --- Comment #1 from Emil Velikov --- Created attachment 92371 --> https://bugs.freedesktop.org/attachment.cgi?id=92371&action=edit handle builds whenever libdrm is not installed Nice catch Vinson, I had to move my libdrm.pc to reproduce this a

[Mesa-dev] [Bug 73776] NameError: name 'libloader' is not defined:

2014-01-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73776 --- Comment #1 from Emil Velikov --- Created attachment 92372 --> https://bugs.freedesktop.org/attachment.cgi?id=92372&action=edit handle builds whenever libdrm is not installed As mentioned in bug 73777 I've folded all three possible fixes in

Re: [Mesa-dev] [PATCH 2/2] i965: Bump BRW_MAX_TEX_UNIT to 32.

2014-01-18 Thread Kenneth Graunke
On 01/17/2014 01:57 PM, Chris Forbes wrote: > Ken, > > Assuming the caches don't completely derail things, you ought to be > able to make this work with pretty minimal impact: > > - Keep the low four bits of the sampler index where they are > - If the 5th bit is set: > - Force message header

Re: [Mesa-dev] [PATCH] i965: Add some informative debug when the X Server botches DRI2 GetBuffers.

2014-01-18 Thread Kenneth Graunke
On 01/18/2014 10:27 AM, Eric Anholt wrote: > We've had various bug reports over the years where miptrees are missing, > and when I screwed it up while adding DRI2 to the modesetting driver, I > figured I should put the info necessary for debug here. > --- > src/mesa/drivers/dri/i965/brw_context.c

Re: [Mesa-dev] [PATCH] i965: Remove redundant check in blitter-based glBlitFramebuffer().

2014-01-18 Thread Kenneth Graunke
On 01/18/2014 10:30 AM, Eric Anholt wrote: > The intel_miptree_blit() code checks the format for us now, plus it > handles xrgb vs argb for us. > --- > src/mesa/drivers/dri/i965/intel_fbo.c | 10 -- > 1 file changed, 10 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/intel_fbo.c

Re: [Mesa-dev] [PATCH] loader: ifdef libdrm specific code and include

2014-01-18 Thread Kenneth Graunke
On 01/18/2014 07:23 PM, Emil Velikov wrote: > Mesa provides the flexibility of building without the > need to have libdrm present on the system. The situation > has regressed with the recent commit > > This is a regression caused by commit > commit 8c2e7fd8460750543367053b1be9368cc38e1d6a > Author

[Mesa-dev] [PATCH 07/10] i965/vec4: Support arbitrarily large sampler state indices on Haswell+.

2014-01-18 Thread Kenneth Graunke
Like the scalar backend, we add an offset to the "Sampler State Pointer" field to select a group of 16 samplers, then use the "Sampler Index" field to select within that group. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_vec4_generator.cpp | 19 ++- src/mesa/

[Mesa-dev] [PATCH 10/10] i965: Support 32 texture image units on Haswell+.

2014-01-18 Thread Kenneth Graunke
The Intel closed source OpenGL driver recently began supporting 32 texture image units on Haswell. This makes the open source driver support 32 as well. Earlier generations don't have the message header field required to support more than 16 sampler states, so we continue to advertise 16 there.

[Mesa-dev] [PATCH 04/10] i965/fs: Support arbitrarily large sampler state indices on Haswell+.

2014-01-18 Thread Kenneth Graunke
The message descriptor's "Sampler Index" field is only 4 bits (on all generations of hardware), so it can only represent indices 0 through 15. Haswell introduced a new field in the message header - "Sampler State Pointer". Normally, this is copied straight from g0, but we can also add a byte offs

[Mesa-dev] [PATCH 09/10] i965/fs: Switch from BRW_MAX_TEX_UNIT to the actual limit.

2014-01-18 Thread Kenneth Graunke
BRW_MAX_TEX_UNIT is about to grow, but only Gen7+ will be able to support the new larger value. On older platforms, we don't want to allocate the extra space - it would just be a waste. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_wm.c | 3 ++- 1 file changed, 2 insertions(+

[Mesa-dev] [PATCH 06/10] i965/vec4: Refactor sampler message setup.

2014-01-18 Thread Kenneth Graunke
The next patch adds an additional case where the message header is necessary. So we want to do the g0 copy if inst->header_present is set, rather than inst->texture_offset. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_vec4_generator.cpp | 39 +--- 1 file

[Mesa-dev] [PATCH 01/10] i965: Use get_element_ud to shorten texture header access.

2014-01-18 Thread Kenneth Graunke
This is shorter, easier to read, and further from the 80 column limit. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_fs_generator.cpp | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_generator.cpp b/src/mesa/drivers/dri/i9

[Mesa-dev] [PATCH 05/10] i965/vec4: Don't set header_present if texel offsets are all 0.

2014-01-18 Thread Kenneth Graunke
In theory, a shader might use textureOffset() but set all the texel offsets to zero. In that case, we don't actually need to set up the message header - zero is the implicit default. By moving the texture_offset setup before the header_present setup, we can easily only set header_present when the

[Mesa-dev] [PATCH 03/10] i965/fs: Plumb sampler index into emit_texture_gen7.

2014-01-18 Thread Kenneth Graunke
We'll need this in the next patch. Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/brw_fs.h | 2 +- src/mesa/drivers/dri/i965/brw_fs_fp.cpp | 2 +- src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/sr

[Mesa-dev] [PATCH 08/10] mesa: Bump MAX_TEXTURE_IMAGE_UNITS to 32.

2014-01-18 Thread Kenneth Graunke
This allows drivers to optionally support more than 16 texture units. Signed-off-by: Kenneth Graunke --- src/mesa/main/config.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/main/config.h b/src/mesa/main/config.h index ff9da77..9df4a6e 100644 --- a/src/mesa/main/co

[Mesa-dev] [PATCH 02/10] i965/fs: Refactor sampler message header to duplicate less code.

2014-01-18 Thread Kenneth Graunke
Previously, the code to copy g0 to the message header existed in two places - one for the texture offset case, and one for any other case. By treating texture_offset as a special case of header_present, we can remove this duplication and shorten the code. Future patches which add new header field

[Mesa-dev] [Bug 73776] NameError: name 'libloader' is not defined:

2014-01-18 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=73776 --- Comment #2 from Jos van Wolput --- The latest git version of mesa fails to build: - make[3]: Entering directory `/home/jos/src/xorg/git/mesa/src/gallium/auxiliary/pipe-loader' CC pipe_loader_drm.lo pipe_loader_drm.c: In function '