* Follow-up to "net: drop bad gso csum_start and offset in virtio_net_hdr" - backport for 5.15 needed
@ 2024-09-03 7:37 Christian Theune
2024-09-03 8:02 ` Greg KH
0 siblings, 1 reply; 10+ messages in thread
From: Christian Theune @ 2024-09-03 7:37 UTC (permalink / raw)
To: regressions; +Cc: stable, netdev
Hi,
the issue was so far handled in https://lore.kernel.org/regressions/ZsyMzW-4ee_U8NoX@eldamar.lan/T/#m390d6ef7b733149949fb329ae1abffec5cefb99b and https://bugzilla.kernel.org/show_bug.cgi?id=219129
I haven’t seen any communication whether a backport for 5.15 is already in progress, so I thought I’d follow up here.
Today we rolled out 5.15.165 and immediately ran into the same issue. There’s at least one other person asking for a 5.15 backport in Bugzilla.
Hugs,
Christian
--
Christian Theune - A97C62CE - 0179 7808366
@theuni - christian@theune.cc
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Follow-up to "net: drop bad gso csum_start and offset in virtio_net_hdr" - backport for 5.15 needed
2024-09-03 7:37 Follow-up to "net: drop bad gso csum_start and offset in virtio_net_hdr" - backport for 5.15 needed Christian Theune
@ 2024-09-03 8:02 ` Greg KH
2024-09-09 4:05 ` Willem de Bruijn
0 siblings, 1 reply; 10+ messages in thread
From: Greg KH @ 2024-09-03 8:02 UTC (permalink / raw)
To: Christian Theune; +Cc: regressions, stable, netdev
On Tue, Sep 03, 2024 at 09:37:30AM +0200, Christian Theune wrote:
> Hi,
>
> the issue was so far handled in https://lore.kernel.org/regressions/ZsyMzW-4ee_U8NoX@eldamar.lan/T/#m390d6ef7b733149949fb329ae1abffec5cefb99b and https://bugzilla.kernel.org/show_bug.cgi?id=219129
>
> I haven’t seen any communication whether a backport for 5.15 is already in progress, so I thought I’d follow up here.
Someone needs to send a working set of patches to apply.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Follow-up to "net: drop bad gso csum_start and offset in virtio_net_hdr" - backport for 5.15 needed
2024-09-03 8:02 ` Greg KH
@ 2024-09-09 4:05 ` Willem de Bruijn
2024-09-09 17:03 ` Greg KH
0 siblings, 1 reply; 10+ messages in thread
From: Willem de Bruijn @ 2024-09-09 4:05 UTC (permalink / raw)
To: Greg KH
Cc: Christian Theune, regressions, stable, netdev, mathieu.tortuyaux,
Willem de Bruijn
On Tue, Sep 3, 2024 at 4:03 AM Greg KH <gregkh@linuxfoundation.org> wrote:
>
> On Tue, Sep 03, 2024 at 09:37:30AM +0200, Christian Theune wrote:
> > Hi,
> >
> > the issue was so far handled in https://lore.kernel.org/regressions/ZsyMzW-4ee_U8NoX@eldamar.lan/T/#m390d6ef7b733149949fb329ae1abffec5cefb99b and https://bugzilla.kernel.org/show_bug.cgi?id=219129
> >
> > I haven’t seen any communication whether a backport for 5.15 is already in progress, so I thought I’d follow up here.
>
> Someone needs to send a working set of patches to apply.
The following stack of patches applies cleanly to 5.15.166
(original SHA1s, git log order, so inverse of order to apply):
89add40066f9e net: drop bad gso csum_start and offset in virtio_net_hdr
9840036786d9 gso: fix dodgy bit handling for GSO_UDP_L4
fc8b2a619469 net: more strict VIRTIO_NET_HDR_GSO_UDP_L4 validation
All three are already present in 6.1.109
Please let me know if I should send that stack using git send-email,
or whether this is sufficient into to backport.
The third commit has one Fixes referencing them:
1382e3b6a350 net: change maximum number of UDP segments to 128
This simple -2/+2 line patch unfortunately cannot be backported
without conflicts without backporting non-stable feature changes.
There is a backport to 6.1.y, but that also won't apply cleanly to
5.15.166 without backporting a feature (e2a4392b61f6 "udp: introduce
udp->udp_flags"), which itself does not apply cleanly.
So simplest is probably to fix up this commit and send it using git
send-email. I can do that as part of the stack with the above 3
patches, or stand-alone if the above can be cherry-picked by SHA1.
Thanks,
Willem
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Follow-up to "net: drop bad gso csum_start and offset in virtio_net_hdr" - backport for 5.15 needed
2024-09-09 4:05 ` Willem de Bruijn
@ 2024-09-09 17:03 ` Greg KH
2024-09-09 18:34 ` Willem de Bruijn
0 siblings, 1 reply; 10+ messages in thread
From: Greg KH @ 2024-09-09 17:03 UTC (permalink / raw)
To: Willem de Bruijn
Cc: Christian Theune, regressions, stable, netdev, mathieu.tortuyaux,
Willem de Bruijn
On Mon, Sep 09, 2024 at 12:05:17AM -0400, Willem de Bruijn wrote:
> On Tue, Sep 3, 2024 at 4:03 AM Greg KH <gregkh@linuxfoundation.org> wrote:
> >
> > On Tue, Sep 03, 2024 at 09:37:30AM +0200, Christian Theune wrote:
> > > Hi,
> > >
> > > the issue was so far handled in https://lore.kernel.org/regressions/ZsyMzW-4ee_U8NoX@eldamar.lan/T/#m390d6ef7b733149949fb329ae1abffec5cefb99b and https://bugzilla.kernel.org/show_bug.cgi?id=219129
> > >
> > > I haven’t seen any communication whether a backport for 5.15 is already in progress, so I thought I’d follow up here.
> >
> > Someone needs to send a working set of patches to apply.
>
> The following stack of patches applies cleanly to 5.15.166
> (original SHA1s, git log order, so inverse of order to apply):
>
> 89add40066f9e net: drop bad gso csum_start and offset in virtio_net_hdr
> 9840036786d9 gso: fix dodgy bit handling for GSO_UDP_L4
> fc8b2a619469 net: more strict VIRTIO_NET_HDR_GSO_UDP_L4 validation
>
> All three are already present in 6.1.109
>
> Please let me know if I should send that stack using git send-email,
> or whether this is sufficient into to backport.
I just tried it, they do not apply cleanly here for me at all :(
> The third commit has one Fixes referencing them:
>
> 1382e3b6a350 net: change maximum number of UDP segments to 128
>
> This simple -2/+2 line patch unfortunately cannot be backported
> without conflicts without backporting non-stable feature changes.
> There is a backport to 6.1.y, but that also won't apply cleanly to
> 5.15.166 without backporting a feature (e2a4392b61f6 "udp: introduce
> udp->udp_flags"), which itself does not apply cleanly.
>
> So simplest is probably to fix up this commit and send it using git
> send-email. I can do that as part of the stack with the above 3
> patches, or stand-alone if the above can be cherry-picked by SHA1.
Please send me a set of working, and tested, patches and we will be glad
to consider it.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Follow-up to "net: drop bad gso csum_start and offset in virtio_net_hdr" - backport for 5.15 needed
2024-09-09 17:03 ` Greg KH
@ 2024-09-09 18:34 ` Willem de Bruijn
2024-09-09 18:38 ` Christian Theune
0 siblings, 1 reply; 10+ messages in thread
From: Willem de Bruijn @ 2024-09-09 18:34 UTC (permalink / raw)
To: Greg KH, Willem de Bruijn
Cc: Christian Theune, regressions, stable, netdev, mathieu.tortuyaux,
Willem de Bruijn
Greg KH wrote:
> On Mon, Sep 09, 2024 at 12:05:17AM -0400, Willem de Bruijn wrote:
> > On Tue, Sep 3, 2024 at 4:03 AM Greg KH <gregkh@linuxfoundation.org> wrote:
> > >
> > > On Tue, Sep 03, 2024 at 09:37:30AM +0200, Christian Theune wrote:
> > > > Hi,
> > > >
> > > > the issue was so far handled in https://lore.kernel.org/regressions/ZsyMzW-4ee_U8NoX@eldamar.lan/T/#m390d6ef7b733149949fb329ae1abffec5cefb99b and https://bugzilla.kernel.org/show_bug.cgi?id=219129
> > > >
> > > > I haven’t seen any communication whether a backport for 5.15 is already in progress, so I thought I’d follow up here.
> > >
> > > Someone needs to send a working set of patches to apply.
> >
> > The following stack of patches applies cleanly to 5.15.166
> > (original SHA1s, git log order, so inverse of order to apply):
> >
> > 89add40066f9e net: drop bad gso csum_start and offset in virtio_net_hdr
> > 9840036786d9 gso: fix dodgy bit handling for GSO_UDP_L4
> > fc8b2a619469 net: more strict VIRTIO_NET_HDR_GSO_UDP_L4 validation
> >
> > All three are already present in 6.1.109
> >
> > Please let me know if I should send that stack using git send-email,
> > or whether this is sufficient into to backport.
>
> I just tried it, they do not apply cleanly here for me at all :(
>
> > The third commit has one Fixes referencing them:
> >
> > 1382e3b6a350 net: change maximum number of UDP segments to 128
> >
> > This simple -2/+2 line patch unfortunately cannot be backported
> > without conflicts without backporting non-stable feature changes.
> > There is a backport to 6.1.y, but that also won't apply cleanly to
> > 5.15.166 without backporting a feature (e2a4392b61f6 "udp: introduce
> > udp->udp_flags"), which itself does not apply cleanly.
> >
> > So simplest is probably to fix up this commit and send it using git
> > send-email. I can do that as part of the stack with the above 3
> > patches, or stand-alone if the above can be cherry-picked by SHA1.
>
> Please send me a set of working, and tested, patches and we will be glad
> to consider it.
Done:
https://lore.kernel.org/stable/20240909182506.270136-1-willemdebruijn.kernel@gmail.com/T/#m0086f42d20bfc4d6226a8cf379590032edfbbe21
The following worked fine for me. I hope I did not overlook anything.
Compile tested only. But as said, the same patches have landed in
6.1-stable.
git fetch linux-stable linux-5.15.y
git checkout linux-stable/linux-5.15.y
git cherry-pick fc8b2a619469
git cherry-pick 9840036786d9
git cherry-pick 89add40066f9e
make defconfig
make -j 64
Unfortunately, the key commit itself has a bug report against it. I am
working on that right now.
But as the existing patch that it refers to in its Fixes has landed in
all stable kernels and is causing reports, it is a backporting case of
damned if we do, damned if we don't.
I intend to have a fix ready ASAP for netdev-net/main. If all stable
branches have all backports, it should apply cleanly everywhere. Just
not ready to be included in this series from the start.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Follow-up to "net: drop bad gso csum_start and offset in virtio_net_hdr" - backport for 5.15 needed
2024-09-09 18:34 ` Willem de Bruijn
@ 2024-09-09 18:38 ` Christian Theune
2024-09-09 20:32 ` Willem de Bruijn
0 siblings, 1 reply; 10+ messages in thread
From: Christian Theune @ 2024-09-09 18:38 UTC (permalink / raw)
To: Willem de Bruijn
Cc: Greg KH, Willem de Bruijn, regressions, stable, netdev,
mathieu.tortuyaux
I can contribute live testing and can quickly reproduce the issue.
If anything is there that should be tested for apart from verifying the fix, I’d be happy to try.
Christian
> On 9. Sep 2024, at 20:34, Willem de Bruijn <willemdebruijn.kernel@gmail.com> wrote:
>
> Greg KH wrote:
>> On Mon, Sep 09, 2024 at 12:05:17AM -0400, Willem de Bruijn wrote:
>>> On Tue, Sep 3, 2024 at 4:03 AM Greg KH <gregkh@linuxfoundation.org> wrote:
>>>>
>>>> On Tue, Sep 03, 2024 at 09:37:30AM +0200, Christian Theune wrote:
>>>>> Hi,
>>>>>
>>>>> the issue was so far handled in https://lore.kernel.org/regressions/ZsyMzW-4ee_U8NoX@eldamar.lan/T/#m390d6ef7b733149949fb329ae1abffec5cefb99b and https://bugzilla.kernel.org/show_bug.cgi?id=219129
>>>>>
>>>>> I haven’t seen any communication whether a backport for 5.15 is already in progress, so I thought I’d follow up here.
>>>>
>>>> Someone needs to send a working set of patches to apply.
>>>
>>> The following stack of patches applies cleanly to 5.15.166
>>> (original SHA1s, git log order, so inverse of order to apply):
>>>
>>> 89add40066f9e net: drop bad gso csum_start and offset in virtio_net_hdr
>>> 9840036786d9 gso: fix dodgy bit handling for GSO_UDP_L4
>>> fc8b2a619469 net: more strict VIRTIO_NET_HDR_GSO_UDP_L4 validation
>>>
>>> All three are already present in 6.1.109
>>>
>>> Please let me know if I should send that stack using git send-email,
>>> or whether this is sufficient into to backport.
>>
>> I just tried it, they do not apply cleanly here for me at all :(
>>
>>> The third commit has one Fixes referencing them:
>>>
>>> 1382e3b6a350 net: change maximum number of UDP segments to 128
>>>
>>> This simple -2/+2 line patch unfortunately cannot be backported
>>> without conflicts without backporting non-stable feature changes.
>>> There is a backport to 6.1.y, but that also won't apply cleanly to
>>> 5.15.166 without backporting a feature (e2a4392b61f6 "udp: introduce
>>> udp->udp_flags"), which itself does not apply cleanly.
>>>
>>> So simplest is probably to fix up this commit and send it using git
>>> send-email. I can do that as part of the stack with the above 3
>>> patches, or stand-alone if the above can be cherry-picked by SHA1.
>>
>> Please send me a set of working, and tested, patches and we will be glad
>> to consider it.
>
> Done:
>
> https://lore.kernel.org/stable/20240909182506.270136-1-willemdebruijn.kernel@gmail.com/T/#m0086f42d20bfc4d6226a8cf379590032edfbbe21
>
> The following worked fine for me. I hope I did not overlook anything.
> Compile tested only. But as said, the same patches have landed in
> 6.1-stable.
>
> git fetch linux-stable linux-5.15.y
> git checkout linux-stable/linux-5.15.y
> git cherry-pick fc8b2a619469
> git cherry-pick 9840036786d9
> git cherry-pick 89add40066f9e
> make defconfig
> make -j 64
>
> Unfortunately, the key commit itself has a bug report against it. I am
> working on that right now.
>
> But as the existing patch that it refers to in its Fixes has landed in
> all stable kernels and is causing reports, it is a backporting case of
> damned if we do, damned if we don't.
>
> I intend to have a fix ready ASAP for netdev-net/main. If all stable
> branches have all backports, it should apply cleanly everywhere. Just
> not ready to be included in this series from the start.
Liebe Grüße,
Christian
--
Christian Theune - A97C62CE - 0179 7808366
@theuni - christian@theune.cc
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Follow-up to "net: drop bad gso csum_start and offset in virtio_net_hdr" - backport for 5.15 needed
2024-09-09 18:38 ` Christian Theune
@ 2024-09-09 20:32 ` Willem de Bruijn
2024-09-10 6:32 ` Christian Theune
0 siblings, 1 reply; 10+ messages in thread
From: Willem de Bruijn @ 2024-09-09 20:32 UTC (permalink / raw)
To: Christian Theune, Willem de Bruijn
Cc: Greg KH, Willem de Bruijn, regressions, stable, netdev,
mathieu.tortuyaux
Christian Theune wrote:
> I can contribute live testing and can quickly reproduce the issue.
>
> If anything is there that should be tested for apart from verifying the fix, I’d be happy to try.
If you perform the repro steps and verify that this solves the issue,
that would be helpful, thanks.
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Follow-up to "net: drop bad gso csum_start and offset in virtio_net_hdr" - backport for 5.15 needed
2024-09-09 20:32 ` Willem de Bruijn
@ 2024-09-10 6:32 ` Christian Theune
2024-09-11 7:39 ` Christian Theune
0 siblings, 1 reply; 10+ messages in thread
From: Christian Theune @ 2024-09-10 6:32 UTC (permalink / raw)
To: Willem de Bruijn
Cc: Greg KH, Willem de Bruijn, regressions, stable, netdev,
mathieu.tortuyaux
Happy to do that. I’ve blocked time tomorrow morning.
> On 9. Sep 2024, at 22:32, Willem de Bruijn <willemdebruijn.kernel@gmail.com> wrote:
>
> Christian Theune wrote:
>> I can contribute live testing and can quickly reproduce the issue.
>>
>> If anything is there that should be tested for apart from verifying the fix, I’d be happy to try.
>
> If you perform the repro steps and verify that this solves the issue,
> that would be helpful, thanks.
Liebe Grüße,
Christian
--
Christian Theune - A97C62CE - 0179 7808366
@theuni - christian@theune.cc
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Follow-up to "net: drop bad gso csum_start and offset in virtio_net_hdr" - backport for 5.15 needed
2024-09-10 6:32 ` Christian Theune
@ 2024-09-11 7:39 ` Christian Theune
2024-09-11 7:43 ` Greg KH
0 siblings, 1 reply; 10+ messages in thread
From: Christian Theune @ 2024-09-11 7:39 UTC (permalink / raw)
To: Willem de Bruijn
Cc: Greg KH, Willem de Bruijn, regressions, stable, netdev,
mathieu.tortuyaux
Hi,
I just took 5.15.166 + the 4 patches by Willem as they’re currently in the queue. The applied cleanly (which you already tested) and I can demonstrate the known error (bad gso: type: 4, size: 1428) on 5.15.166 without the patches and it’s gone on 5.15.166 with the patches.
Thanks everyone!
Hugs,
Christian
> On 10. Sep 2024, at 08:32, Christian Theune <christian@theune.cc> wrote:
>
> Happy to do that. I’ve blocked time tomorrow morning.
>
>> On 9. Sep 2024, at 22:32, Willem de Bruijn <willemdebruijn.kernel@gmail.com> wrote:
>>
>> Christian Theune wrote:
>>> I can contribute live testing and can quickly reproduce the issue.
>>>
>>> If anything is there that should be tested for apart from verifying the fix, I’d be happy to try.
>>
>> If you perform the repro steps and verify that this solves the issue,
>> that would be helpful, thanks.
>
>
> Liebe Grüße,
> Christian
>
> --
> Christian Theune - A97C62CE - 0179 7808366
> @theuni - christian@theune.cc
>
Liebe Grüße,
Christian
--
Christian Theune - A97C62CE - 0179 7808366
@theuni - christian@theune.cc
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: Follow-up to "net: drop bad gso csum_start and offset in virtio_net_hdr" - backport for 5.15 needed
2024-09-11 7:39 ` Christian Theune
@ 2024-09-11 7:43 ` Greg KH
0 siblings, 0 replies; 10+ messages in thread
From: Greg KH @ 2024-09-11 7:43 UTC (permalink / raw)
To: Christian Theune
Cc: Willem de Bruijn, Willem de Bruijn, regressions, stable, netdev,
mathieu.tortuyaux
On Wed, Sep 11, 2024 at 09:39:01AM +0200, Christian Theune wrote:
> Hi,
>
> I just took 5.15.166 + the 4 patches by Willem as they’re currently in the queue. The applied cleanly (which you already tested) and I can demonstrate the known error (bad gso: type: 4, size: 1428) on 5.15.166 without the patches and it’s gone on 5.15.166 with the patches.
Thanks for testing!
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2024-09-11 7:43 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-09-03 7:37 Follow-up to "net: drop bad gso csum_start and offset in virtio_net_hdr" - backport for 5.15 needed Christian Theune
2024-09-03 8:02 ` Greg KH
2024-09-09 4:05 ` Willem de Bruijn
2024-09-09 17:03 ` Greg KH
2024-09-09 18:34 ` Willem de Bruijn
2024-09-09 18:38 ` Christian Theune
2024-09-09 20:32 ` Willem de Bruijn
2024-09-10 6:32 ` Christian Theune
2024-09-11 7:39 ` Christian Theune
2024-09-11 7:43 ` Greg KH
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).