[bind10-dev] BIND 10 master, updated. 376df1c9ac466742afad6681ff2e431ebfd75b63 [master] update ChangeLog for #1001
JINMEI Tatuya / 神明達哉
jinmei at isc.org
Tue Jun 28 06:08:03 UTC 2011
At Tue, 28 Jun 2011 02:32:46 +0000 (UTC),
BIND 10 source code commits <bind10-changes at lists.isc.org> wrote:
> diff --git a/ChangeLog b/ChangeLog
> index c1ff9d0..37bb9bd 100644
> --- a/ChangeLog
> +++ b/ChangeLog
> @@ -1,3 +1,7 @@
> +264. [bug] jerry
> + Fix a busy loop issue in notify-out.
> + (Trac 1001, git fb993ba8c52dca4a3a261e319ed095e5af8db15a)
> +
I'm afraid this changelog entry is not sufficiently helpful.
It doesn't explain which module (program) caused the loop, how severe
it is (whether it could occupy 100% of CPU cycle or it's milder than
that), whether the problem (can) happen regularly or only in a rare,
limited situation, etc.
I'd revise it to something like this:
b10-xfrout: fixed a busy loop in its notify-out subthread. Due to
the loop, the thread previously woke up every 0.5 seconds throughout
most of the lifetime of b10-xfrout, wasting the corresponding CPU
time.
Ticket #1001 doesn't have a discussion about the changelog. Was it
included in the branch and got reviewed? If so, we should blame the
reviewer:-) If not, please make sure changelog entries get reviewed as
well the code (this is also documented in the code review procedure at
http://bind10.isc.org/wiki/CodeReviewProcedure although some part of
seem to be already outdated and many of us don't actually refer to it
anyway).
---
JINMEI, Tatuya
More information about the bind10-dev
mailing list