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

Commit 2beabe5c authored by Tom Cherry's avatar Tom Cherry
Browse files

liblog: remove config_write.cpp/.h 

Now that we only have one transport in a given build configuration,
there is no need to have these lists.

Test: liblog-unit-tests
Change-Id: I3fdcdc33c1224a9522080ea06f36f14c83d946ac
parent 9d350344

liblog/Android.bp

+0 −1
Original line number Diff line number Diff line
@@ -15,7 +15,6 @@ 
//



liblog_sources = [

    "config_write.cpp",

    "log_event_list.cpp",

    "log_event_write.cpp",

    "logger_lock.cpp",

liblog/config_write.cpp

deleted100644 → 0
+0 −79
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2016 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */



#include "config_write.h"

#include "logger.h"



struct listnode __android_log_transport_write = {&__android_log_transport_write,

                                                 &__android_log_transport_write};

struct listnode __android_log_persist_write = {&__android_log_persist_write,

                                               &__android_log_persist_write};



static void __android_log_add_transport(struct listnode* list,

                                        struct android_log_transport_write* transport) {

  uint32_t i;



  /* Try to keep one functioning transport for each log buffer id */

  for (i = LOG_ID_MIN; i < LOG_ID_MAX; i++) {

    struct android_log_transport_write* transp;



    if (list_empty(list)) {

      if (!transport->available || ((*transport->available)(static_cast<log_id_t>(i)) >= 0)) {

        list_add_tail(list, &transport->node);

        return;

      }

    } else {

      write_transport_for_each(transp, list) {

        if (!transp->available) {

          return;

        }

        if (((*transp->available)(static_cast<log_id_t>(i)) < 0) &&

            (!transport->available || ((*transport->available)(static_cast<log_id_t>(i)) >= 0))) {

          list_add_tail(list, &transport->node);

          return;

        }

      }

    }

  }

}



void __android_log_config_write() {

#if (FAKE_LOG_DEVICE == 0)

  extern struct android_log_transport_write logdLoggerWrite;

  extern struct android_log_transport_write pmsgLoggerWrite;



  __android_log_add_transport(&__android_log_transport_write, &logdLoggerWrite);

  __android_log_add_transport(&__android_log_persist_write, &pmsgLoggerWrite);

#else

  extern struct android_log_transport_write fakeLoggerWrite;



  __android_log_add_transport(&__android_log_transport_write, &fakeLoggerWrite);

#endif

}



void __android_log_config_write_close() {

  struct android_log_transport_write* transport;

  struct listnode* n;



  write_transport_for_each_safe(transport, n, &__android_log_transport_write) {

    transport->logMask = 0;

    list_remove(&transport->node);

  }

  write_transport_for_each_safe(transport, n, &__android_log_persist_write) {

    transport->logMask = 0;

    list_remove(&transport->node);

  }

}

liblog/config_write.h

deleted100644 → 0
+0 −52
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2016 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */



#pragma once



#include <cutils/list.h>



#include "log_portability.h"



__BEGIN_DECLS



extern struct listnode __android_log_transport_write;

extern struct listnode __android_log_persist_write;



#define write_transport_for_each(transp, transports)                           \

  for ((transp) = node_to_item((transports)->next,                             \

                               struct android_log_transport_write, node);      \

       ((transp) != node_to_item((transports),                                 \

                                 struct android_log_transport_write, node)) && \

       ((transp) != node_to_item((transp)->node.next,                          \

                                 struct android_log_transport_write, node));   \

       (transp) = node_to_item((transp)->node.next,                            \

                               struct android_log_transport_write, node))



#define write_transport_for_each_safe(transp, n, transports)                   \

  for ((transp) = node_to_item((transports)->next,                             \

                               struct android_log_transport_write, node),      \

      (n) = (transp)->node.next;                                               \

       ((transp) != node_to_item((transports),                                 \

                                 struct android_log_transport_write, node)) && \

       ((transp) !=                                                            \

        node_to_item((n), struct android_log_transport_write, node));          \

       (transp) = node_to_item((n), struct android_log_transport_write, node), \

      (n) = (transp)->node.next)



void __android_log_config_write();

void __android_log_config_write_close();



__END_DECLS

liblog/fake_writer.cpp

+2 −3
Original line number Diff line number Diff line
@@ -20,7 +20,6 @@ 


#include <log/log.h>



#include "config_write.h"

#include "fake_log_device.h"

#include "log_portability.h"

#include "logger.h"
@@ -32,9 +31,9 @@ static int fakeWrite(log_id_t log_id, struct timespec* ts, struct iovec* vec, si 
static int logFds[(int)LOG_ID_MAX] = {-1, -1, -1, -1, -1, -1};



struct android_log_transport_write fakeLoggerWrite = {

    .node = {&fakeLoggerWrite.node, &fakeLoggerWrite.node},

    .context.priv = &logFds,

    .name = "fake",

    .logMask = 0,

    .context.priv = &logFds,

    .available = NULL,

    .open = fakeOpen,

    .close = fakeClose,

liblog/logd_writer.cpp

+2 −3
Original line number Diff line number Diff line
@@ -34,7 +34,6 @@ 
#include <private/android_filesystem_config.h>

#include <private/android_logger.h>



#include "config_write.h"

#include "log_portability.h"

#include "logger.h"

#include "uio.h"
@@ -48,9 +47,9 @@ static void logdClose(); 
static int logdWrite(log_id_t logId, struct timespec* ts, struct iovec* vec, size_t nr);



struct android_log_transport_write logdLoggerWrite = {

    .node = {&logdLoggerWrite.node, &logdLoggerWrite.node},

    .context.sock = -EBADF,

    .name = "logd",

    .logMask = 0,

    .context.sock = -EBADF,

    .available = logdAvailable,

    .open = logdOpen,

    .close = logdClose,