Re: Xen 4.8

[Roy Marples <roy%marples.name@localhost>]

On 17/04/2017 19:43, Roy Marples wrote:
> On 17/04/2017 16:21, Manuel Bouyer wrote:
>> On Mon, Apr 17, 2017 at 10:18:39AM +0100, Roy Marples wrote:
>>> On 30/03/2017 10:27, Manuel Bouyer wrote:
>>>> Hello,
>>>> I've just commited packages for xenkernel and xentools version 4.8.0.
>>>> I've not done extensive testing yet, so for production I'd still
>>>> recommend the 4.6 or 4.5 packages.
>>>
>>> gcc -I/usr/include -I/usr/pkg/include  -m32 -march=i686 -DBUILD_ID
>>> -fno-strict-aliasing -std=gnu99 -Wall -Wstrict-prototypes
>>> -Wno-error=ignored-attributes -no-integrated-as
>>> -Wno-error=tautological-compare -Wno-error=null-dereference
>>> -Wno-error=empty-body
>>>  -Wno-error=self-assign -Wno-error=sometimes-uninitialized
>>> -Wno-error=unused-function -Wno-error=format
>>> -Wno-error=unused-const-variable -Wno-error=gnu-designator
>>> -Wno-error=uninitialized -Wno-error=parentheses-equality
>>> -Wno-error=enum-conversion -Wdecla
>>> ration-after-statement -Wno-unused-local-typedefs   -O2
>>> -fomit-frame-pointer
>>> -D__XEN_INTERFACE_VERSION__=__XEN_LATEST_INTERFACE_VERSION__ -MMD -MF
>>> .tpm_drivers.o.d   -DNDEBUG -Werror -nopie -fno-stack-protector
>>> -fno-exceptions -fno-builtin -msoft-float -
>>> I/usr/obj/pkgsrc/sysutils/xentools48/work/xen-4.8.0/tools/firmware/rombios/32bit/tcgbios/../../../../../tools/include
>>>
>>> -I.. -I../.. -I../../../../libacpi  -c -o tpm_drivers.o tpm_drivers.c
>>> tcgbios.c:1519:34: error: taking address of packed member 'u' of
>>> class or
>>> structure 'pushad_regs_t' may result in an unaligned pointer value
>>> [-Werror,-Waddress-of-packed-member]
>>>                                               &regs->u.r32.edx);
>>>                                                ^~~~~~~~~~~~~~~
>>> 1 error generated.
>>>
>>> This is building on a very recent NetBSD-current w clang, but pkgsrc
>>> pulls
>>> in gcc for some reason when building xen.
>>
>> I'm not seeing this on netbsd-7.
>> Is it using the gcc from base here, or the one from the gcc48 package
>> (as I would expect it to do) ?
> 
> It's not using gcc from base as I've renamed that gcc.old, so it must be
> gcc48 from pkgsrc.

I also removed the gcc dependency and the error is 100% identical, so
maybe it's still using clang?

Roy