[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#39118: closed (3.0.0 JIT segfaults on 64-bit Cygwin)

From: GNU bug Tracking System
Subject: bug#39118: closed (3.0.0 JIT segfaults on 64-bit Cygwin)
Date: Mon, 17 Feb 2020 21:07:01 +0000

Your message dated Mon, 17 Feb 2020 22:05:45 +0100
with message-id <address@hidden>
and subject line Re: Segfault while building on 64-bit Cygwin
has caused the debbugs.gnu.org bug report #39118,
regarding 3.0.0 JIT segfaults on 64-bit Cygwin
to be marked as done.

(If you believe you have received this mail in error, please contact

39118: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=39118
GNU Bug Tracking System
Contact address@hidden with problems
--- Begin Message --- Subject: Re: GNU Guile 2.9.9 Released [beta] Date: Mon, 13 Jan 2020 12:26:27 -0500
Guile 2.9.9, like .8 and .7, does not build on Cygwin (64 bit).  Configure runs without error, but make crashes with this (truncated to just the tail):

Making all in bootstrap
make[2]: Entering directory '/cygdrive/c/Users/rr828893/Downloads/guile-2.9.9/bootstrap'
  BOOTSTRAP GUILEC ice-9/eval.go
  BOOTSTRAP GUILEC ice-9/psyntax-pp.go
  BOOTSTRAP GUILEC language/cps/intmap.go
  BOOTSTRAP GUILEC language/cps/intset.go
  BOOTSTRAP GUILEC language/cps/graphs.go
  BOOTSTRAP GUILEC ice-9/vlist.go
  BOOTSTRAP GUILEC srfi/srfi-1.go
/bin/sh: line 6:  4294 Segmentation fault      (core dumped) GUILE_AUTO_COMPILE=0 ../meta/build-env guild compile --target="x86_64-unknown-cygwin" -O1 -Oresolve-primitives -L "/home/rr828893/Downloads/guile-2.9.9/module" -L "/home/rr828893/Downloads/guile-2.9.9/guile-readline" -o "srfi/srfi-1.go" "../module/srfi/srfi-1.scm"
make[2]: *** [Makefile:1930: srfi/srfi-1.go] Error 139
make[2]: Leaving directory '/cygdrive/c/Users/rr828893/Downloads/guile-2.9.9/bootstrap'
make[1]: *** [Makefile:1849: all-recursive] Error 1
make[1]: Leaving directory '/cygdrive/c/Users/rr828893/Downloads/guile-2.9.9'
make: *** [Makefile:1735: all] Error 2

All previous problems (which were easy to work around) have gone away in this release, which is progress, but it doesn't get me past Guile 2.2.

John Cowan          http://vrici.lojban.org/~cowan        address@hidden
Your worships will perhaps be thinking that it is an easy thing
to blow up a dog? [Or] to write a book?
    --Don Quixote, Introduction

--- End Message ---
--- Begin Message --- Subject: Re: Segfault while building on 64-bit Cygwin Date: Mon, 17 Feb 2020 22:05:45 +0100 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux)
Aah, you all are amazing -- thank you!!  Applied and merged.



On Mon 17 Feb 2020 20:27, Charles Stanhope <address@hidden> writes:

> On 2/16/20, Charles Stanhope <address@hidden> wrote:
>> On 2/16/20, Mike Gran <address@hidden> wrote:
>>> I can confirm that Charles's patch, plus another one line patch
>>> to define CPU_SETSIZE, is enough to get Guile 3.0.x to build and run
>>> on my box.  All tests pass except strptime in French, and the absence
>>> of crypt.  This is a 64-bit build.
>> Mike, thanks for going further with the Guile build. The CPU_SETSIZE
>> issue was what was hanging me up from compiling before Andy's comment
>> got me to look at lightening. I assumed I had some configuration,
>> package, or compiler issue. Good to know there's a simple fix.
>> Just a further warning to anyone watching, that patch I posted is a
>> real hack job just to test my theory of the cause of the segfault. I
>> would expect it to fail when you have fewer than four arguments in a
>> JITed function call. I wouldn't try doing much else with that Guile
>> build besides run the tests. :)
> I had a little bit more time to look into the lightening
> implementation last night. I've attached a patch that is less horrible
> and more correct than my previous one. It reserves the stack space
> regardless of the number of parameters and appears to work. But I'm
> new to the lightening code base, so I'm not convinced it is the
> correct solution. It's just the solution I was left with after my time
> ran out. I wanted to post this patch as a replacement to the prior one
> in case people did want to do more testing with Guile 3.0 on Cygwin
> x64.
> With that, I will let more experienced people come up with the
> appropriate solution. Happy hacking, everybody!
> --
> Charles
> diff --git a/lightening/x86.c b/lightening/x86.c
> index 965191a..bdd26e1 100644
> --- a/lightening/x86.c
> +++ b/lightening/x86.c
> @@ -328,6 +328,10 @@ reset_abi_arg_iterator(struct abi_arg_iterator *iter, 
> size_t argc,
>    memset(iter, 0, sizeof *iter);
>    iter->argc = argc;
>    iter->args = args;
> +#if __CYGWIN__ && __X64
> +  // Reserve slots on the stack for 4 register parameters (8 bytes each).
> +  iter->stack_size = 32;
> +#endif
>  }
>  static void

--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]