The case for this was in the wrong function, and this format's store
func was not set in the table at all.
Signed-off-by: Chris Forbes
---
src/mesa/main/texstore.c | 10 ++
1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/src/mesa/main/texstore.c b/src/mesa/main/texstore.c
i
On 03/28/2014 04:58 PM, Matt Turner wrote:
> It was my understanding that the writemask works in SIMD4x2 mode for
> texturing instructions and doesn't require a message header. Some bit of
> this logic must be wrong, so disable it until it's understood.
>
> Bugzilla: https://bugs.freedesktop.org/s
Am 29.03.2014 00:57, schrieb Ilia Mirkin:
> On Fri, Mar 28, 2014 at 7:43 PM, Roland Scheidegger
> wrote:
>> Am 28.03.2014 23:57, schrieb Ilia Mirkin:
>>> On Fri, Mar 28, 2014 at 6:41 PM, Roland Scheidegger
>>> wrote:
Am 28.03.2014 22:56, schrieb Ilia Mirkin:
> On Fri, Mar 28, 2014 at 5
It was my understanding that the writemask works in SIMD4x2 mode for
texturing instructions and doesn't require a message header. Some bit of
this logic must be wrong, so disable it until it's understood.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=76617
---
v2: Base on master, rather t
On Fri, Mar 28, 2014 at 7:43 PM, Roland Scheidegger wrote:
> Am 28.03.2014 23:57, schrieb Ilia Mirkin:
>> On Fri, Mar 28, 2014 at 6:41 PM, Roland Scheidegger
>> wrote:
>>> Am 28.03.2014 22:56, schrieb Ilia Mirkin:
On Fri, Mar 28, 2014 at 5:47 PM, Roland Scheidegger
wrote:
> Am 28
It was my understanding that the writemask works in SIMD4x2 mode for
texturing instructions and doesn't require a message header. Some bit of
this logic must be wrong, so disable it until it's understood.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=76617
---
src/mesa/drivers/dri/i965/b
Am 28.03.2014 23:57, schrieb Ilia Mirkin:
> On Fri, Mar 28, 2014 at 6:41 PM, Roland Scheidegger
> wrote:
>> Am 28.03.2014 22:56, schrieb Ilia Mirkin:
>>> On Fri, Mar 28, 2014 at 5:47 PM, Roland Scheidegger
>>> wrote:
Am 28.03.2014 22:18, schrieb Ilia Mirkin:
> Hey guys,
>
> I w
_mesa_format_matches_format_and_type() returns true for
GL_RED/GL_RED_INTEGER (with an appropriate type) into an intensity
mesa_format.
We want the `red`-based format instead, regardless of the order we find
them in our walk of the mesa formats list.
Signed-off-by: Chris Forbes
---
src/mesa/sta
GL_INTENSITY has never been valid as a pixel format -- to get the memcpy
pack/unpack paths, the app needs to specify GL_RED as the pixel format
(or GL_RED_INTEGER for the integer formats).
Note: This was briefly merged before, but exposed some breakage in gallium, so
was reverted. Hopefully it wil
On Fri, Mar 28, 2014 at 6:41 PM, Roland Scheidegger wrote:
> Am 28.03.2014 22:56, schrieb Ilia Mirkin:
>> On Fri, Mar 28, 2014 at 5:47 PM, Roland Scheidegger
>> wrote:
>>> Am 28.03.2014 22:18, schrieb Ilia Mirkin:
Hey guys,
I was thinking of taking a shot at implementing ARB_sampl
Reviewed-by: Matt Turner
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Am 28.03.2014 22:56, schrieb Ilia Mirkin:
> On Fri, Mar 28, 2014 at 5:47 PM, Roland Scheidegger
> wrote:
>> Am 28.03.2014 22:18, schrieb Ilia Mirkin:
>>> Hey guys,
>>>
>>> I was thinking of taking a shot at implementing ARB_sample_shading for
>>> nv50 (well, nva3-nva8) this weekend. One of the is
Or else poor programmers might mistakenly use the temporary mem_ctx,
instead of the fs_visitor's mem_ctx and wonder why their code is
crashing.
Also remove the parenting. These contexts are local to the optimization
passes they're in and are freed at the end.
---
src/mesa/drivers/dri/i965/brw_fs_
Otherwise calling dump_instructions() after declaring a new fs_reg would
segfault when calculate_register_pressure()'s loop over reg walked off
the end of the virtual_grf_start[] array that calculate_live_intervals()
would have reallocated for you, if it had known there was a new
register.
---
src
On Fri, Mar 28, 2014 at 5:47 PM, Roland Scheidegger wrote:
> Am 28.03.2014 22:18, schrieb Ilia Mirkin:
>> Hey guys,
>>
>> I was thinking of taking a shot at implementing ARB_sample_shading for
>> nv50 (well, nva3-nva8) this weekend. One of the issues is that it's
>> not implemented in gallium at a
On Wed, Mar 26, 2014 at 5:12 PM, Ian Romanick wrote:
> Tomorrow or Friday I'm going to send out the last of the
> GL_ARB_separate_shader_objects patches. Shortly after that, I will send
> out patches to enable GL_EXT_separate_shader_objects on GLES. This EXT
> is the GLES subset of the ARB exten
On Thu, Mar 27, 2014 at 2:00 PM, Emil Velikov wrote:
> Hello list,
>
> Yet another small series from me, targeting the following
>
> - Revert "allow only shared builds". Static osmesa/libgl-xlib is
> still a used, so rever the commit for now.
>
> - Add -no-undefined, -Wl,--no-undefined and -Wl,-
On Thu, Mar 27, 2014 at 2:00 PM, Emil Velikov wrote:
> Signed-off-by: Emil Velikov
> ---
> src/gallium/targets/gbm/Makefile.am | 5 -
> src/gallium/targets/xa/Makefile.am | 7 ++-
> 2 files changed, 10 insertions(+), 2 deletions(-)
>
> diff --git a/src/gallium/targets/gbm/Makefile.am
>
On Thu, Mar 27, 2014 at 2:00 PM, Emil Velikov wrote:
> Signed-off-by: Emil Velikov
> ---
> src/gallium/targets/pipe-loader/Makefile.am | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/gallium/targets/pipe-loader/Makefile.am
> b/src/gallium/targets/pipe-loader/M
Am 28.03.2014 22:18, schrieb Ilia Mirkin:
> Hey guys,
>
> I was thinking of taking a shot at implementing ARB_sample_shading for
> nv50 (well, nva3-nva8) this weekend. One of the issues is that it's
> not implemented in gallium at all right now, so I need to pipe it
> through somehow. I believe th
On Thu, Mar 27, 2014 at 2:00 PM, Emil Velikov wrote:
> Signed-off-by: Emil Velikov
> ---
> src/gallium/targets/egl-static/Makefile.am | 19 +--
> 1 file changed, 13 insertions(+), 6 deletions(-)
>
> diff --git a/src/gallium/targets/egl-static/Makefile.am
> b/src/gallium/targets/
On Thu, Mar 27, 2014 at 2:00 PM, Emil Velikov wrote:
> We've been copying and pasting this hunk for a while now, only to prevent
> build issues on very old and buggy build toolchains. At this point this
> should no longer be needed, so we can cleanup all the mess and simplify
> our makefiles.
Tha
On Fri, Mar 28, 2014 at 10:32 AM, Ilia Mirkin wrote:
> On Fri, Mar 28, 2014 at 11:14 AM, Aaron Watry wrote:
>>>
>>> Do people have opinions on whether it'd be useful to also gather data
>>> for older hardware? FWIW I threw my TNT2 in there, which is probably
>>> among the oldest hw supported by m
On Thu, Mar 27, 2014 at 2:00 PM, Emil Velikov wrote:
> Cc: Christian König
> Signed-off-by: Emil Velikov
> ---
> src/gallium/Automake.inc | 6 ++
> src/gallium/targets/r600/omx/Makefile.am | 5 +
> src/gallium/targets/radeonsi/omx/Makefile.am | 5 +
> 3 files
On Thu, Mar 27, 2014 at 2:00 PM, Emil Velikov wrote:
> This lets us have only one if HAVE_MESA_LLVM block, rather than
> one for each driver.
>
> Signed-off-by: Emil Velikov
> ---
> src/gallium/targets/pipe-loader/Makefile.am | 34
> ++---
> 1 file changed, 11 insertions
On Thu, Mar 27, 2014 at 2:00 PM, Emil Velikov wrote:
> Mesa does _not_ against libudev.
I think you a word. :)
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On Thu, Mar 27, 2014 at 2:00 PM, Emil Velikov wrote:
> Evidently at least static OSMesa is still used as shared one
> causes substantial increase in the load time for some programs
> that use it (from seconds up-to ~30min).
>
> Rather than forcing everyone to use shared mesa, revert commit
> a6efb
Hey guys,
I was thinking of taking a shot at implementing ARB_sample_shading for
nv50 (well, nva3-nva8) this weekend. One of the issues is that it's
not implemented in gallium at all right now, so I need to pipe it
through somehow. I believe that the only piece of data that needs to
be piped throu
On Fri, Mar 28, 2014 at 6:28 AM, Juha-Pekka Heikkila
wrote:
> v3:
> I took out accumulator flag from backend_instruction::has_side_effects()
> as Matt suggested and rebased my patches on top of master where had shown up
> Matt's patches which were overlapping a bit on dead code elimination.
> This
On Fri, Mar 28, 2014 at 6:28 AM, Juha-Pekka Heikkila
wrote:
> + if (inst->writes_accumulator||inst->writes_flag())
Spaces around ||
> inst->dst = dst_reg(retype(brw_null_reg(), inst->dst.type));
> - break;
> - default:
> - if (inst->writes_flag()) {
> -
On Fri, Mar 28, 2014 at 6:28 AM, Juha-Pekka Heikkila
wrote:
> diff --git a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
> b/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
> index a951459..492ee0d 100644
> --- a/src/mesa/drivers/dri/i965/brw_schedule_instructions.cpp
> +++ b/sr
On Thu, Mar 27, 2014 at 1:29 AM, Samuel Iglesias Gonsalvez
wrote:
> Commit 079bdba05f870807d3ed77fa3093cdb7727aa2fd enabled the use of BLORP
> engine for single sample scaled blit with bilinear filter.
>
> However piglit fails when running fbo-blit-stretch test on SandyBridge.
> This patch makes t
On 28.03.2014 08:01, Kenneth Graunke wrote:
On 03/27/2014 11:34 PM, Kenneth Graunke wrote:
On 03/27/2014 01:59 PM, Rafal Mielniczuk wrote:
Just return and do nothing if query result is not yet available
Signed-off-by: Rafal Mielniczuk
---
src/mesa/main/queryobj.c | 16
1 f
https://bugs.freedesktop.org/show_bug.cgi?id=76753
Priority: medium
Bug ID: 76753
Keywords: regression
CC: bri...@vmware.com, chr...@ijw.co.nz
Assignee: mesa-dev@lists.freedesktop.org
Summary: [swrast] piglit arb_clear_buffer_obj
This allows us to emit ADD/MUL/MAC instead of MUL/ADD/MUL/ADD,
saving one instruction and two temporary registers.
Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/drivers/dri/i965/brw_vec4_visitor.cpp | 22 ++
1 file changed, 6 insertions(+), 16 deletions(-)
diff --git a/src
On 28.03.2014 07:51, Kenneth Graunke wrote:
On 03/27/2014 01:59 PM, Rafal Mielniczuk wrote:
Hello,
This is the second version of the series implementing
ARB_query_buffer_object in mesa.
Main changes to the first version are:
- Enable extension only for software driver
- Fix possible segfau
https://bugs.freedesktop.org/show_bug.cgi?id=76212
Sylvain BERTRAND changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
On Fri, 28 Mar 2014 10:08:53 +
Emil Velikov wrote:
> On 28/03/14 08:35, Lauri Kasanen wrote:
> > Emil Velikov wrote:
> >
> >> By doing GC the linker removes all the symbols that are not referenced
> >> and/or used by the final library. This results in a saving of ~100K
> >> up-to ~600K per
Am 28.03.2014 05:06, schrieb Kenneth Graunke:
>
> I've attached the info for everything on the i965 driver. In case you
> ever want to regenerate things, I just ran:
>
> $ INTEL_DEVID_OVERRIDE=0x29a2 glxinfo -l -s &> gen4
> $ INTEL_DEVID_OVERRIDE=0x2a42 glxinfo -l -s &> gen4.5
> $ INTEL_DEVID_OV
On 03/27/2014 02:33 PM, Ian Romanick wrote:
> From: Ian Romanick
>
> This is the closing for the "\defgroup IR Intermediate representation
> nodes" all the way at the top of the file.
>
> Signed-off-by: Ian Romanick
> ---
> src/glsl/ir.h | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions
On Fri, 28 Mar 2014 09:57:15 +0100
Marc Dietrich wrote:
> lto is broken on many compiler/ld combinations. Even if it is supported I
> won't recomment to enable it. A config option to enable it would be nice
> though.
>
> Lauri, on which compiler/binutils version did you got it going?
GNU ld (
On 03/26/2014 02:23 PM, Eric Anholt wrote:
> Here's a little series I wrote yesterday while tracking down why some
> silly MOVs were generated in a microbenchmark. As usual with
> optimization, the thing I was trying to work on (not present in this
> series) ended up requiring a bunch of other wor
It would be nicer not to use dots to indicate an extension is
supported. I recommend using a high-contrast background color for
of supported extensions and white or other neutral color for
unsupported extensions, a lot like piglit has green for pass and gray
for skip, but each vendor can have thei
>
> Do people have opinions on whether it'd be useful to also gather data
> for older hardware? FWIW I threw my TNT2 in there, which is probably
> among the oldest hw supported by mesa.
>
I'm not sure if it's worthwhile or not, but if you want/need it, I've
got a Radeon x1950 at home that I can po
On 03/26/2014 02:23 PM, Eric Anholt wrote:
> Basically, replace the output_components[] array with per-channel tracking
> of the register storing that channel, or a BAD_FILE undefined reg.
>
> Right now var->data.location_frac is always 0, but I'm going to use that
> in vector_splitting next.
> --
On Fri, Mar 28, 2014 at 10:15 AM, Roland Scheidegger wrote:
> Am 28.03.2014 05:06, schrieb Kenneth Graunke:
>>
>> I've attached the info for everything on the i965 driver. In case you
>> ever want to regenerate things, I just ran:
>>
>> $ INTEL_DEVID_OVERRIDE=0x29a2 glxinfo -l -s &> gen4
>> $ INT
This allows us to generate the MAC (multiply-accumulate) instruction,
which can be used to implement some expressions in fewer instructions
than doing a series of MUL and ADDs.
Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/drivers/dri/i965/brw_fs.cpp | 1 +
src/mesa/drivers/dri/i965/
Our hardware has an "accumulator" register, which can be used to store
intermediate results across multiple instructions. Many instructions
can implicitly write a value to the accumulator in addition to their
normal destination register. This is enabled by the "AccWrEn" flag.
This patch introduc
This allows us to emit ADD/MUL/MAC instead of MUL/ADD/MUL/ADD,
saving one instruction and two temporary registers.
Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/drivers/dri/i965/brw_fs_visitor.cpp | 11 ---
1 file changed, 4 insertions(+), 7 deletions(-)
diff --git a/src/mesa/drivers/
This allows us to generate the MAC (multiply-accumulate) instruction,
which can be used to implement some expressions in fewer instructions
than doing a series of MUL and ADDs.
Signed-off-by: Juha-Pekka Heikkila
---
src/mesa/drivers/dri/i965/brw_eu.h | 1 +
src/mesa/drivers/dri/i96
v3:
I took out accumulator flag from backend_instruction::has_side_effects()
as Matt suggested and rebased my patches on top of master where had shown up
Matt's patches which were overlapping a bit on dead code elimination.
This set does not do anything for the MACH anomaly on vec4_visitor::visit
Am 28.03.2014 13:26, schrieb Emil Velikov:
Unused due to the missing GALLIUM_TRACE define.
Requested-by: Christian König
Signed-off-by: Emil Velikov
Reviewed-by: Christian König
---
Hi Christian
Seems like none of the other video accel targets use the tracer.
I'm assuming that the VDPAU
Unused due to the missing GALLIUM_TRACE define.
Requested-by: Christian König
Signed-off-by: Emil Velikov
---
Hi Christian
Seems like none of the other video accel targets use the tracer.
I'm assuming that the VDPAU issue(s) you've mentioned have been
resolved.
-Emil
src/gallium/targets/r60
On Fri, Mar 28, 2014 at 8:25 AM, Eric Anholt wrote:
> I was looking at a bug report about old software, and wanted to see what
> development branch a quoted commit was on:
>
> anholt@eliezer:anholt/src/mesa-release% git describe
> 97217a40f97cdeae0304798b607f704deb0c3558
> snb-magic-15797-g97217a
On 28/03/14 09:40, Christian König wrote:
> Am 27.03.2014 22:00, schrieb Emil Velikov:
>> Otherwise the omx drivers are explicitly linked but never wrapped in
>> order to use it.
>
> On the other hand I'm not sure if we really need the tracer linked in here,
> referencing it was just to make drm_t
On 28/03/14 08:35, Lauri Kasanen wrote:
> On Thu, 27 Mar 2014 21:00:39 +
> Emil Velikov wrote:
>
>> By doing GC the linker removes all the symbols that are not referenced
>> and/or used by the final library. This results in a saving of ~100K
>> up-to ~600K per (stripped) binary (classic vs ga
https://bugs.freedesktop.org/show_bug.cgi?id=76252
--- Comment #7 from cgerlac...@gmail.com ---
José,
thanks for the feedback.
Our first try was to call the dll llvmpipe.dll and we ran exactly in the issues
you mentioned regarding GDI. Therefore we don't see this as an option for us.
A colleaq
Am Freitag, 28. März 2014, 10:35:00 schrieb Lauri Kasanen:
> On Thu, 27 Mar 2014 21:00:39 +
>
> Emil Velikov wrote:
> > By doing GC the linker removes all the symbols that are not referenced
> > and/or used by the final library. This results in a saving of ~100K
> > up-to ~600K per (stripped)
On Thu, 27 Mar 2014 21:00:39 +
Emil Velikov wrote:
> By doing GC the linker removes all the symbols that are not referenced
> and/or used by the final library. This results in a saving of ~100K
> up-to ~600K per (stripped) binary (classic vs gallium drivers).
>
> If interested one can ask th
On Fri, 2014-03-28 at 19:22 +1100, Timothy Arceri wrote:
> On Wed, 2014-03-26 at 16:27 -0700, Felipe Tonello wrote:
> > Hi Timothy,
> >
> > On Sun, Mar 23, 2014 at 2:11 PM, Timothy Arceri
> > wrote:
> > > On Mon, 2014-03-17 at 11:42 -0700, Felipe Tonello wrote:
> > >> Hi all,
> > >>
> > >> I'm w
On Wed, 2014-03-26 at 16:27 -0700, Felipe Tonello wrote:
> Hi Timothy,
>
> On Sun, Mar 23, 2014 at 2:11 PM, Timothy Arceri wrote:
> > On Mon, 2014-03-17 at 11:42 -0700, Felipe Tonello wrote:
> >> Hi all,
> >>
> >> I'm working on the KHR_debug for OpenGL ES junior job. I recently
> >> submitted pa
Kenneth Graunke writes:
> On 03/26/2014 02:23 PM, Eric Anholt wrote:
>> I need to extend the size of the outputs[] array, and this was going to
>> get ridiculous.
>> ---
>> src/mesa/drivers/dri/i965/brw_fs.cpp | 36
>> ++--
>> 1 file changed, 22 insertions(+), 14
I was looking at a bug report about old software, and wanted to see what
development branch a quoted commit was on:
anholt@eliezer:anholt/src/mesa-release% git describe
97217a40f97cdeae0304798b607f704deb0c3558
snb-magic-15797-g97217a4
That's... not terribly useful. It would be nice i
When doing software rendering (i.e. rendering to the selection buffer) we need
to make sure that we have valid index bounds before calling _tnl_draw_prims(),
otherwise we can crash.
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=59455
---
src/mesa/drivers/dri/i965/brw_draw.c | 3 ++-
1 fi
On 03/27/2014 11:34 PM, Kenneth Graunke wrote:
> On 03/27/2014 01:59 PM, Rafal Mielniczuk wrote:
>> Just return and do nothing if query result is not yet available
>>
>> Signed-off-by: Rafal Mielniczuk
>> ---
>> src/mesa/main/queryobj.c | 16
>> 1 file changed, 16 insertions(+)
>
65 matches
Mail list logo
