Re: [dm-devel] [RFC] block: fix blk_queue_split() resource exhaustion

From: Lars Ellenberg
Date: Fri Jul 08 2016 - 09:59:56 EST


On Fri, Jul 08, 2016 at 07:24:24PM +0600, Pavel Goran wrote:
> Friday, July 8, 2016, 7:00:35 PM, you wrote:
> > On Fri, Jul 08, 2016 at 07:39:36PM +1000, NeilBrown wrote:
> >> I think we just might be in violent agreement.
>
> > I thought so, too :-)
>
> > Should I merge both patches,
> > rename to ".queue" and ".tmp",
> > and submit for inclusion?
>
> Could you please *not* use ".tmp"? I have a feeling it's even worse than "X"
> and "Y". :)
>
> I suggest to leave it as ".recursion", or maybe use something like
> ".downstream".

"Naming is hard" :-)

.recursion and .queue, then,
because .remainder does no longer fit.

Also, what about "Cc: Stable"?
The patch does not strictly fall into the "stable" rules,
(too many lines...).
But it should go there anyways, I think.
I'll just add those tags, and wait for the heat?

Lars Ellenberg