| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
| |
This code has originally matured in r300g and was ported to r600g several
times. It was obvious it's a code duplication.
See also comments in the header file.
|
| |
|
|
|
|
|
| |
We don't have to unmap and recreate the upload buffer when a flush occurs.
This should also prevent buffer allocations from failing.
|
| |
|
|
|
|
|
|
|
| |
This uses the surface format to set the CB up not the underlying texture
format, since these can and do differ.
Fixes piglit fbo-srgb.
|
|
|
|
|
| |
Since we never bind the actual DB to the CB/texture only the flushed one
we don't need to track the tile type at the moment.
|
| |
|
|
|
|
|
|
| |
The type of u_get_transfer_vtbl of the usage argument in u_transfer.h is
unsigned and not enum pipe_transfer_usage. This patch changes the type
of usage to unsigned to match the prototype in the header file.
|
|
|
|
|
|
| |
this fixes the mipmap tests with tiling forced on.
Signed-off-by: Dave Airlie <[email protected]>
|
|
|
|
|
|
| |
Since these textures still have the depth bit set.
Signed-off-by: Dave Airlie <[email protected]>
|
| |
|
|
|
|
|
|
| |
For mipmap generation we need all levels in the flushing texture.
Signed-off-by: Dave Airlie <[email protected]>
|
| |
|
|
|
|
|
|
|
|
|
| |
This just adds a flag to create the texture without doing any
flushing to it. Flushing occurs in the draw function. This avoids
unnecessary flushes when we end up rebinding a CB/DB/texture due
to the blitter just restoring state.
Signed-off-by: Dave Airlie <[email protected]>
|
|
|
|
|
|
|
| |
This checks the color buffer bindings to make sure there is something
to flush.
Signed-off-by: Dave Airlie <[email protected]>
|
|
|
|
|
|
|
| |
If we are going to blit from a depth texture we need to flush
it before we blit from it.
Signed-off-by: Dave Airlie <[email protected]>
|
| |
|
| |
|
|
|
|
| |
Plus other assorted clarifications.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Before, the set_sampler_views() and restore_sampler_views() functions
used MAX2(old,new) to tell the driver how many samplers or sampler
views to set. This could result in cases such as:
pipe->set_fragment_sampler_views(pipe, 4, views={foo, bar, NULL, NULL})
Many/most gallium drivers would take this as-is and set
ctx->num_sampler_views=4 and ctx->sampler_views={foo, bar, NULL, NULL, ...}.
Later, loops over ctx->num_sampler_views would have to check for null
pointers. Worse, the number of sampler views and number of sampler CSOs
could get out of sync:
ctx->num_samplers = 2
ctx->samplers = {foo, bar, ...}
ctx->num_sampler_views = 4
ctx->sampler_views={Foo, Bar, NULL, NULL, ...}
So loops over the num_samplers could run into null sampler_views pointers
or vice versa.
This fixes a failed assertion in the SVGA driver when running the Mesa
engine demo in AA line mode (and possibly other cases).
It looks like all gallium drivers are careful to unreference views
and null-out sampler CSO pointers for the units beyond what's set
with the pipe::bind_x_sampler_states() and pipe::set_x_sampler_views()
functions.
I'll update the gallium docs to explain this as well.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
Leak was introduced when fixing strict aliasing violation in this code:
the reference counting was preserved, but the destructor call on zero
reference count was not.
|
|
|
|
| |
See comment for more details.
|
|
|
|
|
|
|
|
| |
Exactly one half would be the ideal, but this is a soft limit, and one
more byte over brings us to synchronous behavior.
Flushing when the referred GMR exceeds one third of the aperture gives us
statistically better performance.
|
| |
|
|
|
|
|
| |
Only svga_context_flush should do it, to ensure upload commands are not
submitted to hardware in an inconsistent state.
|
| |
|
| |
|
| |
|
|
|
|
|
|
| |
With the previous fixes we can now enabled hw depth copies
Signed-off-by: Dave Airlie <[email protected]>
|
|
|
|
|
|
|
|
|
|
| |
If we get a sw accessible buffer like the S8 texture we end up
doing depth tracking on it when there is no need since we won't
ever bind it to the hardware. This leads to a sw fallback in the
transfer destruction which leads to and endless recusion loop
of fail in transfer destroy.
Signed-off-by: Dave Airlie <[email protected]>
|
|
|
|
|
|
|
|
|
|
|
| |
this adds a flag to keep track of whether the depth texture structure
is the flushed texture or not, so we can avoid doing flushes when
we do a hw rendering from one to the other.
it also renames flushed to dirty_db which tracks if the DB copy
has been dirtied by being bound to the hw.
Signed-off-by: Dave Airlie <[email protected]>
|
|
|
|
| |
This fixes zreaddraw in tiling mode
|
| |
|
|
|
|
| |
To be consistant with vertex, geometry sampler fields.
|
|
|
|
|
|
| |
This consolidates the code duplicated between the fragment sampler
and vertex sampler functions. Plus, it'll make adding support for
geometry shader samplers trivial.
|
|
|
|
| |
To better distinguish from vertex sampler fields.
|
|
|
|
|
|
|
| |
Before we were looping to nr_samplers, which is the number of fragment
samplers, not vertex samplers.
NOTE: This is a candidate for the 7.9 and 7.10 branches.
|
|
|
|
| |
Based on work from Jakob Bornecrantz, Michel Dänzer, and Brian Paul.
|
|
|
|
|
| |
Join multiple exports into just one instruction
instead of exporting each register separately.
|
|
|
|
|
| |
Not 100% sure on this one, but this is how it should work,
the question is whether it will uncover other bugs elsewhere.
|
|
|
|
|
|
|
| |
If we see a MACRO bit on r600g its 2D tiled,
if don't see a MACRO bit and we do see a MICRO bit then its 1D tiled.
Signed-off-by: Dave Airlie <[email protected]>
|
| |
|
|
|
|
|
|
|
|
| |
If the tile type for the buffer is 1 then its been bound to the
DB at some point, we need to decompress it, otherwise its only
been bound as texture/cb so don't do anything.
This fixes 5 piglit tests here on r600g.
|
|
|
|
|
|
|
|
| |
this just adds the ioctl interface and sets the tile type
and array mode in the correct place.
This seems to bring eg 1D tiling to the same level, and issues
as on r600. No idea how to address 2D yet.
|
|
|
|
|
|
|
| |
sampler view on a depth texture.
R600/R700 was using incorrect tiling information from the (compressed) depth
buffer. Evergreen worked anyway because tiling doesn't work.
|
|
|
|
|
| |
Only decrement ref count if r600_upload_const_buffer
really changes the buffer.
|