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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to