[SRU][Kinetic][PATCH 0/3] Fix udpgro_frglist.sh kernel selftest

Andrei Gherzan andrei.gherzan at canonical.com
Fri Feb 10 09:38:52 UTC 2023


Hi Stefan,

Thanks for the review.

On Fri, 10 Feb 2023, 08:44 Stefan Bader, <stefan.bader at canonical.com> wrote:

> Patch #2 is missing the BugLink, which of the 2 should be used when
> applying? Generally, it would be better to either duplicate bug reports
> or submit individually. One thing to fix per submission is simpler to
> manage.
>

I avoided a bug link on that patch because it is a fix of a fix. It is a
fix of the CC support for net bpf progs. Now that I think about it, I
reckon we should go ahead and use the same BugLink with the CC support -
https://bugs.launchpad.net/bugs/1996536. I'm happy to resubmit a V2 with
this change.

I understand and support the maintenance advantage of talking one bug per
submission but in this case there was a tight relationship between these
two entries. One failed because of a dependency disabled by another. So
being able to fix and test the former required work on the latter. I hoped
that this was an acceptable balance of submission and maintenance/review
overhead. Similarly though, I'm happy to split and resubmit if deemed
otherwise.

agh
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20230210/33f5d16f/attachment.html>


More information about the kernel-team mailing list