ions if we can make the ifdef-erry less ugly.
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/target-helpers/drm_helper.h | 236 ++
1 file changed, 236 insertions(+)
create mode 100644 src/gallium/auxiliary/target-helpers/drm_helper.h
diff --git a/src/gallium/auxili
... in favour of HAVE_LIBDRM. After all we solely want to build the code
when the latter is available.
In the not too distant future we will remove the libudev/sysfs
dependency and simplify configure.ac even further.
XXX: Check this wrt the previous 1-2 commits.
Signed-off-by: Emil Velikov
With the next commits we'll introduce a 'static' version, which will
essentially load the statically linked-in pipe-drivers, rather than the
standalone pipe-$foo.so ones.
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/pipe-loader/Makefile.am | 8
src/gallium/targ
Dead code since commit 8f50614910c40366d94964fe2c5da5772aff2f96
Cc: Axel Davy
Cc: Tiziano Bacocco
Signed-off-by: Emil Velikov
---
src/gallium/targets/d3dadapter9/drm.c | 36 ---
1 file changed, 36 deletions(-)
diff --git a/src/gallium/targets/d3dadapter9/drm.c
Cc: Axel Davy
Signed-off-by: Emil Velikov
---
src/gallium/targets/d3dadapter9/Makefile.am | 1 +
src/gallium/targets/d3dadapter9/drm.c | 2 ++
2 files changed, 3 insertions(+)
diff --git a/src/gallium/targets/d3dadapter9/Makefile.am
b/src/gallium/targets/d3dadapter9/Makefile.am
index
Analogous to previous commits.
Cc: Axel Davy
Signed-off-by: Emil Velikov
---
src/gallium/targets/d3dadapter9/Makefile.am | 13 -
src/gallium/targets/d3dadapter9/drm.c | 24 +++-
2 files changed, 7 insertions(+), 30 deletions(-)
diff --git a/src/gallium
We delay the null check only to jump through hoops to work around that.
Check early to make our lives easier.
Signed-off-by: Emil Velikov
---
src/gallium/state_trackers/dri/dri2.c | 22 --
src/gallium/state_trackers/dri/dri_screen.c | 5 -
src/gallium
Otherwise we risk things blowing up due to conflicting symbols.
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/util/u_dl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/util/u_dl.c
b/src/gallium/auxiliary/util/u_dl.c
index aca435d..9b97d8d
As of earlier all the targets use the non inline version. Don't forget
to remove the function prototypes/declarations.
Signed-off-by: Emil Velikov
---
.../auxiliary/target-helpers/inline_drm_helper.h | 339 -
src/gallium/include/state_tracker/drm_driver.h | 6
Ported from an identically named commit in st/xa
commit 35cf3831d71770211f29da6608313dc1f6213d7b
Author: Thomas Hellstrom
Date: Thu Jul 3 02:07:36 2014 -0700
st/xa: Don't close the drm fd on failure v2
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/vl/vl_winsys_dri.c | 2
Add a 'static' pipe-loader build, which will be used with follow-up
commits.
Signed-off-by: Emil Velikov
---
src/gallium/Android.mk | 1 +
src/gallium/auxiliary/pipe-loader/Android.mk | 49
2 files changed, 50 insertions(+)
create m
Add a 'static' pipe-loader build, which will be used with follow-up
commits.
Signed-off-by: Emil Velikov
---
src/gallium/SConscript| 1 +
src/gallium/auxiliary/pipe-loader/Makefile.am | 2 ++
src/gallium/auxiliary/pipe-loader/SConsc
: Francisco Jerez
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/pipe-loader/Makefile.am | 23 ---
1 file changed, 20 insertions(+), 3 deletions(-)
diff --git a/src/gallium/auxiliary/pipe-loader/Makefile.am
b/src/gallium/auxiliary/pipe-loader/Makefile.am
index 974cf08
Move the winsys into the pipe-target, bla bla bla
XXX: separate pipe-drivers are likely to be busted
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 94 +-
src/gallium/include/state_tracker/sw_driver.h | 21 +
src/gallium
Already defined as such in struct pipe_loader_device::ops.
Cc: Tom Stellard
Cc: Francisco Jerez
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c | 4 ++--
src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 4 ++--
2 files changed, 4 insertions(+), 4
XXX: More than happy to rename things - header/functions/etc.
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/pipe-loader/Makefile.am | 6 +-
.../auxiliary/pipe-loader/pipe_loader_drm.c| 119 -
.../auxiliary/target-helpers/drm_helper_public.h | 34
XXX: configure knob ?
Signed-off-by: Emil Velikov
---
.../auxiliary/target-helpers/inline_drm_helper.h | 24 ---
.../auxiliary/target-helpers/inline_sw_helper.h| 28 --
src/gallium/drivers/softpipe/Automake.inc | 3 ---
src/gallium/state_tra
Analogous to previous commits.
Signed-off-by: Emil Velikov
---
src/gallium/state_trackers/xa/Makefile.am | 5 -
src/gallium/state_trackers/xa/xa_tracker.c | 16 ++--
src/gallium/targets/xa/Makefile.am | 7 +--
src/gallium/targets/xa/target.c| 2 +-
4
Cc: Axel Davy
Signed-off-by: Emil Velikov
---
src/gallium/targets/d3dadapter9/drm.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/gallium/targets/d3dadapter9/drm.c
b/src/gallium/targets/d3dadapter9/drm.c
index dc040dc..c9559b9 100644
--- a/src/gallium/targets/d3dadapter9/drm.c
+++ b
i.e. plug some (hard to hit) memory leaks.
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c
b/src/gallium/auxiliary/pipe-loader
Ported from an identically named commit in st/xa
commit 35cf3831d71770211f29da6608313dc1f6213d7b
Author: Thomas Hellstrom
Date: Thu Jul 3 02:07:36 2014 -0700
st/xa: Don't close the drm fd on failure v2
Signed-off-by: Emil Velikov
---
src/gallium/state_trackers/dri/dri2.c | 4 ++
We cannot use this C99 feature here quite yet, as the code needs to be
ok with MSVC prior to 2013.
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/pipe-loader/pipe_loader_drm.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/pipe-loader
Analogous to previous commit.
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/Makefile.am| 6 --
src/gallium/auxiliary/vl/vl_winsys_dri.c | 8
src/gallium/targets/omx/Makefile.am | 7 +--
src/gallium/targets/omx/target.c | 2 +-
src/gallium/targets/va
On 17 October 2015 at 23:57, Emil Velikov wrote:
> Move the winsys into the pipe-target, bla bla bla
>
> XXX: separate pipe-drivers are likely to be busted
>
The XXX: is a left over and the pipe-drivers should be fine.
Upon a second thought this whole "keep the sw winsys+dri
On 17 October 2015 at 23:58, Emil Velikov wrote:
> Cc: Axel Davy
> Signed-off-by: Emil Velikov
> ---
> src/gallium/targets/d3dadapter9/Makefile.am | 1 +
> src/gallium/targets/d3dadapter9/drm.c | 2 ++
> 2 files changed, 3 insertions(+)
>
> diff --git a/src/gal
xt pass */
> + d = ir;
Wouldn't the ir_type_dereference_array case, above, need a similar fix ?
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
ost of the cases the two
devices are tightly coupled.
-Emil
P.S. The half-way argument always holds true, but considering the
limited activity on the etnaviv front, half-way is better than
nothing.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
h
On 18 October 2015 at 13:45, Timothy Arceri wrote:
> On Sun, 2015-10-18 at 11:34 +0100, Emil Velikov wrote:
>> Hi Timothy,
>>
>> On 17 October 2015 at 23:19, Timothy Arceri
>> wrote:
>> > Fixes regression cased by bb5aeb854915ba67abc56257f830d002c956439e
>&
Ping on these two trivial patches ?
-Emil
On 7 October 2015 at 12:38, Emil Velikov wrote:
> There is only one function that can be called, which is well known at
> compilation time.
>
> The abstraction used here seems unnecessary, so let's use a direct call
> to brw_stage_p
On 7 October 2015 at 11:11, Emil Velikov wrote:
> Hi all,
>
> A while back we had a case where NIR had C99 initializers in its public
> headers. That's not desired as ISO C++ does not allow them.
>
> So I took a quick look about wiring up a compilation test at 'ma
Signed-off-by: Emil Velikov
---
docs/relnotes/11.1.0.html| 1 +
src/mesa/drivers/dri/i965/intel_extensions.c | 1 +
2 files changed, 2 insertions(+)
diff --git a/docs/relnotes/11.1.0.html b/docs/relnotes/11.1.0.html
index dcf425e..3d948eb 100644
--- a/docs/relnotes/11.1.0
t the chance to
test this properly. Any reviews and testers will be greatly appreciated.
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 9 +
src/mesa/drivers/dri/i965/brw_vec4_nir.cpp | 10 ++
2 files changed, 19 insertions(+)
diff --git a/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
b/src/mesa/drivers/dri/i965/brw_fs_nir.cpp
index
Signed-off-by: Emil Velikov
---
src/glsl/glcpp/glcpp-parse.y| 3 +++
src/glsl/glsl_parser_extras.cpp | 1 +
src/glsl/glsl_parser_extras.h | 2 ++
3 files changed, 6 insertions(+)
diff --git a/src/glsl/glcpp/glcpp-parse.y b/src/glsl/glcpp/glcpp-parse.y
index 1d7a3af..4acccf7 100644
--- a
We're about to reuse get_timestamp() for the nir_intrinsic_shader_clock.
The in the latter the generalisation does not apply, so move the smear()
where needed. This also makes the function analogous to the vec4 one.
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_fs.cpp
v2: correctly set the return type
Signed-off-by: Emil Velikov
---
src/glsl/builtin_functions.cpp | 43 ++
1 file changed, 43 insertions(+)
diff --git a/src/glsl/builtin_functions.cpp b/src/glsl/builtin_functions.cpp
index aae25f8..509a57b 100644
--- a
Signed-off-by: Emil Velikov
---
src/mesa/main/extensions.c | 1 +
src/mesa/main/mtypes.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/mesa/main/extensions.c b/src/mesa/main/extensions.c
index b2c88c3..64972fa 100644
--- a/src/mesa/main/extensions.c
+++ b/src/mesa/main
v2: Add flags and inline comment/description.
v3: None of the input/outputs are variables
Signed-off-by: Emil Velikov
---
src/glsl/nir/glsl_to_nir.cpp | 6 ++
src/glsl/nir/nir_intrinsics.h | 8
2 files changed, 14 insertions(+)
diff --git a/src/glsl/nir/glsl_to_nir.cpp b/src/glsl
On 19 October 2015 at 15:55, Liu, Leo wrote:
> Ping...
>
FYI adding actual commit message does make reviewers happy.
Then again, although it makes sense, I don't feel qualified to review this :(
-Emil
___
mesa-dev mailing li
>img->pitches[0] = w * 4;
>img->offsets[0] = 0;
> diff --git a/src/gallium/state_trackers/va/va_private.h
> b/src/gallium/state_trackers/va/va_private.h
> index 1ea7be7..3479156 100644
> --- a/src/gallium/state_trackers/va/va_private.h
>
RROR_INVALID_PARAMETER;
> +
> +switch (attrib_list[i].value.value.i) {
> +case VA_SURFACE_ATTRIB_MEM_TYPE_VA:
> +default:{
> +return VA_STATUS_ERROR_UNSUPPORTED_MEMORY_TYPE;}
> +}
> +}
> +
> +if ((attrib_list[i].type == VASurfaceAttribExternalBufferDescriptor)
> &&
> +(attrib_list[i].flags == VA_SURFACE_ATTRIB_SETTABLE)) {
> +if (attrib_list[i].value.type != VAGenericValueTypePointer)
> +return VA_STATUS_ERROR_INVALID_PARAMETER;
> +memory_attibute = (VASurfaceAttribExternalBuffers
> *)attrib_list[i].value.value.p;
> +}
> +}
> +
> +/* support 420 & 422 & RGB32 format, 422 and RGB32 are only used
> + * for post-processing (including color conversion) */
> +if (VA_RT_FORMAT_YUV420 != format &&
> +VA_RT_FORMAT_YUV422 != format &&
> +VA_RT_FORMAT_YUV444 != format &&
> +VA_RT_FORMAT_YUV411 != format &&
> +VA_RT_FORMAT_YUV400 != format &&
We don't have any support for 400 or 411. Why are they in the list ?
Cheers,
Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
tbl;
Did you actually experience the crash or did you notice this while
browsing through ?
Either way - abeit unlikely we can return NULL, so let's avoid the crash.
Cc: "11.0"
Reviewed-by: Emil Velikov
Thanks
Emil
g
___
mesa-dev mailing l
fer_format = old_surf_format;
> + return VA_STATUS_ERROR_ALLOCATION_FAILED;
> + }
> +
>if (surf->buffer)
Drop the if (unconditionally destroy the buffer). We're already
checking it before we get here.
With the above sorted
Reviewed-by: Emil Velikov
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
T(vl_screen);
> +
> +#if GALLIUM_STATIC_TARGETS
> + drv->vscreen->pscreen = dd_create_screen(drm_info->fd);
> +#else
> + int loader_fd = dup(drm_info->fd);
> + if (loader_fd == -1)
> + goto error_screen;
> +
> + i
B_MEM_TYPE_DRM_PRIME:
> +vaStatus = suface_from_external_memory(ctx, surf,
> memory_attibute, i, surfaces, &templat);
> +if (vaStatus != VA_STATUS_SUCCESS)
> + goto no_res;
> +break;
> default:
> assert(0);
> }
> diff --git a/src/gallium/state_trackers/va/va_private.h
> b/src/gallium/state_trackers/va/va_private.h
> index e98f64f..2cdd787 100644
> --- a/src/gallium/state_trackers/va/va_private.h
> +++ b/src/gallium/state_trackers/va/va_private.h
> @@ -33,6 +33,7 @@
>
> #include
> #include
> +#include
>
Please include this where needed.
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
VAProcPipelineCaps *pipeline_cap)
> +{
> + unsigned int i = 0;
> +
> + if (!ctx)
> + return VA_STATUS_ERROR_INVALID_CONTEXT;
> +
> + if (!pipeline_cap)
> + return VA_STATUS_ERROR_INVALID_PARAMETER;
> +
> + if (num_filters && !fil
On 19 October 2015 at 18:17, Ilia Mirkin wrote:
> On Mon, Oct 19, 2015 at 1:10 PM, Emil Velikov
> wrote:
>> On 17 October 2015 at 00:14, Julien Isorce wrote:
>>> if (!ctx)
>>>return VA_STATUS_ERROR_INVALID_CONTEXT;
>>>
>>> - if (!(
i.e. plug some (hard to hit) memory leaks.
v2: really teardown the winsys (it rebase fallout), per Brian.
Signed-off-by: Emil Velikov
---
src/gallium/auxiliary/pipe-loader/pipe_loader_sw.c | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/pipe
On 19 October 2015 at 17:03, Brian Paul wrote:
> On 10/17/2015 04:57 PM, Emil Velikov wrote:
>>
>> Move the winsys into the pipe-target, bla bla bla
>>
>> XXX: separate pipe-drivers are likely to be busted
>>
>> Signed-off-by: Emil Velikov
>&g
On 19 October 2015 at 17:07, Brian Paul wrote:
> On 10/17/2015 04:57 PM, Emil Velikov wrote:
>>
>> Hi all,
>>
>> Here goes a series (which is slightly overdue), that reworks the
>> target-helpers into a pipe-loader like interface. As such the
>> state-trac
On 20 October 2015 at 17:06, Julien Isorce wrote:
>
>
> On 19 October 2015 at 17:16, Emil Velikov wrote:
>>
>> On 17 October 2015 at 00:14, Julien Isorce
>> wrote:
>> > This patch allows to use gallium vaapi without requiring
>> > a X server runni
v2: Add flags and inline comment/description.
v3: None of the input/outputs are variables
v4: Drop clockARB reference, relate code motion barrier comment wrt
intrinsic flag.
Signed-off-by: Emil Velikov
---
Matt,
Hopefully the updated comment makes more sense (the clockARB
reference was an
We're about to reuse get_timestamp() for the nir_intrinsic_shader_clock.
In the latter the generalisation does not apply, so move the smear()
where needed. This also makes the function analogous to the vec4 one.
v2: Tweak the comment - The caller -> We (Matt, Connor).
Signed-off-by: Emil
v2:
- Add a few const qualifiers for good measure.
- Drop unneeded retype()s (Matt)
- Convert timestamp to SIMD8/16, as fs_visitor::get_timestamp() returns
SIMD4 (Connor)
Signed-off-by: Emil Velikov
---
src/mesa/drivers/dri/i965/brw_fs_nir.cpp | 11 +++
src/mesa/drivers/dri/i965
On 19 October 2015 at 19:39, Matt Turner wrote:
> On Mon, Oct 19, 2015 at 7:45 AM, Emil Velikov
> wrote:
>> Hi all,
>>
>> This is a resent of the previous RFC series with a few minor changes
>> - fs_reg:;smear() has been moved out of get_timestamp()
>> - cl
hose modifications.
>
> v2. Actually include ASTC support again.
>
Thanks Nanley.
I take it that with this in place the KHR_texture_compression_astc_ldr
piglits/tests are back online (pass) ?
I have double-checked and this patch does fix the erroneous reverts by c6bf1cd.
Review
On 20 October 2015 at 19:58, Connor Abbott wrote:
> On Tue, Oct 20, 2015 at 12:55 PM, Emil Velikov
> wrote:
[snip]
>> +/*
>> + * Shader clock intrinsic with semantics analogous to the clock2x32ARB()
>> + * GLSL intrinsic.
>> + * The latter can be used as code moti
From: Emil Velikov
v2: Add flags and inline comment/description.
v3: None of the input/outputs are variables
v4: Drop clockARB reference, relate code motion barrier comment wrt
intrinsic flag.
v5: Drop the "thus we can eliminate..." comment (Connor)
Signed-off-by: Emil Velikov
R
From: Emil Velikov
We're about to reuse get_timestamp() for the nir_intrinsic_shader_clock.
In the latter the generalisation does not apply, so move the smear()
where needed. This also makes the function analogous to the vec4 one.
v2: Tweak the comment - The caller -> We (Matt, Con
From: Emil Velikov
v2:
- Add a few const qualifiers for good measure.
- Drop unneeded retype()s (Matt)
- Convert timestamp to SIMD8/16, as fs_visitor::get_timestamp() returns
SIMD4 (Connor)
v3:
- Remove unneeded temporary + MOV (Connor)
Signed-off-by: Emil Velikov
Reviewed-by: Connor
r comments that you would like to share
before the release, please go ahead.
Cheers,
Emil
Mesa stable queue
-
Nominated (18)
==
Boyan Ding (1):
i915: Add XRGB format to intel_screen_make_configs
Brian Paul (1):
configure: don't try to bu
e these in the future ?
Thanks
Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 21 October 2015 at 21:33, Kenneth Graunke wrote:
> On Monday, October 19, 2015 02:54:56 PM Emil Velikov wrote:
>> Ping on these two trivial patches ?
>>
>> -Emil
>
> Oh, sorry, I thought I'd sent R-bs for these...
>
> Both are
> Reviewed-by: Kennet
On 21 October 2015 at 22:44, Matt Turner wrote:
> On Wed, Oct 21, 2015 at 2:16 PM, Emil Velikov
> wrote:
>> On 21 October 2015 at 21:33, Kenneth Graunke wrote:
>>> On Monday, October 19, 2015 02:54:56 PM Emil Velikov wrote:
>>>> Ping on these two trivial pat
is feature.
Alternatively feel free to ask Brian/Jose, as they have a fair bit of
experience in the area.
That aside, I'm in favour of keeping the comments as is. The editing
comment does not apply imho.
-Emil
___
mesa-dev mailing list
mesa-dev@lis
extensions.c \
main/extensions.h \
+ main/extensions_table.h \
main/fbobject.c \
main/fbobject.h \
main/feedback.c \
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
s a bitset. */
> uint8_t api_set;
>
> + /** Minimum version the extension requires for the given API
> +* (see gl_api defined in mtypes.h)
> +*/
> + GLuint version[API_OPENGL_LAST + 1];
> +
Please use uint*t type, like the surrounding code and add a note that
the
amp;&
> + (ctx->Version >= i->version[ctx->API]) &&
> + base[i->offset];
Bikeshed: I realise that you're copying most of these, but wouldn't it
be better if we use more common/intuitive variable names ?
ei -> i or idx
i -> ext
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
o real upside...
Pretty much my line of thinking. Perhaps it was applicable for earlier
iterations of the series ?
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
gt; +struct gl_extensions _mesa_extension_override_enables = {0};
> +struct gl_extensions _mesa_extension_override_disables = {0};
Side note: once ARB_compute_shader lands in for i965 we can even make
these local/static.
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
a less mysterious one (extension_sentinel).
>
That's the first time I see someone calling 0, a "unsigned error
value". Perhaps reword the commit message a bit ? The comment/function
description should be updated as well.
-Emil
__
25, /**< G45 only */
> BRW_OPCODE_NOP =126,
>
> diff --git a/src/mesa/drivers/dri/i965/brw_disasm.c
> b/src/mesa/drivers/dri/i965/brw_disasm.c
> index db23a18..c2dac7c 100644
> --- a/src/mesa/drivers/dri/i965/brw_disasm.c
> +++ b/src/mesa/drivers
are at
different level and we do not have enough information within
brw_validate_instructions(), do we ?
Thanks
Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 22 October 2015 at 15:07, Nicolai Hähnle wrote:
> On 18.10.2015 00:57, Emil Velikov wrote:
>>
>> Add a list of driver descriptors and select one from the list, during
>> probe time.
>>
>> As we'll need to have all the driver pipe_foo_screen_create() funct
On 22 October 2015 at 16:32, Emil Velikov wrote:
> On 22 October 2015 at 15:07, Nicolai Hähnle wrote:
>> On 18.10.2015 00:57, Emil Velikov wrote:
>>>
>>> Add a list of driver descriptors and select one from the list, during
>>> probe time.
>>
On 23 October 2015 at 00:34, Nanley Chery wrote:
> On Thu, Oct 22, 2015 at 3:40 AM, Emil Velikov
> wrote:
>>
>> On 19 October 2015 at 23:44, Nanley Chery wrote:
>> > From: Nanley Chery
>> >
>> > This variable existed to provide an unsigned e
On 22 October 2015 at 23:22, Nanley Chery wrote:
> On Thu, Oct 22, 2015 at 3:06 AM, Emil Velikov
> wrote:
>>
>> That aside, I'm in favour of keeping the comments as is. The editing
>> comment does not apply imho.
>>
>
> Since, there isn't a unanim
possible
nv30: include the header of ffs prototype
Chris Wilson (1):
i965: Remove early release of DRI2 miptree
Dave Airlie (1):
mesa/uniforms: fix get_uniform for doubles (v2)
Emil Velikov (3):
docs: add sha256 checksums for 11.0.3
Update version to 11.0.4
On 23 October 2015 at 19:46, Jordan Justen wrote:
> On 2015-10-22 03:32:58, Emil Velikov wrote:
>> On 19 October 2015 at 23:44, Nanley Chery wrote:
>> > From: Nanley Chery
>> >
>> > Aside from those modified in this commit, all gl_extensions structs are
>
igned oclass = 0;
> int ret, i;
>
> - if (!screen)
> + if (!screen) {
> + nouveau_device_del(&dev);
>return NULL;
> + }
>
Imho having these in screen_create() seems like the wrong 'layer'.
Shouldn't one call nouveau_device_dev() from w
On 27 October 2015 at 10:50, samuel.pitoiset wrote:
> On 27/10/2015 11:37, Emil Velikov wrote:
>>
>> On 22 October 2015 at 00:16, Julien Isorce
>> wrote:
>>>
>>> The real fix is in nouveau_drm_winsys.c by setting dev to 0.
>>> Which means
king about the following but I'm
sure you can think of something better.
"With the said commit the initialization of the Target{,Index} was
moved from glBindTexture to glTextureView. One does not need the
latter for GL VDPAU interop thus things broke."
With those
Revi
reedesktop.org/show_bug.cgi?id=91927
Small suggestion:
For future patches, please mention the commit sha/title as regressions
like these happen.
Not a strict requirement by any means, but quite useful for devs,
testers and users alike.
Thanks
Emil
__
From: Emil Velikov
The drm/ prefix is required, if using the kernel provided headers. As
most distros don't ship them it and we already depend on libdrm (which
adds the relevant -I flag) just drop the drm/ from the include.
Once a libdrm release with the virtgpu_drm.h header is released, w
Signed-off-by: Emil Velikov
---
src/gallium/winsys/virgl/vtest/Makefile.sources | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/gallium/winsys/virgl/vtest/Makefile.sources
b/src/gallium/winsys/virgl/vtest/Makefile.sources
index ab72560..410763f 100644
--- a/src
From: Emil Velikov
The build already sets it as needed.
Signed-off-by: Emil Velikov
---
src/gallium/winsys/virgl/drm/virgl_drm_winsys.c | 3 ---
src/gallium/winsys/virgl/vtest/virgl_vtest_socket.c | 2 --
src/gallium/winsys/virgl/vtest/virgl_vtest_winsys.c | 2 --
3 files changed, 7
Hi all,
Some mostly trivial fixes which I came across whist trying to build
virgl. Build tested only.
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
From: Emil Velikov
Signed-off-by: Emil Velikov
---
src/gallium/winsys/virgl/drm/Makefile.sources | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/gallium/winsys/virgl/drm/Makefile.sources
b/src/gallium/winsys/virgl/drm/Makefile.sources
index c0baed8..eca8eb6 100644
From: Emil Velikov
Use the relevant GALLIUM_foo_CFLAGS which has all the requirements
(not to mention VISIBITY_CFLAGS) and keep ../ out of the include
directives.
Signed-off-by: Emil Velikov
---
src/gallium/drivers/virgl/Makefile.am | 5 +
src/gallium/drivers/virgl/virgl.h
From: Emil Velikov
... and add the missing files while we're at it.
Signed-off-by: Emil Velikov
---
src/gallium/drivers/virgl/Makefile.am | 12 ++--
src/gallium/drivers/virgl/Makefile.sources | 17 +
2 files changed, 19 insertions(+), 10 deletions(-)
create
hod to the madness, I think.
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
hod to the madness, I think.
-Emil
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
hod to the madness, I think.
-Emil
P.S. Apologies if you're getting this multiple times, git send-email
seems to hate me today.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev
On 29 October 2015 at 11:12, Emil Velikov wrote:
> Hi all,
>
> A slightly longer series (that builds on top of the previous sent a
> minute ago), that moves/renames a couple of files, adds a few inline
> wrappers, 'includes what you want' and related fixes.
>
>
g
[sendemail]
smtpuser = ikalvac...@gmail.com
smtpencryption = tls
smtpserverport = 587
smtpserver = smtp.gmail.com
For ease of use you can also set the TO address in
$(mesa_top)/.git/config. You can also move the above setup in here.
[sendemail]
to = mesa-de
Strictly speaking virgl_hw.h should reside in the driver folder, as
it describes the hardware. Moving it allows us to nuke the following
strange dependency
winsys/vtest > driver > winsys/drm
Signed-off-by: Emil Velikov
---
src/gallium/drivers/virgl/Makefile.sources | 1 +
src/g
Include what you want, rather than relying on a header foo.h N levels
down the include chain, to provide something that you need.
Signed-off-by: Emil Velikov
---
src/gallium/drivers/virgl/virgl_buffer.c| 2 ++
src/gallium/drivers/virgl/virgl_context.c | 7 ++-
src
Provide a more meaningful name considering it's purpose.
Signed-off-by: Emil Velikov
---
src/gallium/drivers/virgl/Makefile.sources | 2 +-
src/gallium/drivers/virgl/virgl.h | 51 --
src/gallium/drivers/virgl/virgl_context.c | 2 +-
src/gallium/dr
Signed-off-by: Emil Velikov
---
src/gallium/winsys/virgl/drm/virgl_drm_winsys.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/src/gallium/winsys/virgl/drm/virgl_drm_winsys.c
b/src/gallium/winsys/virgl/drm/virgl_drm_winsys.c
index 11542f5..d9b4d58 100644
--- a
Wrap some of the 'omg it's getting out of hand' long lines, and
re-indent where things feel off.
Signed-off-by: Emil Velikov
---
src/gallium/winsys/virgl/drm/virgl_drm_winsys.c| 83 ---
.../winsys/virgl/vtest/virgl_vtest_socket.c| 13 ++-
.../win
301 - 400 of 8586 matches
Mail list logo
