From: Dan Nicolaescu <dann@ics.uci.edu>
To: Ulrich Mueller <ulm@gentoo.org>
Cc: Chong Yidong <cyd@stupidchicken.com>,
Giorgos Keramidas <keramida@freebsd.org>,
emacs-devel@gnu.org
Subject: Re: configure.in support for FreeBSD ia64/sparc64/powerpc
Date: Fri, 19 Dec 2008 09:16:00 -0800 (PST) [thread overview]
Message-ID: <200812191716.mBJHG0oC016436@mothra.ics.uci.edu> (raw)
In-Reply-To: <18763.42713.941380.795066@a1ihome1.kph.uni-mainz.de> (Ulrich Mueller's message of "Fri, 19 Dec 2008 14:51:21 +0100")
Ulrich Mueller <ulm@gentoo.org> writes:
> >>>>> On Mon, 6 Nov 2006, Giorgos Keramidas wrote:
>
> >On 2006-11-03 22:28, Chong Yidong <cyd@stupidchicken.com> wrote:
> >>Giorgos Keramidas <keramida@ceid.upatras.gr> writes:
> >>> The following change makes Emacs buildable on FreeBSD running on
> >>> sparc64, ia64 and powerpc too. It has been tested for some time
> >>> as a local patch in the editors/emacs-devel port of FreeBSD, so it
> >>> would be nice if it became part of Emacs 22. We won't have to
> >>> maintain local diffs for this part then:
> >>>
> >>> --- a/src/alloc.c Fri Oct 27 15:45:51 2006 +0000
> >>> +++ b/src/alloc.c Sat Oct 28 01:49:45 2006 +0300
> >>> @@ -4560,7 +4560,11 @@ mark_stack ()
> >>> needed on ia64 too. See mach_dep.c, where it also says inline
> >>> assembler doesn't work with relevant proprietary compilers.
> >>> */
> >>> #ifdef sparc
> >>> +#ifdef __sparc64__
> >>> + asm ("flushw");
> >>> +#else
> >>> asm ("ta 3");
> >>> +#endif
> >>> #endif
> >>
> >> Is this change correct for non-FreeBSD sparc64 systems too?
> >> (I don't know anything about this part of the code.)
>
> > Sorry for taking so long to follow up about this. I tried looking
> > at the sparc v9 specification for hints about why this change is
> > needed on 64-bit SPARC platforms, but I couldn't come up with
> > anything in a reasonable amount of time. I am not that proficient
> > with SPARC64 assembly, but I will ask our FreeBSD/sparc64 people for
> > details.
>
> > I'll follow up again when I get a reply from the people who know more
> > about sparc64 assembly :)
>
> Following up to this message from two years ago. ;-)
>
> Gentoo also maintains the above as local patch, both for FreeBSD and
> GNU/Linux. Our Sparc team says that the patch is sane, and without it
> Emacs cannot be built on Sparc/FreeBSD. See Gentoo Bug 159584 for
> further details: <http://bugs.gentoo.org/159584>
>
> Below is a slightly updated patch that will apply to the currect CVS
> trunk. It would be nice if it could make it into Emacs 23.
>
> Ulrich
>
> --- emacs-orig/src/alloc.c
> +++ emacs/src/alloc.c
> @@ -4573,7 +4573,11 @@
> needed on ia64 too. See mach_dep.c, where it also says inline
> assembler doesn't work with relevant proprietary compilers. */
> #ifdef __sparc__
> +#ifdef __sparc64__
> + asm ("flushw");
> +#else
Was this tested on GNU/Linux and Solaris (and all other sparc64
platforms)? Otherwise better make that #ifdef depend on FreeBSD.
> asm ("ta 3");
> +#endif
> #endif
>
> /* Save registers that we need to see on the stack. We need to see
next prev parent reply other threads:[~2008-12-19 17:16 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-11-03 21:45 configure.in support for FreeBSD ia64/sparc64/powerpc Giorgos Keramidas
2006-11-04 3:28 ` Chong Yidong
2006-11-06 14:10 ` Giorgos Keramidas
2008-12-19 13:51 ` Ulrich Mueller
2008-12-19 17:16 ` Dan Nicolaescu [this message]
2008-12-19 18:10 ` Ulrich Mueller
2008-12-19 19:51 ` Giorgos Keramidas
2009-01-22 11:10 ` Ulrich Mueller
2009-01-23 3:31 ` Giorgos Keramidas
2009-01-23 4:39 ` Chong Yidong
2009-01-23 5:42 ` Giorgos Keramidas
2009-02-05 6:37 ` Ulrich Mueller
2009-02-05 13:55 ` Chong Yidong
2009-02-05 14:17 ` Giorgos Keramidas
2009-02-05 14:20 ` Dan Nicolaescu
2009-02-05 15:30 ` Chong Yidong
2009-02-05 15:31 ` Dan Nicolaescu
2009-02-05 16:01 ` Ulrich Mueller
2008-12-21 8:24 ` Ulrich Mueller
2008-12-29 19:42 ` Ulrich Mueller
2008-12-29 19:54 ` Dan Nicolaescu
2008-12-29 20:20 ` Ulrich Mueller
2008-12-29 20:39 ` Dan Nicolaescu
2008-12-29 21:19 ` Ulrich Mueller
2008-12-29 21:41 ` Giorgos Keramidas
2008-12-29 22:06 ` Dan Nicolaescu
2008-12-29 22:36 ` Giorgos Keramidas
2008-12-29 23:03 ` Ulrich Mueller
2008-12-30 3:47 ` Dan Nicolaescu
2008-12-30 12:44 ` Ulrich Mueller
2008-12-30 20:55 ` Giorgos Keramidas
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=200812191716.mBJHG0oC016436@mothra.ics.uci.edu \
--to=dann@ics.uci.edu \
--cc=cyd@stupidchicken.com \
--cc=emacs-devel@gnu.org \
--cc=keramida@freebsd.org \
--cc=ulm@gentoo.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.