|
From: Greg P. <gp...@us...> - 2005-02-02 03:36:18
|
Jeremy Fitzhardinge writes: > I just checked in a series of patches which remove the baseBlock and all > its uses. The main reason for this was to simplify the code (~1000 > lines gone), and to simplify accessing the CPU state. There's now only > one place to look for VCPU state - the thread's ThreadState. While > generated code is running, %ebp points to the thread's entry in > VG_(threads). This is way cool. Understanding the baseBlock mechanism and the various ways VCPU data was stored and moved was difficult for an outsider looking in. The other thing that would be nice to kill (which I think you mentioned already) is the ThreadState vs. ThreadId distinction. -- Greg Parker gp...@us... |