You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2021/04/07 15:28:44 UTC

[trafficserver] branch 9.1.x updated: Rollback LAZY_BUF_ALLOC remove in HttpTunnel (#7583)

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch 9.1.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/9.1.x by this push:
     new d33db23  Rollback LAZY_BUF_ALLOC remove in HttpTunnel (#7583)
d33db23 is described below

commit d33db239046eb984bcb90ce19aa87c197e62b889
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Thu Apr 8 00:28:22 2021 +0900

    Rollback LAZY_BUF_ALLOC remove in HttpTunnel (#7583)
    
    (cherry picked from commit 9ed916c7d08d357e02da8824c3df6d2a526eca7a)
---
 proxy/http/HttpTunnel.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/proxy/http/HttpTunnel.cc b/proxy/http/HttpTunnel.cc
index 6192516..53596f3 100644
--- a/proxy/http/HttpTunnel.cc
+++ b/proxy/http/HttpTunnel.cc
@@ -1234,7 +1234,7 @@ HttpTunnel::consumer_reenable(HttpTunnelConsumer *c)
 {
   HttpTunnelProducer *p = c->producer;
 
-  if (p && p->alive && p->read_buffer->write_avail() > 0) {
+  if (p && p->alive) {
     // Only do flow control if enabled and the producer is an external
     // source.  Otherwise disable by making the backlog zero. Because
     // the backlog short cuts quit when the value is equal (or
@@ -1370,7 +1370,7 @@ HttpTunnel::consumer_handler(int event, HttpTunnelConsumer *c)
     //    the SM since the reenabling has the side effect
     //    updating the buffer state for the VConnection
     //    that is being reenabled
-    if (p->alive && p->read_vio && p->read_buffer->write_avail() > 0) {
+    if (p->alive && p->read_vio) {
       if (p->is_throttled()) {
         this->consumer_reenable(c);
       } else {