Venkatesh Srinivas <[email protected]> added the comment:

As written I think the patch behaves like this:

if (zero_state == 0) /* We've not hit the upper limit */
   if (zeroed pages >= high count) /* hit the upper limit */
      zero_state = 1;
      return STOP_ZEROING_PAGES;
   return KEEP_ZEROING_PAGES;

if (zero_state == 1) /* We've hit the upper limit before */
   if (zeroed_pages >= LOW MARK) /* Still have free pages; also covers high */
      return STOP_ZEROING_PAGES;
   /* We've seen an upper limit; now we're below the lower limit. */
   /* Restore to 0 (not seen high limit) */
   zero_state = 0;
   return KEEP_ZEROING_PAGES;


What was wrong with this?

-- vs

_____________________________________________________
DragonFly issue tracker <[email protected]>
<http://bugs.dragonflybsd.org/issue1771>
_____________________________________________________

Reply via email to