Thread (9 messages) 9 messages, 4 authors, 2010-08-25

Re: [PATCH] powerpc: remove fpscr use from [kvm_]cvt_{fd,df}

From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: 2010-08-25 01:30:56

On Tue, 2010-08-24 at 15:15 +1000, Michael Neuling wrote:
quoted
quoted
Do some 32 bit processors need this? 

In 32 bit before the merge, we use to have code that did:

  #if defined(CONFIG_4xx) || defined(CONFIG_E500)
   #define cvt_fd without save/restore fpscr
  #else
   #define cvt_fd with save/restore fpscr
  #end if

Kumar; does this ring any bells?
I don't see anything in the various 440 docs I have at hand that would
hint at lfd/stfs adffecting FPSCR.
The way the ifdefs are, it's the other way around.  4xx procs don't need
to save/restore fpscr and others do.
Right, my bad. In any case, Paulus reckons it's all his mistake and we
really never need to save/restore fpscr.

Cheers,
Ben.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help