WARNING - OLD ARCHIVES

This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
   
 
 
Xen 
 
Home Products Support Community News
 
   
 

xen-devel

Re: [Xen-devel] Re: A race condition introduced by changeset 15175: Re-i

To: Mukesh <mukesh.rathor@xxxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxx>
Subject: Re: [Xen-devel] Re: A race condition introduced by changeset 15175: Re-init hypercall stubs page after HVM save/restore
From: Keir Fraser <keir.fraser@xxxxxxxxxxxxx>
Date: Tue, 21 Oct 2008 08:05:50 +0100
Cc: "Cui, Dexuan" <dexuan.cui@xxxxxxxxx>
Delivery-date: Tue, 21 Oct 2008 00:06:19 -0700
Envelope-to: www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to: <loom.20081021T033349-715@xxxxxxxxxxxxxx>
List-help: <mailto:xen-devel-request@lists.xensource.com?subject=help>
List-id: Xen developer discussion <xen-devel.lists.xensource.com>
List-post: <mailto:xen-devel@lists.xensource.com>
List-subscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe: <http://lists.xensource.com/mailman/listinfo/xen-devel>, <mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe>
Sender: xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index: AckzS3LlsXfrlJ8+Ed2eKwAWy6hiGQ==
Thread-topic: [Xen-devel] Re: A race condition introduced by changeset 15175: Re-init hypercall stubs page after HVM save/restore
User-agent: Microsoft-Entourage/11.4.0.080122
On 21/10/08 04:38, "Mukesh" <mukesh.rathor@xxxxxxxxxx> wrote:

>> Hi Dexuan,
>> 
>> Are you really sure that this is the problem? The suspend_lock was
>> introduced specifically to solve this problem. Note that the BSP takes this
>> lock before messing with the hypercall page.
>> 
>>  -- Keir
> 
> I'm also looking at this now (I'm on 3.1.4 BTW). I see both hang and panic. it
> appears I see the hang because the "master" vcpu is trying to catch other
> vcpus
> right at the cpu_relax so it can grab the lock in write mode. With many VCPUs
> it's just not happening..... Not sure i like the design of this very much...
> i'm
> gonna try to modify it a bit ....

Hmmm, yes, agreed. We need a completely different approach, or a lock which
doesn't allow in more readers when there is a waiting writer. I'll look into
it.

 -- Keir



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel