[lkml]   [2001]   [Sep]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
Patch in this message
SubjectRe: PnP BIOS + 2.4.9-ac16 = no boot

In -ac15 the PnP BIOS driver was changed so that the proc interface
got the "current" and "boot" configurations the right way 'round.
As a side effect the change made the initialization routine
build its device list from the "current" configuration as opposed
to the "boot" configuration (... which seemed like the right thing
to do at the time, heh heh ... hmmmmm).

You wrote:
> With the patch a third line is printed before the oops:
> PnP: PNP BIOS 13 devices detected (or something like that).

The patch I sent you reverted the build_devlist function to looking
at the "boot" configuration. Your output shows that the devlist
got built this time, which is good. However you still got an oops,
which is bad. My guess is that the oops now occurs in pnp_proc_init().
I attach a revised patch which cuts out proc support. Hopefully
this will allow your kernel to boot. This is just a hack; I'll
submit a proper fix to Alan later.

Alan (if you're listening): The patches I have already sent you are
still okay to go since they don't affect this issue. What I need
to do is submit a patch that, for Sony laptops, (1) causes pnpbios
to build its devlist from the "boot" configuration, and (2) omits
the "current" configuration from /proc/bus/pnp. I'll work on this

Stelian: Your report made me go back and look at something Alan
told me earlier about Vaio laptops. At the time I didn't fully
understand what he meant when he said:
> If you query the current device status on a Vaio and some other boxes
> using the 32bit API your computer dies horribly shortly afterwards.
I didn't realize that he meant the "current" configuration as
opposed to the "boot" configuration. Stupid of me.

P.S. The attached patch is only a temporary hack!--- pnp_bios.c_ORIG Thu Sep 27 15:21:46 2001
+++ pnp_bios.c_vaiofix Fri Sep 28 09:38:16 2001
@@ -614,11 +614,11 @@
pnp_bios_inst_struc = check;
- pnp_proc_init();
+ // pnp_proc_init();
if(kernel_thread(pnp_dock_thread, NULL, CLONE_FS | CLONE_FILES | CLONE_SIGNAL)>0)
unloading = 0;
@@ -845,11 +845,11 @@
for(i=0;i<0xff;i++) {
dev = kmalloc(sizeof (struct pci_dev), GFP_KERNEL);
if (!dev)

- if (pnp_bios_get_dev_node((u8 *)&num, (char )0 , node))
+ if (pnp_bios_get_dev_node((u8 *)&num, (char )1 , node))

 \ /
  Last update: 2005-03-22 13:03    [W:0.045 / U:1.068 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site