kvm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "H. Peter Anvin" <hpa@zytor.com>
To: Jim Keniston <jkenisto@us.ibm.com>
Cc: Masami Hiramatsu <mhiramat@redhat.com>,
	Ingo Molnar <mingo@elte.hu>,
	        Ananth N Mavinakayanahalli <ananth@in.ibm.com>,
	        Andi Kleen <andi@firstfloor.org>,
	kvm@vger.kernel.org,         Steven Rostedt <rostedt@goodmis.org>,
	        Frederic Weisbecker <fweisbec@gmail.com>,
	        Andrew Morton <akpm@linux-foundation.org>,
	        Arnaldo Carvalho de Melo <acme@redhat.com>,
	        systemtap-ml <systemtap@sources.redhat.com>,
	        LKML <linux-kernel@vger.kernel.org>,
	        Vegard Nossum <vegard.nossum@gmail.com>,
	Avi Kivity <avi@redhat.com>,
	        Roland McGrath <roland@redhat.com>
Subject: Re: [PATCH -tip 3/6 V4.1] x86: instruction decorder API
Date: Mon, 06 Apr 2009 15:55:19 -0700	[thread overview]
Message-ID: <49DA8857.8030607@zytor.com> (raw)
In-Reply-To: <1239058135.5212.43.camel@localhost.localdomain>

Jim Keniston wrote:
> 
> For user-space probing, we've been concentrating on native-built
> executables.  Am I correct in thinking that we'll see 16-bit or V86 mode
> only on legacy apps built elsewhere?  In any case, it only makes sense
> to build on the kvm folks' work in this regard.
> 

That's a fair assumption; you will of course need to test it and take
appropriate action if it doesn't pan out.

> 
> As noted, the INAT tables follow the kvm model of one fat bitmap of
> attributes per opcode, rather than the kprobes/uprobes model of one or
> two 256-bit tables per attribute.  (This latter approach was due to the
> gradual accumulation of tables over the years.)
> 
> I like the bitmap-per-opcode approach because it's relatively easy to
> see in one place everything you're saying about a particular opcode.
> But with all the potential clients for this service, it's not clear that
> we'll get by with a single bitmap for every opcode.  (x86 kvm uses 32
> bits per opcode, I think, and the INAT tables use 10.  Seems like we
> could overrun 64 bits pretty quickly.)  So I guess that means we'll have
> to get a little creative as to how we expose these attribute sets to the
> client.
> 

This is another very good reason to use an instruction table which is
preprocessed into a usable format: it means that if the internal data
structures change -- and they almost certainly will have to at some
point -- the raw data isn't lost.

	-hpa

-- 
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel.  I don't speak on their behalf.

  reply	other threads:[~2009-04-06 22:55 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-02 17:24 [PATCH -tip 3/6 V4] x86: instruction decorder API Masami Hiramatsu
2009-04-03 23:29 ` [PATCH -tip 3/6 V4.1] " Masami Hiramatsu
2009-04-03 23:43   ` H. Peter Anvin
2009-04-04  0:37     ` Masami Hiramatsu
2009-04-06 22:48       ` Jim Keniston
2009-04-06 22:55         ` H. Peter Anvin [this message]
2009-04-16 23:31           ` Masami Hiramatsu
2009-04-16 23:39             ` H. Peter Anvin
2009-04-17 13:31               ` Masami Hiramatsu
2009-04-17 18:07                 ` H. Peter Anvin
2009-04-17  0:06             ` Jim Keniston
2009-04-17  0:08               ` H. Peter Anvin
2009-04-22  0:17                 ` Masami Hiramatsu
2009-04-23  0:47                   ` Jim Keniston
2009-04-23 17:29                     ` Masami Hiramatsu
2009-04-23 22:22                       ` Jim Keniston
2009-04-24  3:53                         ` Masami Hiramatsu

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=49DA8857.8030607@zytor.com \
    --to=hpa@zytor.com \
    --cc=acme@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=ananth@in.ibm.com \
    --cc=andi@firstfloor.org \
    --cc=avi@redhat.com \
    --cc=fweisbec@gmail.com \
    --cc=jkenisto@us.ibm.com \
    --cc=kvm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mhiramat@redhat.com \
    --cc=mingo@elte.hu \
    --cc=roland@redhat.com \
    --cc=rostedt@goodmis.org \
    --cc=systemtap@sources.redhat.com \
    --cc=vegard.nossum@gmail.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).