[Top] [All Lists]

Re: FP emulator patch

To: Jun Sun <>
Subject: Re: FP emulator patch
From: Daniel Jacobowitz <>
Date: Thu, 16 Aug 2001 15:37:02 -0700
Cc: "Kevin D. Kissell" <>, "MIPS/Linux List (SGI)" <>
In-reply-to: <>; from on Thu, Aug 16, 2001 at 02:34:45PM -0700
References: <018201c12680$8f13e680$0deca8c0@Ulysses> <> <> <01b001c12693$b4920140$0deca8c0@Ulysses> <>
User-agent: Mutt/1.3.16i
On Thu, Aug 16, 2001 at 02:34:45PM -0700, Jun Sun wrote:
> Yes, that is somewhat the purpose.  Essentially we want to see, at the
> beginning of a signal handler execution, the process appears to have not used
> FPU at all.


> This requirement might be a must, because whether clearing current->used_math
> bit determine which patch we will take in the do_cpu(), when signal handler
> uses FPU for the first time.  See the code below.
>         if (current->used_math) {               /* Using the FPU again.  */
>                 lazy_fpu_switch(last_task_used_math);
>         } else {                                /* First time FPU user.  */
>                 init_fpu();
>                 current->used_math = 1;
>         }
>         last_task_used_math = current;
> Clearly the second path is logically the correct one.

Not really.  Why should it get a clean set of FP registers?  I think
the CORRECT thing would actually be for it to have the app's FP
registers.  Changes should not propogate back to the app, that's all.

> BTW, do I see another bug here in do_cpu()?  It seems that before we call
> init_fpu(), we should check last_task_used_math.  If it is not NULL, we should
> save the FP state to the last_task_used_math.  Hmm, strange ...

I thought I got all of these... <sigh>

Daniel Jacobowitz                           Carnegie Mellon University
MontaVista Software                         Debian GNU/Linux Developer

<Prev in Thread] Current Thread [Next in Thread>