[RFC] GTP-U v0/v1 kernel module

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/OpenBSC@lists.osmocom.org/.

Neels Hofmeyr nhofmeyr at sysmocom.de
Wed Oct 28 13:59:10 UTC 2015


On Wed, Oct 28, 2015 at 04:40:48AM +0100, Pablo Neira Ayuso wrote:
> Try to send a small batch to start with, to avoid a small patchbomb
> thing ;-)

*cough* like my patch bombs...
But, in fact, I would personally prefer to read a large number of small
neat patches than a small number of large convoluted patches.

So me, I'm basically all for patch bombs (presuming sufficiently detailed
commit log messages). They are rather easy to skip in the mail inbox and
allow for easier review.

It can pay off to invest "too much" time in restructuring/splitting
patches, allowing N people to save time reading them. My occasional tiny
cosmetic patches are split off for this reason alone.

> I wonder if you can find a good way to split this in logic changes to
> make the review easier.

There, you said it ;)

But, please do let me know if I should refrain from sending patch bombs...
(My current work being sponsored does produce a lot of patches, and once
Holger is done reviewing the last one, there's a lot more to come.)

~Neels

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.osmocom.org/pipermail/openbsc/attachments/20151028/491c81ec/attachment.bin>


More information about the OpenBSC mailing list