patch 8.0.0097
Problem: When a channel callback consumes a lot of time Vim becomes
unresponsive. (skywind)
Solution: Bail out of checking channel readahead after 100 msec.
diff --git a/src/os_unix.c b/src/os_unix.c
index a63eb6e..12fb33b 100644
--- a/src/os_unix.c
+++ b/src/os_unix.c
@@ -376,21 +376,6 @@
RealWaitForChar(read_cmd_fd, p_wd, NULL, NULL);
}
-#if defined(HAVE_GETTIMEOFDAY) && defined(HAVE_SYS_TIME_H)
-/*
- * Return time in msec since "start_tv".
- */
- static long
-elapsed(struct timeval *start_tv)
-{
- struct timeval now_tv;
-
- gettimeofday(&now_tv, NULL);
- return (now_tv.tv_sec - start_tv->tv_sec) * 1000L
- + (now_tv.tv_usec - start_tv->tv_usec) / 1000L;
-}
-#endif
-
/*
* mch_inchar(): low level input function.
* Get a characters from the keyboard.
@@ -411,10 +396,10 @@
int did_start_blocking = FALSE;
long wait_time;
long elapsed_time = 0;
-#if defined(HAVE_GETTIMEOFDAY) && defined(HAVE_SYS_TIME_H)
- struct timeval start_tv;
+#ifdef ELAPSED_FUNC
+ ELAPSED_TYPE start_tv;
- gettimeofday(&start_tv, NULL);
+ ELAPSED_INIT(start_tv);
#endif
/* repeat until we got a character or waited long enough */
@@ -438,8 +423,8 @@
else
/* going to block after p_ut */
wait_time = p_ut;
-#if defined(HAVE_GETTIMEOFDAY) && defined(HAVE_SYS_TIME_H)
- elapsed_time = elapsed(&start_tv);
+#ifdef ELAPSED_FUNC
+ elapsed_time = ELAPSED_FUNC(start_tv);
#endif
wait_time -= elapsed_time;
if (wait_time < 0)
@@ -1554,18 +1539,16 @@
#ifdef FEAT_X11
-# if defined(HAVE_GETTIMEOFDAY) && defined(HAVE_SYS_TIME_H) \
+# if defined(ELAPSED_TIMEVAL) \
&& (defined(FEAT_XCLIPBOARD) || defined(FEAT_TITLE))
-static void xopen_message(struct timeval *start_tv);
-
/*
* Give a message about the elapsed time for opening the X window.
*/
static void
-xopen_message(struct timeval *start_tv)
+xopen_message(long elapsed_msec)
{
- smsg((char_u *)_("Opening the X display took %ld msec"), elapsed(start_tv));
+ smsg((char_u *)_("Opening the X display took %ld msec"), elapsed_msec);
}
# endif
#endif
@@ -1864,11 +1847,11 @@
#endif
if (x11_display != NULL)
{
-# if defined(HAVE_GETTIMEOFDAY) && defined(HAVE_SYS_TIME_H)
+# ifdef ELAPSED_FUNC
if (p_verbose > 0)
{
verbose_enter();
- xopen_message(&start_tv);
+ xopen_message(ELAPSED_FUNC(start_tv));
verbose_leave();
}
# endif
@@ -4630,8 +4613,8 @@
ga_init2(&ga, 1, BUFLEN);
noread_cnt = 0;
-# if defined(HAVE_GETTIMEOFDAY) && defined(HAVE_SYS_TIME_H)
- gettimeofday(&start_tv, NULL);
+# ifdef ELAPSED_FUNC
+ ELAPSED_INIT(start_tv);
# endif
for (;;)
{
@@ -4666,8 +4649,8 @@
/* Get extra characters when we don't have any.
* Reset the counter and timer. */
noread_cnt = 0;
-# if defined(HAVE_GETTIMEOFDAY) && defined(HAVE_SYS_TIME_H)
- gettimeofday(&start_tv, NULL);
+# ifdef ELAPSED_FUNC
+ ELAPSED_INIT(start_tv);
# endif
len = ui_inchar(ta_buf, BUFLEN, 10L, 0);
}
@@ -4886,10 +4869,10 @@
if (got_int)
break;
-# if defined(HAVE_GETTIMEOFDAY) && defined(HAVE_SYS_TIME_H)
+# ifdef ELAPSED_FUNC
if (wait_pid == 0)
{
- long msec = elapsed(&start_tv);
+ long msec = ELAPSED_FUNC(start_tv);
/* Avoid that we keep looping here without
* checking for a CTRL-C for a long time. Don't
@@ -5632,15 +5615,14 @@
/* May retry getting characters after an event was handled. */
# define MAY_LOOP
-# if defined(HAVE_GETTIMEOFDAY) && defined(HAVE_SYS_TIME_H)
+# ifdef ELAPSED_FUNC
/* Remember at what time we started, so that we know how much longer we
* should wait after being interrupted. */
-# define USE_START_TV
long start_msec = msec;
- struct timeval start_tv;
+ ELAPSED_TYPE start_tv;
if (msec > 0)
- gettimeofday(&start_tv, NULL);
+ ELAPSED_INIT(start_tv);
# endif
/* Handle being called recursively. This may happen for the session
@@ -5947,9 +5929,9 @@
/* We're going to loop around again, find out for how long */
if (msec > 0)
{
-# ifdef USE_START_TV
+# ifdef ELAPSED_FUNC
/* Compute remaining wait time. */
- msec = start_msec - elapsed(&start_tv);
+ msec = start_msec - ELAPSED_FUNC(start_tv);
# else
/* Guess we got interrupted halfway. */
msec = msec / 2;
@@ -7046,11 +7028,11 @@
#if defined(HAVE_SETJMP_H)
int (*oldIOhandler)();
#endif
-# if defined(HAVE_GETTIMEOFDAY) && defined(HAVE_SYS_TIME_H)
- struct timeval start_tv;
+# ifdef ELAPSED_FUNC
+ ELAPSED_TYPE start_tv;
if (p_verbose > 0)
- gettimeofday(&start_tv, NULL);
+ ELAPSED_INIT(start_tv);
# endif
/* Ignore X errors while opening the display */
@@ -7092,11 +7074,11 @@
/* Catch terminating error of the X server connection. */
(void)XSetIOErrorHandler(x_IOerror_handler);
-# if defined(HAVE_GETTIMEOFDAY) && defined(HAVE_SYS_TIME_H)
+# ifdef ELAPSED_FUNC
if (p_verbose > 0)
{
verbose_enter();
- xopen_message(&start_tv);
+ xopen_message(ELAPSED_FUNC(start_tv));
verbose_leave();
}
# endif