PERFORCE change 119371 for review

Attilio Rao attilio at freebsd.org
Wed May 9 19:33:36 UTC 2007


2007/5/9, Attilio Rao <attilio at freebsd.org>:
> 2007/5/9, John Baldwin <jhb at freebsd.org>:
> > On Sunday 06 May 2007 05:10:35 pm Rui Paulo wrote:
> > > http://perforce.freebsd.org/chv.cgi?CH=119371
> > >
> > > Change 119371 by rpaulo at rpaulo_epsilon on 2007/05/06 21:10:15
> > >
> > >       We don't need any scheduler support because:
> > >       1) msrtemp is a child of cpu - this implies that every
> > >          rdmsr/cpuid instruction will be executed on that CPU.
> >
> > No, that isn't true.  You do need to use sched_bind() for that so you are
> > really on the desired CPU when you read the MSR.
>
> I think he just needs msr of the cpu where curthread is executed, so
> any scheduler lock should be needed.
s/should/shouldn't.

Attilio

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


More information about the p4-projects mailing list