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/21 21:36:31 UTC

[trafficserver] branch 9.0.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.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


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

commit 522b06a8b37888d632fe00cc8d89b4e034183a0f
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 1f86437..e92ee98 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 {