[haiku-development] Re: intel_extreme rework status

  • From: Adrien Destugues <pulkomandy@xxxxxxxxxxxxx>
  • To: haiku-development@xxxxxxxxxxxxx
  • Date: Tue, 24 May 2016 22:03:58 +0200

On Tue, May 24, 2016 at 07:35:22PM +0000, Alexander von Gluck IV wrote:

As these cards were working by pure luck, and nothing over Sandy Bridge
was functional, I disabled intel_extreme on these chipsets to preserve
a working system via vesa (which will do native a lot of the time)

[citation needed], none of my laptops get into native mode with VESA.

The previous driver worked because it didn't touch the FDI, which means
it was left however configured by the VESA BIOS. That works as long as
we have such a BIOS or something similar doing the setup for us, and we
don't reset it.


No-one here is questioning the worth of a driver rework which improves the 
code structure. I'm
pretty sure that Adrien's comment about a revert was meaning only a 
temporary revert, until the new
driver is ready to handle all the hardware which worked with the old one, 
in case that will take a
while longer. So this is really not about throwing your work away, just 
giving it more time so
people don't have nasty regressions on their Haiku machines.

Indeed, the refactoring work is appreciated and the way forward, but
it's not acceptable if I have to run Haiku at non-native resolution. If
we had a stable branch, it would be fine to use the trunk as a
playground for experiments, but in the current situation, only complete
and working code should be put there. So the intel driver rework would
have to linger in a branch for some more time until these problems are
solved.

The closest I have an IvyBridge here (and 0 time for patches right now) so 
feel free to give it a shot:

http://cgit.haiku-os.org/haiku/tree/src/add-ons/kernel/drivers/graphics/intel_extreme/driver.cpp#n87

And disable this:

http://cgit.haiku-os.org/haiku/tree/src/add-ons/accelerants/intel_extreme/Pipes.cpp#n67


Thanks for the hints, I'll see if I can get something out of that.

-- 
Adrien.

Other related posts: