From mboxrd@z Thu Jan 1 00:00:00 1970 From: Masami Hiramatsu Subject: Re: [PATCH -tip v6 1/5] x86: instruction decorder API Date: Thu, 21 May 2009 11:52:05 -0400 Message-ID: <4A1578A5.7090608@redhat.com> References: <20090520205758.25003.44632.stgit@localhost.localdomain> <20090520205805.25003.71314.stgit@localhost.localdomain> <40e92d5b0905201501i1548621xe1ee63aa88129a0@mail.gmail.com> <4A1488A2.30706@redhat.com> <4A14EA86.2060609@zytor.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Cc: =?UTF-8?B?UHJ6ZW15c8WCYXcgUGF3ZcWCY3p5aw==?= , Ingo Molnar , Steven Rostedt , lkml , systemtap , kvm , DLE , Jim Keniston , Ananth N Mavinakayanahalli , Frederic Weisbecker , Andi Kleen , Vegard Nossum , Avi Kivity To: "H. Peter Anvin" Return-path: In-Reply-To: <4A14EA86.2060609@zytor.com> List-Unsubscribe: List-Subscribe: List-Post: List-Help: , Sender: systemtap-owner@sourceware.org List-Id: kvm.vger.kernel.org H. Peter Anvin wrote: > Masami Hiramatsu wrote: >>> /usr/bin/env 'awk -f' gen-insn-attr-x86.awk >>> >>> which is obviously wrong. >> Oops, right. Anyway, there is no reason that we make it >> executable because it's always called from Makefile. >> So, I think just removing that line is better way. >> > > I suggest leaving in a #!/bin/awk line just for documentation's sake, > though. OK, I'd leave that, and change usage line to use awk command instead of run the script directly. Thank you! > > -hpa > -- Masami Hiramatsu Software Engineer Hitachi Computer Products (America) Inc. Software Solutions Division e-mail: mhiramat@redhat.com