[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240603171507.2bd5e904@kernel.org>
Date: Mon, 3 Jun 2024 17:15:07 -0700
From: Jakub Kicinski <kuba@...nel.org>
To: Aaron Conole <aconole@...hat.com>
Cc: Adrian Moreno <amorenoz@...hat.com>, netdev@...r.kernel.org, Pravin B
Shelar <pshelar@....org>, "David S. Miller" <davem@...emloft.net>, Eric
Dumazet <edumazet@...gle.com>, Paolo Abeni <pabeni@...hat.com>, Shuah Khan
<shuah@...nel.org>, dev@...nvswitch.org, linux-kselftest@...r.kernel.org,
linux-kernel@...r.kernel.org, Simon Horman <horms@...ge.net.au>
Subject: Re: [PATCH net-next 1/2] selftests: openvswitch: fix action
formatting
On Mon, 03 Jun 2024 15:00:03 -0400 Aaron Conole wrote:
> I agree, this is an issue. BUT I think it might be better to just
> filter by field type up front. See:
>
> https://github.com/apconole/linux-next-work/commit/7262107de7170d44b6dbf6c5ea6f7e6c0bb71d36#diff-3e72e7405c6bb4e9842bed5f63883ca930387086bb40d4034e92ed83a5decb4bR441
>
> That version I think ends up being much easier to follow. If you want
> to take it for your series, feel free. If you disagree, maybe there's
> something I'm not considering about it.
>
> NOTE that version is just a bunch of independent changes that are
> squashed together. I have a cleaner version.
>
> I can also bundle up the series I have so far and submit, but I didn't
> want to do that until I got all the pmtu.sh support working. Maybe it
> makes sense to send it now though. Simon, Jakub - wdyt?
I'd say - hold onto the changes until pmtu.sh works, unless there's
*any* reason for a particular patch to go in early, eg:
- patch fixes existing bug
- someone else needs a patch
- ...
- a patch which falls under some of the criteria above depends
on the patch
Powered by blists - more mailing lists