all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: Paul Eggert <eggert@cs.ucla.edu>
Cc: 24892@debbugs.gnu.org
Subject: bug#24892: {s, }brk removed from FreeBSD 11.x and later, arm64 architecture
Date: Thu, 10 Nov 2016 19:24:31 +0200	[thread overview]
Message-ID: <831syjfdtc.fsf@gnu.org> (raw)
In-Reply-To: <92e2ea15-6663-2311-f780-b256d0fe078d@cs.ucla.edu> (message from Paul Eggert on Thu, 10 Nov 2016 08:59:11 -0800)

> Cc: 24892@debbugs.gnu.org
> From: Paul Eggert <eggert@cs.ucla.edu>
> Date: Thu, 10 Nov 2016 08:59:11 -0800
> 
> On 11/10/2016 08:13 AM, Eli Zaretskii wrote:
> > how can zero be a useful approximation of the memory footprint of a 
> > running process?
> 
> It's not, but memory-limit is not about memory footprint.

It is, in a way, although it only reports part of that footprint.

> > What does memory-limit return on your system?
> 
> 47668, representing about 47 MiB. In contrast, (alist-get 'vsize 
> (process-attributes (emacs-pid))) returns 807344, representing about 788 
> MiB. So 0 is numerically closer to the "correct" answer.

"Numerically closer" doesn't mean "useful".  Depending on what the
user wants, either result could be useful, but zero isn't.

> > I think we should have a function that does this in, say, simple.el, 
> > under a name such as emacs-memory-size, and point to that in the 
> > obsolescence note.
> 
> Something like that could be done in a later patch. However, the notion 
> "memory size" is vague and prone to misinterpretation, so any later 
> patch should be done carefully. And I'm leery of defining a function 
> that nobody is asking for - if nobody has cared for many years that 
> memory-limit has been returning bogus values, then why do we need a 
> function at all?

It only returned bogus results on some platforms.

FWIW, I'm frequently interested in the Emacs memory footprint, for
various reasons, so having a function in Emacs would be handy.

> OK, less-drastic patch attached.

Thanks, LGTM for master.





  reply	other threads:[~2016-11-10 17:24 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-07  6:01 bug#24892: {s, }brk removed from FreeBSD 11.x and later, arm64 architecture Ashish SHUKLA
2016-11-07 15:19 ` Eli Zaretskii
2016-11-08 17:54 ` Paul Eggert
2016-11-08 20:11   ` Eli Zaretskii
2016-11-08 21:41     ` Paul Eggert
2016-11-09  3:34       ` Eli Zaretskii
2016-11-09  8:27         ` Paul Eggert
2016-11-09 15:49           ` Eli Zaretskii
2016-11-10  1:47             ` Paul Eggert
2016-11-10 16:13               ` Eli Zaretskii
2016-11-10 16:59                 ` Paul Eggert
2016-11-10 17:24                   ` Eli Zaretskii [this message]
2016-11-10  0:22       ` Richard Stallman
2016-11-10  1:40         ` Paul Eggert
2016-11-09  4:29 ` Ashish SHUKLA
2016-11-09  8:26   ` Paul Eggert
2016-11-09  9:46     ` Andreas Schwab
2016-11-09 15:50       ` Eli Zaretskii
2016-11-09 17:52       ` Paul Eggert
2016-11-10  9:52         ` Andreas Schwab
2016-11-10 16:23           ` Paul Eggert
2016-11-10 17:00             ` Andreas Schwab
2016-11-10 17:22               ` Paul Eggert
2016-11-10 17:33                 ` Andreas Schwab
2016-11-11  2:48                   ` Paul Eggert
2016-11-17 17:27                     ` Eli Zaretskii
2016-11-18 12:29                       ` Ashish SHUKLA
2016-11-18 16:21                         ` Paul Eggert
2016-11-18 20:20                           ` Ed Maste
2016-11-18 20:23                           ` Brooks Davis
     [not found]                           ` <CAPyFy2AL_OZWYpp+tiLSro_3WK1-H8Pdjy4-ZhUQChMQqageUw@mail.gmail.com>
2016-11-18 22:22                             ` Paul Eggert
2016-11-19  6:57                               ` Eli Zaretskii
2016-11-19  7:20                               ` Eli Zaretskii
2017-09-02 13:37                                 ` Eli Zaretskii
2016-11-09 13:23     ` Ashish SHUKLA

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=831syjfdtc.fsf@gnu.org \
    --to=eliz@gnu.org \
    --cc=24892@debbugs.gnu.org \
    --cc=eggert@cs.ucla.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.