[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