[Intel-gfx] [PATCH] Revive i830_update_front_offset() function overzealously pruned during NoAccel removal.(was Re: [ANNOUNCE] xf86-video-intel 2.7.99.902)

[Fatih Aşıcı]

On Tuesday 14 July 2009 22:34:36 Keith Packard wrote:
> On Tue, 2009-07-14 at 11:13 +0100, Colin Guthrie wrote:
> > Please apply (or change and apply an appropriate fix - it doesn't really
> > need to be in a function now as it's quite simple - or perhaps it can be
> > inlined?) :)
>
> Does something like:
>
> diff --git a/src/i830_memory.c b/src/i830_memory.c
> index f2f3966..cc9d376 100644
> --- a/src/i830_memory.c
> +++ b/src/i830_memory.c
> @@ -1581,6 +1581,9 @@ i830_bind_all_memory(ScrnInfoPtr pScrn)
>  	i830_update_cursor_offsets(pScrn);
>      i830_set_max_gtt_map_size(pScrn);
>
> +    if (pI830->front_buffer)
> +	pScrn->fbOffset = pI830->front_buffer->offset;
> +
>      return TRUE;
>  }
>
> do the trick?

It seems so. A user (owns 945GM) reported a similar crash and it worked when 
he rebuilt with this patch.