qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] hw/core/null-machine: Print error message when


From: Philippe Mathieu-Daudé
Subject: Re: [Qemu-devel] [PATCH] hw/core/null-machine: Print error message when using the -kernel parameter
Date: Tue, 28 Feb 2017 16:41:30 -0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1

On 02/28/2017 10:17 AM, Marcel Apfelbaum wrote:
On 02/28/2017 10:52 AM, Thomas Huth wrote:
If the user currently tries to use the -kernel parameter, simply nothing
happens, and the user might get confused that there is nothing loaded
to memory, but also no error message has been issued. Since there is no
real generic way to load a kernel on all CPU types (but on some targets,
the generic loader can be used instead), issue an appropriate error
message here now to avoid the possible confusion.

Signed-off-by: Thomas Huth <address@hidden>
---
 hw/core/null-machine.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/hw/core/null-machine.c b/hw/core/null-machine.c
index 27c8369..864832d 100644
--- a/hw/core/null-machine.c
+++ b/hw/core/null-machine.c
@@ -40,6 +40,12 @@ static void machine_none_init(MachineState *mch)
         memory_region_allocate_system_memory(ram, NULL, "ram",
mch->ram_size);
         memory_region_add_subregion(get_system_memory(), 0, ram);
     }
+
+    if (mch->kernel_filename) {
+        error_report("The -kernel parameter is not supported "
+                     "(use the generic 'loader' device instead).");
+        exit(1);
+    }
 }

 static void machine_none_machine_init(MachineClass *mc)



Reviewed-by: Marcel Apfelbaum <address@hidden>

Reviewed-by: Philippe Mathieu-Daudé <address@hidden>


Thanks,
Marcel




reply via email to

[Prev in Thread] Current Thread [Next in Thread]