From owner-freebsd-java@freebsd.org Sun Oct 7 16:41:28 2018 Return-Path: Delivered-To: freebsd-java@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 66D4D10C6195 for ; Sun, 7 Oct 2018 16:41:28 +0000 (UTC) (envelope-from glewis@eyesbeyond.com) Received: from mailman.ysv.freebsd.org (mailman.ysv.freebsd.org [IPv6:2001:1900:2254:206a::50:5]) by mx1.freebsd.org (Postfix) with ESMTP id DF1838ACEB for ; Sun, 7 Oct 2018 16:41:27 +0000 (UTC) (envelope-from glewis@eyesbeyond.com) Received: by mailman.ysv.freebsd.org (Postfix) id 9EF2E10C6194; Sun, 7 Oct 2018 16:41:27 +0000 (UTC) Delivered-To: java@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 7BB2F10C6193 for ; Sun, 7 Oct 2018 16:41:27 +0000 (UTC) (envelope-from glewis@eyesbeyond.com) Received: from misty.eyesbeyond.com (gerbercreations.com [71.39.140.16]) by mx1.freebsd.org (Postfix) with ESMTP id E90A78ACE7; Sun, 7 Oct 2018 16:41:24 +0000 (UTC) (envelope-from glewis@eyesbeyond.com) Received: from misty.eyesbeyond.com (localhost.eyesbeyond.com [127.0.0.1]) by misty.eyesbeyond.com (8.15.2/8.15.2) with ESMTP id w97GcpT8022031; Sun, 7 Oct 2018 09:38:51 -0700 (PDT) (envelope-from glewis@eyesbeyond.com) Received: (from glewis@localhost) by misty.eyesbeyond.com (8.15.2/8.15.2/Submit) id w97GcoB5022030; Sun, 7 Oct 2018 09:38:50 -0700 (PDT) (envelope-from glewis@eyesbeyond.com) X-Authentication-Warning: misty.eyesbeyond.com: glewis set sender to glewis@eyesbeyond.com using -f Date: Sun, 7 Oct 2018 09:38:50 -0700 From: Greg Lewis To: Dimitry Andric Cc: java@freebsd.org Subject: Re: Fwd: Request for Approval: Backport of JDK-8205965: SIGSEGV on write to NativeCallStack::EMPTY_STACK Message-ID: <20181007163850.GA22016@misty.eyesbeyond.com> References: <58ed90b3-ee3d-9c8b-70a3-b7f3a7398040@loongson.cn> <19EDB12C-A2AE-4B49-BEE4-5215138985A3@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <19EDB12C-A2AE-4B49-BEE4-5215138985A3@FreeBSD.org> User-Agent: Mutt/1.10.1 (2018-07-13) X-BeenThere: freebsd-java@freebsd.org X-Mailman-Version: 2.1.27 Precedence: list List-Id: Porting Java to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 07 Oct 2018 16:41:28 -0000 No objections to using that instead. It looks cleaner overall. If that gets committed to the official repo then we'll merge it in during the next merge anyway. On Wed, Sep 19, 2018 at 10:13:22AM +0200, Dimitry Andric wrote: > FYI. I'm testing this patch (at http://cr.openjdk.java.net/~zgu/8205965_8u/webrev.00/ ) locally, which reverts the fix we committed for https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=225054 in https://svnweb.freebsd.org/ports?view=revision&revision=459368, and solves the issue in another way. > > -Dimitry > > > Begin forwarded message: > > > > From: Leslie Zhai > > Subject: Re: Request for Approval: Backport of JDK-8205965: SIGSEGV on write to NativeCallStack::EMPTY_STACK > > Date: 19 September 2018 at 03:22:48 CEST > > To: Zhengyu Gu , Seán Coffey > > Cc: jdk8u-dev , Dimitry Andric > > > > Hi Zhengyu, > > > > Thanks for your sponsor! > > > > I will run jtreg towards: > > > > * jdk8u-dev, base on 8722:16b9bbfaa450, applied with the patch, compiled with clang-6 and clang-8-dev > > > > * jdk8u-dev, base on 8722:16b9bbfaa450, compiled with gcc-6.4.3 and gcc-9-dev > > > > Then run jtdiff on Linux X86. > > > > Hi Dimitry, > > > > Please verify the patch on FreeBSD compiled with clang-6, thanks a lot! > > > > Thanks, > > > > Leslie Zhai > > > > > > 在 2018年09月19日 07:06, Zhengyu Gu 写道: > >> Hi Leslie, > >> > >> The backport is approved. > >> > >> Please send me *committed* patch [1] after proper tests, I will sponsor it for you. > >> > >> Thanks, > >> > >> -Zhengyu > >> > >> [1] http://openjdk.java.net/sponsor/ Section 3 > >> > >> You should use commit message from original patch and add yourself as contributor, ex. > >> > >> 8205965: SIGSEGV on write to NativeCallStack::EMPTY_STACK > >> Summary: Made EMPTY_STACK non-const, so it will not be placed in read-only BSS section. > >> Reviewed-by: stuefe, martin > >> Contributed-by: zhaixiang@loongson.cn > >> > >> > >> > >> On 09/18/2018 06:28 PM, Seán Coffey wrote: > >>> Approved for jdk8u-dev. > >>> > >>> regards, > >>> Sean. > >>> > >>> > >>> On 18/09/2018 07:48, Leslie Zhai wrote: > >>>> Hi Zhengyu, > >>>> > >>>> I would like to backport the fix for: > >>>> > >>>> https://bugs.openjdk.java.net/browse/JDK-8205965 > >>>> > >>>> To OpenJDK 8 updates dev: > >>>> > >>>> http://hg.openjdk.java.net/jdk8u/jdk8u-dev > >>>> > >>>> The fix is mostly the same as the version that was committed in 12, here is the patch: > >>>> > >>>> http://cr.openjdk.java.net/~zgu/8205965_8u/webrev.00/ > >>>> > >>>> The patch was reviewed, thanks for your review! Here is the relevant thread: > >>>> > >>>> http://mail.openjdk.java.net/pipermail/jdk8u-dev/2018-September/007860.html > >>>> > >>>> A patch by Loongson! Please sponsor it, thanks a lot! > >>>> > >>> > > > > > -- Greg Lewis Email : glewis@eyesbeyond.com Eyes Beyond Web : http://www.eyesbeyond.com Information Technology FreeBSD : glewis@FreeBSD.org