Subject: Re: [PATCH] mm: Fix alloc_vm_area for noMMU system

From: Konrad Rzeszutek Wilk <[email-masked]>
Date: Wed, 16 Nov 2011 10:32:32 -0500

On Wed, Nov 16, 2011 at 03:25:30PM +0100, Michal Simek wrote:
> Konrad Rzeszutek Wilk wrote:
> >On Wed, Nov 16, 2011 at 02:01:28PM +0000, David Vrabel wrote:
> >>On 16/11/11 13:59, Konrad Rzeszutek Wilk wrote:
> >>>On Wed, Nov 16, 2011 at 08:51:02AM +0100, Michal Simek wrote:
> >>>>Patch:
> >>>>"xen: map foreign pages for shared rings by updating the PTEs directly"
> >>>>(sha1: 5149c0d53f6959d6ab87b4702d544b17ebdc783e) changed number
> >>>>of argument for alloc_vm_area but forget to change it for noMMU systems.
> >>>>This patch fix it.
> >>>Duh!
> >>>>CC: David Vrabel <[email-masked]>
> >>>>CC: Andrew Morton <[email-masked]>
> >>>>CC: Konrad Rzeszutek Wilk <[email-masked]>
> >>>Yes, please. Thank you!
> >>Konrad, I would suggest folding this into the original patch to avoid
> >>breaking bisects for noMMU users unnecessarily.
> >
> >Can't do. We would lose Michal authership that way and that is not good.
> >
> 
> Do it in way you like and it is the best for others. Not sure where exactly your patch

That is very nice of you!

> is and it is hard to suggest you how to do it in connection to your merging strategy
> Anyway you can add my signed-off-by line to origin patch or keep it single patch

Ok. That certainly makes it easier and it does not break the git bisection.

Will do! Thank you.