On Monday 21 Nov 2011 17:04:18 Kenneth MacDonald wrote:
> OK, that did indeed work. I should really have guessed that since I had
> read the patch that renamed the file!
>
> Anyway, on machines it works I see (in red)
>
> Found command line "...my command line..."
> Using command line "...my command line..."
>
> And on machines it doesn't work on I see ...
>
> Found command line ""
> Using command line ""
OK, that shows that cmdline_phys _is_ being set to a non-zero value, but that
it seems to be pointing to an empty string.
Could you try the attached patch (with DEBUG=runtime) on both working and non-
working machines? It should show both the original address of the command
line and the content.
Michael
diff --git a/src/arch/i386/core/runtime.c b/src/arch/i386/core/runtime.c
index 09ba625..84b5e33 100644
--- a/src/arch/i386/core/runtime.c
+++ b/src/arch/i386/core/runtime.c
@@ -119,6 +119,7 @@ static int cmdline_init ( void ) {
DBGC ( colour, "RUNTIME found no command line\n" );
return 0;
}
+ DBG_HDA ( cmdline_phys, phys_to_virt ( cmdline_phys ), 0x100 );
cmdline_user = phys_to_user ( cmdline_phys );
len = ( strlen_user ( cmdline_user, 0 ) + 1 /* NUL */ );
_______________________________________________
ipxe-devel mailing list
[email protected]
https://lists.ipxe.org/mailman/listinfo/ipxe-devel
