On Tue, 2014-08-19 at 15:24 +1000, Paul Mackerras wrote:

On Tue, Aug 19, 2014 at 02:59:29PM +1000, Michael Neuling wrote:

quoted

This cleans up kvmppc_load/save_fp.  It removes unnecessary isyncs.

=20
NAK - they are necessary on PPC970, which we (still) support.  You
could put them in a feature section if they are really annoying you.

I'm not fussed, but we should at least have a comment there for why we
need them.

quoted

 It also
removes the unnecessary resetting of the MSR bits on exit of kvmppc_sav=

e_fp.

=20
... except it doesn't. :)  That got folded into e4e38121507a ("KVM:
PPC: Book3S HV: Add transactional memory support").

Arrh, thanks.  This patch was cleaning up stuff from an old local tree
and couldn't see where it had been upstreamed.  I missed this.

Mikey