From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Richard Stallman Newsgroups: gmane.emacs.devel Subject: [dmantipov@yandex.ru: Re: Your Emacs changes] Date: Thu, 30 Aug 2007 03:16:02 -0400 Message-ID: Reply-To: rms@gnu.org NNTP-Posting-Host: lo.gmane.org Content-Type: text/plain; charset=ISO-8859-15 X-Trace: sea.gmane.org 1188458504 18354 80.91.229.12 (30 Aug 2007 07:21:44 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Thu, 30 Aug 2007 07:21:44 +0000 (UTC) To: emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Thu Aug 30 09:21:43 2007 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by lo.gmane.org with esmtp (Exim 4.50) id 1IQeLW-0004u9-Vx for ged-emacs-devel@m.gmane.org; Thu, 30 Aug 2007 09:21:43 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1IQeLW-0005Br-OA for ged-emacs-devel@m.gmane.org; Thu, 30 Aug 2007 03:21:42 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1IQeKw-0004Y5-0l for emacs-devel@gnu.org; Thu, 30 Aug 2007 03:21:06 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1IQeKu-0004Wh-Rg for emacs-devel@gnu.org; Thu, 30 Aug 2007 03:21:05 -0400 Original-Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1IQeKu-0004W3-EY for emacs-devel@gnu.org; Thu, 30 Aug 2007 03:21:04 -0400 Original-Received: from fencepost.gnu.org ([140.186.70.10]) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1IQeKu-0007ey-1N for emacs-devel@gnu.org; Thu, 30 Aug 2007 03:21:04 -0400 Original-Received: from rms by fencepost.gnu.org with local (Exim 4.60) (envelope-from ) id 1IQeG2-0005Xq-LP; Thu, 30 Aug 2007 03:16:02 -0400 X-Detected-Kernel: Linux 2.6, seldom 2.4 (older, 4) X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:77392 Archived-At: [I sent this message a week ago but did not get a response.] Would someone please install this in the trunk, then ack? ------- Start of forwarded message ------- X-Spam-Status: No, score=0.0 required=5.0 tests=none autolearn=failed version=3.1.0 Date: Tue, 21 Aug 2007 18:42:20 +0400 From: Dmitry Antipov MIME-Version: 1.0 To: rms@gnu.org CC: emacs-devel@gnu.org Subject: Re: Your Emacs changes In-Reply-To: Content-Type: multipart/mixed; boundary="------------040802050101020103000601" This is a multi-part message in MIME format. - --------------040802050101020103000601 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Richard Stallman wrote: > Now that we have your papers, what changes should we install now? Probably the thing described at http://lists.gnu.org/archive/html/emacs-devel/2007-07/msg00094.html. Attached is the latest stuff on this. Note the same optimization may be performed on sweeping floats. But, since an amount of allocated floats is too small in comparison with conses, an effect is expected to be very negligible. Dmitry - --------------040802050101020103000601 Content-Type: text/x-patch; name="fast_cons_sweep.patch" Content-Transfer-Encoding: 7bit Content-Disposition: inline; filename="fast_cons_sweep.patch" Index: alloc.c =================================================================== RCS file: /sources/emacs/emacs/src/alloc.c,v retrieving revision 1.414 diff -u -r1.414 alloc.c - --- alloc.c 19 Aug 2007 00:15:26 -0000 1.414 +++ alloc.c 21 Aug 2007 14:31:37 -0000 @@ -5974,23 +5974,53 @@ for (cblk = cons_block; cblk; cblk = *cprev) { - - register int i; + register int i = 0; int this_free = 0; - - for (i = 0; i < lim; i++) - - if (!CONS_MARKED_P (&cblk->conses[i])) - - { - - this_free++; - - cblk->conses[i].u.chain = cons_free_list; - - cons_free_list = &cblk->conses[i]; + + while (1) + { + if (cblk->gcmarkbits[i] == -1) + { + /* Fast path - everything is marked. */ + cblk->gcmarkbits[i++] = 0; + num_used += BITS_PER_INT; + } + else + { + /* Slow path - scan over each bit, from the beginning + of current word to 'min (word boundary, LIM)'. */ + int start, stop; + + start = i * BITS_PER_INT; + stop = lim - start; + if (stop > BITS_PER_INT) + stop = BITS_PER_INT; + stop += start; + + while (start < stop) + { + if (!CONS_MARKED_P (&cblk->conses[start])) + { + this_free++; + cblk->conses[start].u.chain = cons_free_list; + cons_free_list = &cblk->conses[start]; #if GC_MARK_STACK - - cons_free_list->car = Vdead; + cons_free_list->car = Vdead; #endif - - } - - else - - { - - num_used++; - - CONS_UNMARK (&cblk->conses[i]); - - } + } + else + { + num_used++; + CONS_UNMARK (&cblk->conses[start]); + } + start++; + } + if (stop < (++i) * BITS_PER_INT) + /* Whole bitmap is scanned or LIM is reached. */ + break; + } + } + lim = CONS_BLOCK_SIZE; /* If this block contains only free conses and we have already seen more than two blocks worth of free conses then deallocate - --------------040802050101020103000601-- ------- End of forwarded message -------