Subject: Re: CVS commit: src/sys/arch/newsmips/newsmips
To: None <source-changes@netbsd.org>
From: Christos Zoulas <christos@astron.com>
List: source-changes
Date: 08/27/2006 05:41:17
In article <20060827052547.A35D0211CA@cvs.netbsd.org>,
Izumi Tsutsui  <tsutsui@netbsd.org> wrote:
>
>Module Name:	src
>Committed By:	tsutsui
>Date:		Sun Aug 27 05:25:47 UTC 2006
>
>Modified Files:
>	src/sys/arch/newsmips/newsmips: news3400.c
>
>Log Message:
>Make badaddr_flag volatile to avoid unintended optimization by gcc4.
>Now GENERIC compiled by gcc4 works on NWS-3470D.
>
>XXX: Is this gcc4 optimization bug, or volitile is required in this case?

It is not a gcc bug. You need volatile, since gcc cannot intuit that the
pointer deref will end up changing its value.

christos