Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Norman Maurer <norman.maurer@googlemail.com>
Content-Type: text/plain;
        charset=utf-8
Content-Transfer-Encoding: quoted-printable
Mime-Version: 1.0 (Mac OS X Mail 14.0 \(3654.60.0.2.21\))
Subject: Re: [PATCH] udp: Add support for getsockopt(..., ..., UDP_GRO, ...,
 ...)
Date:   Wed, 31 Mar 2021 15:10:01 +0200
References: <20210325195614.800687-1-norman_maurer@apple.com>
 <8eadc07055ac1c99bbc55ea10c7b98acc36dde55.camel@redhat.com>
 <CF78DCAD-6F2C-46C4-9FF1-61DF66183C76@apple.com>
 <2e667826f183fbef101a62f0ad8ccb4ed253cb75.camel@redhat.com>
To:     Paolo Abeni <pabeni@redhat.com>, netdev@vger.kernel.org,
        linux-kernel@vger.kernel.org, dsahern@kernel.org,
        davem@davemloft.net
In-Reply-To: <2e667826f183fbef101a62f0ad8ccb4ed253cb75.camel@redhat.com>
Message-Id: <71BBD1B0-FA0A-493D-A1D2-40E7304B0A35@googlemail.com>
Precedence: bulk

Friendly ping=E2=80=A6=20

As this missing change was most likely an oversight in the original =
commit I do think it should go into 5.12 and subsequently stable as =
well. That=E2=80=99s also the reason why I didn=E2=80=99t send a v2 and =
changed the commit message / subject for the patch. For me it clearly is =
a bug and not a new feature.


Thanks
Norman


> On 26. Mar 2021, at 13:22, Paolo Abeni <pabeni@redhat.com> wrote:
>=20
> On Fri, 2021-03-26 at 11:22 +0100, Norman Maurer wrote:
>> On 26. Mar 2021, at 10:36, Paolo Abeni <pabeni@redhat.com> wrote:
>>> One thing you can do to simplifies the maintainer's life, would be =
post
>>> a v2 with the correct tag (and ev. obsolete this patch in =
patchwork).
>>=20
>> I am quite new to contribute patches to the kernel so I am not sure
>> how I would =E2=80=9Cobsolete=E2=80=9D this patch and make a v2. If =
you can give me
>> some pointers I am happy to do so.
>=20
> Well, I actually gave you a bad advice about fiddling with patchwork.
>=20
> The autoritative documentation:
>=20
> Documentation/networking/netdev-FAQ.rst
>=20
> (inside the kernel tree) suggests to avoid it.
>=20
> Just posting a v2 will suffice.
>=20
> Thanks!
>=20
> Paolo
>=20