From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.bugs Subject: bug#12450: Remove configure's --without-sync-input option. Date: Sun, 16 Sep 2012 09:33:02 +0300 Message-ID: <83fw6iscip.fsf@gnu.org> References: <50543449.1070306@cs.ucla.edu> <83k3vvtyw0.fsf@gnu.org> <5054551A.1070207@cs.ucla.edu> <837grvtuo5.fsf@gnu.org> <5054DE30.1020806@cs.ucla.edu> Reply-To: Eli Zaretskii NNTP-Posting-Host: plane.gmane.org X-Trace: ger.gmane.org 1347777719 7522 80.91.229.3 (16 Sep 2012 06:41:59 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Sun, 16 Sep 2012 06:41:59 +0000 (UTC) Cc: lekktu@gmail.com, 12450@debbugs.gnu.org To: Paul Eggert , Richard Stallman Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Sun Sep 16 08:42:02 2012 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1TD8YJ-0001ZS-Fv for geb-bug-gnu-emacs@m.gmane.org; Sun, 16 Sep 2012 08:41:59 +0200 Original-Received: from localhost ([::1]:36257 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TD8YF-0005E5-Fo for geb-bug-gnu-emacs@m.gmane.org; Sun, 16 Sep 2012 02:41:55 -0400 Original-Received: from eggs.gnu.org ([208.118.235.92]:50539) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TD8YC-0005Cs-UL for bug-gnu-emacs@gnu.org; Sun, 16 Sep 2012 02:41:53 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TD8QV-0001fG-2g for bug-gnu-emacs@gnu.org; Sun, 16 Sep 2012 02:33:55 -0400 Original-Received: from debbugs.gnu.org ([140.186.70.43]:54964) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TD8QU-0001fB-VP for bug-gnu-emacs@gnu.org; Sun, 16 Sep 2012 02:33:54 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.72) (envelope-from ) id 1TD8RZ-0003Ap-K1 for bug-gnu-emacs@gnu.org; Sun, 16 Sep 2012 02:35:01 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Eli Zaretskii Original-Sender: debbugs-submit-bounces@debbugs.gnu.org Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Sun, 16 Sep 2012 06:35:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 12450 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 12450-submit@debbugs.gnu.org id=B12450.134777725012137 (code B ref 12450); Sun, 16 Sep 2012 06:35:01 +0000 Original-Received: (at 12450) by debbugs.gnu.org; 16 Sep 2012 06:34:10 +0000 Original-Received: from localhost ([127.0.0.1]:36277 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.72) (envelope-from ) id 1TD8Qj-00039i-KP for submit@debbugs.gnu.org; Sun, 16 Sep 2012 02:34:10 -0400 Original-Received: from mtaout22.012.net.il ([80.179.55.172]:51416) by debbugs.gnu.org with esmtp (Exim 4.72) (envelope-from ) id 1TD8Qh-00039Z-NM for 12450@debbugs.gnu.org; Sun, 16 Sep 2012 02:34:08 -0400 Original-Received: from conversion-daemon.a-mtaout22.012.net.il by a-mtaout22.012.net.il (HyperSendmail v2007.08) id <0MAF00A00J9AL600@a-mtaout22.012.net.il> for 12450@debbugs.gnu.org; Sun, 16 Sep 2012 09:32:59 +0300 (IDT) Original-Received: from HOME-C4E4A596F7 ([87.69.4.28]) by a-mtaout22.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0MAF00A7ZJIY96B0@a-mtaout22.012.net.il>; Sun, 16 Sep 2012 09:32:59 +0300 (IDT) In-reply-to: <5054DE30.1020806@cs.ucla.edu> X-012-Sender: halo1@inter.net.il X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.13 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6 (newer, 2) X-Received-From: 140.186.70.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:64389 Archived-At: > Date: Sat, 15 Sep 2012 12:59:44 -0700 > From: Paul Eggert > CC: 12450@debbugs.gnu.org, lekktu@gmail.com > > >> -#ifdef REL_ALLOC > >> - malloc_hysteresis = 32; > >> -#else > >> - malloc_hysteresis = 0; > >> -#endif > > malloc_hysteresis is used only by the non-SYNC_INPUT code. No, it is also used indirectly, via __malloc_extra_blocks, in gmalloc.c and in ralloc.c. Since only one platform I know of that uses SYNC_INPUT also uses ralloc.c, I'm not sure it's safe to make this change to __malloc_extra_blocks. But there's something I don't understand about this snippet. Why does it set the value to 32 conditioned by REL_ALLOC, and just a few days after the change which introduced this snippet (18 years ago) there was a change committed in ralloc.c that overrides the value of __malloc_extra_blocks with 64? If using ralloc.c always sets __malloc_extra_blocks to 64, why setting it to 32 in alloc.c is conditioned by REL_ALLOC? Richard, you made both changes -- can you perhaps shed some light on this?