lkml.org 
[lkml]   [2008]   [Aug]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [revert] mysql+oltp regression

* Gregory Haskins <ghaskins@novell.com> wrote:

> Ingo Molnar wrote:
>> * Mike Galbraith <efault@gmx.de> wrote:
>>
>>
>>> Greetings,
>>>
>>> During regression testing of tip/sched/clock fixes, a regression in
>>> low client count throughput turned up, which I traced this back to
>>> the commit below. I don't see anything wrong with it, but suspect
>>> that it is preventing client/server pairs from staying together on
>>> the same CPU as buddies, which mysql definitely likes quite a lot.
>>> (I suspect that this is the case, because I've seen this same
>>> performance curve while tinkering with wakeup affinity and breaking
>>> it all to pieces;)
>>>
>>> Changelog and test results below in case nobody sees a problem with
>>> the commit itself.
>>>
>>
>> i've applied your fix to tip/sched/urgent for the time being, thanks
>> Mike for tracking it down. We can re-try newer iterations of Greg's
>> patch in tip/sched/devel.
>>
>>
>
> Hmm.. The patch still looks correct afaict. I fear we are just
> papering over some other issue by reverting it, but I will try to see
> if I can track this down. We will, of course, now be skipping trying
> to balance the (effectively random) last task in the queue which may
> or may not result in better performance on sheer luck instead of
> algorithmic intelligence. This makes me nervous.

yeah - but we had that behavior for quite some time.

This is how the patch cycle works normally: we had a fair chance to
discover this problem in your testing then in -tip testing and then in
linux-next or -mm but we didnt find it at any stage.

Now we are in the upstream release cycle so unless there's some
immediate fix available (or there are _really_ strong reasons against
the revert) doing the revert is the right approach.

A revert is not necessarily the indicator of the quality of the change
in question, it is a tester-driven exception event that guarantees that
the kernel improves in a monotonic way. (for all testers who opt to help
us in doing so)

And given that the problem was readily reproducible for Mike, it should
be reproducible for you as well - so we dont actually make the bug
harder to fix by doing the revert.

Perhaps we should introduce the notion of "Defer-to-next-release"
reverts - which this really is - in contrast to "Revert-because-bad",
which your change definitely is not.

> Speaking of this: Another patch I submitted to you Ingo (had to do
> with updating the load_weight inside task_setprio) seems to also have
> this phenomenon: e.g. its technically correct but further testing has
> revealed negative repercussions elsewhere. So please ignore that
> patch (or revert if you already pulled in, but I don't think you
> have). Ill try to look into this issue as well.

ok, under which thread/subject is that? Not queued in tip/sched/* yet,
correct?

Ingo


\
 
 \ /
  Last update: 2008-08-11 14:51    [W:0.069 / U:0.268 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site