Re: Next round of console patches

Tom Eastep (eastep@loc1.tandem.com)
Mon, 31 Aug 1998 13:50:37 -0700


Richard Gooch wrote:

> That's quite bad. Can you do two separate tests, please? Firstly,
> insert a printk just before the call to mtrr_add() and send me a copy
> of your kernel logs?

Ok, maybe this will help. I moved the call to mtrr_add() down to after the
framebuffer console has been established:

request_region(0x3c0, 32, "vga+");

strcpy(fb_info.modename, "VESA VGA");
fb_info.changevar = NULL;
fb_info.node = -1;
fb_info.fbops = &vesafb_ops;
fb_info.disp=&disp;
fb_info.switch_con=&vesafb_switch;
fb_info.updatevar=&vesafb_update_var;
fb_info.blank=&vesafb_blank;
vesafb_set_disp(-1);

if (register_framebuffer(&fb_info)<0)
return;

#ifdef CONFIG_MTRR
mtrr_add((unsigned long)video_base, video_size, MTRR_TYPE_WRCOMB, 1);
#endif

printk("fb%d: %s frame buffer device\n",
GET_FB_IDX(fb_info.node), fb_info.modename);

With this code, I get:

Kernel panic:mtrr: timed out waiting for other CPUs.

A hard reset is required.

-Tom

--
Tom Eastep
COMPAQ Computer Corporation
Enterprise Computing Group
Tandem Division
tom.eastep@compaq.com

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.altern.org/andrebalsa/doc/lkml-faq.html