Re: [Xen-devel] [Patch] continue_hypercall_on_cpu rework using t

To:	Jan Beulich <JBeulich@xxxxxxxxxx>
Subject:	Re: [Xen-devel] [Patch] continue_hypercall_on_cpu rework using tasklets
From:	Juergen Gross <juergen.gross@xxxxxxxxxxxxxx>
Date:	Tue, 13 Apr 2010 15:49:52 +0200
Cc:	"xen-devel@xxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxx>
Delivery-date:	Tue, 13 Apr 2010 06:51:00 -0700
Dkim-signature:	v=1; a=rsa-sha256; c=simple/simple; d=ts.fujitsu.com; i=juergen.gross@xxxxxxxxxxxxxx; q=dns/txt; s=s1536b; t=1271166586; x=1302702586; h=message-id:date:from:mime-version:to:cc:subject: references:in-reply-to:content-transfer-encoding; z=Message-ID:=20<4BC47680.60700@xxxxxxxxxxxxxx>\|Date:=20Tu e,=2013=20Apr=202010=2015:49:52=20+0200\|From:=20Juergen =20Gross=20<juergen.gross@xxxxxxxxxxxxxx>\|MIME-Version: =201.0\|To:=20Jan=20Beulich=20<JBeulich@xxxxxxxxxx>\|CC:=20 "xen-devel@xxxxxxxxxxxxxxxxxxx"=20<xen-devel@xxxxxxxxxxxx rce.com>\|Subject:=20Re:=20[Xen-devel]=20[Patch]=20continu e_hypercall_on_cpu=20rework=20using=09tasklets \|References:=20<4BC46F58.3030805@xxxxxxxxxxxxxx>=20<4BC49 08B020000780003A008@xxxxxxxxxxxxxxxxxx>\|In-Reply-To:=20<4 BC4908B020000780003A008@xxxxxxxxxxxxxxxxxx> \|Content-Transfer-Encoding:=207bit; bh=dqqxeo0D5//4vxtTAHOM9h0NntiVepUnXkZ5oenBS14=; b=SDQCELIrSwMuwtePl5SqP50OzPlQpraycEkUZvSKQ5kXPqOFV4ALgPOn azySfuUcuH1Qo63NMtxvh8oJmJ4f8c1FwBx2aiZC0nOTOaBwNJAV9f+8K 0X+cBbfnjnvgC0k7EPrQUzSQGFeqd8xd1mdvpItq9BNJ9vvsgxME3b4Fq gYU2PQCOZLzax2+2+uB1agCk5Atcp5ld93TswSLlt3HOlBWiMqLJ283j9 7OhCmrolZdsANam+QRbALvqvibGR+;
Domainkey-signature:	s=s1536a; d=ts.fujitsu.com; c=nofws; q=dns; h=X-SBRSScore:X-IronPort-AV:Received:X-IronPort-AV: Received:Received:Message-ID:Date:From:Organization: User-Agent:MIME-Version:To:CC:Subject:References: In-Reply-To:X-Enigmail-Version:Content-Type: Content-Transfer-Encoding; b=UQrgkOpYM0PR0mGpIfChByu0OHyYglalaOM0+fnkwd0AEe0iOsMg79OC kcBC6fSF66ColFnDi1Yn7698/3tX1Qf9aJtqOD4XEHUCs0jPsXzbKcGmt A7yU5dSt1cWyfhjkbs0N9jx7SKHga+tPIDoynVLTYw8HJempW353xvbFR T/VUMxLFkm+18aSvxk6dcdCmVY8cqhuau2YX+dftHwCmlXAOER271L7JL TUmPC8mMcW+exdGiDaVWK5uo3s3v/;
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxxx
In-reply-to:	<4BC4908B020000780003A008@xxxxxxxxxxxxxxxxxx>
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>
Organization:	Fujitsu Technology Solutions
References:	<4BC46F58.3030805@xxxxxxxxxxxxxx> <4BC4908B020000780003A008@xxxxxxxxxxxxxxxxxx>
Sender:	xen-devel-bounces@xxxxxxxxxxxxxxxxxxx
User-agent:	Mozilla-Thunderbird 2.0.0.22 (X11/20090707)

Jan Beulich wrote:
>>>> Juergen Gross <juergen.gross@xxxxxxxxxxxxxx> 13.04.10 15:19 >>>
>> cpupools are using continue_hypercall_on_cpu, but this function is defined 
>> for
>> x86 only, not for ia64. I see 2 possible solutions:
>>
>> 1. make continue_hypercall_on_cpu available on ia64, too
>> 2. make cpupools a pure x86 feature
> 
> Is there anything meaningful in the new code that's really x86-specific
> (i.e. can't the whole code chunk be moved to xen/common/)?

My main concern is the difference in the handling of schedule_tail. I just
can't tell how to change the ia64 variant to make it work correctly.

The other x86-specific part is the setting of the return register, but this
is rather easy doable by using an architecture specific macro.


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@xxxxxxxxxxxxxx
Domagkstr. 28                           Internet: ts.fujitsu.com
D-80807 Muenchen                 Company details: ts.fujitsu.com/imprint.html

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

WARNING - OLD ARCHIVES

xen-devel

Re: [Xen-devel] [Patch] continue_hypercall_on_cpu rework using tasklets