Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
From:   jackygam2001 <jacky_gam_2001@163.com>
To:     edumazet@google.com
Cc:     davem@davemloft.net, kuba@kernel.org, pabeni@redhat.com,
        rostedt@goodmis.org, mingo@redhat.com, yoshfuji@linux-ipv6.org,
        dsahern@kernel.org, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org, yhs@fb.com, ping.gan@dell.com
Subject: Re: [PATCH net-next] tcp: Add tracepoint for tcp_set_ca_state
Date:   Fri,  8 Apr 2022 08:40:49 +0800
Message-Id: <20220408004050.27292-1-jacky_gam_2001@163.com>
In-Reply-To: <CANn89iKXK6m45eBDCDw8Pij1H=iybZTTi2Yf9DFYu96KDpnyPA@mail.gmail.com>
References: <CANn89iKXK6m45eBDCDw8Pij1H=iybZTTi2Yf9DFYu96KDpnyPA@mail.gmail.com>
Reply-To: edumazet@google.com
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Precedence: bulk

At 2022-04-07 14:05:42, "Eric Dumazet" <edumazet@google.com> wrote:
> On Tue, Apr 5, 2022 at 6:12 PM jackygam2001 <jacky_gam_2001@163.com> wrote:
> >
> > From: Ping Gan <jacky_gam_2001@163.com>
> >
> > The congestion status of a tcp flow may be updated since there
> > is congestion between tcp sender and receiver. It makes sense to
> > add tracepoint for congestion status set function to summate cc
> > status duration and evaluate the performance of network
> > and congestion algorithm. the backgound of this patch is below.
> >
> > Link: https://github.com/iovisor/bcc/pull/3899
> >
> > Signed-off-by: Ping Gan <jacky_gam_2001@163.com>
> > ---
>
> This seems good to me, thanks.
>
> Reviewed-by: Eric Dumazet <edumazet@google.com>
Thanks for you comment and review!