[coreboot] K8 RAMinit problematic code

ron minnich rminnich at gmail.com
Mon Sep 1 17:39:25 CEST 2008

On Mon, Sep 1, 2008 at 8:12 AM, Carl-Daniel Hailfinger
<c-d.hailfinger.devel.2006 at gmx.net> wrote:
> Hi,
> this patch is not to be applied, but an annotation from my tree.
> Index: northbridge/amd/k8/dqs.c
> ===================================================================
> --- northbridge/amd/k8/dqs.c    (Revision 853)
> +++ northbridge/amd/k8/dqs.c    (Arbeitskopie)
> @@ -2001,6 +2001,7 @@
>  static inline void train_ram_on_node(unsigned nodeid, unsigned coreid, struct sys_info *sysinfo, unsigned retcall)
>  {
>        if(coreid) return; // only do it on core0
> +#error This is broken beyond repair. We need to use the generic global variable infrastructure, especially if we relocate the global variables without telling anybody.

These fixes come in to play once single core is done.

We're going to write this code as always SMP safe, and I want to
remove the CONFIG_*_SMP conditionals. They make no sense for a bios.


More information about the coreboot mailing list