Re: [coreboot] [v2] r4323 - trunk/coreboot-v2/src/arch/i386/lib

[Luc Verhaegen]

On Fri, May 29, 2009 at 03:08:27PM +0200, [e-mail] wrote:
> Author: stepan
> Date: 2009-05-29 15:08:27 +0200 (Fri, 29 May 2009)
> New Revision: 4323
>
> Modified:
>    trunk/coreboot-v2/src/arch/i386/lib/c_start.S
> Log:
> drop most of the crappy vm86 code and replace it with a rewritten
> version that has all assembler in a .S file and all C code in a .c
> file. Also, remove requirement to move around between GDTs.
>
> This version includes the suggestions from Peter to clean up CR0 manipulation
> and to guard critical code paths by cli/sti. Tested and working on my hardware.
>
> Signed-off-by: Stefan Reinauer <[e-mail]>
> Acked-by: Peter Stuge <[e-mail]>
>
> Modified: trunk/coreboot-v2/src/arch/i386/lib/c_start.S
> ===================================================================
> --- trunk/coreboot-v2/src/arch/i386/lib/c_start.S	2009-05-29 03:44:47 UTC (rev 4322)
> +++ trunk/coreboot-v2/src/arch/i386/lib/c_start.S	2009-05-29 13:08:27 UTC (rev 4323)
> @@ -253,7 +253,7 @@
>
> /* This is the gdt for GCC part of coreboot.
>    * It is different from the gdt in ROMCC/ASM part of coreboot
> -	 * which is defined in entry32.inc */
> +	 * which is defined in entry32.inc */ /* BUT WHY?? */
>  gdt:
>   /* selgdt 0, unused */
>   .word	0x0000, 0x0000		/* dummy */
> @@ -275,6 +275,13 @@
>   .word	0x0000, 0x0000		/* dummy */
>   .byte	0x00, 0x00, 0x00, 0x00
>
> +	/* selgdt 0x28 16-bit 64k code at 0x00000000 */
> +	.word   0xffff, 0x0000
> +	.byte   0, 0x9a, 0, 0
> +
> +	/* selgdt 0x30 16-bit 64k data at 0x00000000 */
> +	.word   0xffff, 0x0000
> +	.byte   0, 0x92, 0, 0
>  gdt_end:
>
> idtarg:

Is this correct?

Aren't some files supposed to be added and some supposed to be removed 
with this commit? At least i seem to remember something like that from 
the patch that was sent in a week or so ago, and the log message also 
points in this direction.

Luc Verhaegen.

[v2] r4323 - trunk/coreboot-v2/src/arch/i386/lib

[Stefan Reinauer]

On 02.06.2009 14:05 Uhr, Luc Verhaegen wrote:
> Is this correct?
>
> Aren't some files supposed to be added and some supposed to be removed 
> with this commit? At least i seem to remember something like that from 
> the patch that was sent in a week or so ago, and the log message also 
> points in this direction.
>
> Luc Verhaegen.

check the following two commits. the added / removed stuff is
svn:external'ed in from another repository. (what a pain!)




-- 
coresystems GmbH • Brahmsstr. 16 • D-79104 Freiburg i. Br.
      Tel.: [phone] • Fax: [phone]
Email: [e-mail]  • http://www.coresystems.de/
Registergericht: Amtsgericht Freiburg • HRB 7656
Geschäftsführer: Stefan Reinauer • Ust-IdNr.: DE245674866

[v2] r4323 - trunk/coreboot-v2/src/arch/i386/lib

[Luc Verhaegen]

On Tue, Jun 02, 2009 at 04:37:54PM +0200, Stefan Reinauer wrote:
> On 02.06.2009 14:05 Uhr, Luc Verhaegen wrote:
> > Is this correct?
> >
> > Aren't some files supposed to be added and some supposed to be removed 
> > with this commit? At least i seem to remember something like that from 
> > the patch that was sent in a week or so ago, and the log message also 
> > points in this direction.
> >
> > Luc Verhaegen.
>
> check the following two commits. the added / removed stuff is
> svn:external'ed in from another repository. (what a pain!)

Ah, that's why it didn't show in the git-log of this tree.

Sorry for the noise,

Luc Verhaegen.