Return-Path: Delivered-To: apmail-harmony-dev-archive@www.apache.org Received: (qmail 34799 invoked from network); 20 Apr 2008 07:01:40 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 20 Apr 2008 07:01:40 -0000 Received: (qmail 59870 invoked by uid 500); 20 Apr 2008 07:01:40 -0000 Delivered-To: apmail-harmony-dev-archive@harmony.apache.org Received: (qmail 59827 invoked by uid 500); 20 Apr 2008 07:01:40 -0000 Mailing-List: contact dev-help@harmony.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@harmony.apache.org Delivered-To: mailing list dev@harmony.apache.org Received: (qmail 59818 invoked by uid 99); 20 Apr 2008 07:01:40 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 20 Apr 2008 00:01:39 -0700 X-ASF-Spam-Status: No, hits=-0.0 required=10.0 tests=SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of alexei.fedotov@gmail.com designates 209.85.146.181 as permitted sender) Received: from [209.85.146.181] (HELO wa-out-1112.google.com) (209.85.146.181) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 20 Apr 2008 07:00:54 +0000 Received: by wa-out-1112.google.com with SMTP id k22so2273770waf.18 for ; Sun, 20 Apr 2008 00:01:07 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from:to:subject:mime-version:content-type:content-transfer-encoding:content-disposition; bh=OniDSKUKAoXQ4JAq/+YmqaCppm/ICJANuOOfYmwj5gk=; b=I6hxFOmab4b+H6YnxSOIzyEzrS8MoIyVFIT33OxE7a3Odqywqr4Ds/6Mwnapa9Y+fHiIn1kSd7jDUaecdX/h1YwEDaS8bC8UNfvEtcwbwdCTo5QuC1Y3hbfTB1+vX6K+Ae+qIvIc4xYo5CetkSLFx3usMCiosqZjNURENzAxK/g= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:mime-version:content-type:content-transfer-encoding:content-disposition; b=g/f44y5VXPSQXKlmEhaVPHO6/vB7ar9/5CXBwp/ZtTSKvr+tLWKRPt4PBuEz0LY3CgmUhrtPspBLcwspRfGu7Os+Gbd0ybBu7nMvDiYuis4Bqw6Ta1iTGoRyC/14qiVsGM0GL4QkaIEGeBSwlv1F9+OFZMhnP6YmI4dSWhELuzU= Received: by 10.115.93.16 with SMTP id v16mr4472636wal.185.1208674867598; Sun, 20 Apr 2008 00:01:07 -0700 (PDT) Received: by 10.114.92.17 with HTTP; Sun, 20 Apr 2008 00:01:07 -0700 (PDT) Message-ID: Date: Sun, 20 Apr 2008 11:01:07 +0400 From: "Alexei Fedotov" To: dev@harmony.apache.org Subject: [drlvm][verifier-ext] need a test Was: Releasing scheduling MIME-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Content-Disposition: inline X-Virus-Checked: Checked by ClamAV on apache.org Hello Vasily, I'm trying to understand how to made the simple verifier extension test from your static instrumentator. Or, in other words, I'm trying to understand the verifier extension interface and how to reproduce your bug [1]. I downloaded the probekit sources from dev.eclipse.org and tried to understand how do they use a verifier extension: $ cvs -d :pserver:anonymous@dev.eclipse.org:/cvsroot/tptp co platform/org.eclipse.hyades.probekit $ grep -rI recompute_stackmaptable platform/ I cannot find anything. I started to think that your patch to the bug report [2] may contain stack map re-computation example and checked your attachments. But it seems that the recompute_stackmaptable function is missed there as well. Could you please give more directions? For example, could you point to the part of your work related to the stack map re-computation? I also wonder if recompute_stackmaptable is a part of public interfaces, or I should look for something else. Thank you in advance, Alexei [1] https://issues.apache.org/jira/browse/HARMONY-5764 [2] https://bugs.eclipse.org/bugs/show_bug.cgi?id=148629 On Fri, Apr 18, 2008 at 7:31 PM, Vasily Levchenko wrote: > Hello folks, > Have you got any updates about commitment of > H-5750. > > > About testing. > We've discussed it with some folks, but I don't know how it complex for > testing system of Harmony. > Actually the functionality we need is used for recalculating stack maps > after instrumentation. There is a subproject of TPTP called Probekit that > injects probes into compiled code. But for re-calculation requires valid JNI > pointer (you can find some details in > https://bugs.eclipse.org/bugs/show_bug.cgi?id=148629). So minimize test it > possible re-use static instrumentator with introduced in the same bugzilla > java6 support but for static instrumentation. > > Is it ok with you? > > On Fri, Apr 18, 2008 at 10:34 AM, Alexei Fedotov > wrote: > > > > > Nathan, thanks for a question! > > > > > Is VS6 needed to appropriately test this issue [1]? [...] are there > > specific tests that could be run to get a general > > > assurance of the passivity? > > > > I was asking Vasily to prepare at least one stand-alone test to be > > included (by me) in a Harmony test base. While the whole TPTP requires > > VC6, I believe I will be able to recompile the only test with a newer > > compiler. > > > > With best regards, Alexei > > > > > > > > On Fri, Apr 18, 2008 at 5:27 AM, Nathan Beyer wrote: > > > Is VS6 needed to appropriately test this issue [1]? I'm not intimate > > with > > > the verifier; are there specific tests that could be run to get a > > general > > > assurance of the passivity? > > > > > > -Nathan > > > > > > [1] https://issues.apache.org/jira/browse/HARMONY-5750 > > > > > > On Thu, Apr 17, 2008 at 2:22 PM, Vasily Levchenko < > > > > > > vasily.v.levchenko@gmail.com> wrote: > > > > > > > > > > Greetings, > > > > We've finally established source base and launched our test, > > demonstrating > > > > stability and reliable of verifier code. would you mind to initiate > > with > > > > releasing milestone HDK-M5.5_Eclipse_TPTP? > > > > > > > > As I've mentioned earlier but wasn't able point to JIRA ( > > > > https://issues.apache.org/jira/browse/HARMONY-5750) we're extremely > > > > interested this patch to be included. > > > > > > > > On Mon, Apr 14, 2008 at 9:15 AM, Stepan Mishura < > > stepan.mishura@gmail.com< > > https://mail.google.com/mail?view=cm&tf=0&to=stepan.mishura@gmail.com> > > > > > > > > > wrote: > > > > > > > > > On 4/14/08, Vasily Levchenko > https://mail.google.com/mail?view=cm&tf=0&to=vasily.v.levchenko@gmail.com > > >> > > > > > > > wrote: > > > > > > On Mon, Apr 14, 2008 at 8:32 AM, Stepan Mishura < > > > > > stepan.mishura@gmail.com< > > https://mail.google.com/mail?view=cm&tf=0&to=stepan.mishura@gmail.com> > > > > > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > Hi folks, > > > > > > > > > > > > > > As I understood the thread it is doable to make interim release > > > > > > > targeted to assist inclusion of Harmony verifier to the nearest > > > > > > > Eclipse TPTP release. Let me share my understanding of the > > request > > > > and > > > > > > > a possible way to resolve it (please correct me if I'm wrong): > > The > > > > > > > Eclipse team needs an 'official' (i.e. published on the > > web-site as > > > > > > > milestone build) Harmony release. The Eclipse team only > > interesting > > > > in > > > > > > > changes in verifier since M5 so the main criteria for the > > interim > > > > > > > release is no regressions in verifier functionality (i.e. I > > assume > > > > > > > that not critical regressions are acceptable for interim > > release. I > > > > > > > believe that is important for having a shorten > > freeze/test/release > > > > > > > period for the interim release) > > > > > > > > > > > > > > So I think we may consider: > > > > > > > - making sure that all artifacts required are in place (i.e. > > > > committed > > > > > > > to the trunk) > > > > > > > - declaring short code freeze > > > > > > > - running testing cycle to see if there are any issues with > > verifier > > > > > > > and overall code. (BTW, are there any know issues with > > verifier > > > > that > > > > > > > needs to be fixed?) > > > > > > > > > > > > > > Having said that I thought that we are testing up to 6 > > snapshots per > > > > > > > week so we may pick up any interim snapshot that has everything > > > > > > > required and shows good testing results, make it 'official' - > > i.e. > > > > > > > publish it ... with proper label - M5.5_Eclipse or something > > else to > > > > > > > avoid confusions and to state clearly that the release it > > targeted > > > > to > > > > > > > the Eclipse TPTP release. > > > > > > > > > > > > > > Does it make sense and works for all parties? > > > > > > > > > > > > > > The only issue that still unclear for me is ABI requirements: > > has > > > > the > > > > > > > Harmony team build/test the code to satisfy ABI or you can do > > it? > > > > > > > (Alexey Petrenko asked this before but I don't see any answer) > > > > > > > > > > > > > > > > > > I suppose we can do it, but it should be in the released package > > too. > > > > If > > > > > > we're going to share building of the module how it will looks > > like? > > > > > > 1. you'll give us revision > > > > > > 2. and we'll return the compiled libraries > > > > > > > > > > > > or some other way? > > > > > > > > > > > > > > > > OK as I see from your answer - the Harmony team has to build > > binaries > > > > > that satisfy ABI (because we publish binaries that are created only > > by > > > > > Harmony committers.) > > > > > > > > > > -Stepan. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > --vvl > > > > > > > > > > > > > > > -- > > With best regards, > > Alexei > > > > > > -- > --vvl > -- With best regards, Alexei