On 4/7/06, Gavin Maltby <[EMAIL PROTECTED]> wrote: > Hi, > > On 04/06/06 00:51, Alexander Kolbasov wrote: > > Hello, > > > > here is you chance to bite a piece of the scheduler code. > > > > The purpose of the fix is to make idle thread less less aggressive in > > stealin > > work from other CPUs when the work was only placed on the run-queue > > recently. > > > > The diffs are available at > > > > http://cr.grommit.com/~akolb/sticky/index.html > > The changes look solid. > > I have one question. It looks like disp_getwork will traverse the > other cpus (relative to the argument) in a deterministic order > for a given cpu and lgroup configuration (I may be wrong there - > it's not obvious!).
I have a related comment: I think it may be better to change the traverse code to scan local CPU first - in the order local core, local board, local machine and then all remaining boards - to avoid massive communication overhead in NUMA/COMA machines. -- Holger _______________________________________________ perf-discuss mailing list perf-discuss@opensolaris.org
