unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Yuan Fu <casouri@gmail.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: emacs-devel@gnu.org
Subject: Re: Existing redisplay profiling tool?
Date: Sun, 15 Sep 2024 23:16:48 -0700	[thread overview]
Message-ID: <2A088262-BE9A-4289-A55A-128FB3D8E229@gmail.com> (raw)
In-Reply-To: <86y13udpbi.fsf@gnu.org>



> On Sep 14, 2024, at 12:10 PM, Eli Zaretskii <eliz@gnu.org> wrote:
> 
>> From: Yuan Fu <casouri@gmail.com>
>> Date: Sat, 14 Sep 2024 11:20:15 -0700
>> 
>> I’m doing some optimization for tree-sitter font-lock, and want to measure the difference in a real editing session (rather than fontifying the whole buffer 10 times). I’m sure there’s some tool for measuring time spent in redisplay some where, but couldn’t find anything poking around.
> 
> How about using the scrolling benchmark through xdisp.c while
> profiling?
> 
> (defun scroll-up-benchmark ()
>  (interactive)
>  (let ((oldgc gcs-done)
>        (oldtime (float-time)))
>    (condition-case nil (while t (scroll-up) (redisplay))
>      (error (message "GCs: %d Elapsed time: %f seconds"
>                      (- gcs-done oldgc) (- (float-time) oldtime))))))
> 
> Evaluate the above, then turn on profiling, then type
> "M-x scroll-up-benchmark RET" with point at beginning
> of buffer that visits xdisp.c under c-ts-mode.

Thanks! I’m hoping to measure the perceived responsiveness when typing text in the buffer, so here’s what I came up with:

(let ((prev-time (current-time))
      (measurements nil))
  (dotimes (_ 100)
    (insert "s")
    (redisplay)
    (push (float-time (time-subtract (current-time) prev-time))
          measurements)
    (setq prev-time (current-time)))
  (message "Average time: %f"
           (/ (apply #'+ measurements) (length measurements))))

Do you think this accurately measures the redisplay time between each keystroke? (Obviously this doesn’t take account of post-command-hook, I only want to measure repose & redisplay here.)

If that’s an accurate measurements, then my optimization doesn’t make any significant difference :-)
In xdisp.c, inserting 100 “s” takes about 17.8ms per “keystroke” on average. With optimization it’s about 16.7ms, not much difference.

Obviously the time differs depending on where are you inserting the “s”, but I tried a few places (close to beginning of buffer, close to end, inside a function, outside a function, etc), and the measurements doesn’t vary too much (12–16ms with optimization on)

I also measured a base performance by enabling fundamental-mode in xdisp.c. It takes about 2.2ms between each “keystroke”.

In a very small file, the average time between each “keystroke” is 3.2ms and 3.5ms, for optimized and unoptimized run respectively.

Finally, c-mode in xdisp.c takes about 10.7ms per “keystroke” ;-)

I guess the takeaway is a) my new optimization doesn’t do anything, and b) tree-sitter font-lock doesn’t add human-perceivable latency when typing.

Yuan

My measurements:

xdisp.c:
optimization off: Average time: 0.017755 -> 17.8ms
optimization  on: Average time: 0.016701 -> 16.7ms
fundamental mode: Average time: 0.002188 -> 2.2ms

optimization on w/ post-command-hook: Average time: 0.033086 -> 33.1ms


small file:
optimization  on: Average time: 0.003207 -> 3.2ms
optimization off: Average time: 0.003473 -> 3.5ms

optimization on w/ post-command-hook: Average time: 0.013553 -> 13.6ms

c-mode:
xdisp.c : Average time: 0.010691 -> 10.7ms
small file: Average time: 0.005141 -> 5.1ms


  reply	other threads:[~2024-09-16  6:16 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-09-14 18:20 Existing redisplay profiling tool? Yuan Fu
2024-09-14 19:10 ` Eli Zaretskii
2024-09-16  6:16   ` Yuan Fu [this message]
2024-09-16 12:19     ` Eli Zaretskii
2024-09-17  2:49       ` Yuan Fu
2024-09-17 12:38         ` Eli Zaretskii

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

  List information: https://www.gnu.org/software/emacs/

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

  git send-email \
    --in-reply-to=2A088262-BE9A-4289-A55A-128FB3D8E229@gmail.com \
    --to=casouri@gmail.com \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    /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 public inbox

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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).