Re: [Xen-devel] [XTF PATCH] build: disable PIE during linking if necessary

2017-02-01 Thread Andrew Cooper
On 01/02/17 16:27, Jan Beulich wrote: On 01.02.17 at 17:21, wrote: >> --- a/build/common.mk >> +++ b/build/common.mk >> @@ -50,6 +50,15 @@ obj-perarch := >> obj-perenv := >> include $(ROOT)/build/files.mk >> >> + >> +cc-option = $(shell if [ -z "`echo 'int p=1;' | $(CC) $(1) -S -o /dev/n

Re: [Xen-devel] [XTF PATCH] build: disable PIE during linking if necessary

2017-02-01 Thread Jan Beulich
>>> On 01.02.17 at 17:21, wrote: > --- a/build/common.mk > +++ b/build/common.mk > @@ -50,6 +50,15 @@ obj-perarch := > obj-perenv := > include $(ROOT)/build/files.mk > > + > +cc-option = $(shell if [ -z "`echo 'int p=1;' | $(CC) $(1) -S -o /dev/null > -x c - 2>&1`" ]; \ > +

Re: [Xen-devel] [XTF PATCH] build: disable PIE during linking if necessary

2017-02-01 Thread Wei Liu
On Wed, Feb 01, 2017 at 04:23:53PM +, Andrew Cooper wrote: > On 01/02/17 16:21, Wei Liu wrote: > > Starting from ee3e265688, $(CC) is used for linking. That means all > > default $(CC) flags coming from distro takes effect. > > > > On Debian Stretch, gcc contains -pie by default, which makes th

Re: [Xen-devel] [XTF PATCH] build: disable PIE during linking if necessary

2017-02-01 Thread Andrew Cooper
On 01/02/17 16:21, Wei Liu wrote: > Starting from ee3e265688, $(CC) is used for linking. That means all > default $(CC) flags coming from distro takes effect. > > On Debian Stretch, gcc contains -pie by default, which makes the final > object fail to link. We need to explicitly disable PIE when lin