On Thu, Dec 5, 2024 at 1:02 PM Max Kellermann max.kellermann@ionos.com wrote:
- What "other thread"? I can't find anything on LKML and ceph-devel.
Just in case you mean this patch authored by you: https://github.com/ceph/ceph-client/commit/2a802a906f9c89f8ae492dbfcd82ff412...
I don't think that's a good patch, and if I had the power, I would certainly reject it, because:
- it's big and confusing; hard to review - it's badly documented; the commit message is just "fixing a race condition when a file shrinks" but other than that, doesn't explain anything; a proper explanation is necessary for such a complicated diff - the patch changes many distinct things in one patch, but it should really be split - this patch is about the buffer overflow for which my patch is much simpler: https://lore.kernel.org/lkml/20241127212130.2704804-1-max.kellermann@ionos.c... which I suggested merging instead of all the other candicate patches https://lore.kernel.org/lkml/CAKPOu+9kdcjMf36bF3HAW4K8v0mHxXQX3_oQfGSshmXBKt... but you did not reply (as usual, sigh!) - deeply hidden in this patch is also a fix for the memory leak, but instead of making one large patch which fixes everything, you should first merge my trivial leak bug fix and then the fix for the buffer overflow on top