From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 25275 invoked by alias); 12 Dec 2011 14:06:07 -0000 Received: (qmail 25252 invoked by uid 22791); 12 Dec 2011 14:06:02 -0000 X-SWARE-Spam-Status: No, hits=-2.2 required=5.0 tests=AWL,BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FROM,RCVD_IN_DNSWL_LOW,TW_SV X-Spam-Check-By: sourceware.org Received: from mail-ww0-f43.google.com (HELO mail-ww0-f43.google.com) (74.125.82.43) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Mon, 12 Dec 2011 14:05:43 +0000 Received: by wgbds11 with SMTP id ds11so9216709wgb.12 for ; Mon, 12 Dec 2011 06:05:41 -0800 (PST) Received: by 10.216.229.24 with SMTP id g24mr1999479weq.67.1323698741498; Mon, 12 Dec 2011 06:05:41 -0800 (PST) MIME-Version: 1.0 Received: by 10.223.89.73 with HTTP; Mon, 12 Dec 2011 06:05:00 -0800 (PST) In-Reply-To: References: From: Hui Zhu Date: Mon, 12 Dec 2011 14:06:00 -0000 Message-ID: Subject: Re: KGTP (Linux Kernel debugger and tracer) 20111111 release (Android support OK) To: Geunsik Lim Cc: linux-kernel@vger.kernel.org, "gdb@sourceware.org" , kgtp , Marc Khouzam , Thiago Jung Bauermann , Steven , colyli@gmail.com, Christoph Hellwig , Steven Rostedt , Randy Dunlap Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes Mailing-List: contact gdb-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-owner@sourceware.org X-SW-Source: 2011-12/txt/msg00017.txt.bz2 Hi Geunsik, Thanks for your patch. But Kernel upstream: struct perf_event * perf_event_create_kernel_counter(struct perf_event_attr *attr, int cpu, struct task_struct *task, perf_overflow_handler_t overflow_handler, void *context) I am sorry that gtp.patch is just for the Kernel upstream (Actually, it just for review). I suggest you use LKM if it is OK for you. If you want include KGTP into your kernel tree or something, maybe I can supply a special patch for you. BTW looks I need release an gtp_auto.patch that can patch to most of Linux kernel. Thanks, Hui On Mon, Dec 12, 2011 at 19:13, Geunsik Lim wrote: > On Mon, Nov 14, 2011 at 2:15 PM, Hui Zhu wrote: >> KGTP is a realtime and lightweight Linux Kernel GDB debugger and tracer. >> >> It makes Linux Kernel supply a GDB remote debug interface. Then GDB in >> current machine or remote machine can debug and trace Linux through >> GDB tracepoint without stopping the Linux Kernel. >> And even if the board doesn't have GDB on it and doesn't have >> interface for remote debug. It can debug the Linux Kernel using >> offline debug. >> Now, it supports X86-32, X86-64, MIPS and ARM. >> >> Now, KGTP 20111111 release. >> >> The change of this release is: >> Add a doc for use KGTP with Android in >> https://code.google.com/p/kgtp/wiki/HowToUseKGTPinAndroid >> post_handler will make kprobes-optimization cannot be used. So make >> gtp_kp_post_handler be call only when tpe->step is true. >> Make rdtsc_current to x86 special. >> Register easy Kprobe handler to speed up it when no tracepoint access >> $cooked_clock, $cooked_rdtsc or "pc_pe" tvariable. >> Fix a bug of ARM build. >> To get other change info please goto https://code.google.com/p/kgtp/wiki= /UPDATE >> >> According to the comments of Christoph. =A0I make a patch for Linux >> Kernel and make it looks OK with checkpatch.pl. =A0The file gtp.patch is >> include in the source of KGTP code. > Dear Hui Zhu, > > Recently, I synchronized KGTP kernel source with svn 838 to test > built-in kernel > after adjusting recent bug-fix. I still had a compilation error. > Please, adjust the below patch content to support =A0X86/ARM > architecture normally. > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > From 39d58cc7dab8df5e57fa106e8e7f00e562b45212 Mon Sep 17 00:00:00 2001 > From: Geunsik Lim > Date: Mon, 10 Dec 2011 18:08:03 +0900 > Subject: [PATCH] Modify perf related func interface. > > Signed-off-by: Geunsik Lim > Signed-off-by: Kyungmin Park > --- > =A0lib/gtp.c | =A0 =A02 +- > =A01 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/lib/gtp.c b/lib/gtp.c > index 9bdf82c..9a2670e 100644 > --- a/lib/gtp.c > +++ b/lib/gtp.c > @@ -4480,7 +4480,7 @@ restart: > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0tve->pts->event =3D > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0perf_event_create_kernel_c= ounter(&(tve->pts->attr), > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 = =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 tve->pts->cpu, > - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0= =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0NULL, NULL, NULL); > + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0= =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0NULL, NULL); > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0if (IS_ERR(tve->pts->event)) { > =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0int =A0 =A0 ret =3D PTR_ER= R(tve->pts->event); > > -- > 1.7.3.4 > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D > > Thank you for your contribution for embedded kernel debugging > >> >> Thanks, >> Hui >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-kernel" = in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at =A0http://vger.kernel.org/majordomo-info.html >> Please read the FAQ at =A0http://www.tux.org/lkml/ > > > > -- > Best regards, > Geunsik Lim ( Samsung Electronics ) > Blog : http://blog.naver.com/invain/ > Homepage: http://leemgs.fedorapeople.org > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at=A0 http://vger.kernel.org/majordomo-info.html > Please read the FAQ at=A0 http://www.tux.org/lkml/