Return-Path: Delivered-To: apmail-incubator-harmony-dev-archive@www.apache.org Received: (qmail 84767 invoked from network); 17 Oct 2006 19:13:12 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 17 Oct 2006 19:13:12 -0000 Received: (qmail 4741 invoked by uid 500); 17 Oct 2006 19:13:08 -0000 Delivered-To: apmail-incubator-harmony-dev-archive@incubator.apache.org Received: (qmail 4627 invoked by uid 500); 17 Oct 2006 19:13:07 -0000 Mailing-List: contact harmony-dev-help@incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: harmony-dev@incubator.apache.org Delivered-To: mailing list harmony-dev@incubator.apache.org Received: (qmail 4567 invoked by uid 99); 17 Oct 2006 19:13:07 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 17 Oct 2006 12:13:07 -0700 X-ASF-Spam-Status: No, hits=2.5 required=10.0 tests=DNS_FROM_RFC_ABUSE,HTML_MESSAGE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (asf.osuosl.org: domain of pavel.n.afremov@gmail.com designates 72.14.204.236 as permitted sender) Received: from [72.14.204.236] (HELO qb-out-0506.google.com) (72.14.204.236) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 17 Oct 2006 12:13:04 -0700 Received: by qb-out-0506.google.com with SMTP id a33so492466qbd for ; Tue, 17 Oct 2006 12:12:43 -0700 (PDT) DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:references; b=gw2ootZ6UAhya4a3tJg8DjlyJAduoNem3JZkvjd8tTRKpmrJS9XWFIIJzE69k9ZnmCqsuhpCgtYToiB4RCD7YJtPDEbklL58g4dLJcO0khxYkHg5t59SedRSwvbM324KNequbxv3xJEvNBxnWZSOaYVi8jUciXBC8gzSPjI1iBY= Received: by 10.48.220.15 with SMTP id s15mr2175927nfg; Tue, 17 Oct 2006 12:12:42 -0700 (PDT) Received: by 10.49.27.15 with HTTP; Tue, 17 Oct 2006 12:12:42 -0700 (PDT) Message-ID: <783bf8b0610171212n41d1f47cm37fd77670039c9ec@mail.gmail.com> Date: Tue, 17 Oct 2006 23:12:42 +0400 From: "Pavel Afremov" To: harmony-dev@incubator.apache.org Subject: Re: [drlvm] Calling native methods from Java code: implementation details In-Reply-To: <51d555c70610171014gd507b14hce9f2c80b6096c04@mail.gmail.com> MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_156119_5531036.1161112362654" References: <51d555c70610171014gd507b14hce9f2c80b6096c04@mail.gmail.com> X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N ------=_Part_156119_5531036.1161112362654 Content-Type: text/plain; charset=WINDOWS-1252; format=flowed Content-Transfer-Encoding: quoted-printable Content-Disposition: inline Mikhail proposal is OK. But I think that following solution is better 1) The special interface MagicNativeCall should be created. The interface has only one method CallAddress getCallAddress(String methodName)= . All native methods of classes which implement this interface should be called by JIT via special "magic" way. 2) If JIT find call of native method, it tries to find call address using getCallAddress method. If null is returned by the method, usual call of JNI method should be processed. In other case magic native call can be included instead JNI call. Result returned by getCallAddress can and shoul= d be cashed by JIT for performance reasons. 3) I propose for the firs time use fastcall calling convention. In future new additional method can be added MagicNativeCall interface. It can be named as getCallType for example. If it returns JNI =96 jit should use stdcall, if FastWay =96 fast call. Also we can add special Magic type of calling convention. In this type of call one can select Register and stack slot for every parameter and return value. It can looks like: ReturnValue methodName(InParameterList ) or void methodName(InParameterList, OutParameterList). ReturValue is an void or OutValue, InParameterList is an empty or separate= d by comma "," list of the InValue, OutParameterList is an empty or separated by comma "," list of the OutValue. The OutValue can by one of OutRegisterParametr## or OutStackParametr##, where ## number of register or call in the stack, which are possible on the current platform. The InValue can be one of InRegisterParametr## or InStackParametr##. Register## and Stack## is the short aliases for InRegisterParametr## and InStackParametr## correspondently. Result is the short alias for OutRegisterParametr00 (EAX on ia32). Generics can be used for type safety. Thanks. Pavel Afremov. On 10/17/06, Rana Dasgupta wrote: > > Mikhail, > All this looks reasonable to me. At least to go ahead. Regarding 2A, > could the jit cache this information for re-use? > Alternatively, the JIT can do all this at startup...by going thru the > contract class of fastpath java methods and querying the component manage= r > for the native addresses of the slow counterparts. > > Thanks, > Rana > > > On 10/17/06, Mikhail Fursov wrote: > > > > All, > > Finally we have almost everything finished to post "helper's fast-path > > inlining" framework into JIRA. > > > > The issue is left is how to call native slow-path versions of the from > > Java > > code. We already discussed some of the aspects, but there was no > detailed > > discussion with a final agreement what API we will use. > > > > Let's make a final decision so I can add the code into JIRA. > > > > I'm sending my vision of the solution. Correct me or advise another > > approach. > > > > Step 1: How JIT will know if a Java method must be replaced with a > native > > helper call. > > Solution 1.A: Every Java method that must be replaced with native call > > must > > be a static method and must have special "Native" runtime method > > annotation. > > > > Step 2: How JIT will get the address of the native method to generate a > > direct call? > > Solution 2.A: Every Java method that must be replaced with native call > > must > > have special "Component" annotation. > > JIT will use this annotation to ask Component Manager to access to the > > specified component by it's name and call "void* getAddress(const char* > > methodName)" component's method to get the address of a helper. That is > > every component that have native calls from Java must provide this > > interface. > > > > Step 3: How JIT will know which calling convention to use? > > Solution 3.A: Use method's annotation again. > > > > > > These were all of the problems with native calls. Now we need to agree > if > > the solution proposed is OK or find another one. > > Please note, that this is just the first implementation. We can extend > it > > with more features in the future. > > > > > > > > -- > > Mikhail Fursov > > > > > > ------=_Part_156119_5531036.1161112362654--