Return-Path: Delivered-To: apmail-incubator-harmony-dev-archive@www.apache.org Received: (qmail 74882 invoked from network); 10 Nov 2006 07:49:48 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 10 Nov 2006 07:49:48 -0000 Received: (qmail 48265 invoked by uid 500); 10 Nov 2006 07:49:57 -0000 Delivered-To: apmail-incubator-harmony-dev-archive@incubator.apache.org Received: (qmail 48226 invoked by uid 500); 10 Nov 2006 07:49:57 -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 48213 invoked by uid 99); 10 Nov 2006 07:49:57 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 09 Nov 2006 23:49:57 -0800 X-ASF-Spam-Status: No, hits=2.0 required=10.0 tests=HTML_MESSAGE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (herse.apache.org: domain of pavel.ozhdikhin@gmail.com designates 64.233.166.176 as permitted sender) Received: from [64.233.166.176] (HELO py-out-1112.google.com) (64.233.166.176) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 09 Nov 2006 23:49:43 -0800 Received: by py-out-1112.google.com with SMTP id h31so248741pyc for ; Thu, 09 Nov 2006 23:49:22 -0800 (PST) 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=Hw75INrgJn9xyqqNR/w7zAxyGGmAZsN1iqyBqwYYEoY+SKsN2gIkHnrP51gu3pDiV4LuHxU7nq0656qurzzQ6BeIZHB3saYmGrQWqYApzwoMif1uvueHCLEEFhUDKJbm+IRVwdJ3yv19AsbEiAkYB3iYxk1WCsh3ZCCBcgpmrbM= Received: by 10.35.65.17 with SMTP id s17mr2637242pyk.1163144962534; Thu, 09 Nov 2006 23:49:22 -0800 (PST) Received: by 10.35.123.9 with HTTP; Thu, 9 Nov 2006 23:49:22 -0800 (PST) Message-ID: <469bff730611092349g468da753v25f38bc1def07b3a@mail.gmail.com> Date: Fri, 10 Nov 2006 13:49:22 +0600 From: "Pavel Ozhdikhin" To: harmony-dev@incubator.apache.org Subject: Re: [drlvm] release vs. debug In-Reply-To: <51d555c70611092316v3fd40dd8o1def241fc75bebf5@mail.gmail.com> MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_50430_4849640.1163144962439" References: <2c9597b90611090803s2efaf1c8v473b48a4db3e9625@mail.gmail.com> <455353C4.2090507@pobox.com> <4553968C.2060508@pobox.com> <469bff730611092207v23ae08dpb62caa16ae5fadf5@mail.gmail.com> <51d555c70611092316v3fd40dd8o1def241fc75bebf5@mail.gmail.com> X-Virus-Checked: Checked by ClamAV on apache.org ------=_Part_50430_4849640.1163144962439 Content-Type: text/plain; charset=WINDOWS-1252; format=flowed Content-Transfer-Encoding: quoted-printable Content-Disposition: inline Sure, contributors should check debug or even both debug and release builds and comment about this in JIRA. I'm talking about committers, will they test debug build which takes 5 time= s more time? And does it mean they will be able to apply 5 times less patches= ? Actually I'm fine if the committers will check both debug and release build= s and I encourage them to check on all supported platforms. But I'm not a committer. :) Thanks, Pavel On 11/10/06, Rana Dasgupta wrote: > > I can understand that the contributor may not have access to multiple > platforms, but surely he can test using both debug and release builds :-) > What do we gain by asking the contributor to test less? > > Rana > > > On 11/9/06, Pavel Ozhdikhin wrote: > > > > +1 for debug testing before submitting a patch. > > > > But, for pre-commit testing we should be careful saying we'll test all > the > > patches in debug mode. Though it imprves quality of checks, debug build > is > > significantly slower then release, especially when running with > > interpreter > > or Jitrino.OPT. I ran "build test" on the DRLVM built in debug mode and > it > > takes about 5 times more time than release version on my laptop, The > times > > were as follows: > > > > "build test", Windows XP/ IA32: > > DRLVM release: 22 min 55 sec > > DRLVM debug: 99 min 23 sec > > > > So, may be the good choice will be to ask patch submitters to check > their > > patches on the debug build and, if the JIRA issue contains comments tha= t > > this check is passed, committers may test it on release build only. > > > > Thanks, > > Pavel > > > > On 11/10/06, Geir Magnusson Jr. wrote: > > > > > > Well, since the problem is repeatable.... :) > > > > > > > > > Gregory Shimansky wrote: > > > > Geir Magnusson Jr. wrote: > > > >> > > > >> > > > >> Alexei Zakharov wrote: > > > >>> Hi DRLVM fans, > > > >>> > > > >>> I encountered a rather strange problem while working on some clas= s > > > >>> library tests. At least two tests from the beans module hang (or > > > >>> crash) while running on DRLVM debug builds but work fine on DRLVM > > > >>> release builds. I thought that debug build is something about > adding > > > >>> debug symbols to VM binaries and this should not affect the > > > >>> functionality. Can someone shed a light on this? > > > >> > > > >> I would think it's just an assertion firing... > > > > > > > > Actually it is more than that. In debug mode TRACE statements are > > > > compiled and therefore produce executable code. There may also be > some > > > > bugs in compiler generating code in different modes (although this > > > > usually happens for release). > > > > > > > > I don't know why hanging happens, but the difference in generated > code > > > > is actually quite big. Also assertions or any crashes go to > > > > exceptions/signal handlers which may happen to loop infinitely, > there > > > > were bugs like this happening on the current version of sources, > look > > at > > > > HARMONY-2018 and HADMONY-2006. > > > > > > > >>> This is the first > > > >>> question. The second question - what we should do with such tests= . > > The > > > >>> tests pass on the downloadable HDK and JRE snapshots as well as o= n > > > >>> classlib + j9. What should be the commit criteria for DRLVM =96 i= .e. > > > >>> what is the *true* build? :) I think many people here currently > use > > > >>> snapshots to test their patches. > > > >> > > > >> debug :) don't sweep the problems under the rug... > > > > > > > > +1 > > > > > > > > > > > > > ------=_Part_50430_4849640.1163144962439--