From: storm@cua.dk (Kim F. Storm)
Cc: emacs-devel@gnu.org
Subject: Re: x86_64 problems/fix (was: alloc.c problem when GC_MARK_STACK is GC_USE_GCPROS_AS_BEFORE)
Date: 12 Jul 2004 11:33:14 +0200 [thread overview]
Message-ID: <m3d631po2t.fsf@kfs-l.imdomain.dk> (raw)
In-Reply-To: <m37jtgtyi3.fsf@ecube.site>
Barry Fishman <barry_fishman@att.net> writes:
> I was unable to make any change to alloc.c that would fix the GC_MARK_STACK
> problem without later crashes in the build while compiling the lisp
> files.
Does the following patch work (if you undo the patch you suggested below)?
*** alloc.c 23 Jun 2004 10:30:52 +0200 1.346
--- alloc.c 12 Jul 2004 11:29:19 +0200
***************
*** 5033,5038 ****
--- 5033,5039 ----
break;
case Lisp_Misc_Save_Value:
+ #if GC_MARK_STACK
{
register struct Lisp_Save_Value *ptr = XSAVE_VALUE (obj);
/* If DOGC is set, POINTER is the address of a memory
***************
*** 5045,5050 ****
--- 5046,5052 ----
mark_maybe_object (*p);
}
}
+ #endif
break;
case Lisp_Misc_Overlay:
>
> However, I suspect the attached change to src/s/gnu-linux.h is
> approprate. This produces a working emacs (for me) on a amd64
> gnu-linux system (SuSE 9.1), although it sidesteps the problem in
> alloc.c.
This is a good change -- have you been using that for some time
now without problems? If so, I will commit your patch.
>
> Index: gnu-linux.h
> ===================================================================
> RCS file: /cvsroot/emacs/emacs/src/s/gnu-linux.h,v
> retrieving revision 1.89
> diff -c -r1.89 gnu-linux.h
> *** gnu-linux.h 1 Sep 2003 15:45:58 -0000 1.89
> --- gnu-linux.h 6 Jul 2004 23:49:36 -0000
> ***************
> *** 355,361 ****
>
> #if defined __i386__ || defined __sparc__ || defined __mc68000__ \
> || defined __alpha__ || defined __mips__ || defined __s390__ \
> ! || defined __arm__ || defined __powerpc__
> #define GC_SETJMP_WORKS 1
> #define GC_MARK_STACK GC_MAKE_GCPROS_NOOPS
> #ifdef __mc68000__
> --- 355,361 ----
>
> #if defined __i386__ || defined __sparc__ || defined __mc68000__ \
> || defined __alpha__ || defined __mips__ || defined __s390__ \
> ! || defined __arm__ || defined __powerpc__ || defined __amd64__
> #define GC_SETJMP_WORKS 1
> #define GC_MARK_STACK GC_MAKE_GCPROS_NOOPS
> #ifdef __mc68000__
>
> --
> Barry Fishman
> _______________________________________________
> Emacs-devel mailing list
> Emacs-devel@gnu.org
> http://lists.gnu.org/mailman/listinfo/emacs-devel
--
Kim F. Storm <storm@cua.dk> http://www.cua.dk
next prev parent reply other threads:[~2004-07-12 9:33 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-07-06 22:37 alloc.c problem when GC_MARK_STACK is GC_USE_GCPROS_AS_BEFORE Barry Fishman
2004-07-07 1:08 ` x86_64 problems/fix (was: alloc.c problem when GC_MARK_STACK is GC_USE_GCPROS_AS_BEFORE) Barry Fishman
2004-07-12 9:33 ` Kim F. Storm [this message]
2004-07-12 14:24 ` Barry Fishman
2004-07-21 21:26 ` Kim F. Storm
2004-07-21 22:34 ` Kim F. Storm
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=m3d631po2t.fsf@kfs-l.imdomain.dk \
--to=storm@cua.dk \
--cc=emacs-devel@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.