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 10:39:27 +0200 [thread overview]
Message-ID: <1187339967.7007.71.camel@chaos> (raw)
In-Reply-To: <1187323117.23789.55.camel@imap.mvista.com>
On Thu, 2007-08-16 at 20:58 -0700, Daniel Walker wrote:
> On Thu, 2007-08-16 at 21:37 -0300, Luis Claudio R. Goncalves wrote:
> > Fixes spurious system load spikes observed in /proc/loadavgrt, as described in:
> >
> > Bug 253103: /proc/loadavgrt issues weird results
> > https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253103
>
> 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.
tglx
next prev parent reply other threads:[~2007-08-17 8:39 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 [this message]
2007-08-17 16:24 ` Daniel Walker
2007-08-17 21:37 ` Thomas Gleixner
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=1187339967.7007.71.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.