Re: [Xen-devel] [PATCH] xsm/flask: handle XEN_DOMCTL_memory_mapping for all architectures

[Arianna Avanzini <avanzini.arianna@xxxxxxxxx>]

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.
> 

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel