Marcin Ślusarz
44ddc28fa8
Add new rules to .gitattributes
...
This will make sure that all source files have native line endings
when checked out locally.
Reviewed-by: Erik Faye-Lund <erik.faye-lund@collabora.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11779 >
2022-01-19 15:17:17 +00:00
Marcin Ślusarz
a4b8f69bb5
radv/ci: add line endings exception for files generated with wine
...
Reviewed-by: Samuel Pitoiset <samuel.pitoiset@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11779 >
2022-01-19 15:17:17 +00:00
Marcin Ślusarz
745af88783
ci/windows: normalize line endings
...
Reviewed-by: Erik Faye-Lund <erik.faye-lund@collabora.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11779 >
2022-01-19 15:17:17 +00:00
Marcin Ślusarz
f547122772
microsoft/compiler: normalize line endings
...
Reviewed-by: Erik Faye-Lund <erik.faye-lund@collabora.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11779 >
2022-01-19 15:17:17 +00:00
Marcin Ślusarz
ed0edcc338
freedreno/rnn: normalize line endings in rules-ng.xsd
...
Reviewed-by: Eric Engestrom <eric@engestrom.ch >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/11779 >
2022-01-19 15:17:17 +00:00
Rhys Perry
b59764a9fc
aco: use p_extract for SGPR nir_op_unpack_half_2x16_split_y
...
fossil-db (Sienna Cichlid):
Totals from 7264 (5.40% of 134627) affected shaders:
VGPRs: 548152 -> 548184 (+0.01%)
SpillSGPRs: 7615 -> 7607 (-0.11%)
CodeSize: 71025152 -> 70993036 (-0.05%); split: -0.05%, +0.00%
Instrs: 13386799 -> 13298580 (-0.66%); split: -0.66%, +0.00%
Latency: 177464497 -> 177091693 (-0.21%); split: -0.21%, +0.00%
InvThroughput: 32185148 -> 32151873 (-0.10%); split: -0.10%, +0.00%
VClause: 233167 -> 233166 (-0.00%); split: -0.00%, +0.00%
SClause: 468423 -> 468426 (+0.00%); split: -0.00%, +0.01%
Copies: 950727 -> 942753 (-0.84%); split: -0.85%, +0.02%
Branches: 455919 -> 455901 (-0.00%); split: -0.01%, +0.00%
fossil-db (Vega):
Totals from 7264 (5.39% of 134762) affected shaders:
SGPRs: 738800 -> 738816 (+0.00%)
VGPRs: 550264 -> 550344 (+0.01%)
SpillSGPRs: 11149 -> 11157 (+0.07%)
CodeSize: 67487104 -> 67466772 (-0.03%); split: -0.03%, +0.00%
Instrs: 13142106 -> 13061767 (-0.61%)
Latency: 209278575 -> 208438854 (-0.40%); split: -0.40%, +0.00%
InvThroughput: 81486405 -> 81265773 (-0.27%); split: -0.27%, +0.00%
VClause: 222293 -> 222291 (-0.00%); split: -0.00%, +0.00%
SClause: 447783 -> 447800 (+0.00%); split: -0.00%, +0.01%
Copies: 1243760 -> 1238842 (-0.40%); split: -0.43%, +0.03%
Signed-off-by: Rhys Perry <pendingchaos02@gmail.com >
Reviewed-by: Daniel Schürmann <daniel@schuermann.dev >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14592 >
2022-01-19 12:41:48 +00:00
Matti Hamalainen
bbc4ca5d7d
aux/trace: cosmetic cleanup
...
Fix up some function argument indentation alignments and
adjust few other small cosmetics.
Signed-off-by: Matti Hamalainen <ccr@tnsp.org >
Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14398 >
2022-01-19 11:57:17 +00:00
Matti Hamalainen
85a75e42db
aux/trace: implement missing trace calls
...
Some call traces (resource_from_handle, resource_get_handle and
resource_get_param) were TODO, so implement them while we are here.
Signed-off-by: Matti Hamalainen <ccr@tnsp.org >
Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14398 >
2022-01-19 11:57:17 +00:00
Matti Hamalainen
32d40f7c80
aux/trace: print enum names instead of integer values in gallium traces
...
Having only magic constants instead of human-readable strings in
traces not only hinders readability, but also may affect trace
comparision of old and new traces if new enums have been added
or modified (thus possibly changing the values of existing ones.)
So we implement printing of enum names as strings instead.
In order to have those strings, we need to add some new helper
functions, which we will automatically generate from header file
src/gallium/include/pipe/p_defines.h via a new Python script
enums2names.py.
We also bolt this all into the Meson build system.
Link: https://gitlab.freedesktop.org/mesa/mesa/-/issues/4609
Signed-off-by: Matti Hamalainen <ccr@tnsp.org >
Reviewed-by: Dylan Baker <dylan@pnwbakers.com >
Reviewed-By: Mike Blumenkrantz <michael.blumenkrantz@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14398 >
2022-01-19 11:57:17 +00:00
Lionel Landwerlin
51f6288a2d
genxml: reduce amount of generated code
...
$ wc -l build/src/intel/genxml/genX_bits.h
250581 build/src/intel/genxml/genX_bits.h (before)
5748 build/src/intel/genxml/genX_bits.h (after)
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Acked-by: Kenneth Graunke <kenneth@whitecape.org >
Acked-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14469 >
2022-01-19 10:57:48 +00:00
Lionel Landwerlin
acebea9cf1
anv: fix missing descriptor copy of bufferview/surfacestate content
...
When doing copies of descriptors from one set to another, that contain
either a UNIFORM_BUFFER or STORAGE_BUFFER, both the buffer view &
surface state are allocated from the source descriptor. Therefore we
need to copy their content otherwise we could run into lifecycle
issues when the source descriptor is destroyed.
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Cc: mesa-stable
Reviewed-by: Tapani Pälli <tapani.palli@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14585 >
2022-01-19 10:40:37 +00:00
Lionel Landwerlin
2c875e6ad7
intel/dev: fix ppipe_mask computation
...
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Fixes: e86ce98c6a ("intel/devinfo: deal with i915 topology query change")
Reviewed-by: Francisco Jerez <currojerez@riseup.net >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14608 >
2022-01-19 10:22:36 +00:00
Thomas H.P. Andersen
9e839620da
meson: add check kwarg to run_command
...
run_command will change the default for the check arg to true
in the future. If it is true then meson will exit if the command
fails. It must be false here as we check the return code to
provide a meaningful error message.
With meson 0.61 we get the following warning:
WARNING: You should add the boolean check kwarg to the run_command call.
It currently defaults to false,
but it will default to true in future releases of meson.
See also: https://github.com/mesonbuild/meson/issues/9300
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14602 >
2022-01-19 04:55:55 +00:00
Mike Blumenkrantz
71673cd2d4
zink: add deqp ci baseline for nv
...
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14603 >
2022-01-19 04:41:55 +00:00
Mike Blumenkrantz
f9892b0f4f
zink: update nv ci baseline
...
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14603 >
2022-01-19 04:41:55 +00:00
Mike Blumenkrantz
806946b094
lavapipe: replace hard pointer calcs in push descriptors with ptralloc
...
Acked-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13678 >
2022-01-19 04:10:58 +00:00
Mike Blumenkrantz
48fde98b79
lavapipe: replace hard pointer calcs in dynamic render with ptralloc
...
Acked-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13678 >
2022-01-19 04:10:58 +00:00
Mike Blumenkrantz
171ccdbf1c
util: add ptralloc
...
many times it will be the case that an allocation for a block of data
needs to be done in one alloc() call such that the members of a struct as well
as some extra trailing data are all in the same allocation like
```
struct Test {
unsigned a[4];
unsigned c;
};
unsigned *b; //ptr to uint[8]
```
should be allocated as a single block of (13 * sizeof(unsigned)) memory using
C pointer offsets to allocate the memory as
```
| Test | b |
```
with something like
```
struct Test *t = malloc(sizeof(struct Test) + (8 * sizeof(unsigned)));
```
and then set `b` with
```
t->b = ((uint8_t*)t) + sizeof(struct Test);
```
this is annoying, awful to read, and (at least for dum-dums like me) prone to errors,
however, so having some utility functions which can deliver the same
functionality with better readability helps out this case by transforming it to
```
unsigned *b;
void **ptrs[] = {(void*)&b};
size_t sizes[] = {8 * sizeof(unsigned));
struct Test *t = ptralloc(sizeof(struct Test), 1, sizes, ptrs);
```
where `b` is now set to the appropriate offset in memory
Acked-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13678 >
2022-01-19 04:10:58 +00:00
Kenneth Graunke
0bc7562466
iris: Do primitive ID overrides in 3DSTATE_SBE not SBE_SWIZ
...
Broadwell introduced new fields in 3DSTATE_SBE which allow us to ask
the hardware to override Primitive ID for us, rather than requiring us
to turn on attribute swizzling and specify per-attribute overrides in
3DSTATE_SBE_SWIZ. We unconditionally enable attribute swizzling today,
but this is a step toward letting us think about disabling it in the
future.
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14210 >
2022-01-19 01:31:47 +00:00
Kenneth Graunke
223edb1ec1
iris: Use prog_data->inputs rather than shader info in SBE code.
...
This should be the same thing, but requires looking up less data.
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14210 >
2022-01-19 01:31:47 +00:00
Kenneth Graunke
d475e839da
intel/fs: Reuse the same FS input slot for VUE header fields.
...
VARYING_SLOT_{VIEWPORT,LAYER,PSIZ} all live in the same VUE header slot,
and the FS is already set up to read the x/y/z/w component of that vec4.
However, we were setting up the SBE to pass each of those items as a
separate FS input, so hypothetically if a shader read all three, we
would burn 3 FS inputs with redundant data. Not only was this passing
extra data to the FS, but it would count as extra input slots for the
"Do we have 16 or fewer attributes?" check for using SBE swizzling to
rearrange them in a convenient manner.
Now we make them share a single FS attribute and only count them once.
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14210 >
2022-01-19 01:31:47 +00:00
Emma Anholt
5f0bf2113e
r300: Add consts (uniforms) count to the shader-db output.
...
This is one of the critical metrics for this driver.
Reviewed-by: Matt Turner <mattst88@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14600 >
2022-01-19 00:49:57 +00:00
Emma Anholt
cd7b260eeb
r300: Drop unused r300_get_stats() call.
...
Unused since we switched to shader-db.
Reviewed-by: Matt Turner <mattst88@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14600 >
2022-01-19 00:49:57 +00:00
Jordan Justen
695ba644ab
intel/gem: Return length from intel_i915_query_alloc
...
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com >
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13866 >
2022-01-19 00:29:35 +00:00
Jordan Justen
83da7dc487
intel/dev: Recalculate max_cs_threads after applying hwconfig changes
...
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com >
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13866 >
2022-01-19 00:29:35 +00:00
Jordan Justen
ab22fb488b
intel/dev: Apply settings from hwconfig if devinfo::apply_hwconfig is set
...
During debug builds, if apply_hwconfig is not set, then the devinfo
value will be compared with the hwconfig value. If they don't match
then a warning message will be logged to stderr.
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com >
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13866 >
2022-01-19 00:29:35 +00:00
Jordan Justen
59f32c62c7
intel/dev: Set intel_device_info::apply_hwconfig for DG2
...
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com >
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13866 >
2022-01-19 00:29:35 +00:00
Jordan Justen
a15f18c886
intel/dev: Add intel_device_info::apply_hwconfig
...
This will be used to conditionally use hwconfig values to update
intel_device_info at runtime.
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com >
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13866 >
2022-01-19 00:29:35 +00:00
Jordan Justen
0300351028
intel/dev: Print urb size with intel_dev_info
...
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com >
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13866 >
2022-01-19 00:29:35 +00:00
Jordan Justen
2dd78b9a4f
intel/dev: Add intel_print_hwconfig_table()
...
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com >
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13866 >
2022-01-19 00:29:35 +00:00
Jordan Justen
3a119f0c6d
intel/dev: Add intel_hwconfig_types.h from random post on the internet
...
There is no spec for any of this, but we'll need this to interpret a
blob that the kernel is giving us.
Ref: https://lists.freedesktop.org/archives/intel-gfx/2021-September/277488.html
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com >
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13866 >
2022-01-19 00:29:35 +00:00
Jordan Justen
f0692365a2
anv,blorp,crocus,i965,iris: Use devinfo->max_threads_per_psd for gfx8+
...
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com >
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13866 >
2022-01-19 00:29:35 +00:00
Jordan Justen
08a897fa38
intel/dev: Add max_threads_per_psd field to devinfo for gfx8+
...
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com >
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Reviewed-by: Caio Oliveira <caio.oliveira@intel.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/13866 >
2022-01-19 00:29:35 +00:00
Pavel Ondračka
5e2834754e
r300: fix translate_LRP
...
This was broken by 7daba1fe65
Signed-off-by: Pavel Ondračka <pavel.ondracka@gmail.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14596 >
2022-01-19 00:19:46 +00:00
Neha Bhende
e6b60c2764
svga: enable GL43 on SVGA GL43 capable device
...
This patch enables GL43 feature for VMware's future GL43 capable
SVGA device.
Tested with various GL43 apps.
Reviewed-by: Charmaine Lee <charmainel@vmware.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14270 >
2022-01-18 23:50:36 +00:00
Neha Bhende
1942c06f9c
svga: add GL43 resource validation at draw time
...
This patch adds GL43 tracked states stack and supports GL43 resource
validation at draw time. This patch is squash of in house patches
to support GL43 on VMware driver.
Reviewed-by: Charmaine Lee <charmainel@vmware.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14270 >
2022-01-18 23:50:36 +00:00
Neha Bhende
1e99a30738
svga: shader translation for compute, image views and shader buffers
...
This patch handles shader translation for compute, image views and shader
buffers and updates the corresponding shader compile keys.
It also includes support of using shader raw buffer for shader buffer used
as constant buffer.
This patch is squash of numerous in house patches.
Reviewed-by: Charmaine Lee <charmainel@vmware.com >
v2: As pointed out by Thomas, fix revert of 64292c0f caused by this patch.
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14270 >
2022-01-18 23:50:36 +00:00
Neha Bhende
247c61f2d0
svga: Add support for compute shader, shader buffers and image views
...
This commit is squash of commits which handles resource creation and management
for compute shader, shader buffers and image views. It creates uavs for shader
buffers and image views.
Reviewed-by: Charmaine Lee <charmainel@vmware.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14270 >
2022-01-18 23:50:36 +00:00
Neha Bhende
533a09541d
tgsi: Add hw_atomic_declared in tgsi_info
...
This patch also adds hw_atomic_declared info in tgsi_info.
Reviewed-by: Charmaine Lee <charmainel@vmware.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14270 >
2022-01-18 23:50:36 +00:00
Neha Bhende
dc512f11ea
svga: Add utility to check for GL43 support
...
GL43 support in SVGA driver requires vmwgfx kernel version 2.20 and
GL43 capable SVGA device.
Reviewed-by: Charmaine Lee <charmainel@vmware.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14270 >
2022-01-18 23:50:36 +00:00
Neha Bhende
391a8bbc77
svga: Add GL43 commands support
...
This patch updates SVGA header files and adds support for uav related commands.
Reviewed-by: Charmaine Lee <charmainel@vmware.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14270 >
2022-01-18 23:50:36 +00:00
Mike Blumenkrantz
440beb01d7
zink: enable EXT_external_objects pipe caps
...
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14498 >
2022-01-18 23:31:01 +00:00
Mike Blumenkrantz
4ed30be31e
zink: implement external memory object resource handling
...
this is super simple and can almost fully reuse the existing codepaths
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14498 >
2022-01-18 23:31:01 +00:00
Mike Blumenkrantz
32597e116d
zink: implement GL semaphores
...
this is basically just a wrapper around vulkan semaphores, so it maps
fairly well
the existing fence function was a big ??? and should never have been triggered
like it was
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14498 >
2022-01-18 23:31:01 +00:00
Mike Blumenkrantz
29285a0e85
zink: add driver/device uuid screen hooks
...
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14498 >
2022-01-18 23:31:01 +00:00
Mike Blumenkrantz
65e34fa70e
zink: add VK_KHR_external_memory_capabilities to instance exts
...
the props for this are weird so they have to be fetched "manually"
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14498 >
2022-01-18 23:31:01 +00:00
Mike Blumenkrantz
f0f1a94fcf
zink: add VK_KHR_external_semaphore_fd to device exts
...
Reviewed-by: Dave Airlie <airlied@redhat.com >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14498 >
2022-01-18 23:31:01 +00:00
Jordan Justen
a11dfc11cf
iris: Use mi_builder for load/store reg/mem/imm functions
...
Ref: 06cf838cbd ("intel/mi_builder: Support gen11 command-streamer based register offsets")
Ref: 6ffdcc335e ("iris: Use mi_builder in iris_load_indirect_location()")
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com >
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14340 >
2022-01-18 23:11:38 +00:00
Jordan Justen
e29ed39d63
iris: Use mi_builder to set 3DPRIM registers for draws
...
Signed-off-by: Jordan Justen <jordan.l.justen@intel.com >
Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com >
Reviewed-by: Kenneth Graunke <kenneth@whitecape.org >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14340 >
2022-01-18 23:11:38 +00:00
Dave Airlie
fbcbbae662
crocus: only clamp point size on last stage.
...
This fixes a regression in tests that pass unclamped point size
values between stages.
This keeps xfb broken since the real way it should work is to have
the hw clamp after xfb, but this seems the least evil path.
Fixes: 3077d96856 ("crocus: Clamp VS point sizes to the HW limits as required.")
Reviewed-by: Emma Anholt <emma@anholt.net >
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/14359 >
2022-01-18 22:53:45 +00:00