On 09/19/2013 09:57 PM, Mj Embd wrote: > two quick points > (a) xen already has a mode_switch code, so AFAIK xen might not use it > (as suggested by comment in another patch in this patch set) Just a few days ago Ian sent out patches to remove this code from Xen. That code was never meant to survive, just to have something working to let people play with Xen easily. Xen now requires to be started in HYP mode, just like KVM. http://lists.xen.org/archives/html/xen-devel/2013-09/msg01690.html Patch 5/7 has more details. Regards, Andre. > (b) There are 2 methods of switching from Secure to Hyp mode > one you have proposed another implemented in xen. I was suggesting > take the best approach > > > > On 9/20/13, Christoffer Dall <christoffer.dall@xxxxxxxxxx> wrote: >> On Thu, Sep 19, 2013 at 10:00:03PM +0530, Mj Embd wrote: >>> Hi Andre, >>> >>> There is another approach taken in xen. (xen/arch/arm/mode_switch.S) >>> Which do you think is the better approach >>> >> Hi there, >> >> I'm not sure I completely understand your question. Do you think this >> patch series should be changed to take something from Xen? If so, can >> you please clarify why? >> >> For the record, this patch series has been reviewed and tested quite a >> lot for U-Boot and we would very much like to get this merged and avoid >> further churn, unless there's a compelling technical reason to change >> it. >> >> Thanks, >> -Christoffer >> > > _______________________________________________ kvmarm mailing list kvmarm@xxxxxxxxxxxxxxxxxxxxx https://lists.cs.columbia.edu/cucslists/listinfo/kvmarm
