[bind10-dev] BIND 10 master, updated. 376df1c9ac466742afad6681ff2e431ebfd75b63 [master] update ChangeLog for #1001

Jerry jerry.zzpku at gmail.com
Tue Jun 28 06:51:30 UTC 2011


The ChangeLog entry hasn't been reviewed yet.
I'll update it and add more details, thanks for your suggestion.

--
Jerry

> 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
> _______________________________________________
> bind10-dev mailing list
> bind10-dev at lists.isc.org
> https://lists.isc.org/mailman/listinfo/bind10-dev




More information about the bind10-dev mailing list