[Qemu-devel] Issues building seabios

[Qemu-devel] Issues building seabios

[Avi Kivity]

Trying to debug the cdrom issue, I see

   Compiling whole program out/ccode32.o
src/util.c: In function ‘__end_thread’:
src/util.c:183: internal compiler error: in simplify_subreg, at 
simplify-rtx.c:5055

(with F12's gcc (GCC) 4.4.2 20091027 (Red Hat 4.4.2-7))

The issue seems to be with the pos variable; removing everything except 
the declaration + initialization retains the error, converting the 
variable to static removes it.  Presumably you've seen many of these?

With that out of the way, I get:

   Linking (no relocs) out/rom32.o
   Linking (no relocs) out/rom16.o
   Linking out/rom.o
`cdemu_drive' referenced in section `.text32' of out/rom32.o: defined in 
discarded section `.discard.var16.src/cdrom.c.107' of out/rom32.o
`cdemu_drive' referenced in section `.text32' of out/rom32.o: defined in 
discarded section `.discard.var16.src/cdrom.c.107' of out/rom32.o
`cdemu_drive' referenced in section `.text32' of out/rom32.o: defined in 
discarded section `.discard.var16.src/cdrom.c.107' of out/rom32.o

Given we're chasing a cdom related problem, this is suspicious, even 
though we aren't using cdemu.  Presumably access to uninitialized memory 
can confuse disk.c.

out/rom32.o: In function `check_irqs32':
/build/home/tlv/akivity/qemu-kvm/roms/seabios/src/util.c:109: undefined 
reference to `trampoline_checkirqs'

No idea where that came from.

-- 
error compiling committee.c: too many arguments to function

[Qemu-devel] Re: Issues building seabios

[Kevin O'Connor]

On Thu, Nov 19, 2009 at 03:10:20PM +0200, Avi Kivity wrote:
> Trying to debug the cdrom issue, I see
>
>   Compiling whole program out/ccode32.o
> src/util.c: In function ‘__end_thread’:
> src/util.c:183: internal compiler error: in simplify_subreg, at  
> simplify-rtx.c:5055
>
> (with F12's gcc (GCC) 4.4.2 20091027 (Red Hat 4.4.2-7))

I think this should be fixed in gcc - see:

https://bugzilla.redhat.com/show_bug.cgi?id=531385

Long story short - this is the result of gcc's "-combine" being
fragile.  It's possible to avoid by compiling seabios with "make
COMPSTRAT=1".  We may need to change the default.

Ughh - did FC12 go out without this fix?

-Kevin

[Qemu-devel] Re: Issues building seabios

[Avi Kivity]

On 11/19/2009 03:39 PM, Kevin O'Connor wrote:
> On Thu, Nov 19, 2009 at 03:10:20PM +0200, Avi Kivity wrote:
>    
>> Trying to debug the cdrom issue, I see
>>
>>    Compiling whole program out/ccode32.o
>> src/util.c: In function ‘__end_thread’:
>> src/util.c:183: internal compiler error: in simplify_subreg, at
>> simplify-rtx.c:5055
>>
>> (with F12's gcc (GCC) 4.4.2 20091027 (Red Hat 4.4.2-7))
>>      
> I think this should be fixed in gcc - see:
>
> https://bugzilla.redhat.com/show_bug.cgi?id=531385
>
> Long story short - this is the result of gcc's "-combine" being
> fragile.  It's possible to avoid by compiling seabios with "make
> COMPSTRAT=1".  We may need to change the default.
>
>    

That works.  Still complains about cdemu_drive and trampoline_checkirqs.

> Ughh - did FC12 go out without this fix?
>    

Yes, and 4.4.2-8 is not on updates-testing or rawhide.

-- 
error compiling committee.c: too many arguments to function

[Qemu-devel] Re: Issues building seabios

[Kevin O'Connor]

On Thu, Nov 19, 2009 at 04:02:06PM +0200, Avi Kivity wrote:
> On 11/19/2009 03:39 PM, Kevin O'Connor wrote:
>> Long story short - this is the result of gcc's "-combine" being
>> fragile.  It's possible to avoid by compiling seabios with "make
>> COMPSTRAT=1".  We may need to change the default.
>
> That works.  Still complains about cdemu_drive and trampoline_checkirqs.

I updated SeaBIOS' gcc check to test for the broken fc12 compiler -
commit 3133e382.

With that commit, I can build on fc12 and fc12beta without any errors
or warnings.  Did you run a "make clean" before rebuilding?

-Kevin

[Qemu-devel] Re: Issues building seabios

[Avi Kivity]

On 11/21/2009 12:34 AM, Kevin O'Connor wrote:
> On Thu, Nov 19, 2009 at 04:02:06PM +0200, Avi Kivity wrote:
>    
>> On 11/19/2009 03:39 PM, Kevin O'Connor wrote:
>>      
>>> Long story short - this is the result of gcc's "-combine" being
>>> fragile.  It's possible to avoid by compiling seabios with "make
>>> COMPSTRAT=1".  We may need to change the default.
>>>        
>> That works.  Still complains about cdemu_drive and trampoline_checkirqs.
>>      
> I updated SeaBIOS' gcc check to test for the broken fc12 compiler -
> commit 3133e382.
>
> With that commit, I can build on fc12 and fc12beta without any errors
> or warnings.  Did you run a "make clean" before rebuilding?
>    

I did.  Latest seabios builds for me, though, so I can confirm the 
problem is fixed.

-- 
error compiling committee.c: too many arguments to function