On 7/30/24 19:13, Jakub Sitnicki wrote:
On Fri, Jul 26, 2024 at 10:29 PM +02, Michal Luczaj wrote:
On 7/26/24 19:23, Jakub Sitnicki wrote:
I was going to suggest that a single return path for success is better than two (diff below), but I see that this is what you ended up with after patch 6.
So I think we can leave it as is. [...]
And speaking of which, would you rather have patch 1 and 6 squashed?
Don't have a straight answer, sorry . Would have to see if the diff is clear enough after squashing it. Use your best judgement.
It's certainly fine with me to review the steps that were taken to massage the code.
That's what I've assumed, thanks. So here's the bpf-next based respin: https://lore.kernel.org/bpf/20240731-selftest-sockmap-fixes-v2-0-08a0c73abed...