[Mesa-dev] [PATCH 09/17] st/mesa: add a second pipeline for compute

[Marek Olšák]

On Sun, Jan 24, 2016 at 10:09 PM, Samuel Pitoiset
<samuel.pitoiset at gmail.com> wrote:
> Compute needs a new and different validation path.
>
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> ---
>  src/gallium/include/state_tracker/st_api.h |  8 +++++
>  src/mesa/state_tracker/st_atom.c           | 48 ++++++++++++++++++++++++------
>  src/mesa/state_tracker/st_atom.h           |  4 ++-
>  src/mesa/state_tracker/st_cb_bitmap.c      |  2 +-
>  src/mesa/state_tracker/st_cb_clear.c       |  2 +-
>  src/mesa/state_tracker/st_cb_drawpixels.c  |  4 +--
>  src/mesa/state_tracker/st_cb_drawtex.c     |  2 +-
>  src/mesa/state_tracker/st_cb_msaa.c        |  2 +-
>  src/mesa/state_tracker/st_cb_rasterpos.c   |  2 +-
>  src/mesa/state_tracker/st_cb_readpixels.c  |  2 +-
>  src/mesa/state_tracker/st_context.c        |  6 ++++
>  src/mesa/state_tracker/st_context.h        |  1 +
>  src/mesa/state_tracker/st_draw.c           |  4 +--
>  src/mesa/state_tracker/st_draw_feedback.c  |  2 +-
>  14 files changed, 68 insertions(+), 21 deletions(-)
>
> diff --git a/src/gallium/include/state_tracker/st_api.h b/src/gallium/include/state_tracker/st_api.h
> index 356863d..3f91d0b 100644
> --- a/src/gallium/include/state_tracker/st_api.h
> +++ b/src/gallium/include/state_tracker/st_api.h
> @@ -156,6 +156,14 @@ enum st_context_resource_type {
>  };
>
>  /**
> + * Enumeration of state tracker pipelines.
> + */
> +enum st_pipeline {
> +   ST_PIPELINE_RENDER,
> +   ST_PIPELINE_COMPUTE,
> +};

st_api.h should only contain interfaces for communication with the
windowing system support code (such as st/dri).

Since this work doesn't contain any such thing, the definition should
be in st_context.h, which is private to mesa/state_tracker.

Marek