Return-Path: Delivered-To: apmail-harmony-dev-archive@www.apache.org Received: (qmail 57640 invoked from network); 18 Mar 2007 22:39:34 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 18 Mar 2007 22:39:34 -0000 Received: (qmail 83732 invoked by uid 500); 18 Mar 2007 22:39:40 -0000 Delivered-To: apmail-harmony-dev-archive@harmony.apache.org Received: (qmail 83698 invoked by uid 500); 18 Mar 2007 22:39:40 -0000 Mailing-List: contact dev-help@harmony.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@harmony.apache.org Delivered-To: mailing list dev@harmony.apache.org Received: (qmail 83689 invoked by uid 99); 18 Mar 2007 22:39:40 -0000 Received: from herse.apache.org (HELO herse.apache.org) (140.211.11.133) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 18 Mar 2007 15:39:40 -0700 X-ASF-Spam-Status: No, hits=2.9 required=10.0 tests=HTML_10_20,HTML_MESSAGE,SPF_PASS X-Spam-Check-By: apache.org Received-SPF: pass (herse.apache.org: domain of weldonwjw@gmail.com designates 64.233.182.185 as permitted sender) Received: from [64.233.182.185] (HELO nf-out-0910.google.com) (64.233.182.185) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 18 Mar 2007 15:39:31 -0700 Received: by nf-out-0910.google.com with SMTP id d4so549332nfe for ; Sun, 18 Mar 2007 15:39:10 -0700 (PDT) DKIM-Signature: a=rsa-sha1; c=relaxed/relaxed; d=gmail.com; s=beta; h=domainkey-signature:received:received:message-id:date:from:to:subject:mime-version:content-type; b=E/1JPjektmXGdqTy+rspGie08IYk+YiEorSO8YYq5hp9BJcOYMQnojArCSorDtFJVp3GXrQfd0RBU0JiSTMLkq+y4fA5ZtihsV7OjvZt9HWG3QavnUwfZhzLGoyFZB+waUQp7A8kj4D3klXTudTG/4MRRabQTioZ5C1GjowThUY= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:to:subject:mime-version:content-type; b=ipFZ4F2q62/z9BqdEiDquGaq7lxVtTZmyMqV3BOM+v7pZXEre+FlVBnhq9oOkADjKTt8npIgY+cDnjM1WzjssnSseAaSlUM4I2KxRnLZp7XLtZnLGbKZcJUnYR9hubifaofGKIxFf99TDVpDV8z23KITUjwEZJKmNQHcxI8tfME= Received: by 10.78.193.19 with SMTP id q19mr2159200huf.1174257550192; Sun, 18 Mar 2007 15:39:10 -0700 (PDT) Received: by 10.78.137.2 with HTTP; Sun, 18 Mar 2007 15:39:10 -0700 (PDT) Message-ID: <4dd1f3f00703181539i76a0550fse56a77c4d45fa383@mail.gmail.com> Date: Sun, 18 Mar 2007 15:39:10 -0700 From: "Weldon Washburn" To: dev@harmony.apache.org Subject: [drlvm][threading] questions about H3413, Refactoring exception safepoint callback MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_91816_1638110.1174257550155" X-Virus-Checked: Checked by ClamAV on apache.org ------=_Part_91816_1638110.1174257550155 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Content-Disposition: inline Pavel, The code in this patch looks much better than the original code. Thanks for focusing on cleaning this code up. A question about the use of the disable_count field in struct hythread_t. I notice in hythread_exception_safe_point() that the old value of this field is saved. The field is then specifically set to a value of one and once the exception callback is processed, the field is returned to its original value. Is this a case where disable_count is actually be overloaded with another use? Would it make sense to split the two uses into two seperate, distinct fields?? -- Weldon Washburn Intel Enterprise Solutions Software Division ------=_Part_91816_1638110.1174257550155--