[coreboot] SeaBIOS and boot of SATA raid card

[Kevin O'Connor]

Hi Ruik!

On Wed, Dec 17, 2008 at 12:55:54AM +0100, Rudolf Marek wrote:
> Hello,
> 
> I had some time today. I have PCI SATA RAID (fake raid) with Silicon Image chip
> on it. The option rom runs fine but SeaBIOS complains that EBDA was relocated. I
> fixed that in the patch. Also I fixed the stuff so now I get:

Yeah, SeaBIOS needs to handle a relocatable ebda.

The simple way to do this (reload the ebda_seg from the bda) on every
access is going to lead to terrible code generation.  It may be
simpler to implement the Post Memory Manager (PMM) interface - in
theory, an option rom shouldn't relocate the ebda if it can allocate
memory via PMM.  Another possibility would be to assign a segment to
the current ebda location (eg, fs/gs) at every entry to the C code.

[...]

Hrmm.  I'm not sure what the following part of your patch does:

> @@ -341,13 +343,29 @@
>              callrom(rom, OPTION_ROM_INITVECTOR, 0);
>              continue;
>          }
> -        // PnP rom.
> -        if (pnp->bev)
> -            // Can boot system - add to IPL list.
> -            add_ipl(rom, pnp);
> -        else if (pnp->bcv)
> -            // Has BCV - run it now.
> -            callrom(rom, pnp->bcv, 0);
> +	/* rebuild the PNP block, init migh have change that */
> +	pnp = get_pnp_rom(rom);
> +
> +	/* for further details check BIOS Boot Specification */
> +	while (pnp) {
> +			dprintf(1, "$PnP at %p\n",pnp);
> +
> +		if ((pnp->bev) && (!pnp->bcv))
> +		        add_ipl(rom, pnp, IPL_TYPE_BEV);
> +
> +		if ((!pnp->bev) && (pnp->bcv)) {
> +			dprintf(1, "Running BCV %x\n",pnp->bcv);
> +        	    callrom(rom, pnp->bcv, 0);
> +		        add_ipl(rom, pnp, pnp->type_lo);
> +		}
> +
> +		if (pnp->nextoffset)
> +			pnp = (struct pnp_data *) (((u8 *) pnp) + pnp->nextoffset);
> +		else
> +			pnp = NULL;
> +	}
> +
> +
>      }
>  }
>  

-Kevin