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

Commit 89bbefcb authored by Corey Tabaka's avatar Corey Tabaka
Browse files

libvrflinger: Move some tracing to a more verbose level. 

Quiet the logs when using TRACE=1.

Bug: None
Test: Set TRACE=1; build; observe traces do not appear.
Change-Id: I521fa5cf4b0ab18e73f89a1898bf2d58fc2d1f88
parent 0b485c91

libs/vr/libvrflinger/display_surface.cpp

+3 −2
Original line number Diff line number Diff line
@@ -127,7 +127,8 @@ void DisplaySurface::SurfaceUpdated(display::SurfaceUpdateFlags update_flags) { 
}



void DisplaySurface::ClearUpdate() {

  ALOGD_IF(TRACE, "DisplaySurface::ClearUpdate: surface_id=%d", surface_id());

  ALOGD_IF(TRACE > 1, "DisplaySurface::ClearUpdate: surface_id=%d",

           surface_id());

  update_flags_ = display::SurfaceUpdateFlags::None;

}
@@ -333,7 +334,7 @@ void DirectDisplaySurface::DequeueBuffersLocked() { 
    auto buffer_status = direct_queue_->Dequeue(0, &slot, &acquire_fence);

    if (!buffer_status) {

      ALOGD_IF(

          TRACE && buffer_status.error() == ETIMEDOUT,

          TRACE > 1 && buffer_status.error() == ETIMEDOUT,

          "DirectDisplaySurface::DequeueBuffersLocked: All buffers dequeued.");

      ALOGE_IF(buffer_status.error() != ETIMEDOUT,

               "DirectDisplaySurface::DequeueBuffersLocked: Failed to dequeue "

libs/vr/libvrflinger/epoll_event_dispatcher.cpp

+2 −2
Original line number Diff line number Diff line
@@ -101,12 +101,12 @@ void EpollEventDispatcher::EventThread() { 
    if (num_events < 0 && errno != EINTR)

      break;



    ALOGD_IF(TRACE, "EpollEventDispatcher::EventThread: num_events=%d",

    ALOGD_IF(TRACE > 1, "EpollEventDispatcher::EventThread: num_events=%d",

             num_events);



    for (int i = 0; i < num_events; i++) {

      ALOGD_IF(

          TRACE,

          TRACE > 1,

          "EpollEventDispatcher::EventThread: event %d: handler=%p events=0x%x",

          i, events[i].data.ptr, events[i].events);

libs/vr/libvrflinger/hardware_composer.cpp

+1 −1
Original line number Diff line number Diff line
@@ -429,7 +429,7 @@ void HardwareComposer::PostLayers() { 
    ATRACE_INT("frame_skip_count", 0);

  }



#if TRACE

#if TRACE > 1

  for (size_t i = 0; i < active_layer_count_; i++) {

    ALOGI("HardwareComposer::PostLayers: layer=%zu buffer_id=%d composition=%s",

          i, layers_[i].GetBufferId(),

libs/vr/libvrflinger/vsync_service.cpp

+2 −2
Original line number Diff line number Diff line
@@ -200,12 +200,12 @@ pdx::Status<int64_t> VSyncWaiter::OnWait(pdx::Message& /*message*/) { 
}



void VSyncChannel::Ack() {

  ALOGD_IF(TRACE, "VSyncChannel::Ack: pid=%d cid=%d\n", pid_, cid_);

  ALOGD_IF(TRACE > 1, "VSyncChannel::Ack: pid=%d cid=%d\n", pid_, cid_);

  service_.ModifyChannelEvents(cid_, POLLPRI, 0);

}



void VSyncChannel::Signal() {

  ALOGD_IF(TRACE, "VSyncChannel::Signal: pid=%d cid=%d\n", pid_, cid_);

  ALOGD_IF(TRACE > 1, "VSyncChannel::Signal: pid=%d cid=%d\n", pid_, cid_);

  service_.ModifyChannelEvents(cid_, 0, POLLPRI);

}