[Xorg-driver-geode] [PATCH] Allow compilation under 64 bit systems

[Gaetan Nadon]

On Wed, 2010-06-23 at 18:10 +0300, Martin-Éric Racine wrote:

> On Wed, Jun 23, 2010 at 5:57 PM, Gaetan Nadon <memsize at videotron.ca> wrote:
> > On Wed, 2010-06-23 at 14:35 +0800, Huang, FrankR wrote:
> >
> > 	Is that to say this solve method will let the geode compilation successful,
> > but in un-use status? Only for the whole build success of the X.org tree?
> >
> > Exactly.
> >
> > - We (as in X.Org) get bug reports as compilation fails on 64 bit. People
> > don't know so they ask.
> 
> For this, I prefer your initial idea of issuing a warning and
> gracefully skipping src/ on non-x86 architectures.
> 
> > - People other than pure Geode developers make contributions and many have
> > 64 bit systems. It could be in Autotools only, like I do,
> > it could be fixing code due to server API changes, like yesterday. Basically
> > anything that is safe to do without the need of loading the driver on the
> > real hardware.
> >
> > It is a simple matter of making it easy for others to contribute.
> 
> For that, wouldn't prepending -m32 to the compiler flags ensure that
> the driver can be built on x86_64 and still generate x86_32
> instructions always?
> 

I am open to an equivalent solution. I tried -m32 on durango.c:

/usr/include/gnu/stubs.h:7:27: error: gnu/stubs-32.h: No such file or
directory


The problem isn't generated code but the inline assembler in the source
code:

durango.c:203: Error: suffix or operands invalid for `push'

203:     " push %%ebx\n"      It should be %%rbx  for 64 bit.

There may be an easy way to tell the compiler to handle this situation.



> Martin-Éric
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.x.org/archives/xorg-driver-geode/attachments/20100623/235abc61/attachment.htm>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
URL: <http://lists.x.org/archives/xorg-driver-geode/attachments/20100623/235abc61/attachment.pgp>