All of lore.kernel.org
 help / color / mirror / Atom feed
From: Paolo Abeni <pabeni@redhat.com>
To: Geliang Tang <geliangtang@gmail.com>, mptcp@lists.linux.dev
Subject: Re: [MPTCP][PATCH v2 mptcp-next 0/5] data checksum support cleanups
Date: Thu, 06 May 2021 15:09:42 +0200	[thread overview]
Message-ID: <d98c77f57836940c9af061478afffc214209a89e.camel@redhat.com> (raw)
In-Reply-To: <cover.1620286893.git.geliangtang@gmail.com>

On Thu, 2021-05-06 at 15:48 +0800, Geliang Tang wrote:
> v2:
>  - add two new squash-to patches to rename the mib DSSCSUMERR to
>    DATACSUMERR.
> 
> Please insert patch 4 (mptcp: dump csum fields in mptcp_dump_mpext) between the
> commits "mptcp: add a new sysctl checksum_enabled" and "selftests: mptcp: 
> enable checksum in mptcp_connect.sh".
> 
> Geliang Tang (5):
>   Squash to "mptcp: add csum_reqd in mptcp_out_options"
>   Squash to "mptcp: add the mib for data checksum"
>   Squash to "mptcp: add a new sysctl checksum_enabled"
>   mptcp: dump csum fields in mptcp_dump_mpext
>   Squash to "selftests: mptcp: enable checksum in mptcp_join.sh"
> 
>  Documentation/networking/mptcp-sysctl.rst       |  2 +-
>  include/net/mptcp.h                             |  4 ++--
>  include/trace/events/mptcp.h                    | 17 +++++++++++------
>  net/mptcp/ctrl.c                                |  8 +++++---
>  net/mptcp/mib.c                                 |  2 +-
>  net/mptcp/mib.h                                 |  2 +-
>  net/mptcp/subflow.c                             |  2 +-
>  tools/testing/selftests/net/mptcp/mptcp_join.sh |  4 ++--
>  8 files changed, 24 insertions(+), 17 deletions(-)
> 

I had a very minor comment on patch 4/5, but I think this can be merged
as is and ev. squash an additional change later, if needed/there is
agreement.

Cheers,

Paolo


  parent reply	other threads:[~2021-05-06 13:09 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-06  7:48 [MPTCP][PATCH v2 mptcp-next 0/5] data checksum support cleanups Geliang Tang
2021-05-06  7:48 ` [MPTCP][PATCH v2 mptcp-next 1/5] Squash to "mptcp: add csum_reqd in mptcp_out_options" Geliang Tang
2021-05-06  7:48   ` [MPTCP][PATCH v2 mptcp-next 2/5] Squash to "mptcp: add the mib for data checksum" Geliang Tang
2021-05-06  7:48     ` [MPTCP][PATCH v2 mptcp-next 3/5] Squash to "mptcp: add a new sysctl checksum_enabled" Geliang Tang
2021-05-06  7:48       ` [MPTCP][PATCH v2 mptcp-next 4/5] mptcp: dump csum fields in mptcp_dump_mpext Geliang Tang
2021-05-06  7:48         ` [MPTCP][PATCH v2 mptcp-next 5/5] Squash to "selftests: mptcp: enable checksum in mptcp_join.sh" Geliang Tang
2021-05-06 13:08         ` [MPTCP][PATCH v2 mptcp-next 4/5] mptcp: dump csum fields in mptcp_dump_mpext Paolo Abeni
2021-05-07  2:50     ` [MPTCP][PATCH v2 mptcp-next 2/5] Squash to "mptcp: add the mib for data checksum" Geliang Tang
2021-05-07  8:07       ` Matthieu Baerts
2021-05-06 13:09 ` Paolo Abeni [this message]
2021-05-06 14:28 ` [MPTCP][PATCH v2 mptcp-next 0/5] data checksum support cleanups Matthieu Baerts

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=d98c77f57836940c9af061478afffc214209a89e.camel@redhat.com \
    --to=pabeni@redhat.com \
    --cc=geliangtang@gmail.com \
    --cc=mptcp@lists.linux.dev \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.