<html>
    <head>
      <base href="https://bugs.webkit.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Web process never leaves memory pressured state if caused by per-process limit"
   href="https://bugs.webkit.org/show_bug.cgi?id=188299#c5">Comment # 5</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Web process never leaves memory pressured state if caused by per-process limit"
   href="https://bugs.webkit.org/show_bug.cgi?id=188299">bug 188299</a>
              from <span class="vcard"><a class="email" href="mailto:koivisto@iki.fi" title="Antti Koivisto <koivisto@iki.fi>"> <span class="fn">Antti Koivisto</span></a>
</span></b>
        <pre><span class="quote">> > Source/WTF/wtf/cocoa/MemoryPressureHandlerCocoa.mm:96
> > +                return MemoryPressureHandler::singleton().isUnderMemoryPressure();

> Is this line only about simulated memory pressure? If so, would be nice to
> rename that function since it's really confusing that the input to
> setUnderMemoryPressure() uses isUnderMemoryPressure().</span >

No, that line is about making DISPATCH_MEMORYPRESSURE_PROC_LIMIT_* notifications to have no effect to the memory pressure state. When we get those we just stay in the current state.

<span class="quote">> I'm still not sure how you get out of memory pressure with this code, since
> isUnderVMMemoryPressure doesn't consult
> DISPATCH_MEMORYPRESSURE_PROC_LIMIT_CRITICAL.</span >

Only DISPATCH_MEMORYPRESSURE_CRITICAL puts us into the pressured state, we get out when we receive DISPATCH_MEMORYPRESSURE_NORMAL or DISPATCH_MEMORYPRESSURE_WARN.

<span class="quote">> These NSLogs should be WTFLogAlways, so they show in iOS syslog. Are they
> even useful with all the release logging we have? Should they be release
> logs?</span >

I suppose release logging changes in pressure state might be useful.</pre>
        </div>
      </p>


      <hr>
      <span>You are receiving this mail because:</span>

      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>