Changeset 37d188f


Ignore:
Timestamp:
Mar 25, 2011, 3:46:46 AM (10 years ago)
Author:
David Benjamin <davidben@mit.edu>
Children:
0af5f9d
Parents:
87833a8
git-author:
David Benjamin <davidben@mit.edu> (02/26/11 14:38:04)
git-committer:
David Benjamin <davidben@mit.edu> (03/25/11 03:46:46)
Message:
Allow interrupts to be taken at any point

This way we can catch SIGINT in the middle of a search if we care.
(Though ideally we wouldn't block the event loop when searching like
this.)
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • functions.c

    rfafb842 r37d188f  
    29952995      i--;
    29962996    }
    2997 #if 0
    2998     /* FIXME!!! */
    2999     owl_function_mask_sigint(NULL);
    3000     if(owl_global_is_interrupted(&g)) {
    3001       owl_global_unset_interrupted(&g);
    3002       owl_function_unmask_sigint(NULL);
     2997    if (owl_global_take_interrupt(&g)) {
    30032998      owl_function_makemsg("Search interrupted!");
    30042999      owl_mainwin_redisplay(owl_global_get_mainwin(&g));
    30053000      return;
    30063001    }
    3007     owl_function_unmask_sigint(NULL);
    3008 #endif
    30093002  }
    30103003  owl_mainwin_redisplay(owl_global_get_mainwin(&g));
     
    30893082          ret=ZLocateUser(zstr(user), &numlocs, ZAUTH);
    30903083
    3091 #if 0
    3092           /* FIXME!! */
    3093           owl_function_mask_sigint(NULL);
    3094           if(owl_global_is_interrupted(&g)) {
     3084          if (owl_global_take_interrupt(&g)) {
    30953085            interrupted = 1;
    3096             owl_global_unset_interrupted(&g);
    3097             owl_function_unmask_sigint(NULL);
    30983086            owl_function_makemsg("Interrupted!");
    30993087            break;
    31003088          }
    3101 
    3102           owl_function_unmask_sigint(NULL);
    3103 #endif
    31043089
    31053090          if (ret!=ZERR_NONE) {
     
    35063491}
    35073492
    3508 void owl_function_mask_sigint(sigset_t *oldmask) {
    3509   sigset_t intr;
    3510 
    3511   sigemptyset(&intr);
    3512   sigaddset(&intr, SIGINT);
    3513   sigprocmask(SIG_BLOCK, &intr, oldmask);
    3514 }
    3515 
    3516 void owl_function_unmask_sigint(sigset_t *oldmask) {
    3517   sigset_t intr;
    3518 
    3519   sigemptyset(&intr);
    3520   sigaddset(&intr, SIGINT);
    3521   sigprocmask(SIG_UNBLOCK, &intr, oldmask);
    3522 }
    3523 
    35243493void _owl_function_mark_message(const owl_message *m)
    35253494{
  • global.c

    rfafb842 r37d188f  
    113113  g->timerlist = NULL;
    114114  g->kill_buffer = NULL;
     115
     116  g->interrupt_count = 0;
     117  g->interrupt_lock = g_mutex_new();
    115118}
    116119
     
    930933  g->kill_buffer = g_strndup(kill, len);
    931934}
     935
     936void owl_global_add_interrupt(owl_global *g) {
     937  /* TODO: This can almost certainly be done with atomic
     938   * operations. Whatever. */
     939  g_mutex_lock(g->interrupt_lock);
     940  g->interrupt_count++;
     941  g_mutex_unlock(g->interrupt_lock);
     942}
     943
     944bool owl_global_take_interrupt(owl_global *g) {
     945  bool ans = false;
     946  g_mutex_lock(g->interrupt_lock);
     947  if (g->interrupt_count > 0) {
     948    ans = true;
     949    g->interrupt_count--;
     950  }
     951  g_mutex_unlock(g->interrupt_lock);
     952  return ans;
     953}
  • owl.c

    r87833a8 r37d188f  
    369369  } else if (sig == SIGTERM || sig == SIGHUP) {
    370370    owl_function_quit();
    371   } else if (sig == SIGINT) {
     371  } else if (sig == SIGINT && owl_global_take_interrupt(&g)) {
    372372    owl_input in;
    373373    in.ch = in.uch = owl_global_get_startup_tio(&g)->c_cc[VINTR];
     
    381381  GSource *source;
    382382
    383   /* TODO: Special-case SIGINT so that it can interrupt outside the
    384    * event loop. */
    385 
     383  /* If it was an interrupt, set a flag so we can handle it earlier if
     384   * needbe. sig_handler_main_thread will check the flag to make sure
     385   * no one else took it. */
     386  if (sig == SIGINT) {
     387    owl_global_add_interrupt(&g);
     388  }
    386389  /* Send a message to the main thread. */
    387390  source = g_idle_source_new();
  • owl.h

    r96ade35 r37d188f  
    633633  FILE *debug_file;
    634634  char *kill_buffer;
     635  int interrupt_count;
     636  GMutex *interrupt_lock;
    635637} owl_global;
    636638
Note: See TracChangeset for help on using the changeset viewer.