* Matthew Garrett <mj...@srcf.ucam.org> wrote:
> On Thu, Feb 04, 2010 at 06:08:26PM +0100, Ingo Molnar wrote:
>
> > Well, once i applied the revert i got no more hangs or crashes today, in
> > lots of bootups. This is fully repeatable - if i re-apply that commit
> > with the config i sent the hang happens again.
>
> If you leave the commit applied, use that config and then enable Radeon KMS
> under staging, do you get the hang?
drivers/staging/ is generally an exception from all sorts of regression (and
ABI compatibility, etc.) policies so i dont auto-test it. (because simpy i
cannot do it in a sane way)
I do test the other thousands of regular mainline drivers though - more than
a hundred of which get activated for real on my various test-systems.
So if you move a driver (or a piece of a driver) from drivers/staging to
drivers/ in the -rc6 timeframe, and it causes trouble, i see it as a new
regression.
But you could claim that it's not a regression because 1) technically the
code got introduced in drivers/staging/, and staging drivers are not on the
regression list 2) the Kconfig value is default-off so it can only harm those
who got lured by a new Kconfig value popping up in -rc7 in a well working
driver they already have enabled.
So the moving of driver functionality from drivers/staging/ to drivers/ is a
grey area it appears. Wouldnt it have been better to do this in the next
merge window, as all other drivers do? It's not new hardware enablement
either, it's feature enablement for an existing driver.
Thanks,
Ingo
------------------------------------------------------------------------------
The Planet: dedicated and managed hosting, cloud storage, colocation
Stay online with enterprise data centers and the best network in the business
Choose flexible plans and management services without long-term contracts
Personal 24x7 support from experience hosting pros just a phone call away.
http://p.sf.net/sfu/theplanet-com
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel
