Re: loader causes reboot

On 05/08/05 22:19, David Gurvich wrote:
> On Sunday 08 May 2005 23:57, Jonathan Noack wrote:
>>On 05/08/05 14:29, David Gurvich wrote:
>>>cdrom /boot/loader from 5.3 has no problem.  However, after updating
>>>world from kernel, /boot/loader is replaced with cvs version.  This one
>>>goes into endless cycle of reboots.  When replaced with /boot/loader from
>>>cdrom boots normally.
>>
>>Are you setting CPUTYPE?  A few people have reported an endless reboot
>>with the athlon-xp or pentium-m settings.  I experience this on my
>>athlon-xp and the only workaround I've found is just to not set CPUTYPE.
>>  This problem is way beyond my feeble skills to track down and fix, but
>>it doesn't seem to affect many people and no one has stepped up to
>>resolve it.  As it is easily worked around, I've brought it up a few
>>times but haven't made too big of a fuss.  Come to think of it, why
>>didn't I ever open a PR?  Hmm... perhaps I'll do that at work tomorrow
>>(this is on a machine at work).
> 
> I am setting CPUTYPE?=athlon-xp
> I never considered that might be the cause.  If it is then there is clearly a 
> bug in the configuration of the loader.  Perhaps when the loader is being 
> compiled it should use the most basic flags possible for just that.

Please don't top post.

There was an attempt to resolve this issue a few months ago.  The commit 
was rev. 1.10 of src/sys/boot/i386/Makefile.inc and the log message was, 
"Ensure GCC does not use FP registers in integer code."  This fix was 
MFC'd to RELENG_5 a day later and is present in 5.4.

For the life of me I can't remember if I tested the change when it was 
made.  I think it didn't work for me, but I'll test it again tomorrow.

-- 
Jonathan Noack | noackjr_at_alumni.rice.edu | OpenPGP: 0x991D8195