From mboxrd@z Thu Jan 1 00:00:00 1970 From: Juergen Gross Subject: Re: [Patch] continue_hypercall_on_cpu rework using tasklets Date: Thu, 15 Apr 2010 09:57:59 +0200 Message-ID: <4BC6C707.4080709@ts.fujitsu.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Keir Fraser Cc: "xen-devel@lists.xensource.com" List-Id: xen-devel@lists.xenproject.org Keir Fraser wrote: > On 15/04/2010 07:31, "Juergen Gross" wrote: > >>>> Okay, attached is the modified patch again. >>> I cleaned up some more and applied as xen-unstable:21165 and >>> xen-unstable:21166. >> Thanks. Unfortunately, your modifications are not working. Microcode update >> hangs. My version worked without problems. > > What revision did you test? I put in some fixes as c/s 21173. My highest c/s was 21167. c/s 21173 is hanging, too (sorry for the delay, but I had to remove my cpupool stuff due to the scheduler changes for credit2). Is a call of sync_vcpu_execstate() fro a tasklet really allowed? I don't think the ASSERTs in __sync_lazy_execstate() are all fulfilled in this case. Juergen -- Juergen Gross Principal Developer Operating Systems TSP ES&S SWE OS6 Telephone: +49 (0) 89 3222 2967 Fujitsu Technology Solutions e-mail: juergen.gross@ts.fujitsu.com Domagkstr. 28 Internet: ts.fujitsu.com D-80807 Muenchen Company details: ts.fujitsu.com/imprint.html