[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Xen-devel] Need help with fixing the Xen waitqueue feature
- To: Jan Beulich <JBeulich@xxxxxxxx>
- From: Keir Fraser <keir@xxxxxxx>
- Date: Thu, 24 Nov 2011 09:51:27 +0000
- Cc: Olaf Hering <olaf@xxxxxxxxx>, xen-devel@xxxxxxxxxxxxxxxxxxx
- Delivery-date: Thu, 24 Nov 2011 09:52:36 +0000
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=sender:user-agent:date:subject:from:to:cc:message-id:thread-topic :thread-index:in-reply-to:mime-version:content-type :content-transfer-encoding; bh=VNHm9VXHiciaAbqCPmqbLGdhIzTCslLXvGjZCxlEUdI=; b=B8ODNAVTVLkpn4WJrkATnG6m79NZbYGb+sBC5p+driRjv48sNUirN5cRcMC5iJBB4q K/FMZuHsbmXNLWbJoMiykK10sEwPPDEgsugVCccH5y8GxMcM4J18ZaqT1JnSSBiXwkR5 J+051nc1rjSvHU3WENOITrPWOYiuNqdBIvd4E=
- List-id: Xen developer discussion <xen-devel.lists.xensource.com>
- Thread-index: AcyqjqIvGnKiwPFDFUeERlqmWp9uSA==
- Thread-topic: [Xen-devel] Need help with fixing the Xen waitqueue feature
On 24/11/2011 09:15, "Jan Beulich" <JBeulich@xxxxxxxx> wrote:
>>
>> Attached three patches for you to try. They apply in sequence.
>> 00: A fixed version of "domain_crash on stack overflow"
>> 01: Reorders prepare_to_wait so that the vcpu will always be on the
>> waitqueue on exit (even if it has just been woken).
>> 02: Ensures the vcpu wakes up on the same cpu that it slept on.
>
> Didn't we (long ago) settle on not permitting new calls to
> domain_crash_synchronous()? Is it really impossible to just
> domain_crash() in any of the instances these add?
It's safe because you must be in a context that is safe to preempt. That's a
pre-condition for using a waitqueue. It's not safe to use domain_crash()
because the caller of wait_event() may not handle the exceptional return.
-- Keir
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel