On Tue, Sep 14, 2021 at 5:22 AM Konstantin Ryabitsev <[email protected]> wrote: > > On Mon, Sep 13, 2021 at 04:46:47PM -0700, Dan Williams wrote: > > > In the ideal world I'd like to have seen this as a noop patch going from > > > devm > > > to non devm for cleanup followed by new stuff. meh, the world isn't ideal > > > and all that sort of nice stuff takes time! > > > > It would also require a series resend since I can't use the in-place > > update in a way that b4 will recognize. > > BTW, b4 0.7+ can do partial series rerolls. You can just send a single > follow-up patch without needing to reroll the whole series, e.g.: > > [PATCH 1/3] > [PATCH 2/3] > \- [PATCH v2 2/3] > [PATCH 3/3] > > This is enough for b4 to make a v2 series where only 2/3 is replaced.
Oh, yes, I use that liberally, istr asking for it originally. What I was referring to here was feedback that alluded to injecting another patch into the series, ala: [PATCH 1/3] [PATCH 2/3] \- [PATCH v2 2/4] \- [PATCH v2 3/4] [PATCH 3/3] <-- this one would be 4/4 I don't expect b4 to handle that case, and would expect to re-roll the series with the new numbering. > > -K > > (Yes, I am monitoring all mentions of "b4" on lore.kernel.org/all in a totally > non-creepy way, I swear.) I still need to do that for my sub-systems.
