Re: [patch] CFS scheduler, -v8

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi!

*** Balbir Singh <[email protected]> wrote:

> The problem is with comparing a s64 values with (s64)ULONG_MAX, which > evaluates to -1. Then we check if exec_delta64 and fair_delta64 are > greater than (s64)ULONG_MAX (-1), if so we assign (s64)ULONG_MAX to > the respective values.

ah, indeed ...

> The fix is to compare these values against (s64)LONG_MAX and assign > (s64)LONG_MAX to exec_delta64 and fair_delta64 if they are greater > than (s64)LONG_MAX. > > Tested on PowerPC, the regression is gone, tasks are load balanced as > they were in v7.

thanks, applied!

	Ingo

I started up 12 glxgears to see the effect of CFS v8
on my Athlon64 X2 4200.

Without this patch all the GL load was handled by the second core,
the system only stressed the first core if other processes were also
started, i.e. a kernel compilation. With this patch  the load is evenly
balanced across the two cores all the time. And while doing make -j4
on the kernel, the 12 gears are still spinning about 185+ FPS and
there are only slightly visible hiccups. Switching between workspaces,
i.e. refreshing the large windows of Thunderbird and Firefox are
done very quickly, the whole system is exceptionally responsive.

Thanks for this great work!

Best regards,
Zoltán Böszörményi

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[Index of Archives]     [Kernel Newbies]     [Netfilter]     [Bugtraq]     [Photo]     [Stuff]     [Gimp]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Video 4 Linux]     [Linux for the blind]     [Linux Resources]
  Powered by Linux