On 01/19/2012 10:54 AM, Keith Packard wrote: > On Thu, 19 Jan 2012 10:50:05 -0800, Eric Anholt <eric at anholt.net> wrote: > >> - if (IS_GEN6(dev) || IS_GEN7(dev)) >> - mode |= MI_FLUSH_ENABLE << 16 | MI_FLUSH_ENABLE; > > This seems better than setting random bits that don't do anything but > annoy the simulator. The simulator complains unless both bits are set iirc. I can double check, but it's been a while since I've run without my patch. Ben