On 2002.06.15 18:25 Leif Delgass wrote:
> On Sat, 15 Jun 2002, Keith Whitwell wrote:
> 
> > ...
> >
> > Is this necessary if the toggle is really just a write?  Jose, you're
> not
> > doing a read-modify-write operation on that flag are you?
> 
> Actually, with the current implementation it is a read-modify-write.
> Jose, we could avoid that by caching the ring offset and data for
> BM_COMMAND from the (previous) tail as my original patch did.
> 

As Leif said, I'm doing a read-modify-write. But there is no need to keep 
track of the previous written value to void the read since (as I said in a 
previous post) the linux kernel function clear_bit does the necessary job 
atomically (with no read too).

Nevertheless I still fail to understand why the engine would fail here. If 
he got either the previous value, or the new one it shouldn't fail as both 
situations are accounted for. My concern is if it can gets values 
different than these two due to a simultaneous read/write, but I'm not 
sure if this is even possible as the modify consists of a single bit!

> > I suppose to some extent we're relying on the atomicity of the mach64's
> use of
> > that flag as well -- I don't know if there are any obvious ways they
> could get
> > it wrong (I assume they just fetch it once...) -- so that's probably
> not a
> > huge problem.
> >

There is the possibility of I'm missing something... well, I have no other 
choice than keep trying and eliminating hypothesis.

José Fonseca

_______________________________________________________________

Don't miss the 2002 Sprint PCS Application Developer's Conference
August 25-28 in Las Vegas - 
http://devcon.sprintpcs.com/adp/index.cfm?source=osdntextlink

_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to