From: "Kevin D. Kissell" <kevink@paralogos.com>
To: Ralf Baechle <ralf@linux-mips.org>
Cc: Shane McDonald <mcdonald.shane@gmail.com>,
linux-mips@linux-mips.org, Atsushi Nemoto <anemo@mba.ocn.ne.jp>
Subject: Re: [MIPS] FPU emulator: allow Cause bits of FCSR to be writeable by ctc1
Date: Wed, 05 May 2010 08:43:16 -0700 [thread overview]
Message-ID: <4BE19214.4010209@paralogos.com> (raw)
In-Reply-To: <20100505091159.GA4016@linux-mips.org>
Ralf Baechle wrote:
>> I'm glad to hear that the patch is functional. It was pretty clear that
>> it would solve your problem, but I was concerned that the inability to
>> write the Cause bits was done as a way around some other problem.
>> Someone went to a certain amount of trouble to not accept them as
>> inputs, in violation of spec. I just looked back, and the bug was
>> introduced as part of a patch of Ralf's from April 2005 entitled "Fix
>> Preemption and SMP problems in the FP emulator". It's unlikely that
>> overriding CTC semantics actually fixed any underlying problems, but it
>> may have masked symptoms of problems that we can only hope have been
>> fixed in the mean time. Anyone run this patch on an FPUless SMP? Oh,
>> for a 34Kf with a bunch of TCs! ;o)
>>
>
> That's commit ID 72402ec9efdb2ea7e0ec6223cf20e7301a57da02 and the patch
> was comitted during the CVS days which only records the comitter but not
> the author. The actual author is Atsushi Nemoto.
>
Well,. I certainly understood that you were simply the guy who did the
commit. Didn't mean to make it sound like an accusation, but it was
marginally easier to type your name and date than to find, cut, and
paste the commit ID. Sorry if it came off wrong. It would be cool if
Atsushi remembered the reasoning behind the change, but empirical proof
that undoing it doesn't create a subtle problem for current SMP kernels
would be better still.
Regards,
Kevin K.
next prev parent reply other threads:[~2010-05-05 15:43 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-05 4:02 [MIPS] FPU emulator: allow Cause bits of FCSR to be writeable by ctc1 Shane McDonald
2010-05-05 7:48 ` Kevin D. Kissell
2010-05-05 9:11 ` Ralf Baechle
2010-05-05 15:43 ` Kevin D. Kissell [this message]
2010-05-05 16:22 ` Atsushi Nemoto
2010-05-05 19:20 ` Kevin D. Kissell
2010-05-06 11:24 ` Sergei Shtylyov
2010-05-06 15:46 ` Kevin D. Kissell
[not found] ` <o2yb2b2f2321005061142v431dbc78n2a21722676a72501@mail.gmail.com>
2010-05-06 18:47 ` Ralf Baechle
2010-05-06 19:06 ` Kevin D. Kissell
2010-05-08 16:46 ` Atsushi Nemoto
2010-05-06 9:01 ` Ralf Baechle
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4BE19214.4010209@paralogos.com \
--to=kevink@paralogos.com \
--cc=anemo@mba.ocn.ne.jp \
--cc=linux-mips@linux-mips.org \
--cc=mcdonald.shane@gmail.com \
--cc=ralf@linux-mips.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.