public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Keith Owens <kaos@ocs.com.au>
To: Hugh Dickins <hugh@veritas.com>
Cc: Horst von Brand <vonbrand@inf.utfsm.cl>,
	Dave Jones <davej@redhat.com>, Andrew Morton <akpm@osdl.org>,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH] WTF is VLI?
Date: Tue, 16 Nov 2004 17:00:10 +1100	[thread overview]
Message-ID: <13456.1100584810@kao2.melbourne.sgi.com> (raw)
In-Reply-To: Your message of "Mon, 15 Nov 2004 21:18:43 -0000." <Pine.LNX.4.44.0411152110460.4171-100000@localhost.localdomain>

On Mon, 15 Nov 2004 21:18:43 +0000 (GMT), 
Hugh Dickins <hugh@veritas.com> wrote:
>On Fri, 12 Nov 2004, Horst von Brand wrote:
>> Dave Jones <davej@redhat.com> said:
>> > On Thu, Nov 11, 2004 at 09:05:11PM +0000, Hugh Dickins wrote:
>> >  > What is this "VLI" that 2.6.9 started putting after the taint string
>> >  > in i386 oopses?  Vick Library Index?  Vineyard Leadership Institute?
>> > 
>> > "Variable length instructions".  I think newer ksymoops looks
>> > for this tag and does something magical when doing disassembly.
>> 
>> Huh? Either an architecture has them (i386) or doesn't (RISCs). 
>> Or am I seriously misunderstanding here?
>
>I share your surprise, it does seem rather odd.  I think what it's
>really trying to do is distinguish how 2.6.9 starts the "Code:" bytes
>at eip - 43, where 2.6.8 started at eip; but flag that since it's VLI
>then it's got a bit of guessing to do.  I'd have preferred to work it
>out from i386 and the new "<%02x>" around the eip byte itself, rather
>than stick a "VLI" somewhere else; but let's not interfere now it's so.

ksymoops has to work with lots of different log formats from lots of
different architectures.  Some arch's already print the code around the
oops and enclose the failing instruction in <> or [], some do not.

Just looking at a code string, you cannot tell if the arch has variable
length instructions or not (don't forget that ksymoops also works cross
architecture).  The VLI tag will work for _all_ architectures that have
variable length instructions, not just i386.  At the very least, s390
can use it as well.

There are enough ambiguity problems in ksymoops, without adding new
ones.


  reply	other threads:[~2004-11-16  6:04 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-11-11 21:05 [PATCH] WTF is VLI? Hugh Dickins
2004-11-11 21:10 ` Dave Jones
2004-11-12 23:26   ` Horst von Brand
2004-11-15 21:18     ` Hugh Dickins
2004-11-16  6:00       ` Keith Owens [this message]
2004-11-16  9:56         ` Hugh Dickins
2004-11-17  5:55           ` Keith Owens
2004-11-18 15:47             ` Avi Kivity
2004-11-18 18:08               ` Horst von Brand
2004-11-18 23:38                 ` Avi Kivity
2004-11-21  3:58               ` Keith Owens
2004-11-11 21:14 ` Andrew Morton
     [not found] <fa.inbtt12.195ed02@ifi.uio.no>
     [not found] ` <fa.cg6f09j.ji89hv@ifi.uio.no>
2004-11-18 18:28   ` Bodo Eggert
2004-11-18 23:44     ` Avi Kivity

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=13456.1100584810@kao2.melbourne.sgi.com \
    --to=kaos@ocs.com.au \
    --cc=akpm@osdl.org \
    --cc=davej@redhat.com \
    --cc=hugh@veritas.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=vonbrand@inf.utfsm.cl \
    /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