samsung: pm-check: Cleanup chunk exclude code to use suspend_volatile
Recent changes mark memory that may change across suspend/resume
as suspend_volatile. We can now use this in pm-check, so do so.
That means we can remove some extra globals from printk.c.
This change also cleans up handling of sleep_save_sp:
* Don't exclude *sleep_save_sp. That was a physical address and is
also already handled by current_thread_info().
* Don't exclude sleep_save_sp. Instead: we zero it out before out
code runs. This is a transitory value and zeroing it is fine. Now
we won't exclude chunks of memory stored near sleep_save_sp.
...and a few other minor fixes:
* Skips regions in s3c_pm_makecheck() too.
* Avoids goto in s3c_pm_runcheck() loop.
BUG=chrome-os-partner:15914
TEST=suspend_stress_test
Change-Id: I5eadc7631c3f31f91138170c5e37430da7e3520b
Signed-off-by: Doug Anderson <dianders@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/37359
Reviewed-by: Jon Kliegman <kliegs@chromium.org>