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 > <funfunc...@folklore1984.net <mailto:funfunc...@folklore1984.net>> wrote: > > From: Mike Lothian <m...@fireburn.co.uk <mailto:m...@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 <v...@freedesktop.org > <mailto:v...@freedesktop.org>> (v1) > Cc: Marek Olšák <marek.ol...@amd.com <mailto:marek.ol...@amd.com>> > Cc: Axel Davy <axel.d...@ens.fr <mailto:axel.d...@ens.fr>> > Signed-off-by: Mike Lothian <m...@fireburn.co.uk > <mailto:m...@fireburn.co.uk>> > Signed-off-by: Edward O'Callaghan <funfunc...@folklore1984.net > <mailto:funfunc...@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 >
signature.asc
Description: OpenPGP digital signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev