Issue 6634003: This change fixes the platform_KernelErrorCodePaths test so that it

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Issue 6634003: This change fixes the platform_KernelErrorCodePaths test so that it (Closed)

Created:
9 years, 9 months ago by thutt

Modified:
9 years, 5 months ago

Reviewers:
vadimb, sjg, Mandeep Singh Baines, diandersAtChromium

Base URL:
ssh://git@gitrw.chromium.org:9222/autotest.git@master

Visibility:
Public.

Description

This change fixes the platform_KernelErrorCodePaths test so that it does not hang on Tegra2 systems. On those systems, there is no 'preserved memory' support, so the sysctl settings to cause an immediate reboot are not set. The result now is that the test will fail (rather than hang) on Tegra2 boards. It completes successfully on Mario machines. Change-Id: Id5cf4ff3f0e08bb29bed8dcbdac717f93c9b3a9d BUG= TEST= Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=b23a645

Patch Set 1 #

Created: 9 years, 9 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+13 lines, -1 line)			Patch
	M	server/site_tests/platform_KernelErrorPaths/platform_KernelErrorPaths.py	View		2 chunks	+13 lines, -1 line	0 comments	Download

Messages

Total messages: 2 (0 generated)

Expand Messages | Collapse Messages

sjg

9 years, 9 months ago (2011-03-14 18:55:32 UTC) #2

On 2011/03/11 21:30:04, thutt wrote:
> Anyone have a chance to look at this trivial change?

Sorry I am so late. This is fine as a temporary but it should probably go away
with preserved finally lands. Perhaps a TODO(thutt) to remove it later?

Expand Messages | Collapse Messages