[Mesa-dev] [PATCH] st/nine: Drop USER_INDEX_BUFFERS check
Edward O'Callaghan
funfunctor at folklore1984.net
Sat Feb 25 11:31:43 UTC 2017
oh yes I missed that, you want to have drop the negation in your patch Mike.
On 02/25/2017 10:28 PM, Mike Lothian wrote:
> I think my question regarding the double negative still stands. Is it
> "!This->cmst_active" ot "This->cmst_active"
>
> On Sat, 25 Feb 2017 at 11:23 Edward O'Callaghan
> <funfunctor at folklore1984.net <mailto:funfunctor at folklore1984.net>> wrote:
>
> From: Mike Lothian <mike at fireburn.co.uk <mailto:mike at fireburn.co.uk>>
>
> This fixes 4a883966c1f74f43afc145d2c3d27af7b8c5e01a where the
> PIPE_CAP was removed.
>
> Now USER_INDEX_BUFFERS are always enabled remove the check and only
> check for cmst_active directly.
>
> v2: Axel pointed out the code was still needed when cmst was inactive,
> Rebase on master too
> v3: drop struct member user_ibufs also && fixup shortlog (Edward).
>
> Fixes: 4a883966c1f7 ("gallium: remove PIPE_CAP_USER_INDEX_BUFFERS")
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=99953
> Reported-and-tested-by: Vinson Lee <vlee at freedesktop.org
> <mailto:vlee at freedesktop.org>> (v1)
> Cc: Marek Olšák <marek.olsak at amd.com <mailto:marek.olsak at amd.com>>
> Cc: Axel Davy <axel.davy at ens.fr <mailto:axel.davy at ens.fr>>
> Signed-off-by: Mike Lothian <mike at fireburn.co.uk
> <mailto:mike at fireburn.co.uk>>
> Signed-off-by: Edward O'Callaghan <funfunctor at folklore1984.net
> <mailto:funfunctor at folklore1984.net>>
> ---
> src/gallium/state_trackers/nine/device9.c | 3 +--
> src/gallium/state_trackers/nine/device9.h | 1 -
> 2 files changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/src/gallium/state_trackers/nine/device9.c
> b/src/gallium/state_trackers/nine/device9.c
> index c3924a2..30ab8de 100644
> --- a/src/gallium/state_trackers/nine/device9.c
> +++ b/src/gallium/state_trackers/nine/device9.c
> @@ -473,7 +473,6 @@ NineDevice9_ctor( struct NineDevice9 *This,
> /* Allocate upload helper for drivers that suck (from st pov ;). */
>
> This->driver_caps.user_vbufs = GET_PCAP(USER_VERTEX_BUFFERS) &&
> !This->csmt_active;
> - This->driver_caps.user_ibufs = GET_PCAP(USER_INDEX_BUFFERS) &&
> !This->csmt_active;
> This->driver_caps.user_cbufs = GET_PCAP(USER_CONSTANT_BUFFERS);
> This->driver_caps.user_sw_vbufs =
> This->screen_sw->get_param(This->screen_sw,
> PIPE_CAP_USER_VERTEX_BUFFERS);
> This->driver_caps.user_sw_cbufs =
> This->screen_sw->get_param(This->screen_sw,
> PIPE_CAP_USER_CONSTANT_BUFFERS);
> @@ -2896,7 +2895,7 @@ NineDevice9_DrawIndexedPrimitiveUP( struct
> NineDevice9 *This,
> vbuf.buffer_offset -= base;
> vbuf.user_buffer = NULL;
> }
> - if (!This->driver_caps.user_ibufs) {
> + if (!This->cmst_active) {
> u_upload_data(This->context.pipe->stream_uploader,
> 0,
> (prim_count_to_vertex_count(PrimitiveType,
> PrimitiveCount)) * ibuf.index_size,
> diff --git a/src/gallium/state_trackers/nine/device9.h
> b/src/gallium/state_trackers/nine/device9.h
> index 71ebbdc..4b1630c 100644
> --- a/src/gallium/state_trackers/nine/device9.h
> +++ b/src/gallium/state_trackers/nine/device9.h
> @@ -127,7 +127,6 @@ struct NineDevice9
>
> struct {
> boolean user_vbufs;
> - boolean user_ibufs;
> boolean user_cbufs;
> boolean user_sw_vbufs;
> boolean user_sw_cbufs;
> --
> 2.9.3
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20170225/a78aa9b8/attachment.sig>
More information about the mesa-dev
mailing list