Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit cf983bcb authored by Mark Salyzyn's avatar Mark Salyzyn
Browse files

liblog: add android_log_logger_list management

Add list of android_log_logger_list entities as a way of
accessing the list of transport contexts from within
one of the transports.  This will enable us to iterate back
to an internal transport context which may house a last index
or signalling semaphore to propagate a wakeup on a blocking
read.

Test: gTest libcutils-tests, logd-unit-tests, liblog-unit-tests,
      logcat-unit-tests
Bug: 27405083
Change-Id: I0a5e793946c020d97ffe10192369998e4ed92a83
parent 62d0d2d6
Loading
Loading
Loading
Loading
+36 −0
Original line number Diff line number Diff line
@@ -99,6 +99,7 @@ struct android_log_transport_read {
};

struct android_log_logger_list {
  struct listnode node;
  struct listnode logger;
  struct listnode transport;
  int mode;
@@ -146,6 +147,41 @@ struct android_log_transport_context {
         (logp) = node_to_item((logp)->node.next,                   \
                             struct android_log_logger, node))

/*
 *    Global list of log readers.
 *
 * Usage case: search out transport contexts for all readers
 */

LIBLOG_HIDDEN struct listnode __android_log_readers;

#if defined(_WIN32)
#define logger_list_rdlock()
#define logger_list_wrlock()
#define logger_list_unlock()
#else
LIBLOG_HIDDEN pthread_rwlock_t __android_log_readers_lock;

#define logger_list_rdlock() pthread_rwlock_rdlock(&__android_log_readers_lock)
#define logger_list_wrlock() pthread_rwlock_wrlock(&__android_log_readers_lock)
#define logger_list_unlock() pthread_rwlock_unlock(&__android_log_readers_lock)
#endif

/* Must be called with logger_list_rdlock() or logger_list_wrlock() held */
#define logger_list_for_each(logger_list)                              \
    for ((logger_list) = node_to_item(&__android_log_readers,          \
                                      struct android_log_logger_list,  \
                                      node);                           \
         (logger_list) != node_to_item(&__android_log_readers,         \
                                       struct android_log_logger_list, \
                                       node) &&                        \
         (logger_list) != node_to_item((logger_list)->node.next,       \
                                       struct android_log_logger_list, \
                                       node);                          \
         (logger_list) = node_to_item((logger_list)->node.next,        \
                                      struct android_log_logger_list,  \
                                      node))

/* OS specific dribs and drabs */

#if defined(_WIN32)
+19 −0
Original line number Diff line number Diff line
@@ -228,6 +228,13 @@ LIBLOG_ABI_PUBLIC int android_logger_set_prune_list(
    LOGGER_LIST_FUNCTION(logger_list, -ENODEV, setPrune, buf, len);
}

LIBLOG_HIDDEN struct listnode __android_log_readers =
    { &__android_log_readers, &__android_log_readers };
#if !defined(_WIN32)
LIBLOG_HIDDEN pthread_rwlock_t __android_log_readers_lock =
    PTHREAD_RWLOCK_INITIALIZER;
#endif

LIBLOG_ABI_PUBLIC struct logger_list *android_logger_list_alloc(
        int mode,
        unsigned int tail,
@@ -246,6 +253,10 @@ LIBLOG_ABI_PUBLIC struct logger_list *android_logger_list_alloc(
    logger_list->tail = tail;
    logger_list->pid = pid;

    logger_list_wrlock();
    list_add_tail(&__android_log_readers, &logger_list->node);
    logger_list_unlock();

    return (struct logger_list *)logger_list;
}

@@ -267,6 +278,10 @@ LIBLOG_ABI_PUBLIC struct logger_list *android_logger_list_alloc_time(
    logger_list->start = start;
    logger_list->pid = pid;

    logger_list_wrlock();
    list_add_tail(&__android_log_readers, &logger_list->node);
    logger_list_unlock();

    return (struct logger_list *)logger_list;
}

@@ -502,6 +517,10 @@ LIBLOG_ABI_PUBLIC void android_logger_list_free(struct logger_list *logger_list)
        return;
    }

    logger_list_wrlock();
    list_remove(&logger_list_internal->node);
    logger_list_unlock();

    while (!list_empty(&logger_list_internal->transport)) {
        struct listnode *node = list_head(&logger_list_internal->transport);
        struct android_log_transport_context *transp =