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

Commit 3f625c85 authored by Myles Watson's avatar Myles Watson Committed by Automerger Merge Worker
Browse files

Merge "gd hci: Handle overflow of acl credits" am: be74cf8d 

Original change: https://android-review.googlesource.com/c/platform/system/bt/+/1652247

Change-Id: I8011397f11dea5baa5e19cdf6da10d98afa28899
parents 61f641b4 be74cf8d

system/gd/hci/acl_manager/round_robin_scheduler.cc

+16 −3
Original line number Diff line number Diff line
@@ -205,14 +205,27 @@ void RoundRobinScheduler::incoming_acl_credits(uint16_t handle, uint16_t credits 
    LOG_INFO("Dropping %hx received credits to unknown connection 0x%0hx", credits, handle);

    return;

  }



  if (acl_queue_handler->second.number_of_sent_packets_ >= credits) {

    acl_queue_handler->second.number_of_sent_packets_ -= credits;

  } else {

    LOG_WARN("receive more credits than we sent");

    acl_queue_handler->second.number_of_sent_packets_ = 0;

  }



  if (acl_queue_handler->second.connection_type_ == ConnectionType::CLASSIC) {

    acl_packet_credits_ += credits;

    if (acl_packet_credits_ > max_acl_packet_credits_) {

      acl_packet_credits_ = max_acl_packet_credits_;

      LOG_WARN("acl packet credits overflow due to receive %hx credits", credits);

    }

  } else {

    le_acl_packet_credits_ += credits;

    if (le_acl_packet_credits_ > le_max_acl_packet_credits_) {

      le_acl_packet_credits_ = le_max_acl_packet_credits_;

      LOG_WARN("le acl packet credits overflow due to receive %hx credits", credits);

    }

  }

  ASSERT(acl_packet_credits_ <= max_acl_packet_credits_);

  ASSERT(le_acl_packet_credits_ <= le_max_acl_packet_credits_);

  if (acl_packet_credits_ == credits || le_acl_packet_credits_ == credits) {

    start_round_robin();

  }