svn commit: r220387 - head/sys/vm

Garrett Cooper yanegomi at gmail.com
Wed Apr 6 17:22:33 UTC 2011


2011/4/6 Edward Tomasz Napierała <trasz at freebsd.org>:
> Wiadomość napisana przez Garrett Cooper w dniu 2011-04-06, o godz. 18:57:
>> On Wed, Apr 6, 2011 at 9:27 AM, Edward Tomasz Napierala
>> <trasz at freebsd.org> wrote:
>>> Author: trasz
>>> Date: Wed Apr  6 16:27:04 2011
>>> New Revision: 220387
>>> URL: http://svn.freebsd.org/changeset/base/220387
>>>
>>> Log:
>>>  In vm_daemon(), do not skip processes stopped with SIGSTOP.
>>
>>    Did you run this by anyone else before you committed the change?
>
> The whole racct patchset was reviewed by kib@, and I seem to remember
> that he said this might cause problems.  However, I didn't encounter
> any problems with this, neither did any person testing the patchset.
>
> So, what's wrong with this?

My concern was just that "Reviewed by" was missing and that this might
introduce some unexpected functional issues (then again it's now going
to kill stopped processes, right?).
-Garrett


More information about the svn-src-head mailing list