On 04/11/2016 16:57, Borislav Petkov wrote: > On Fri, Nov 04, 2016 at 04:13:24PM +0100, Paolo Bonzini wrote: >> I mean that the changes to scattered.c are small, so it makes no sense >> to split them. With an Acked-by I could simply take the patch into my >> tree. > > On no, it is not about the size or which tree it goes thru - rather that > having stuff separated conceptually makes the patches much more clear. > > The patch order would be > > * modify scattered.c to return leaf > * add user of new function > > which also makes review almost trivial. That's fine by me of course. I would probably take both patches anyway. Paolo -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html