Re: [PATCH] kvm-userspace: add ibm archs to unifdef header

2008-11-04 Thread Avi Kivity
Christian Borntraeger wrote: Am Montag, 3. November 2008 schrieb Ehrhardt Christian: +#ifdef __s390__ +#ifndef CONFIG_S390 +#define CONFIG_S390 1 +#endif +#endif + +#endif It seems that currently no header needs this, but better save than sorry. __s390__ is defined by gcc for s390 an

Re: [PATCH] kvm-userspace: add ibm archs to unifdef header

2008-11-04 Thread Christian Borntraeger
Am Montag, 3. November 2008 schrieb Ehrhardt Christian: > +#ifdef __s390__ > +#ifndef CONFIG_S390 > +#define CONFIG_S390 1 > +#endif > +#endif > + > +#endif > It seems that currently no header needs this, but better save than sorry. __s390__ is defined by gcc for s390 and s390x, CONFIG_S390 is s

Re: [PATCH] kvm-userspace: add ibm archs to unifdef header

2008-11-04 Thread Avi Kivity
Ehrhardt Christian wrote: This adds powerpc to the unifdef header avi introduced. I think s390 should work the same way and is needed as they have libkvm already. Christian/Carsten could you ack that after a quick test if it is working? I don't think it's really needed since we don't depend