On Tue, Jul 28, 2009 at 09:17:05AM +0300, Avi Kivity wrote:
> On 07/28/2009 03:48 AM, Glauber Costa wrote:
>> On Mon, Jul 27, 2009 at 06:43:47PM +0300, Avi Kivity wrote:
>>
>>> On 07/22/2009 01:13 AM, Glauber Costa wrote:
>>>
>>>> qemu CPUState already provides "stop" and "stopped" states. And they
>>>> mean exactly that. There is no need for us to provide our own.
>>>>
>>>>
>>>>
>>> This patch (known as dd0e1c1a589 in qemu-kvm.git) breaks reboot. My
>>> test case is FC6 i386 -smp 2, running the reboot command in rc.local.
>>> In about 15 minutes qemu hangs hard. Please check what's gone wrong.
>>>
>> I found out that doing kill -38<your_pid> makes it run again, so we're
>> likely
>> hanging somewhere while holding qemu_mutex. The state of the process is "D",
>> so we're holding qemu_mutex, and then calling something that can block.
>>
>
> Sounds like we call a vcpu ioctl from the iothread (or from a different
> vcpu thread).
>
>> It's hard for me to believe that this patch introduced it. At best, it might
>> have
>> made it more likely. Also, I also verified that it sometimes takes a while
>> until
>> it happen for the first time. Are you sure this is the first patch that
>> makes it happen?
>>
>
> I haven't been able to reproduce it before this patch. Maybe this patch
> doesn't introduce it, only exposes it.
>
What are backtraces of all threads when it happens?
--
Gleb.
--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
