Andrea Arcangeli wrote:
>
> On Wed, Oct 25, 2000 at 02:46:19PM -0400, Doug Ledford wrote:
> > @@ -214,8 +215,8 @@
> > movb ready,%al # First CPU if 0
> > orb %al,%al
> > jz 4f # First CPU skip this stuff
> > - movl %cr4,%eax # Turn on 4Mb pages
> > - orl $16,%eax
> > + movl %cr4,%eax # Set CR4 register to match first CPU
> > + orl X86_MMU_CR4,%eax
> > movl %eax,%cr4
> > movl %cr3,%eax # Intel specification clarification says
> > movl %eax,%cr3 # to do this. Maybe it makes a difference.
>
> I had a fast look and I don't see why you do that change. Infact
> I think this whole section could be removed:
>
> movl %cr4,%eax # Turn on 4Mb pages
> orl $16,%eax
> movl %eax,%cr4
>
> We just reload the cr4 at the very early part of the boot stage.
I thought the same thing while reviewing this code, but I didn't have time to
test the change so I didn't want to do so just yet. I was being safe...
--Doug Ledford <dledford@redhat.com> http://people.redhat.com/dledford Please check my web site for aic7xxx updates/answers before e-mailing me about problems - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Tue Oct 31 2000 - 21:00:17 EST