Re: [PATCH] ACPI PM: Restore the 2.6.24 suspend ordering

From: Felix Möller
Date: Tue Apr 01 2008 - 10:45:37 EST


Hi,
For the reasons outlined above, the change of the suspend ordering
should be reverted, which is done by the patch below.
But this will break those few nvidia-based systems, no?

this may have been a good idea in -rc1 days, but we are in -rc7
now... and the patch is slightly big.
It's quite obvious, though.
Yes, but breaking systems between -rc7 and final is _very_ unnice.
Breaking systems between 2.6.24 and 2.6.25 is even worse, which is why
I've posted this patch.

IOW, we tried to fix systems that were broken with 2.6.24, but it didn't work,
because our "fix" broke systems that were OK with 2.6.24. Solution: revert
the "fix" and go back to the design board. That's all we can do so late in
the release cycle, IMO.

Well, I agree that regression from 2.6.24 is worse, but it is
_slightly_ worse... -rcs are really expected to improve...

...plus it no longer looks like macbook regression is caused by _PTS
ordering?
I am the reporter from the original Novell Bug: https://bugzilla.novell.com/show_bug.cgi?id=374217

I just tried current git head (two hours ago) with the patch (the one from the beginning of this thread) from Rafael and without it. With the patch my MacBook does suspend without it does not.

HTH
Felix Möller
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/