From: Thomas Gleixner <tglx@linutronix.de>
To: Daniel Walker <dwalker@mvista.com>
Cc: "Luis Claudio R. Goncalves" <lclaudio@uudg.org>,
Ingo Molnar <mingo@elte.hu>,
linux-rt-users@vger.kernel.org
Subject: Re: [PATCH] Fixes spurious system load spikes in /proc/loadavgrt
Date: Fri, 17 Aug 2007 23:37:12 +0200 [thread overview]
Message-ID: <1187386632.7007.100.camel@chaos> (raw)
In-Reply-To: <1187367845.23789.62.camel@imap.mvista.com>
On Fri, 2007-08-17 at 09:24 -0700, Daniel Walker wrote:
> > > It would be nice if you explained better what the code defect was, and
> > > how you fixed it .. One could extract that from the code, but it's nice
> > > if it's backed up with a text description.. Also you should CC lkml next
> > > time.
> >
> > It would be nice if you'd stop being uber schoolmasterly. Luis
> > explanation of the defect is entirely clear and the resulting fix is
> > obvious. .. You could really look at the patch and figure out your self,
> > that it needs no text description at all. Also you should comment on the
> > nice patch itself next time.
>
> If all you've said was true I wouldn't have commented on it. There was
> no text description of the code defect, and yes as I said I can get it
> from the code but it's _nice_ when it's backed up with a text
> description ..
You really start to annoy me. There was a detailed bug description and
the code defect is easy to see. If you neither have the interest nor the
time to look at the actual _trivial_ _self explaining_ patch, then
please look for some other playground to live out your wisenheimer airs
and graces.
> If your just going to start flames Thomas take it off list.
Flames are only flames when they happen in public.
tglx
next prev parent reply other threads:[~2007-08-17 21:37 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-17 0:37 [PATCH] Fixes spurious system load spikes in /proc/loadavgrt Luis Claudio R. Goncalves
2007-08-17 3:58 ` Daniel Walker
2007-08-17 8:39 ` Thomas Gleixner
2007-08-17 16:24 ` Daniel Walker
2007-08-17 21:37 ` Thomas Gleixner [this message]
2007-08-17 21:43 ` Daniel Walker
2007-08-17 22:21 ` Thomas Gleixner
2007-08-17 22:43 ` Daniel Walker
2007-08-17 23:33 ` Thomas Gleixner
2007-08-23 15:33 ` Sven-Thorsten Dietrich
2007-08-23 15:51 ` Ingo Molnar
2007-08-23 15:57 ` Sven-Thorsten Dietrich
2007-08-23 16:13 ` Luis Claudio R. Goncalves
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1187386632.7007.100.camel@chaos \
--to=tglx@linutronix.de \
--cc=dwalker@mvista.com \
--cc=lclaudio@uudg.org \
--cc=linux-rt-users@vger.kernel.org \
--cc=mingo@elte.hu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.