xen-ia64-devel

[Top] [All Lists]

RE: [Xen-ia64-devel] pv_ops: move binary patching to later after CPU ini

from [Dong, Eddie]

[Permanent Link][Original]

To:	"Isaku Yamahata" <yamahata@xxxxxxxxxxxxx>
Subject:	RE: [Xen-ia64-devel] pv_ops: move binary patching to later after CPU initialization
From:	"Dong, Eddie" <eddie.dong@xxxxxxxxx>
Date:	Thu, 27 Mar 2008 19:44:14 +0800
Cc:	xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
Delivery-date:	Thu, 27 Mar 2008 04:45:20 -0700
Envelope-to:	www-data@xxxxxxxxxxxxxxxxxx
In-reply-to:	<20080327105050.GC30410%yamahata@xxxxxxxxxxxxx>
List-help:	<mailto:xen-ia64-devel-request@lists.xensource.com?subject=help>
List-id:	Discussion of the ia64 port of Xen <xen-ia64-devel.lists.xensource.com>
List-post:	<mailto:xen-ia64-devel@lists.xensource.com>
List-subscribe:	<http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=subscribe>
List-unsubscribe:	<http://lists.xensource.com/cgi-bin/mailman/listinfo/xen-ia64-devel>, <mailto:xen-ia64-devel-request@lists.xensource.com?subject=unsubscribe>
References:	<10EA09EFD8728347A513008B6B0DA77A02F76F3F@xxxxxxxxxxxxxxxxxxxxxxxxxxxx> <20080327105050.GC30410%yamahata@xxxxxxxxxxxxx>
Sender:	xen-ia64-devel-bounces@xxxxxxxxxxxxxxxxxxx
Thread-index:	AciP+LuU11ceqopQTGi6f1HH9JjaogABqxog
Thread-topic:	[Xen-ia64-devel] pv_ops: move binary patching to later after CPU initialization

Isaku Yamahata wrote:
> I guess you just followed x86 way, but delaying until check_bug()
> is too late for IA64 case because of at least ia64_get_cpuid().

No. Binary patching is just optimization, while pv_ops hook
is installed at very beginning.

> At this moment I'm not sure how late binary patching can
> be delayed, though.
> Presumably it is necessary to revise boot protocol.

Any time as if SMC sequence is considered.
Putting together with original Linux patching code will make things
simple.

> 
> Renaming xen_paravirt_patch() to xen_patch() seems reasonable,
> so I applied only the renaming part.
> 

I strongly suggest you check in entry.S patch first and then everything
will
be very simple.

And your above concern can be solved too.

Thanks, eddie

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

[More with this subject...]

<Prev in Thread]	Current Thread	[Next in Thread>
[Xen-ia64-devel] pv_ops: move binary patching to later after CPU initialization, Dong, Eddie Re: [Xen-ia64-devel] pv_ops: move binary patching to later after CPU initialization, Isaku Yamahata RE: [Xen-ia64-devel] pv_ops: move binary patching to later after CPU initialization, Dong, Eddie <=

Previous by Date:	Re: [Xen-ia64-devel] pv_ops: move binary patching to later after CPU initialization, Isaku Yamahata
Next by Date:	[Xen-ia64-devel] [Patch][0/2]implement dump_execution_state() on ia64, Akio Takebe
Previous by Thread:	Re: [Xen-ia64-devel] pv_ops: move binary patching to later after CPU initialization, Isaku Yamahata
Next by Thread:	[Xen-ia64-devel] Which brash do you use?, Akio Takebe
Indexes:	[Date] [Thread] [Top] [All Lists]