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
Hi Timothy,
On 17 October 2015 at 23:19, Timothy Arceri wrote:
> Fixes regression cased by bb5aeb854915ba67abc56257f830d002c956439e
>
> We don't care about the swizzle when building the name so just skip over it.
> ---
>
> New piglit test: http://patchwork.freedesktop.org/patch/62111/
>
> src/g
Hi Lucas,
Pardon for jumping it like this.
On 19 October 2015 at 10:25, Lucas Stach wrote:
[snip]
> This series implements a way to fuse multiple DRM devices into a single
> EGL device, solely on the base that current EGL users expect that a
> single device is able to render and scanout at the s
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
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()
- clock2x32ARB() has proper return value
- the nir intrinsic no longer claims to have a variable
I'm still rocking a SNB hardware so I did not really got the cha
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 list
mesa-dev@lists.freedeskto
On 17 October 2015 at 00:14, Julien Isorce wrote:
> Also add RGBA, RGBX and BGRX.
> Also extend ChromaToPipe and implement PipeToYCbCr.
>
> Note that gstreamer-vaapi check all the VAImageFormat fields.
>
> Signed-off-by: Julien Isorce
> ---
> src/gallium/state_trackers/va/image.c | 10 +
On 17 October 2015 at 00:14, Julien Isorce wrote:
> Inspired from
> http://cgit.freedesktop.org/vaapi/intel-driver/tree/src/i965_drv_video.c
>
Please mention the actual usecase here and/or how you've tested this.
> Signed-off-by: Julien Isorce
> ---
> src/gallium/state_trackers/va/context.c
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
On 17 October 2015 at 00:14, Julien Isorce wrote:
> This patch allows to use gallium vaapi without requiring
> a X server running for your second graphic card.
>
I've sent a lengthy series which should mitigate the need of some hunks.
> Signed-off-by: Julien Isorce
> ---
> src/gallium/state_tra
On 17 October 2015 at 00:14, Julien Isorce wrote:
> For now it is limited to RGBA, BGRA, RGBX, BGRX surfaces.
>
> Signed-off-by: Julien Isorce
> ---
> src/gallium/state_trackers/va/surface.c| 90
> +-
> src/gallium/state_trackers/va/va_private.h | 1 +
> 2 files
On 17 October 2015 at 00:14, Julien Isorce wrote:
> Improve following functions to support VA_PROFILE_NONE profile (vpp):
> vlVaQueryConfigProfiles
> vlVaQueryConfigEntrypoints
> vlVaCreateConfig
> vlVaQueryConfigAttributes
>
> Add VADriverVTableVPP and improve following functions to support vpp:
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
ild gallium DRI drivers if --disable-dri is set
Emil Velikov (3):
i965: store reference to the context within struct brw_fence
egl/dri2: expose srgb configs when KHR_gl_colorspace is available
mesa; add get-extra-pick-list.sh script into bin/
Ian Romanick (1):
i965: Fix is-rend
On 20 October 2015 at 05:08, Matt Turner wrote:
> Otherwise we'd emit a MOV from the null register (which isn't allowed).
>
Would you say it's a good idea to push the check down to the MOV()
implementation ? If not perhaps we should add an assert() to easily
catch cases like these in the future ?
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
On 20 October 2015 at 16:43, Nanley Chery wrote:
> On Tue, Oct 20, 2015 at 8:16 AM, Marek Olšák wrote:
>>
>> Also, the FIXME comment should be on its own line.
>>
>
> I moved it aside to make editing the table easier. However, since the
> formatting of the
> table is unlikely to change much after
On 19 October 2015 at 23:36, Nanley Chery wrote:
> From: Nanley Chery
>
> With this infrastructure set in place, we can now reuse the entries to
> generate useful code.
>
> Signed-off-by: Nanley Chery
> ---
> src/mesa/main/extensions.c | 321
> +--
> s
On 19 October 2015 at 23:36, Nanley Chery wrote:
> From: Nanley Chery
>
> Enable limiting advertised extension support by context version with
> finer granularity. GLuint is chosen over smaller datatypes because,
> when this field is eventually used, usage of this datatype provides
> the smallest
On 19 October 2015 at 23:36, Nanley Chery wrote:
> From: Nanley Chery
>
> Create a function which determines if an extension is supported in the
> current context.
>
> Signed-off-by: Nanley Chery
> ---
> src/mesa/main/extensions.c | 17 +
> 1 file changed, 17 insertions(+)
>
> d
On 22 October 2015 at 07:43, Erik Faye-Lund wrote:
> On Tue, Oct 20, 2015 at 12:44 AM, Nanley Chery wrote:
>> From: Nanley Chery
>>
>> Since the version numbers being compared are integral and we don't ever
>> expect gl_context::Version to be equal to 0, subtract 1 from the rhs of
>> the equatio
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
> zero-initialized by default. There is therefore no need to memset the
> structs to 0. Also, remove the open-coded memset in
> _mesa_init_extensions()
On 19 October 2015 at 23:44, Nanley Chery wrote:
> From: Nanley Chery
>
> This variable existed to provide an unsigned error value for
> name_to_offset(). Since o(extension_sentinel) is also a valid unsigned
> error value, save space and replace this mysterious variable with
> a less mysterious o
On 21 October 2015 at 23:58, Matt Turner wrote:
> Add some instructions: illegal, movi, sends, sendsc.
>
> Remove some instructions with reused opcodes: msave, mrestore, push,
> pop, goto. I did have some gross code for disassembling opcodes
> per-generation, but there's very little meaningful ove
On 21 October 2015 at 23:58, Matt Turner wrote:
> Initially just checks that sources are non-NULL, which would have
> alerted us to the problem fixed by commit 6c846dc5.
What are the chances of combining this with and/or removing
fs_visitor::validate() ? Afaict both of these validations are at
di
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
>
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 dev's ownership has been passed to previous call.
> Other changes are there to be consistent with what the
> screen_create functions already do on errors.
>
> Encountered
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
Hi Nanley,
On 28 October 2015 at 21:50, Nanley Chery wrote:
> From: Nanley Chery
>
> A prior, literal reading of the ASTC spec led to the prohibition
> of some compressed formats being used against the targets:
> TEXTURE_CUBE_MAP_ARRAY and TEXTURE_3D. Since the spec does not specify
> interactio
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
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.
As some of these can be seen as bikeshedding, although rest assured
there is a method to the ma
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.
As some of these can be seen as bikeshedding, although rest assured
there is a method to the ma
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.
As some of these can be seen as bikeshedding, although rest assured
there is a method to the ma
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.
>
>
On 29 October 2015 at 09:24, Ivan Kalvachev wrote:
> On 10/29/15, Ilia Mirkin wrote:
>> On Wed, Oct 28, 2015 at 8:52 PM, Ivan Kalvachev
>> wrote:
>>> I'm attaching v3 of the patch. Same as v2, but without the extra empty
>>> line.
>>
>> FYI, there's a lot of overhead to reviewing an attached pat
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
The screen already has a pointer to the (base) winsys object.
With the latter of which implemented/sub-classed as either drm or sw
based one, depending on the target.
Signed-off-by: Emil Velikov
---
src/gallium/drivers/virgl/virgl_public.h | 1 -
src/gallium/drivers/virgl/virgl_screen.c | 1
Signed-off-by: Emil Velikov
---
src/gallium/winsys/virgl/drm/virgl_drm_winsys.h | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/gallium/winsys/virgl/drm/virgl_drm_winsys.h
b/src/gallium/winsys/virgl/drm/virgl_drm_winsys.h
index eac1d3e..a547654 100644
--- a/src/gallium/winsys/virgl/drm
301 - 400 of 8584 matches
Mail list logo