lkml.org 
[lkml]   [2006]   [Nov]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 20/20] x86_64: Move CPU verification code to common file
Date

> However, this test should probably be pushed earlier, into setup.S,
> where executing a BIOS-clean reboot is much easier.

It's already in there in fact. It wasn't originally, until we discovered
that there is no way to output a message in head.S when you're
using vesafb. The only way to give a visible error is to do it
before the video switched.

The old test was kept, although it's redundant.

This means Vivek/Eric added it now to the SMP trampoline and ACPI
S3 resume too, but there it is technically redundant too.

But you have to spin, otherwise the user cannot see what is wrong
(and that is much more important than your obscure possibility
of automatic fallback -- inserting the wrong CD is pretty common)

Finding panic=.. would require writing a command line parser in 16bit assembly.
I have my doubts that's a good use of anyone's time.

-Andi (who wonders why he wastes so much time writing about this thing)
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-11-18 09:09    [W:0.081 / U:0.528 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site