From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arianna Avanzini Subject: Re: [PATCH] xsm/flask: handle XEN_DOMCTL_memory_mapping for all architectures Date: Wed, 10 Sep 2014 22:07:59 +0200 Message-ID: <20140910200757.GA1040@gmail.com> References: <54072895.8010104@tycho.nsa.gov> <1409831392-2468-1-git-send-email-avanzini.arianna@gmail.com> <1410171085.10156.143.camel@kazak.uk.xensource.com> <20140908122444.GA922@gmail.com> <1410179923.10156.207.camel@kazak.uk.xensource.com> <20140908125053.GA586@gmail.com> <1410266129.8217.155.camel@kazak.uk.xensource.com> <20140909131302.GA961@gmail.com> <1410273905.8217.217.camel@kazak.uk.xensource.com> Reply-To: avanzini.arianna@gmail.com Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1410273905.8217.217.camel@kazak.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: paolo.valente@unimore.it, stefano.stabellini@eu.citrix.com, dario.faggioli@citrix.com, xen-devel@lists.xen.org, julien.grall@citrix.com, etrudeau@broadcom.com, dgdegra@tycho.nsa.gov, viktor.kleinik@globallogic.com, andrii.tseglytskyi@globallogic.com List-Id: xen-devel@lists.xenproject.org On Tue, Sep 09, 2014 at 03:45:05PM +0100, Ian Campbell wrote: > On Tue, 2014-09-09 at 15:13 +0200, Arianna Avanzini wrote: > > On Tue, Sep 09, 2014 at 01:35:29PM +0100, Ian Campbell wrote: > > > On Mon, 2014-09-08 at 14:50 +0200, Arianna Avanzini wrote: > > > > On Mon, Sep 08, 2014 at 01:38:43PM +0100, Ian Campbell wrote: > > > > > On Mon, 2014-09-08 at 14:24 +0200, Arianna Avanzini wrote: > > > > > > On Mon, Sep 08, 2014 at 11:11:25AM +0100, Ian Campbell wrote: > > > > > > > On Thu, 2014-09-04 at 13:49 +0200, Arianna Avanzini wrote: > > > > > > > > - Reworded commit message so that it explains what the patch does and > > > > > > > > not the side effects of its absence. > > > > > > > > > > > > > > Thanks. > > > > > > > > > > > > > > Am I correct in assuming that this is replacing the previous #10/14 in > > > > > > > v12 but that a v13 is not on its way. IOW I can do the switcharoo during > > > > > > > commit (assuming the rest is acked, I didn't check yet)? > > > > > > > > > > > > Yes, this patch is replacing #10/14. Unfortunately patches #11 to #14 still > > > > > > have to be acked. I was waiting for Andrii's feedback on Daniel De Graaf's > > > > > > comments about patch #11, and I'll resend patches from #12 to #14 (with > > > > > > your comments addressed) within today. > > > > > > > > > > Thanks, it looks to me like #12-#14 can (once resent) go ahead without > > > > > #11 for the time being, is that right? > > > > > > > > Yes exactly, that's what I thought too. > > > > > > I've now applied this new replacement #10 plus the little three patch > > > series you sent corresponding to #12-#14 in this one. > > > > > > > Thank you for taking the time to review them! > > Thanks for the contribution (and commitment!), I hadn't realised when I > committed that batch that it was effectively everything but some > xsm/flask loose ends, congrats! > Thank you! I'll follow the discussion on Andrii's patch to see what is to be done at the end. Arianna > Ian. >