Edit

kc3-lang/libevent/defer-internal.h

Branch :

  • Show log

    Commit

  • Author : Nick Mathewson
    Date : 2012-05-09 11:06:06
    Hash : c0e425ab
    Message : Restore our priority-inversion-prevention code with deferreds Back when deferred_cb stuff had its own queue, the queue was always executed, but we never ran more than 16 callbacks per iteration. That made for two problems: 1: Because deferred_cb stuff would always run, and had no priority, it could cause priority inversion. 2: It doesn't respect the max_dispatch_interval code. Then, when I refactored deferred_cb to be a special case of event_callback, that solved the above issues, but made for two more issues: 3: Because deferred_cb stuff would always get the default priority, it could could low-priority bufferevents to get too much priority. 4: With code like bufferevent_pair, it's easy to get into a situation where two deferreds keep adding one another, preventing the event loop from ever actually scanning for more events. This commit fixes the above by giving deferreds a better notion of priorities, and by limiting the number of deferreds that can be added to the _current_ loop iteration's active queues. (Extra deferreds are put into the active_later state.) That isn't an all-purpose priority inversion solution, of course: for that, you may need to mess around with max_dispatch_interval.

  • defer-internal.h
  • /*
     * Copyright (c) 2009-2012 Niels Provos and Nick Mathewson
     *
     * Redistribution and use in source and binary forms, with or without
     * modification, are permitted provided that the following conditions
     * are met:
     * 1. Redistributions of source code must retain the above copyright
     *    notice, this list of conditions and the following disclaimer.
     * 2. Redistributions in binary form must reproduce the above copyright
     *    notice, this list of conditions and the following disclaimer in the
     *    documentation and/or other materials provided with the distribution.
     * 3. The name of the author may not be used to endorse or promote products
     *    derived from this software without specific prior written permission.
     *
     * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR
     * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
     * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
     * IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT,
     * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
     * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
     * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
     * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
     * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
     * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
     */
    #ifndef DEFER_INTERNAL_H_INCLUDED_
    #define DEFER_INTERNAL_H_INCLUDED_
    
    #ifdef __cplusplus
    extern "C" {
    #endif
    
    #include "event2/event-config.h"
    #include "evconfig-private.h"
    
    #include <sys/queue.h>
    
    struct event_callback;
    typedef void (*deferred_cb_fn)(struct event_callback *, void *);
    
    /**
       Initialize an empty, non-pending event_callback.
    
       @param deferred The struct event_callback structure to initialize.
       @param priority The priority that the callback should run at.
       @param cb The function to run when the struct event_callback executes.
       @param arg The function's second argument.
     */
    void event_deferred_cb_init_(struct event_callback *, ev_uint8_t, deferred_cb_fn, void *);
    /**
       Change the priority of a non-pending event_callback.
     */
    void event_deferred_cb_set_priority_(struct event_callback *, ev_uint8_t);
    /**
       Cancel a struct event_callback if it is currently scheduled in an event_base.
     */
    void event_deferred_cb_cancel_(struct event_base *, struct event_callback *);
    /**
       Activate a struct event_callback if it is not currently scheduled in an event_base.
    
       Return true iff it was not previously scheduled.
     */
    int event_deferred_cb_schedule_(struct event_base *, struct event_callback *);
    
    #ifdef __cplusplus
    }
    #endif
    
    #endif /* EVENT_INTERNAL_H_INCLUDED_ */