On Thu, 2010-09-16 at 13:37 +0800, Zhenyu Wang wrote:
> On 2010.09.16 13:10:29 +0800, Xiang, Haihao wrote:
> > > Or can't that be done in init function by using advance_ring?
> > advance_ring uses ring->tail to set TAIL register, i965_reset() also
> > invokes ring->init() to re-init ring buffer, how to guarantee ring->tail
> > is 0? BTW advance_ring can be implemented as set_tail(dev, ring->tail).
> > 
> 
> oh, that's bad, not only just for render ring, but ring tail should
> always be inited correctly, not just in static variable..
> 
init_ring_common() always inits ring tail. so it is not just in static
variable.





_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to