[Webkit-unassigned] [Bug 61847] New: Worker may not be stopped after LT.

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Jun 1 01:48:01 PDT 2011


           Summary: Worker may not be stopped after LT.
           Product: WebKit
           Version: 528+ (Nightly build)
          Platform: Unspecified
        OS/Version: Unspecified
            Status: UNCONFIRMED
          Severity: Normal
          Priority: P2
         Component: Tools / Tests
        AssignedTo: webkit-unassigned at lists.webkit.org
        ReportedBy: zhenghao at chromium.org
                CC: tony at chromium.org, dpranke at chromium.org

In worker.py, only call to cleanup() is in __del__, which is not guaranteed to be called when the python interpreter exits. And self._driver is not killed after LT. After many runs of LT, there will be many zombie processes of driver. I think we need to run cleanup() explicitly for worker.

I'm not sure where is better to put cleanup():

    def handle_stop(self, src):
        self._done = True
**      self.cleanup()


    def run(self, port):

        exception_msg = ""
        _log.debug("%s starting" % self._name)

            if not self.is_done():
                raise AssertionError("%s: ran out of messages in worker queue."
                                     % self._name)
        except KeyboardInterrupt:
            exception_msg = ", interrupted"
            exception_msg = ", exception raised"
            _log.debug("%s done%s" % (self._name, exception_msg))
            if exception_msg:
                exception_type, exception_value, exception_traceback = sys.exc_info()
                stack_utils.log_traceback(_log.debug, exception_traceback)
                # FIXME: Figure out how to send a message with a traceback.
                    (exception_type, exception_value, None))
**          self.cleanup()

I'd like to hear your comments.

Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

More information about the webkit-unassigned mailing list