[Mesa-dev] [PATCH 1/5] svga: Ignore pipe_rasterizer_state::line_last_pixel.

[Brian Paul]

On 09/14/2015 05:09 AM, Jose Fonseca wrote:
> Altough SVGA3D_RS_LASTPIXEL renderstate exists, most backends ignore it.
> ---
>   src/gallium/drivers/svga/svga_context.h         | 1 -
>   src/gallium/drivers/svga/svga_pipe_rasterizer.c | 1 -
>   src/gallium/drivers/svga/svga_state_rss.c       | 2 +-
>   3 files changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/src/gallium/drivers/svga/svga_context.h b/src/gallium/drivers/svga/svga_context.h
> index e8575f3..3f8d437 100644
> --- a/src/gallium/drivers/svga/svga_context.h
> +++ b/src/gallium/drivers/svga/svga_context.h
> @@ -138,7 +138,6 @@ struct svga_rasterizer_state {
>      unsigned scissortestenable:1;
>      unsigned multisampleantialias:1;
>      unsigned antialiasedlineenable:1;
> -   unsigned lastpixel:1;
>      unsigned pointsprite:1;
>
>      unsigned linepattern;
> diff --git a/src/gallium/drivers/svga/svga_pipe_rasterizer.c b/src/gallium/drivers/svga/svga_pipe_rasterizer.c
> index a7aadac..12e93d7 100644
> --- a/src/gallium/drivers/svga/svga_pipe_rasterizer.c
> +++ b/src/gallium/drivers/svga/svga_pipe_rasterizer.c
> @@ -182,7 +182,6 @@ svga_create_rasterizer_state(struct pipe_context *pipe,
>      rast->scissortestenable = templ->scissor;
>      rast->multisampleantialias = templ->multisample;
>      rast->antialiasedlineenable = templ->line_smooth;
> -   rast->lastpixel = templ->line_last_pixel;
>      rast->pointsprite = templ->sprite_coord_enable != 0x0;
>
>      if (templ->point_smooth) {
> diff --git a/src/gallium/drivers/svga/svga_state_rss.c b/src/gallium/drivers/svga/svga_state_rss.c
> index d43894d..56c5d42 100644
> --- a/src/gallium/drivers/svga/svga_state_rss.c
> +++ b/src/gallium/drivers/svga/svga_state_rss.c
> @@ -224,7 +224,7 @@ emit_rss_vgpu9(struct svga_context *svga, unsigned dirty)
>         EMIT_RS( svga, cullmode, CULLMODE, fail );
>         EMIT_RS( svga, curr->scissortestenable, SCISSORTESTENABLE, fail );
>         EMIT_RS( svga, curr->multisampleantialias, MULTISAMPLEANTIALIAS, fail );
> -      EMIT_RS( svga, curr->lastpixel, LASTPIXEL, fail );
> +      EMIT_RS( svga, FALSE, LASTPIXEL, fail );
>         EMIT_RS_FLOAT( svga, curr->pointsize, POINTSIZE, fail );
>         EMIT_RS_FLOAT( svga, point_size_min, POINTSIZEMIN, fail );
>         EMIT_RS_FLOAT( svga, screen->maxPointSize, POINTSIZEMAX, fail );
>


The series look OK to me.

Reviewed-by: Brian Paul <brianp at vmware.com>