bytes(struct util_dynarray *buf, unsigned nelts, size_t
> eltsize)
> {
> + if (unlikely(nelts > UINT_MAX / eltsize)) {
> + util_dynarray_fini(buf);
> + return 0;
> + }
> +
> + unsigned newsize = nelts * eltsize;
> void *p = util_dynarray_ensu
czw., 25 kwi 2019 o 20:11 Gustaw Smolarczyk napisał(a):
>
> czw., 25 kwi 2019 o 19:42 Emil Velikov napisał(a):
> >
> > The function is analogous to lp_fence_wait() while taking at timeout
> > (ns) parameter, as needed for EGL fence/sync.
> >
> > v2:
>
turn (f->count >= f->rank && ret == thrd_success);
Is checking for ret == thrd_success here really necessary? If the
first part is true we already know that the fence has been signalled.
With this changed or not:
Reviewed-by: Gustaw Smolarczyk
> +}
> +
> +
wt., 16 kwi 2019 o 12:11 Emil Velikov napisał(a):
>
> On Thu, 11 Apr 2019 at 17:55, Gustaw Smolarczyk wrote:
> >
> > czw., 11 kwi 2019 o 18:06 Emil Velikov
> > napisał(a):
> > >
> > > The function is analogous to lp_fence_wait() while taking at timeo
mp;f->mutex);
> + assert(f->issued);
> + while (f->count < f->rank) {
> + ret = cnd_timedwait(&f->signalled, &f->mutex, &ts);
Shouldn't ret be checked for thrd_busy here as well? Otherwise, the
function will busy-wait after the timeout is rea
FWIW,
Reviewed-by: Gustaw Smolarczyk
pon., 11 lut 2019 o 10:15 Samuel Pitoiset
napisał(a):
>
> "The C standard says that compound literals which occur inside of
> the body of a function have automatic storage duration associated
> with the enclosing block. Older GCC releases
dth;
> sin.height = in->height;
> sin.numSlices = in->numSlices;
> sin.numMipLevels = in->numMipLevels;
> sin.numSamples = in->numSamples;
> sin.numFrags = in->numFrags;
>
> - if (flags & RADEON_SURF_SCANOUT) {
Isn
ssert((uintptr_t)&ptr[1] % SUBALLOC_ALIGNMENT == 0);
> > > return &ptr[1];
> > > }
> >
> > These patches are:
> >
> > Reviewed-by: Eric Anholt
>
> Thanks a bunch! I hope this is useful f
śr., 31 paź 2018 o 00:23 Marek Olšák napisał(a):
>
> On Tue, Oct 30, 2018 at 7:11 PM Gustaw Smolarczyk
> wrote:
>>
>> wt., 30 paź 2018 o 23:55 Marek Olšák napisał(a):
>> >
>> > On Tue, Oct 30, 2018 at 6:32 PM Gustaw Smolarczyk
>> > wrote:
wt., 30 paź 2018 o 23:55 Marek Olšák napisał(a):
>
> On Tue, Oct 30, 2018 at 6:32 PM Gustaw Smolarczyk
> wrote:
>>
>> wt., 30 paź 2018, 23:01 Marek Olšák :
>>>
>>> On Mon, Oct 29, 2018 at 12:43 PM Michel Dänzer wrote:
>>>>
>>>> On
wt., 30 paź 2018, 23:01 Marek Olšák :
> On Mon, Oct 29, 2018 at 12:43 PM Michel Dänzer wrote:
>
>> On 2018-10-28 11:27 a.m., Gustaw Smolarczyk wrote:
>> > pon., 17 wrz 2018 o 18:24 Michel Dänzer
>> napisał(a):
>> >>
>> >> On 2018-09-15 3:04 a
wt., 30 paź 2018, 06:08 :
> From: Mathias Fröhlich
>
> On finishing a display list playback the VBO_SAFE_FALLBACK bit
>
s/SAFE/SAVE/g (here and in the title)
Regards,
Gustaw Smolarczyk
is still kept in vbo_save_context::replay_flags. But examining
> replay_flags and the dis
| Mesa and X developer
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
Hi,
What was the conclusion of this discussion? I don't see any
whitelist
radeonsi/si_pipe.c
> @@ -1026,23 +1026,24 @@ struct pipe_screen *radeonsi_screen_create(struct
> radeon_winsys *ws,
> sscreen->has_msaa_sample_loc_bug = (sscreen->info.family >=
> CHIP_POLARIS10 &&
> sscreen->
ret == -1)
>
> if (ret == -1 && errno != EEXIST) ?
Won't EEXIST be returned even in case the path already exists but is
not a directory? [1]
Regards,
Gustaw Smolarczyk
[1] http://man7.org/linux/man-pages/man2/mkdir.2.html
>
> > + return false
riptorPoolCreateInfo::pPoolSizes with a member equal to that
type.
What it implies (I think), is that VkDescriptorPoolCreateInfo::maxSets
and descriptorCount of each VkDescriptorPoolCreateInfo::pPoolSizes are
treated separately. I don't think you sh
*/
> +optimized = GL_TRUE;
> +/* rotate only around x-axis */
> +M(1,1) = c;
> +M(2,2) = c;
> +if (x < 0.0F) {
Isn't x guaranteed to be 0.0F here? And I think you wanted to treat it
as 1.0F in that case, so that
2018-06-18 11:19 GMT+02:00 Iago Toral :
> On Mon, 2018-06-18 at 10:45 +0200, Gustaw Smolarczyk wrote:
> > 2018-06-18 10:39 GMT+02:00 Iago Toral :
> > > On Mon, 2018-06-18 at 09:43 +0200, Gustaw Smolarczyk wrote:
> > > > 2018-06-18 4:39 GMT+02:00 Timothy Arceri :
2018-06-18 10:39 GMT+02:00 Iago Toral :
> On Mon, 2018-06-18 at 09:43 +0200, Gustaw Smolarczyk wrote:
>
> 2018-06-18 4:39 GMT+02:00 Timothy Arceri :
>
> This is required for compatibility profile support.
> ---
> src/mesa/main/ff_fragment_shader.cpp | 6 +-
> 1 fi
hange when the vertex shader is missing
(unless that was intended - I am not really familiar with how fixed
function shaders work alongside new features).
You could also move or update the comment that is just above your change.
Regards,
Gustaw Smolarczyk
>vprog = ctx->_Shader->
line layouts. These have nothing to do with the pipeline. They
> are
> * just muttiple descriptor set layouts pasted together
> diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c
> index 00bb70612e..593cfc9a36 100644
> --- a/src/amd/vulkan/radv_device.c
> +++ b/src/amd/vulkan/radv_device.c
> @@ -875,6 +87
gt; > > > > +from __future__ import print_function
> > > > >
> > > > > CopyRight = '''
> > > > > /*
> > > > > @@ -60,7 +61,7 @@ class StringTable:
> > > > > """
> > > > &g
o holds for INT_MIN, for example for 32-bit
int, INT_MIN == -2^31 and -(unsigned)INT_MIN == (unsigned)INT_MIN == 2^31.
In other words - by first converting to unsigned, we make any overflow
happening during computation defined as wrapping, and we want the result to
be unsigned anyway.
As
ir_factory body(&instructions, mem_ctx);
>
> for (unsigned i = 0; i < num_operands; i++) {
> - expand_source(body, ir->operands[i], src[i]);
> + if (ir->operands[i]->type->is_64bit())
> + expand_source(body, ir->operands[i], src[i]);
>
st_drawpixels_cache_entry(struct st_context *st)
> > +{
> > + unsigned oldest_age = ~0u, oldest_index = ~0u;
> > + unsigned i;
> > +
> > + /* Find entry with oldest (lowest) age */
> > + for (i = 0; i < ARRAY_SIZE(st->drawpix_cache.entries); i++)
2018-01-16 8:33 GMT+01:00 Gert Wollny :
> Hello Jean,
>
> Am Montag, den 15.01.2018, 20:15 + schrieb Jean Hertel:
> > I have written a simply application like DRI Conf tool.
> > It is written using GTKmm and C++.
> Great!
>
> Unfortunately, it didn't link properly, so I send you a pull request
2017-11-27 14:43 GMT+01:00 Marek Olšák :
> On Mon, Nov 27, 2017 at 12:54 PM, Nicolai Hähnle
> wrote:
> > On 23.11.2017 20:35, Marek Olšák wrote:
> >>
> >> From: Marek Olšák
> >>
> >> The next commit will reduce the size even more.
> >>
> >> v2: typecast to uint64_t manually
> >> ---
> >> src/a
2017-11-14 15:04 GMT+01:00 Marek Olšák :
> On Mon, Nov 13, 2017 at 10:19 PM, Ian Romanick wrote:
>> On 11/08/2017 07:16 PM, Marek Olšák wrote:
>>> From: Marek Olšák
>>>
>>> For lower CPU cache usage. All enums fit within 2 bytes.
>>
>> Have you benchmarked this on anything? My recollection is th
2017-09-28 20:21 GMT+02:00 Nicolai Hähnle :
> On 28.09.2017 19:18, Gustaw Smolarczyk wrote:
>>
>> 2017-09-28 18:52 GMT+02:00 Marek Olšák :
>>>
>>> A clearer comment would be: "Don't destroy the fence when it's in the
>>> middle of util_queue
2017-09-28 18:52 GMT+02:00 Marek Olšák :
> A clearer comment would be: "Don't destroy the fence when it's in the
> middle of util_queue_fence_signal (signalled but not unlocked yet
> because util_queue_fence_is_signalled doesn't lock). Instead, wait
> until util_queue_fence_signal returns and then
t (whether 0.0 and -0.0 should be
indistinguishable), but is that ok?
Regards,
Gustaw Smolarczyk
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
2017-09-06 15:53 GMT+02:00 Samuel Pitoiset :
> Useless to track which one has been updated because we
> re-upload all the vertex buffers in one shot.
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/amd/vulkan/radv_cmd_buffer.c | 5 +++--
> src/amd/vulkan/radv_private.h| 2 +-
> 2 files changed
2017-08-02 20:57 GMT+02:00 Samuel Pitoiset :
> Other ones are either unsupported or don't have any helper
> function checks.
>
> v2: - fix ARB_shader_draw_parameters system value names
>
> Signed-off-by: Samuel Pitoiset
> ---
> src/compiler/glsl/builtin_functions.cpp | 6 --
> src/compiler/g
$ ./piglit run gpu -t arb_draw_indirect results/indirect --dmesg
[12/12] pass: 11, fail: 1
Thank you for running Piglit!
Results have been written to /home/bigos/src/piglit/results/indirect
The only one failing is
spec/arb_draw_indirect/arb_draw_indirect-draw-arrays-prim-restart.
However, it is al
Hello,
While looking at the extension list on my Tahiti GPU (SI) I have found
that ARB_draw_parameters is missing. Looking at the code, it requires
ME firmware version >= 87 and PFP firmware version >= 121. While the
first is satisfied, the second is not [1]. I believe I use the newest
TAHITI firm
since v5:
>> >Actually include changes from v5 in patch
>> >
>> > Changes since v4:
>> >Gustaw Smolarczyk
>> > - Make util_strlcpy have the same behaviour as strlcpy
>> >
>> > Changes since v3:
>> >Matt Turner
2017-07-05 8:57 GMT+02:00 Robert Foss :
> Add local strlcpy implementation.
>
> Signed-off-by: Robert Foss
> ---
> Changes since v5:
> Actually include changes from v5 in patch
>
> Changes since v4:
> Gustaw Smolarczyk
>- Make util_strlcpy have the same be
4 lip 2017 14:25 "Robert Foss" napisał(a):
Add local strlcpy implementation.
Signed-off-by: Robert Foss
---
Changes since v3:
Matt Turner
- Change name of util_strncpy to util_strlcpy
Changes since v2:
Brian Paul
- Patch added
src/util/u_string.h | 9 +
1 file changed, 9
2017-07-03 6:10 GMT+02:00 Robert Foss :
> On Sun, 2017-07-02 at 21:06 -0700, Matt Turner wrote:
>> Otherwise known as strlcpy()?
>
> I didn't realize this, a wrapper would be needed for all platformsou
> that don't support it natively.
>
> Do you know which platforms support strlcpy?
>
>
> Rob.
I
2017-06-29 4:41 GMT+02:00 Robert Foss :
> Switch to using strncopy to avoid potential overflow of
> name array in struct hud_graph.
>
> Coverity-id: 1413760
>
> Signed-off-by: Robert Foss
> ---
> src/gallium/auxiliary/hud/hud_cpu.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff
2017-06-13 12:07 GMT+02:00 Michel Dänzer :
> On 13/06/17 06:51 PM, Timothy Arceri wrote:
>> On 13/06/17 19:22, Michel Dänzer wrote:
>>> From: Michel Dänzer
>>>
>>> It calling itself recursively prevented it from being inlined, resulting
>>> in a copy being generated in every compilation unit refer
Same problem as in the previous patch.
Regards,
Gustaw
1 cze 2017 15:07 "Samuel Pitoiset" napisał(a):
Signed-off-by: Samuel Pitoiset
---
src/mesa/main/blit.c | 105 +++---
-
1 file changed, 56 insertions(+), 49 deletions(-)
diff --git a/src/mesa/m
This time, send it to the list too.
Gustaw
1 cze 2017 15:45 "Gustaw Smolarczyk" napisał(a):
1 cze 2017 15:07 "Samuel Pitoiset" napisał(a):
Signed-off-by: Samuel Pitoiset
---
src/mesa/main/blit.c | 111 +++---
-
1 file ch
2017-05-18 23:01 GMT+02:00 Jason Ekstrand :
> ---
> src/intel/vulkan/anv_device.c | 42 --
> 1 file changed, 36 insertions(+), 6 deletions(-)
>
> diff --git a/src/intel/vulkan/anv_device.c b/src/intel/vulkan/anv_device.c
> index 6ea8dfe..8eed7f3 100644
> ---
2017-05-16 18:48 GMT+02:00 Ian Romanick :
> On 05/15/2017 11:38 AM, Gustaw Smolarczyk wrote:
>> 2017-05-15 20:28 GMT+02:00 Ian Romanick :
>>> With this patch I applied, I still see this bit used in several
>>> places, including the i915 driver. Did you tes
2017-05-15 20:28 GMT+02:00 Ian Romanick :
> With this patch I applied, I still see this bit used in several
> places, including the i915 driver. Did you test that?
>
> rc/compiler/shader_enums.h:#define VARYING_BIT_FOGC
> BITFIELD64_BIT(VARYING_SLOT_FOGC)
> src/mesa/swrast/s_context.c:
2017-05-01 14:52 GMT+02:00 Marek Olšák :
> From: Marek Olšák
SNIP
>
> /**
> * Derived from Mesa gl_program:
> */
> -struct st_geometry_program
> +struct st_common_program
> {
> struct gl_program Base; /**< The Mesa geometry program */
I think you should remove the word "geometry" fro
olai
>
>
>>}
>> }
>>
>> @@ -579,9 +576,8 @@ emit_combine(texenv_fragment_program *p,
>> case TEXENV_MODE_ADD_PRODUCTS_SIGNED_NV:
>>return add(add(mul(src[0], src[1]), mul(src[2], src[3])),
>> new(p->mem_ctx) ir_constant(-0.5f));
>> - default:
>> - assert(0);
>> - return src[0];
>> + default:
>> + unreachable("Invalid TexEnv Combine mode");
>> }
>> }
With Nicolai's suggestion or not:
Reviewed-by: Gustaw Smolarczyk
Regards,
Gustaw
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
2017-04-21 14:08 GMT+02:00 Emil Velikov :
> Note: GCC 5/6 crashes with internal compiler error at seemingly random
> stages of the build. Unless any of the SWR devs sort it out, we might
> want to disable it for now.
>
> A sample report can be seen at
> https://travis-ci.org/evelikov/Mesa/builds/22
2017-04-08 18:37 GMT+02:00 Marek Olšák :
> On Sat, Apr 8, 2017 at 12:53 AM, Gustaw Smolarczyk
> wrote:
>> 2017-04-07 23:56 GMT+02:00 Marek Olšák :
>>> On Fri, Apr 7, 2017 at 6:54 PM, Gustaw Smolarczyk
>>> wrote:
>>>> 2017-04-07 16:31 GMT+02:00 Marek
2017-04-07 23:56 GMT+02:00 Marek Olšák :
> On Fri, Apr 7, 2017 at 6:54 PM, Gustaw Smolarczyk
> wrote:
>> 2017-04-07 16:31 GMT+02:00 Marek Olšák :
>>> On Thu, Mar 30, 2017 at 8:09 PM, Gustaw Smolarczyk
>>> wrote:
>>>> Signed-off-by: Gustaw Smolarczyk
2017-04-07 16:40 GMT+02:00 Marek Olšák :
> On Fri, Apr 7, 2017 at 4:35 PM, Marek Olšák wrote:
>> On Thu, Mar 30, 2017 at 8:09 PM, Gustaw Smolarczyk
>> wrote:
>>> Signed-off-by: Gustaw Smolarczyk
>>> ---
>>> src/mesa/main/mtypes.h | 83 ++
2017-04-07 16:31 GMT+02:00 Marek Olšák :
> On Thu, Mar 30, 2017 at 8:09 PM, Gustaw Smolarczyk
> wrote:
>> Signed-off-by: Gustaw Smolarczyk
>> ---
>> src/mesa/main/enable.c | 1 +
>> src/mesa/main/fog.c| 9 +
>> src/mesa/main/mtypes.h | 14 ++
Signed-off-by: Gustaw Smolarczyk
---
src/mesa/state_tracker/st_atifs_to_tgsi.c | 6 +++---
src/mesa/state_tracker/st_atom_shader.c | 17 +
2 files changed, 4 insertions(+), 19 deletions(-)
diff --git a/src/mesa/state_tracker/st_atifs_to_tgsi.c
b/src/mesa/state_tracker
Signed-off-by: Gustaw Smolarczyk
---
src/mesa/main/mtypes.h | 83 ++
src/mesa/main/texstate.c | 103 +++
2 files changed, 186 insertions(+)
diff --git a/src/mesa/main/mtypes.h b/src/mesa/main/mtypes.h
index
Along the way, add missing GL_ONE source support and drop non-existing
GL_ZERO and GL_ONE operand support.
Signed-off-by: Gustaw Smolarczyk
---
src/mesa/main/ff_fragment_shader.cpp | 335 +++
1 file changed, 104 insertions(+), 231 deletions(-)
diff --git a/src
Signed-off-by: Gustaw Smolarczyk
---
src/mesa/main/ff_fragment_shader.cpp | 15 ++-
1 file changed, 6 insertions(+), 9 deletions(-)
diff --git a/src/mesa/main/ff_fragment_shader.cpp
b/src/mesa/main/ff_fragment_shader.cpp
index 2b4d99c879..e1fe9b58c0 100644
--- a/src/mesa/main
It's not used.
Signed-off-by: Gustaw Smolarczyk
---
src/mesa/main/ff_fragment_shader.cpp | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/mesa/main/ff_fragment_shader.cpp
b/src/mesa/main/ff_fragment_shader.cpp
index 95c74e2b92..05641997de 100644
--- a/src/mesa
Signed-off-by: Gustaw Smolarczyk
---
src/mesa/main/ff_fragment_shader.cpp | 6 --
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/mesa/main/ff_fragment_shader.cpp
b/src/mesa/main/ff_fragment_shader.cpp
index 1b76f40e9a..717f39e9d3 100644
--- a/src/mesa/main
Signed-off-by: Gustaw Smolarczyk
---
src/mesa/main/ff_fragment_shader.cpp | 18 +-
1 file changed, 1 insertion(+), 17 deletions(-)
diff --git a/src/mesa/main/ff_fragment_shader.cpp
b/src/mesa/main/ff_fragment_shader.cpp
index e1fe9b58c0..bdbefc7880 100644
--- a/src/mesa/main
Signed-off-by: Gustaw Smolarczyk
---
src/mesa/main/enable.c | 1 +
src/mesa/main/fog.c| 9 +
src/mesa/main/mtypes.h | 14 ++
3 files changed, 24 insertions(+)
diff --git a/src/mesa/main/enable.c b/src/mesa/main/enable.c
index d9d63a6b4b..ef278a318a 100644
--- a/src
Instead of computing it once again using _mesa_tex_target_to_index.
Signed-off-by: Gustaw Smolarczyk
---
src/mesa/main/ff_fragment_shader.cpp | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/mesa/main/ff_fragment_shader.cpp
b/src/mesa/main/ff_fragment_shader.cpp
index
Change it into filter_fp_input_mask transform function that instead of
returning a mask, transforms input.
Also, simplify the case of vertex program handling by assuming that
fp_inputs is always a combination of VARYING_BIT_COL* and VARYING_BIT_TEX*.
Signed-off-by: Gustaw Smolarczyk
---
src
Since fixed-function shaders are restricted to MAX_TEXTURE_COORD_UNITS
texture units, use this constant instead of MAX_TEXTURE_UNITS. This
reduces the array size from 32 to 8.
Signed-off-by: Gustaw Smolarczyk
Reviewed-by: Eric Anholt
---
src/mesa/main/ff_fragment_shader.cpp | 2 +-
1 file
Signed-off-by: Gustaw Smolarczyk
---
src/mesa/main/ff_fragment_shader.cpp | 8
1 file changed, 8 deletions(-)
diff --git a/src/mesa/main/ff_fragment_shader.cpp
b/src/mesa/main/ff_fragment_shader.cpp
index 9b00c36534..95c74e2b92 100644
--- a/src/mesa/main/ff_fragment_shader.cpp
+++ b
Its only usage is easily replaced by nr_enabled_units. As for cache key
part, unit[i].enabled should be enough.
Signed-off-by: Gustaw Smolarczyk
Reviewed-by: Eric Anholt
---
src/mesa/main/ff_fragment_shader.cpp | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/mesa
Since it holds values from 0 to 8, 4 bits will suffice.
Signed-off-by: Gustaw Smolarczyk
Reviewed-by: Eric Anholt
---
src/mesa/main/ff_fragment_shader.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/ff_fragment_shader.cpp
b/src/mesa/main
pped from ~0.78% to ~0.37%.
Standard deviation here is ~0.025% so the performance gain is statistically
significant.
Regards,
Gustaw
Gustaw Smolarczyk (14):
mesa/main/ff_frag: Use correct constant.
mesa/main/ff_frag: Remove enabled_units.
mesa/main/ff_frag: Reduce the size of nr_enabled_u
28 mar 2017 06:35 "Timothy Arceri" napisał(a):
---
src/mesa/main/getstring.c | 12
1 file changed, 12 insertions(+)
diff --git a/src/mesa/main/getstring.c b/src/mesa/main/getstring.c
index 6e90511..50140cf 100644
--- a/src/mesa/main/getstring.c
+++ b/src/mesa/main/getstring.c
@@ -2
2017-03-27 21:27 GMT+02:00 Gustaw Smolarczyk :
> 2017-03-27 21:10 GMT+02:00 Jason Ekstrand :
>> On March 27, 2017 10:24:47 AM Gustaw Smolarczyk
>> wrote:
>>
>>> Hello,
>>>
>>> I was playing with profiling Minecraft on radeonsi in perf and found that
2017-03-27 21:59 GMT+02:00 Edmondo Tommasina :
> Introduce a new MESA_USER_DRIRC environment variable to load a customized
> drirc file.
>
> This can be used mostly for two things:
> 1. Force the load of a different user drirc configuration for an application
>without touching/creating a user g
2017-03-27 21:10 GMT+02:00 Jason Ekstrand :
> On March 27, 2017 10:24:47 AM Gustaw Smolarczyk
> wrote:
>
>> Hello,
>>
>> I was playing with profiling Minecraft on radeonsi in perf and found that
>> _mesa_get_fixed_func_fragment_program was a little bit too high
Since it holds values from 0 to 8, 4 bits will suffice.
Signed-off-by: Gustaw Smolarczyk
---
src/mesa/main/ff_fragment_shader.cpp | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/mesa/main/ff_fragment_shader.cpp
b/src/mesa/main/ff_fragment_shader.cpp
index 75189733ba
Its only usage is easily replaced by nr_enabled_units. As for cache key
part, unit[i].enabled should be enough.
Signed-off-by: Gustaw Smolarczyk
---
src/mesa/main/ff_fragment_shader.cpp | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/mesa/main/ff_fragment_shader.cpp
Since fixed-function shaders are restricted to MAX_TEXTURE_COORD_UNITS
texture units, use this constant instead of MAX_TEXTURE_UNITS. This
reduces the array size from 32 to 8.
Signed-off-by: Gustaw Smolarczyk
---
src/mesa/main/ff_fragment_shader.cpp | 2 +-
1 file changed, 1 insertion(+), 1
nside the gl context be more similar to ff cache key. This way make_state_key
would be more about copying stuff around than translating it from one
representation to another.
As I don't have commit access, please push the changes after you deem them
ready for that.
Regards,
Gustaw
Gustaw Smo
26 mar 2017 22:32 "Edmondo Tommasina"
napisał(a):
Define a new MESA_USER_DRIRC environment variable to load a customized
drirc file.
When the variable is not defined, nothing changes and the ${HOME}/.drirc
file will be loaded.
If the variable is set to a file, this file will be loaded instead o
27 mar 2017 08:47 "Alejandro Piñeiro" napisał(a):
On 27/03/17 01:51, Edward O'Callaghan wrote:
> V.1:
> We memset number of elements without multiplication by the
> element size.
It is not usual to summarize v1. The idea is explain just the changes.
>
> V.2:
> We explicitly set each member to -1
18 mar 2017 05:24 "Jason Ekstrand" napisał(a):
Instead of just advertising the aperture size, we do something more
intelligent. On systems with a full 48-bit PPGTT, we can address 100%
of the available system RAM from the GPU. In order to keep clients from
burning 100% of your available RAM for
17 mar 2017 10:01 "Alejandro Piñeiro" napisał(a):
On 17/03/17 06:26, Edward O'Callaghan wrote:
> We memset number of elements without multiplication by the
> element size.
>
> Signed-off-by: Edward O'Callaghan
> ---
> src/mesa/main/formatquery.c | 2 +-
> 1 file changed, 1 insertion(+), 1 delet
21 lut 2017 03:47 "Jason Ekstrand" napisał(a):
Fine by me
Reviewed-by: Jason Ekstrand
On Mon, Feb 20, 2017 at 6:26 PM, Dave Airlie wrote:
> From: Dave Airlie
>
> For prime support I need to access this, so move it in advance.
>
> Signed-off-by: Dave Airlie
> ---
> src/vulkan/wsi/wsi_commo
2017-02-20 11:19 GMT+01:00 Edward O'Callaghan :
>
> On 02/20/2017 09:15 PM, Edward O'Callaghan wrote:
>> The name has become a little misleading now that it applies
>> to both r600g and radeonsi.
>>
>> V.2: Michel Dänzer - R600_DEBUG must continue to work.
>>
>> Signed-off-by: Edward O'Callaghan
>
I think you meant "shared" and not "shader" in the first line of the commit
message.
Gustaw
16 lut 2017 04:55 "Dave Airlie" napisał(a):
From: Dave Airlie
Signed-off-by: Dave Airlie
---
src/amd/common/ac_nir_to_llvm.c | 18 +-
1 file changed, 1 insertion(+), 17 deletions(-)
2017-02-07 14:11 GMT+01:00 Eero Tamminen :
> Hi,
>
> On 06.02.2017 22:26, Samuel Pitoiset wrote:
>>
>> On 02/06/2017 04:45 PM, Eero Tamminen wrote:
>>>
>>> Results from quick try on Ubuntu 16.04 with today's version of the game
>>> (ARK: Survival Evolved)...
>>>
>>> With (Ubuntu 16.04 default) Mesa
2017-01-26 17:04 GMT+01:00 Marek Olšák :
> From: Marek Olšák
>
> this cleanup is based on the vulkan driver, which seems to do the same thing
Is this also ok for r600g? If I'm right, the amdgpu-pro Vulkan driver
doesn't have any support for pre-GCN hardware.
Regards,
Gustaw
_
7 sty 2017 20:04 "Vladislav Egorov" napisał(a):
To find newlines the line continuations removal pass uses strchr()
twice -- first time to find '\n', second time to find '\r'. Now,
if the shader uses Unix line separators '\n', the file could contain
no '\r' at all, so each time it will scan to the
2017-01-02 7:01 GMT+01:00 Ilia Mirkin :
> Signed-off-by: Ilia Mirkin
> ---
> src/gallium/docs/source/screen.rst | 2 ++
> src/gallium/drivers/freedreno/freedreno_screen.c | 1 +
> src/gallium/drivers/i915/i915_screen.c | 1 +
> src/gallium/drivers/ilo/ilo_screen.c
2016-12-20 6:32 GMT+01:00 Jason Ekstrand :
> This sequence shows up The Talos Principal, at least under Vulkan,
> and prevents loop analysis from properly computing trip counts in a
> few loops.
> ---
> src/compiler/nir/nir_opt_algebraic.py | 8
> 1 file changed, 8 insertions(+)
>
> diff
2016-12-16 18:12 GMT+01:00 Gustaw Smolarczyk :
> 2016-12-16 17:57 GMT+01:00 Emil Velikov :
>> On 16 December 2016 at 15:27, Gustaw Smolarczyk wrote:
>>> 2016-12-16 14:50 GMT+01:00 Emil Velikov :
>>>>
>>>> On 5 October 2016 at 23:12, Gustaw Smolarczyk
&g
2016-12-16 17:57 GMT+01:00 Emil Velikov :
> On 16 December 2016 at 15:27, Gustaw Smolarczyk wrote:
>> 2016-12-16 14:50 GMT+01:00 Emil Velikov :
>>>
>>> On 5 October 2016 at 23:12, Gustaw Smolarczyk
>>> wrote:
>>> > 2016-10-06 0:05 GMT+02:00
2016-12-16 14:50 GMT+01:00 Emil Velikov :
> On 5 October 2016 at 23:12, Gustaw Smolarczyk
> wrote:
> > 2016-10-06 0:05 GMT+02:00 Emil Velikov :
> >> On 5 October 2016 at 21:45, Gustaw Smolarczyk
> wrote:
> >>> Hello,
> >>>
> >>> I h
15 gru 2016 18:19 "Jason Ekstrand" napisał(a):
This keeps some of Connor's original code. However, while I was at it,
I updated this very old pass to a bit more modern NIR.
---
src/compiler/nir/nir_lower_regs_to_ssa.c | 582
---
1 file changed, 155 insertions(+), 427
15 gru 2016 18:19 "Jason Ekstrand" napisał(a):
---
src/compiler/nir/nir.h | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/compiler/nir/nir.h b/src/compiler/nir/nir.h
index f1c99ce..4a9fe65 100644
--- a/src/compiler/nir/nir.h
+++ b/src/compiler/nir/nir.h
@@ -373,6 +373,11 @@ typedef s
They are picked automatically by the provided llvm-config flags, but are
not needed.
Fixes loading radv through a vulkan loader.
Cc: 13.0
---
It's work-around for:
https://lists.freedesktop.org/archives/mesa-dev/2016-October/130765.html
Since there are other people than me dealing with this is
2016-12-02 21:39 GMT+01:00 Marek Olšák :
> From: Marek Olšák
>
> It really happens.
> ---
> src/gallium/drivers/radeonsi/si_descriptors.c | 1 +
> src/gallium/drivers/radeonsi/si_pipe.h| 3 +++
> src/gallium/drivers/radeonsi/si_state.c | 5 +
> 3 files changed, 9 insertions(+)
2016-11-24 18:25 GMT+01:00 Emil Velikov :
> Hello list,
>
> The candidate for the Mesa 13.0.2 is now available. Currently we have:
> - 49 queued
> - 4 nominated (outstanding)
> - and 1 rejected patch(es)
>
>
> With this series we have - fixes for vc4, i965 and radeon drivers. In
> addition
> to
2016-11-23 6:28 GMT+01:00 Dave Airlie :
> From: Dave Airlie
>
> This is kind of a gross hacks, but vulkan doesn't specify anything
> but it would be nice to let people with prime systems at least
> see some stuff rendering for now.
>
> This creates a linear shadow image in GART that gets blitted
2016-11-16 0:05 GMT+01:00 Brian Paul :
> A number of drivers report useful debug/perf information accessible
> through GL_ARB_debug_output and with debug contexts (i.e. setting the
> GLX_CONTEXT_DEBUG_BIT_ARB flag). But few applications actually use
> the GL_ARB_debug_output extension.
>
> This c
Looks like a follow-up to 24815bd7b3b50d2e634b56ca607856ecf08ec4ee
Note sure if my rb is helpful, but:
Reviewed-by: Gustaw Smolarczyk
2016-11-09 8:43 GMT+01:00 Bas Nieuwenhuizen :
> Reviewed-by: Bas Nieuwenhuizen
>
> On Wed, Nov 9, 2016 at 2:22 AM, Dave Airlie wrote:
> > Fr
I am not really that much familiar with nir, so I apologize if what I write
below is wrong.
What if we had something like:
a = 0;
if (x) {
discard_if(y);
a = 1;
}
Then if (x && !y), we must not discard and both at the same time set a to
1. Your transformation would probably change it to:
a
1 - 100 of 148 matches
Mail list logo
