On Fri, Aug 14, 2015 at 04:51:52PM -0700, Anuj Phogat wrote:
> This change is required by the later patches.
>
> Cc: Ben Widawsky
> Signed-off-by: Anuj Phogat
> ---
> src/mesa/drivers/dri/i965/brw_blorp.cpp | 3 ++-
> src/mesa/drivers/dri/i965/brw_misc_state.c| 8 +---
> src/mesa/
On 08/14/2015 04:01 PM, Timothy Arceri wrote:
On Fri, 2015-08-14 at 08:55 +0300, Tapani Pälli wrote:
On 08/13/2015 11:54 AM, Timothy Arceri wrote:
I've sent a couple of follow-up patches I notice when reviewing
this.
On Thu, 2015-08-13 at 09:30 +0300, Tapani Pälli wrote:
v2: code cleanup
From: Dave Airlie
It appears CTS does a lot of
glGetFrameBufferAttachmentParameteriv with GL_FRONT_LEFT,
but gallium doesn't create a GL_FRONT_LEFT attachment,
so these all fail badly, now I'd like to argue the test suite
should probably only hit GL_BACK_LEFT but I'm guessing that
won't go down
Reviewed-by: Edward O'Callaghan
--
Edward O'Callaghan
edward.ocallag...@koparo.com
On Sat, Aug 15, 2015, at 08:30 AM, Connor Abbott wrote:
> Signed-off-by: Connor Abbott
> ---
> src/glsl/nir/nir_opcodes.py | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/
Reviewed-by: Edward O'Callaghan
--
Edward O'Callaghan
edward.ocallag...@koparo.com
On Sat, Aug 15, 2015, at 08:30 AM, Connor Abbott wrote:
> This makes intelligent autocomplete plugins much happier.
>
> Signed-off-by: Connor Abbott
> ---
> src/glsl/nir/nir_builder.h | 2 ++
> 1 file cha
Reviewed-by: Edward O'Callaghan
--
Edward O'Callaghan
edward.ocallag...@koparo.com
On Sat, Aug 15, 2015, at 09:52 AM, Timothy Arceri wrote:
> Reviewed-by: Timothy Arceri
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lis
Reviewed-by: Edward O'Callaghan
--
Edward O'Callaghan
edward.ocallag...@koparo.com
On Sat, Aug 15, 2015, at 09:56 AM, Timothy Arceri wrote:
> Reviewed-by: Timothy Arceri
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://list
HI Ilia
2015-08-14 1:31 GMT-03:00 Ilia Mirkin :
> On Fri, Aug 14, 2015 at 12:25 AM, Marcos Souza
> wrote:
> > Hi Ilia,
> >
> > 2015-08-14 1:02 GMT-03:00 Ilia Mirkin :
> >>
> >> On Thu, Aug 13, 2015 at 11:55 PM, Marcos Souza
> >> wrote:
> >> > Hi Ilia,
> >> >
> >> > So i found the point here it
Hi Ilia,
So i found the point here it addrs that double brackets, and the following
patch solves it, but this is a right solution? If someone could guide me
here, I could fix it :)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_dump.c
b/src/gallium/auxiliary/tgsi/tgsi_dump.c
index 8ceb5b4..046471e
2015-08-14 1:55 GMT-03:00 Ilia Mirkin :
> On Fri, Aug 14, 2015 at 12:52 AM, Marcos Paulo de souza
> wrote:
> > Now I'll take a look about the last problem of LRP and MOV.
>
> That should ideally have solved itself too... if not, do you have the
> full shader that demonstrates the problem?
>
Yes,
Hi Ilia,
Em 14-08-2015 01:45, Ilia Mirkin escreveu:
On Fri, Aug 14, 2015 at 12:43 AM, Marcos Souza
wrote:
HI Ilia
2015-08-14 1:31 GMT-03:00 Ilia Mirkin :
On Fri, Aug 14, 2015 at 12:25 AM, Marcos Souza
wrote:
Hi Ilia,
2015-08-14 1:02 GMT-03:00 Ilia Mirkin :
On Thu, Aug 13, 2015 at 11:55 P
Hi Ilia,
2015-08-14 1:02 GMT-03:00 Ilia Mirkin :
> On Thu, Aug 13, 2015 at 11:55 PM, Marcos Souza
> wrote:
> > Hi Ilia,
> >
> > So i found the point here it addrs that double brackets, and the
> following
> > patch solves it, but this is a right solution? If someone could guide me
> > here, I co
On Tue, Aug 11, 2015 at 1:45 PM, Emil Velikov wrote:
> On 29/07/15 19:20, Emil Velikov wrote:
>> Hi all,
>>
>> Below is the preliminary release schedule for Mesa 11.0.0
>>
>> August 21st 2015 - Feature freeze/Release candidate 1
>> August 28th 2015 - Release candidate 2
>> September 04th 2015 - Re
On Sunday 09 August 2015 11:52:13 orome wrote:
> On Sun, 2015-08-09 at 15:39 +0200, Zoltan Gilian wrote:
> > There is no MDOperand in llvm 3.5.
> >
> > v2: Check if kernel metadata is present to avoid crash (EdB).
>
> still builds for me.
Sadly, it's still crashing for me.
auto kernel_node = ge
https://bugs.freedesktop.org/show_bug.cgi?id=91646
--- Comment #6 from Serge Martin ---
Hello
This remind me this. Something similar have happen to ocl-icd, see
https://bugzilla.redhat.com/show_bug.cgi?id=1219646
--
You are receiving this mail because:
You are the QA Contact for the bug.
You
From: Rob Clark
No one was implementing this driver hook, so let's first turn it into
something more useful and better matching the rest of the Driver API.
Signed-off-by: Rob Clark
---
src/mesa/drivers/common/driverfuncs.c | 2 +-
src/mesa/main/dd.h| 6 +++---
src/mesa/ma
From: Rob Clark
Turn it into call to optional pipe->invalidate_resource().
Signed-off-by: Rob Clark
---
src/mesa/state_tracker/st_cb_fbo.c | 12
1 file changed, 12 insertions(+)
diff --git a/src/mesa/state_tracker/st_cb_fbo.c
b/src/mesa/state_tracker/st_cb_fbo.c
index 5707590..5
From: Rob Clark
We can use same Driver hook for these, in addition to
DiscardFramebufferEXT()
Signed-off-by: Rob Clark
---
src/mesa/main/fbobject.c | 25 ++---
1 file changed, 22 insertions(+), 3 deletions(-)
diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
From: Rob Clark
Currently, these extension entry points were implemented, but other than
error-checking, they were no-op. (Technically, DiscardFramebuffer had a
driver hook, but no one implemented it.) So refactor the driver API
slightly, and turn these all into calls to DiscardTexture, which i
The last line of the commit message should say:
GL_RGBA4_S3TC (0x83A3) -> COMPRESSED_RGBA_S3TC_DXT5_EXT (0x83F3)
On Wed, Aug 12, 2015 at 4:19 PM, Nanley Chery wrote:
> From: Nanley Chery
>
> This function's cases for non-generic compressed formats duplicate
> the GL to MESA translation in
https://bugs.freedesktop.org/show_bug.cgi?id=89018
Daniel Exner changed:
What|Removed |Added
CC||dex+fdobugzilla@dragonslave
From: Marcos Paulo de Souza
Before this patch, the tgsi_dumps was printing declaration as:
DCL IN[][0][0], GENERIC[0]
and now it is parsed correctly:
DCL IN[][0], GENERIC[0]
In the same way, for tess stages, the output addr now is parsed correctly,
doing like src parser from:
LRP OUT[0][3], TE
From: Marcos Paulo de Souza
---
src/gallium/drivers/nouveau/nouveau_compiler.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/drivers/nouveau/nouveau_compiler.c
b/src/gallium/drivers/nouveau/nouveau_compiler.c
index 8660498..495450b 100644
--- a/src/gallium/drivers/nouveau/
From: Marcos Paulo de Souza
Hi guys,
first of all, sorry for the flood... my last bunch of patches were sent as a
mistake... but
now the compiling problem was solved :)
I'm happy o send my first patches to mesa, and I have to say a huge thank you
to Ilia Mirkin for all the
help and tips to fi
From: Marcos Paulo de Souza
---
src/gallium/auxiliary/tgsi/tgsi_text.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_text.c
b/src/gallium/auxiliary/tgsi/tgsi_text.c
index a6675c5..24e2dbd 100644
--- a/src/gallium/auxiliary/tgsi/tgsi_tex
From: Marcos Paulo de Souza
---
src/gallium/drivers/nouveau/nouveau_compiler.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/drivers/nouveau/nouveau_compiler.c
b/src/gallium/drivers/nouveau/nouveau_compiler.c
index 8660498..495450b 100644
--- a/src/gallium/drivers/nouveau/
From: Marcos Paulo de Souza
---
src/gallium/auxiliary/tgsi/tgsi_text.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/gallium/auxiliary/tgsi/tgsi_text.c
b/src/gallium/auxiliary/tgsi/tgsi_text.c
index a6675c5..24e2dbd 100644
--- a/src/gallium/auxiliary/tgsi/tgsi_tex
From: Marcos Paulo de Souza
Before this patch, the tgsi_dumps was printing declaration as:
DCL IN[][0][0], GENERIC[0]
and now it is parsed correctly:
DCL IN[][0], GENERIC[0]
In the same way, for tess stages, the output addr now is parsed correctly,
doing like src parser from:
LRP OUT[0][3], TE
It is very common for d3d9 apps to set again the constants
they need before every draw call, even if nothing changed.
Since we are mostly gpu bound, it is better to check
for change, and upload constants again (and thus use
gpu bandwith) only if the constants changed.
Signed-off-by: Axel Davy
--
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_state.c| 188 +++-
src/gallium/state_trackers/nine/nine_state.h| 4 +-
src/gallium/state_trackers/nine/pixelshader9.c | 30 +++-
src/gallium/state_trackers/nine/pixelshader9.h | 41 +-
src/gall
The number of texture stages is 8.
'tex_stage' array was too big, and thus
the checks with 'Elements(state->ff.tex_stage)' were passing,
causing some invalid API calls to pass, and crash because of
out of bounds write since bumpmap_vars was just the correct size.
Signed-off-by: Axel Davy
---
sr
Wine tests were used to get it right.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_ff.c | 161 ++--
src/gallium/state_trackers/nine/nine_ff.h | 81
src/gallium/state_trackers/nine/vertexshader9.h | 2 +-
3 files changed, 174 in
The CSO cache unbinds views that are not needed anymore,
which we don't do.
It checks for change before committing the views.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/device9.c| 3 ---
src/gallium/state_trackers/nine/nine_pipe.c | 4 ++--
src/gallium/state_trackers/nin
Taken from wine tests
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_ff.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/gallium/state_trackers/nine/nine_ff.c
b/src/gallium/state_trackers/nine/nine_ff.c
index 6740274..59ea2cb 100644
--- a/src/gallium/state_tracke
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_ff.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/nine_ff.c
b/src/gallium/state_trackers/nine/nine_ff.c
index 9638d9c..4428ce6 100644
--- a/src/gallium/state_trackers/nine/
Signed-off-by: Axel Davy
Reviewed-by: David Heidelberg
---
src/gallium/state_trackers/nine/pixelshader9.h | 1 -
src/gallium/state_trackers/nine/vertexshader9.h | 1 -
2 files changed, 2 deletions(-)
diff --git a/src/gallium/state_trackers/nine/pixelshader9.h
b/src/gallium/state_trackers/nine
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_state.c | 139 +++
1 file changed, 76 insertions(+), 63 deletions(-)
diff --git a/src/gallium/state_trackers/nine/nine_state.c
b/src/gallium/state_trackers/nine/nine_state.c
index a439217..04b5f18 100644
-
Automatically switch to userbuf path when
we would need to upload fog or bumpmat
constants
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_state.c | 11 +++
1 file changed, 11 insertions(+)
diff --git a/src/gallium/state_trackers/nine/nine_state.c
b/src/gallium/state_
non-debug build was complaining
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_ff.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/state_trackers/nine/nine_ff.c
b/src/gallium/state_trackers/nine/nine_ff.c
index 59ea2cb..fe8933b 100644
--- a/src/gallium/stat
non-debug build was complaining
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_shader.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_trackers/nine/nine_shader.c
index 9680c5a..28f2787 100
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/device9.c | 3 ++
src/gallium/state_trackers/nine/nine_shader.c | 59 --
src/gallium/state_trackers/nine/nine_shader.h | 1 +
src/gallium/state_trackers/nine/nine_state.c | 5 ++-
src/gallium/state_tr
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_ff.c | 68 +++
1 file changed, 61 insertions(+), 7 deletions(-)
diff --git a/src/gallium/state_trackers/nine/nine_ff.c
b/src/gallium/state_trackers/nine/nine_ff.c
index 1033050..6740274 100644
--- a/sr
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_shader.c | 47 +++--
src/gallium/state_trackers/nine/nine_shader.h | 47 +
src/gallium/state_trackers/nine/nine_state.c| 43 +++---
src/gallium/state_trackers/nine
There were flags all sm3 cards do advertise,
and we weren't.
Some games can trigger buggy rendering path
if the caps are not what they expect.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/adapter9.c | 26 +++---
1 file changed, 11 insertions(+), 15 deletions(-
There was an unused variable
---
src/gallium/state_trackers/nine/nine_state.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/nine_state.c
b/src/gallium/state_trackers/nine/nine_state.c
index 847cf1b..558d07a 100644
--- a/src/gallium/state_trackers/nine/nine_sta
Separate state preparation and state commit
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_pipe.c | 4 ++--
src/gallium/state_trackers/nine/nine_pipe.h | 2 +-
src/gallium/state_trackers/nine/nine_state.c | 23 ---
src/gallium/state_trackers/nine/nine_s
The compiler was complaining the value may be uninitialised
when it is used (which is wrong). Initialize to NULL to silent
the warning.
---
src/gallium/state_trackers/nine/cubetexture9.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/cubetexture
Instead of mixing state preparation (filling pipe_)
and state commit (pipe->set_*),
begin doing so in two separate functions.
This will allow to implement efficient Stateblocks,
and eventually lead to optimisation where the complete
pipe_*** structure is only partially updated.
Signed-off
From: Patrick Rudolph
Texture with pool D3DPOOL_SCRATCH and D3DPOOL_MANAGED cannot be used with flag
D3DUSAGE_DYNAMIC.
Signed-off-by: Patrick Rudolph
---
src/gallium/state_trackers/nine/basetexture9.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trac
From: Patrick Rudolph
In case NineBaseTexture9_ctor returns an error This->surfaces[l] might be NULL.
Reviewed-by: Axel Davy
Signed-off-by: Patrick Rudolph
---
src/gallium/state_trackers/nine/texture9.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_tr
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/device9.c | 6 +-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/device9.c
b/src/gallium/state_trackers/nine/device9.c
index 2ac49a1..28daeba 100644
--- a/src/gallium/state_trackers/nin
We have two paths:
. One that uses a fixed constant buffer, and updates it when needed
. One that uses a user constant buffer, and upload it when needed.
This patch separates the preparation of the constant buffer
and the commit.
It also removes NineDevice9_RestoreNonCSOState, which was
used to r
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_ff.c | 53 +++
1 file changed, 19 insertions(+), 34 deletions(-)
diff --git a/src/gallium/state_trackers/nine/nine_ff.c
b/src/gallium/state_trackers/nine/nine_ff.c
index c08f5d3..9638d9c 100644
--- a/s
From: Christoph Bumiller
This type of blending is used for gallium nine software cursor
Signed-off-by: David Heidelberg
---
src/gallium/include/pipe/p_state.h | 1 +
src/gallium/state_trackers/nine/device9.c| 1 +
src/gallium/state_trackers/nine/swapchain9.c | 2 ++
src/mesa/stat
Always use a user constant buffer for ff.
It means we have to:
. commit the user constant buffer for ff when we use it
. commit back the non-ff constant buffer when we stop using it
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/device9.c| 8 ++
src/gallium/state_trackers/nin
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/device9.c| 21 ++---
src/gallium/state_trackers/nine/device9.h| 3 ++-
src/gallium/state_trackers/nine/nine_state.c | 35
3 files changed, 24 insertions(+), 35 deletions(-)
diff --git a
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_shader.c | 49 +++
1 file changed, 42 insertions(+), 7 deletions(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_trackers/nine/nine_shader.c
index a11c4c7..d9a20a5 100644
Separate state preparation and state commit
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_pipe.c | 5 +++--
src/gallium/state_trackers/nine/nine_pipe.h | 2 +-
src/gallium/state_trackers/nine/nine_state.c | 27 +++
src/gallium/state_trackers/nine/n
It was only used to discriminate update framebuffer vs update
everything. Instead use two functions.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/device9.c| 14 +++---
src/gallium/state_trackers/nine/nine_state.c | 23 ++-
src/gallium/state_tracker
From: Patrick Rudolph
For the case of D3DPOOL_MANAGED textures, This->base.resource can be NULL
at the start of the function. In This case, UploadSelf will take care
of the defining. Assign resource after the UploadSelf call
to prevent NULL pointer exception.
Reviewed-by: Axel Davy
Signed-off-b
From: Tiziano Bacocco
Signed-off-by: Tiziano Bacocco
---
src/gallium/state_trackers/nine/device9.c | 32 ++-
src/gallium/state_trackers/nine/nine_shader.c | 126 +++--
src/gallium/state_trackers/nine/nine_shader.h | 1 +
src/gallium/state_trackers/nine/nine_sta
From: Patrick Rudolph
Allow more than two errors, and return D3DERR_INVALIDCALL for failed display
resolution changes.
Reviewed-by: Axel Davy
Signed-off-by: Patrick Rudolph
---
src/gallium/state_trackers/nine/device9.c| 4 ++--
src/gallium/state_trackers/nine/swapchain9.c | 4 +++-
2 fil
It wasn't giving the expected result.
This fixes some object being transparents
in games like FEAR.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_pipe.c | 12 +++-
src/gallium/state_trackers/nine/nine_state.c | 20
src/gallium/state_trackers/nin
This is quite rare that an app does set again
the same shaders, but it isn't an expensive check
either.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/device9.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/gallium/state_trackers/nine/device9.c
b/src/gallium/state_
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_shader.c | 36 ---
1 file changed, 27 insertions(+), 9 deletions(-)
diff --git a/src/gallium/state_trackers/nine/nine_shader.c
b/src/gallium/state_trackers/nine/nine_shader.c
index 7e76d7d..9680c5a 100644
From: Patrick Rudolph
Return 0 for non MANAGED textures and surfaces.
Fixes failing wine d3d9 tests device.c test_resource_priority.
Reviewed-by: Axel Davy
Signed-off-by: Patrick Rudolph
---
src/gallium/state_trackers/nine/resource9.c | 10 --
1 file changed, 8 insertions(+), 2 deleti
Separate state preparation and state commit
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_pipe.c | 8
src/gallium/state_trackers/nine/nine_pipe.h | 2 +-
src/gallium/state_trackers/nine/nine_state.c | 23 ---
src/gallium/state_trackers/nine/ni
For now the path updated is only used by Amd drivers, but a later
patch will make it used by all drivers. Some drivers like llvmpipe
doesn't support the uploading of constants from user buffers, so improve
the path to work for all drivers
Inspired from the gl state tracker.
Signed-off-by: Axel Da
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/adapter9.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/gallium/state_trackers/nine/adapter9.c
b/src/gallium/state_trackers/nine/adapter9.c
index c5ffcb1..ff1c33c 100644
--- a/src/gallium/state_t
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/surface9.c | 10 ++
src/gallium/state_trackers/nine/volume9.c | 9 +
2 files changed, 11 insertions(+), 8 deletions(-)
diff --git a/src/gallium/state_trackers/nine/surface9.c
b/src/gallium/state_trackers/nine/surface
From: Patrick Rudolph
Mimic WindowsXp behaviour and allow negative values in the rectangle passed.
Add comment to point out behaviour used.
Reviewed-by: Axel Davy
Signed-off-by: Patrick Rudolph
---
src/gallium/state_trackers/nine/surface9.c | 8 ++--
1 file changed, 2 insertions(+), 6 del
From: Patrick Rudolph
Set all values to 0 after allocation. Found using valgrind.
Reviewed-by: Axel Davy
Signed-off-by: Patrick Rudolph
---
src/gallium/state_trackers/nine/device9.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/device9.
Signed-off-by: Axel Davy
Reviewed-by: David Heidelberg
---
src/gallium/state_trackers/nine/device9.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/src/gallium/state_trackers/nine/device9.c
b/src/gallium/state_trackers/nine/device9.c
index 4aa5892..a327bf8 1006
If the texture is bound and dirty_mip is true,
BASETEX_REGISTER_UPDATE adds the texture to the list
of things to update before the next draw call.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/cubetexture9.c | 4 +++-
src/gallium/state_trackers/nine/texture9.c | 4 +++-
2 file
From: Patrick Rudolph
Move the assert to return error codes in the correct order.
Always set the pSizeOfData to the required buffer size.
Fixes failing wine test device.c test_private_data()
Reviewed-by: Axel Davy
Signed-off-by: Patrick Rudolph
---
src/gallium/state_trackers/nine/resource9.c
From: Patrick Rudolph
When using D3DRS_POINTSIZE make sure the value is at least
D3DRS_POINTSIZE_MIN but not greater than D3DRS_POINTSIZE_MAX.
Reviewed-by: Axel Davy
Signed-off-by: Patrick Rudolph
---
src/gallium/state_trackers/nine/nine_pipe.c | 8 +++-
src/gallium/state_trackers/nine/
D3DRS_DITHERENABLE was assigned to the rasterizer state
group, but it was used for the blend group.
Assign it to the blend group.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_state.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_track
We had red and green in the wrong channels
for the ATI2 format (RGTC2).
Found thanks to wine tests.
Signed-off-by: Axel Davy
Reviewed-by: David Heidelberg
---
src/gallium/state_trackers/nine/basetexture9.c | 5 +
1 file changed, 5 insertions(+)
diff --git a/src/gallium/state_trackers/nine
Signed-off-by: Axel Davy
Reviewed-by: David Heidelberg
---
src/gallium/state_trackers/nine/device9.c | 18 --
1 file changed, 12 insertions(+), 6 deletions(-)
diff --git a/src/gallium/state_trackers/nine/device9.c
b/src/gallium/state_trackers/nine/device9.c
index 55948cb..fce19
This is the expected behaviour.
Fixes more than 1 wine tests failures.
Signed-off-by: Axel Davy
Reviewed-by: David Heidelberg
---
src/gallium/state_trackers/nine/cubetexture9.c | 7 +++
src/gallium/state_trackers/nine/device9.c| 7 +++
src/gallium/state_trackers/nine/textu
From: Patrick Rudolph
Align texture memory on 32 byte boundry to allow SSE/AVX memcpy to work on
locked rects.
This fixes issue #88.
Reviewed-by: David Heidelberg
Reviewed-by: Axel Davy
Signed-off-by: Patrick Rudolph
---
src/gallium/state_trackers/nine/cubetexture9.c | 2 +-
src/gallium/sta
Signed-off-by: Axel Davy
Reviewed-by: David Heidelberg
---
src/gallium/state_trackers/nine/device9.c | 12 +---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/src/gallium/state_trackers/nine/device9.c
b/src/gallium/state_trackers/nine/device9.c
index fce19b2..9f6c90e 1006
From: David Heidelberg
Fixes bug: fdo #89978
Signed-off-by: David Heidelberg
Cc: "10.4 10.5 10.6"
---
configure.ac | 4
1 file changed, 4 insertions(+)
diff --git a/configure.ac b/configure.ac
index 4e751e3..c355092 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1632,6 +1632,10 @@ if
We had only one usage for this function.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/device9.c | 4 +-
src/gallium/state_trackers/nine/volume9.c | 90 ++-
src/gallium/state_trackers/nine/volume9.h | 8 +--
3 files changed, 21 insertions(+), 81 delet
NineSurface9_CopySurface was supporting more cases than what
we needed, and doing checks that were innapropriate for
some NineSurface9_CopySurface use cases.
This patch splits it into two for the two use cases, and moves
the checks to the caller.
This patch also adds a few checks to NineDevice9_U
UpdateTexture is supposed to optimise by uploading only for the
dirty region of the source (d3d9 doc, wine tests).
This patch adds the behaviour for surfaces, but not entirely for
volumes.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/device9.c | 47 -
From: Marek Olšák
---
src/gallium/auxiliary/util/u_blitter.c | 54 ++---
src/gallium/auxiliary/util/u_blitter.h | 3 +-
src/gallium/auxiliary/util/u_surface.c | 3 ++
src/gallium/drivers/i915/i915_surface.c | 3 +-
src/gallium/drivers/r300/r300_blit.c| 3 +-
IT is better check if the surface was created with RT flag,
instead of checking capability (llvmpipe was complaining)
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/device9.c | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/src/gallium/state_trackers/nine/
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/basetexture9.c | 15 +++
src/gallium/state_trackers/nine/basetexture9.h | 3 +++
src/gallium/state_trackers/nine/device9.c | 12
3 files changed, 26 insertions(+), 4 deletions(-)
diff --git a/src/gallium/
From: Patrick Rudolph
Tests showed, that in case of errors, the pBits pointer is set to NULL.
The pBits field isn't set to NULL in case of an already locked object.
Reviewed-by: Axel Davy
Signed-off-by: Patrick Rudolph
---
src/gallium/state_trackers/nine/surface9.c | 11 +++
src/galli
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/basetexture9.c | 7 ++-
src/gallium/state_trackers/nine/basetexture9.h | 3 ++-
src/gallium/state_trackers/nine/device9.c | 1 +
src/gallium/state_trackers/nine/device9.h | 1 +
4 files changed, 10 insertions(+), 2 deletio
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/cubetexture9.c | 6 +-
src/gallium/state_trackers/nine/texture9.c | 3 +++
src/gallium/state_trackers/nine/volumetexture9.c | 3 +++
3 files changed, 11 insertions(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/n
For sw cursor we do not tell wine the cursor position (the app
tells us directly). We shouldn't use ID3DPresent_GetCursorPos.
device->cursor.pos already contains the coordinates the app
gave us.
Signed-off-by: Axel Davy
Reviewed-by: David Heidelberg
---
src/gallium/state_trackers/nine/swapchai
From: Patrick Rudolph
Add support for multiple cards and fill in Windows like card name, driver name
and version info.
Use fallback for unknown vendors and unknown card names.
Reviewed-by: Axel Davy
Signed-off-by: Patrick Rudolph
---
src/gallium/targets/d3dadapter9/Makefile.am | 1 +
src
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/cubetexture9.c | 6 --
src/gallium/state_trackers/nine/surface9.c | 4 ++--
src/gallium/state_trackers/nine/texture9.c | 10 ++
src/gallium/state_trackers/nine/volumetexture9.c | 8 +---
4 files changed,
Both Points and Point Sprites are rasterized like quads,
according to d3d9 doc and gallium rasterizer doc.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/nine_pipe.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/gallium/state_trackers/nine/nine_pipe.c
b/s
Similar to what was done for Surface9, track the dirty region
only in VolumeTexture9.
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/basetexture9.c | 9 +--
src/gallium/state_trackers/nine/volume9.c | 82 ++
src/gallium/state_trackers/nine/volume9.h
Signed-off-by: Axel Davy
---
src/gallium/state_trackers/nine/basetexture9.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/gallium/state_trackers/nine/basetexture9.c
b/src/gallium/state_trackers/nine/basetexture9.c
index cc74cc9..728aafd 100644
--- a/src/gallium/state_trackers/nine/ba
Signed-off-by: Axel Davy
Reviewed-by: David Heidelberg
---
src/gallium/state_trackers/nine/device9.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/gallium/state_trackers/nine/device9.c
b/src/gallium/state_trackers/nine/device9.c
index 9f6c90e..38af8e3 100644
--- a/
https://bugs.freedesktop.org/show_bug.cgi?id=91643
--- Comment #6 from Tobias Klausmann ---
Oh right, i was not aware of this. I guess mesa-demo should make this the
default then.
--
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
_
1 - 100 of 107 matches
Mail list logo