Re: svn commit: r198868 - in head/sys: amd64/amd64 i386/i386

[ Available lists | Index of svn-src-all | Month of Nov 2009 | Week of 4 Nov 2009 | Raw email | View thread | Wrap long lines | Reply | Tag ]
From
Attilio Rao <attilio@freebsd.org>
Date
4 Nov 2009 16:15:30
Subject
Re: svn commit: r198868 - in head/sys: amd64/amd64 i386/i386
Message-ID
3bbf2fe10911040815u780ea5d5h8252b46ddf0c5de7@mail.gmail.com


[ Hide this part ]
2009/11/4, Dag-Erling Smrgrav <des@des.no>:
> Attilio Rao <attilio@freebsd.org> writes:
> > Dag-Erling Smrgrav <des@des.no> writes:
> > > According to the reference you gave, OpenSolaris has a workaround for
> > > this issue. Perhaps we should adopt it?
> > IIRC, based on what I readed on-line, it could be still a WARNING msg,
> > but I didn't double-check for it.
>
> Uh, apparently, it can be worked around with an additional lfence
> instruction... See the first couple of comments on that bugzilla page
> you linked to. If the cost is too high, we could make it conditional on
> "options BROKEN_OPTERON_E" or something.

Yeah but the lfence is completely impratical from the overhead POV --
and actually I don't like the idea to add a compile time option based
on family/model.
We could maybe do a mixed approach: printout the WARNING msg only if
the BROKEN_OPTERON_E is not used otherwise be ok.
I still don't like it because it is going to dirty the atomic.h code a
bit, but it can be an acceptable compromise.

Attilio


--
Peace can only be achieved by understanding - A. Einstein


Elapsed time: 0.852 seconds