qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH v2] Add remove_boot_device_path() function for h


From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH v2] Add remove_boot_device_path() function for hot-unplug device
Date: Mon, 28 Apr 2014 14:48:46 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.4.0

Il 28/04/2014 14:43, Marcel Apfelbaum ha scritto:
> Yeah, but dangling pointers are bad anyway.  I think we should include
> Jun's patch.
I have nothing against it, but maybe you/someone answer this question
which I honestly don't have the answer for:
After this patch you can replace a boot-able disk at index x.
What happens if you do so and the user reboots and selects index x?
Until now, the index was 'in use', even if the disk was hot-unplugged.
What are the consequences now? If we can live with them, I am all for using 
this patch.

Firmware does not use indices, it uses paths. If boards do not update the paths on reset, firmware behavior will be unmodified before/after this patch.

With or without this patch, firmware will look for a device at the address where the "first" index x was. If there is no device there, either firmware will crash or it will ignore the entry. If there is a device, it will boot from that device no matter what the bootindex was on QEMU's command line.

Paolo



reply via email to

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