Patchwork Remove failover/fallback/normal decision from mainboards' romstage.c

2010-03-17 21:42:02

Hi,

attached patch removes the failover/fallback/normal decision from
romstage.c files of mainboards that did failover. It is the first part
of a larger effort.

This mechanism was used by newconfig style builds and is not useful
anymore since we dropped newconfig. In fact, in rare circumstances it
might lead to coreboot hanging early on boot (when normal is selected -
this should fix itself by three reboots, as coreboot reverts to fallback
then)

The code assumes that various places are visited twice on boot. That's
not true anymore for Kconfig builds, no matter if the boards is using
romstage as bootblock, or the tiny bootblock.
Removing these assumptions simplifies the code and should help the user
when trying to track the code flow.

The change looks big, but it's really a couple of simple and relatively
mechanical steps repeated for lots of boards:

1. In failover_process(), I removed the fallback/normal selection logic
and kept the remaining hardware init in. The if-clauses' conditions are
reverted to match.
Remove #if failover||fallback guard.

2. Change cache_as_ram_main() to first call failover_process, then
real_main unconditionally.

3. Move failover_process's code to the beginning of real_main, remove
failover_process and its call in cache_as_ram_main.

4. Remove cache_as_ram_main, rename real_main to cache_as_ram_main (same
arguments, so no problem with that)

As those changes are quite mechanical, but not easily automated, I'd
prefer some additional eyeballs that look if the end result of the
boards looks alike (they should be more similar to each other than before).

The patch is build tested and boot tested on amd/serengeti_cheetah in
SimNow.

After the changes, the entire hardware init is ran on cold boot, on warm
reset the hardware init that used to be in failover_process is skipped.

The affected boards do not use __fallback_image or __normal_image anymore.

The next step will a comparable change to the remaining boards.

After that, I'll aim for the removal of the remaining uses of
HAVE_FALLBACK_BOOT, USE_FALLBACK_IMAGE,
HAVE_FAILOVER_BOOT, USE_FAILOVER_IMAGE and their definition in Kconfig.

Signed-off-by: Patrick Georgi <patrick.georgi@coresystems.de>
Index: src/arch/i386/lib/failover_failover.lds (deleted)
===================================================================

Submitter	Patrick Georgi
Date	2010-03-17 21:42:02
Message ID	<4BA14CAA.8040807@georgi-clan.de>
Download	mbox \| patch
Permalink	/patch/1081/
State	Accepted
Headers	show Return-path: <coreboot-bounces@coreboot.org> Envelope-to: patchwork@coreboot.org Received: from localhost ([127.0.0.1] helo=ra.coresystems.de) by ra.coresystems.de with esmtp (Exim 4.71) (envelope-from <coreboot-bounces@coreboot.org>) id 1Ns10O-0006Ey-7j; Wed, 17 Mar 2010 22:42:20 +0100 Received: from dslb-088-068-242-178.pools.arcor-ip.net ([88.68.242.178] helo=t-stueck.streichelzoo.local) by ra.coresystems.de with esmtps (TLSv1:AES256-SHA:256) (Exim 4.71) (envelope-from <patrick@georgi-clan.de>) id 1Ns10D-000680-2W for coreboot@coreboot.org; Wed, 17 Mar 2010 22:42:17 +0100 Received: from [192.168.1.112] (dslb-088-068-242-178.pools.arcor-ip.net [88.68.242.178]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) (Authenticated sender: patrick) by t-stueck.streichelzoo.local (Postfix) with ESMTPSA id ECC4F215EF40B for <coreboot@coreboot.org>; Wed, 17 Mar 2010 22:42:07 +0100 (CET) Message-ID: <4BA14CAA.8040807@georgi-clan.de> Date: Wed, 17 Mar 2010 22:42:02 +0100 From: Patrick Georgi <patrick@georgi-clan.de> User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.1; de; rv:1.9.1.8) Gecko/20100227 Lightning/1.0b1 Thunderbird/3.0.3 MIME-Version: 1.0 To: coreboot@coreboot.org Content-Type: multipart/mixed; boundary="------------050809020500040404010304" X-Spam-Score: -0.5 (/) Subject: [coreboot] [PATCH]Remove failover/fallback/normal decision from mainboards' romstage.c X-BeenThere: coreboot@coreboot.org X-Mailman-Version: 2.1.13 Precedence: list List-Id: coreboot project mailing list <coreboot.coreboot.org> List-Unsubscribe: <http://www.coreboot.org/mailman/options/coreboot>, <mailto:coreboot-request@coreboot.org?subject=unsubscribe> List-Archive: <http://www.coreboot.org/pipermail/coreboot> List-Post: <mailto:coreboot@coreboot.org> List-Help: <mailto:coreboot-request@coreboot.org?subject=help> List-Subscribe: <http://www.coreboot.org/mailman/listinfo/coreboot>, <mailto:coreboot-request@coreboot.org?subject=subscribe> Sender: coreboot-bounces@coreboot.org Errors-To: coreboot-bounces@coreboot.org

Patchwork Remove failover/fallback/normal decision from mainboards' romstage.c

Comments

Patch