All of lore.kernel.org
 help / color / mirror / Atom feed
From: "陳韋任 (Wei-Ren Chen)" <chenwj@iis.sinica.edu.tw>
To: "Johnson, Eric" <ericj@mips.com>
Cc: "qemu-trivial@nongnu.org" <qemu-trivial@nongnu.org>,
	"qemu-devel@nongnu.org" <qemu-devel@nongnu.org>,
	"陳韋任 (Wei-Ren Chen)" <chenwj@iis.sinica.edu.tw>,
	"Jia Liu" <proljc@gmail.com>
Subject: Re: [Qemu-trivial] [Qemu-devel] [PATCH] target-mips: Clean up microMIPS32 major opcode
Date: Thu, 15 Nov 2012 11:26:52 +0800	[thread overview]
Message-ID: <20121115032652.GA17050@cs.nctu.edu.tw> (raw)
In-Reply-To: <AD431C1EF67D4B4B801AEC089EB7ED48014488ABEC@exchdb03.mips.com>

On Thu, Nov 15, 2012 at 02:34:31AM +0000, Johnson, Eric wrote:
> Hi Chen,
> 
> Please only remove the POOL48A opcode.
> 
> The others are documented in the microMIPS64 Instruction Set manual ( http://www.mips.com/secure-download/index.dot?product_name=/auth/MD00087-2B-MIPS64BIS-AFP-03.51.pdf ).  See http://www.mips.com/products/architectures/mips64/ for other relavent docs.
> 
> Instead of removing them please surround the POOL32S, DADDIU32, SD32, and LD32 opcodes with
> #if defined(TARGET_MIPS64)
 
  Just want to make sure I am reading the right manual, are you refering
to [1] or [2]? The link you gave me is about MIPS64 not microMIPS64, I 
am not sure which one I should look into. I can find DADDIU, SD, and LD
in [1] and [2], all *without* 32 suffix. Should I change their name?
I cannot find POOL32S in both document, would you like to point it out?
Thanks.

Regards,
chenwj

[1] MIPS Architecture For Programmers
      Volume I-B: Introduction to the microMIPS64 Architecture

    MD00743-2B-microMIPS64INT-AFP-03.02.pdf
 
[2] MIPS Architecture For Programmers
      Volume II-A: The MIPS64 Instruction Set

    MD00087-2B-MIPS64BIS-AFP-03.51.pdf

-- 
Wei-Ren Chen (陳韋任)
Computer Systems Lab, Institute of Information Science,
Academia Sinica, Taiwan (R.O.C.)
Tel:886-2-2788-3799 #1667
Homepage: http://people.cs.nctu.edu.tw/~chenwj


  reply	other threads:[~2012-11-15  3:27 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-15  2:15 [Qemu-trivial] [PATCH] target-mips: Clean up microMIPS32 major opcode 陳韋任 (Wei-Ren Chen)
2012-11-15  2:34 ` [Qemu-trivial] [Qemu-devel] " Johnson, Eric
2012-11-15  3:26   ` 陳韋任 (Wei-Ren Chen) [this message]
2012-11-15  4:01     ` Johnson, Eric
2012-11-15  5:51       ` 陳韋任 (Wei-Ren Chen)
2012-11-15 17:30         ` Johnson, Eric
2012-11-15 14:04   ` Aurelien Jarno
2012-11-15 17:23     ` Johnson, Eric

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=20121115032652.GA17050@cs.nctu.edu.tw \
    --to=chenwj@iis.sinica.edu.tw \
    --cc=ericj@mips.com \
    --cc=proljc@gmail.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-trivial@nongnu.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.