On 07/16/2015 12:05 PM, Christoffer Dall wrote: > 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. That's right 1/2 and 0/2 header need little bit of editing. > > 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