From: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
To: Michal Simek <monstr@monstr.eu>
Cc: David Vrabel <david.vrabel@citrix.com>,
"linux-next@vger.kernel.org" <linux-next@vger.kernel.org>,
"sfr@canb.auug.org.au" <sfr@canb.auug.org.au>,
Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH] mm: Fix alloc_vm_area for noMMU system
Date: Wed, 16 Nov 2011 10:32:32 -0500 [thread overview]
Message-ID: <20111116153232.GD3314@phenom.dumpdata.com> (raw)
In-Reply-To: <4EC3C7DA.8090600@monstr.eu>
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 <david.vrabel@citrix.com>
> >>>>CC: Andrew Morton <akpm@linux-foundation.org>
> >>>>CC: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
> >>>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.
next prev parent reply other threads:[~2011-11-16 15:32 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-16 7:51 [PATCH] mm: Fix alloc_vm_area for noMMU system Michal Simek
2011-11-16 13:59 ` Konrad Rzeszutek Wilk
2011-11-16 14:01 ` David Vrabel
2011-11-16 14:18 ` Konrad Rzeszutek Wilk
2011-11-16 14:25 ` Michal Simek
2011-11-16 15:32 ` Konrad Rzeszutek Wilk [this message]
2011-11-16 14:11 ` Konrad Rzeszutek Wilk
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20111116153232.GD3314@phenom.dumpdata.com \
--to=konrad.wilk@oracle.com \
--cc=akpm@linux-foundation.org \
--cc=david.vrabel@citrix.com \
--cc=linux-next@vger.kernel.org \
--cc=monstr@monstr.eu \
--cc=sfr@canb.auug.org.au \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).