On Thu, Jul 16, 2015 at 11:23:08AM -0700, Mario Smarduch wrote: > On 07/16/2015 08:52 AM, Christoffer Dall wrote: > > On Fri, Jul 10, 2015 at 06:19:05PM -0700, Mario Smarduch wrote: > >> This is a followp to previous iteration but implemented on top of VHE patches. > >> Only non-VHE path is addressied by this patch. In second patch 32-bit handler > >> is updated to keep exit handling consistent with 64-bit code, and nothing > >> has changed. > >> > > Why not simply preserve this the way it was in v3 and have it merged > > first - after all we have reviewed it and I thought it was more or less > > ready to be merged - I suspect the VHE patches may have a way to go > > still ? > > Definitely, that's a better path. After looking at VHE patches, > I would probably leave V3 the way it is (keeping deactivate_xxxx: > symmetric). Marc has Reviewed V3 and you commented either way was > fine with you, so V3 should be ok. Yes, but there was a comment in the assembly file to fix up IIRC. Can you do a quick respin with that commentary changed and then Marc can queue that if he agrees? Thanks, -Christoffer -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html