From 1d3fba86a3561c93cfbb42137c89fa85c65b0f2b Mon Sep 17 00:00:00 2001 From: Nicholas Marriott Date: Sat, 7 Feb 2009 19:16:25 +0000 Subject: [PATCH] Make process name guessing pick up SSTOP. --- TODO | 2 ++ osdep-freebsd.c | 14 +++++++++++--- osdep-openbsd.c | 18 ++++++++++++------ 3 files changed, 25 insertions(+), 9 deletions(-) diff --git a/TODO b/TODO index 3fc90ed7..76f69f76 100644 --- a/TODO +++ b/TODO @@ -86,3 +86,5 @@ - 88 colour support; new grid cell flag, and 256<->88 88<->16 translation tables - some fix for SF feature request 2527847 - now remain-by-default has gone cannot control it per-session +- if the child is suspended in window with eg ^Z it should be restarted, or + ^Z should be ignored diff --git a/osdep-freebsd.c b/osdep-freebsd.c index 1795058a..c7864e4b 100644 --- a/osdep-freebsd.c +++ b/osdep-freebsd.c @@ -1,4 +1,4 @@ -/* $Id: osdep-freebsd.c,v 1.4 2009-01-27 23:10:18 nicm Exp $ */ +/* $Id: osdep-freebsd.c,v 1.5 2009-02-07 19:16:25 nicm Exp $ */ /* * Copyright (c) 2009 Nicholas Marriott @@ -36,6 +36,11 @@ char *get_proc_argv0(pid_t); #define nitems(_a) (sizeof((_a)) / sizeof((_a)[0])) +#define is_runnable(p) \ + ((p)->ki_stat == SRUN || (p)->ki_stat == SIDL) +#define is_stopped(p) \ + ((p)->ki_stat == SSTOP || (p)->ki_stat == SZOMB || (p)->ki_stat == SDEAD) + char * get_argv0(__attribute__ ((unused)) int fd, char *tty) { @@ -78,8 +83,11 @@ retry: if (bestp == NULL) bestp = p; - if (p->ki_stat != SRUN && p->ki_stat != SIDL) - continue; + if (is_runnable(p) && !is_runnable(bestp)) + bestp = p; + if (!is_stopped(p) && is_stopped(bestp)) + bestp = p; + if (p->ki_estcpu < bestp->ki_estcpu) continue; if (p->ki_slptime > bestp->ki_slptime) diff --git a/osdep-openbsd.c b/osdep-openbsd.c index 4ec03dc8..8cbf6f3d 100644 --- a/osdep-openbsd.c +++ b/osdep-openbsd.c @@ -1,4 +1,4 @@ -/* $Id: osdep-openbsd.c,v 1.7 2009-01-30 21:18:26 nicm Exp $ */ +/* $Id: osdep-openbsd.c,v 1.8 2009-02-07 19:16:25 nicm Exp $ */ /* * Copyright (c) 2009 Nicholas Marriott @@ -30,6 +30,11 @@ #define nitems(_a) (sizeof((_a)) / sizeof((_a)[0])) +#define is_runnable(p) \ + ((p)->p_stat == SRUN || (p)->p_stat == SIDL || (p)->p_stat == SONPROC) +#define is_stopped(p) \ + ((p)->p_stat == SSTOP || (p)->p_stat == SZOMB || (p)->p_stat == SDEAD) + char *get_argv0(int, char *); char *get_proc_argv0(pid_t); @@ -76,17 +81,18 @@ retry: if (bestp == NULL) bestp = p; - if (p->p_stat != SRUN && - p->p_stat != SIDL && - p->p_stat != SONPROC) - continue; + if (is_runnable(p) && !is_runnable(bestp)) + bestp = p; + if (!is_stopped(p) && is_stopped(bestp)) + bestp = p; + if (p->p_estcpu < bestp->p_estcpu) continue; if (p->p_slptime > bestp->p_slptime) continue; if (!(p->p_flag & P_SINTR) && bestp->p_flag & P_SINTR) continue; - if (p->p_pid < bestp->p_pid) + if (LIST_FIRST(&p->p_children) != NULL) continue; bestp = p; }