From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752844AbcGOHuU (ORCPT ); Fri, 15 Jul 2016 03:50:20 -0400 Received: from foss.arm.com ([217.140.101.70]:47088 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752749AbcGOHuM (ORCPT ); Fri, 15 Jul 2016 03:50:12 -0400 Date: Fri, 15 Jul 2016 08:50:02 +0100 From: Catalin Marinas To: William Cohen Cc: David Long , Mark Rutland , Petr Mladek , Zi Shen Lim , Will Deacon , Andrey Ryabinin , yalin wang , Li Bin , John Blackwood , Pratyush Anand , Daniel Thompson , Huang Shijie , Dave P Martin , Jisheng Zhang , Vladimir Murzin , Steve Capper , Suzuki K Poulose , Marc Zyngier , Yang Shi , Mark Brown , Sandeepa Prabhu , Alex =?iso-8859-1?Q?Benn=E9e?= , Adam Buchbinder , linux-arm-kernel@lists.infradead.org, Ard Biesheuvel , linux-kernel@vger.kernel.org, James Morse , Masami Hiramatsu , Andrew Morton , Robin Murphy , Jens Wiklander , Christoffer Dall Subject: Re: [PATCH v15 00/10] arm64: Add kernel probes (kprobes) support Message-ID: <20160715075001.GA13360@e104818-lin.cambridge.arm.com> References: <1467995754-32508-1-git-send-email-dave.long@linaro.org> <20160714162208.GA22235@e104818-lin.cambridge.arm.com> <6b43c28b-df68-112d-f8a3-6f952a50f54f@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6b43c28b-df68-112d-f8a3-6f952a50f54f@redhat.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jul 14, 2016 at 01:09:08PM -0400, William Cohen wrote: > On 07/14/2016 12:22 PM, Catalin Marinas wrote: > > On Fri, Jul 08, 2016 at 12:35:44PM -0400, David Long wrote: > >> David A. Long (3): > >> arm64: Add HAVE_REGS_AND_STACK_ACCESS_API feature > >> arm64: Add more test functions to insn.c > >> arm64: add conditional instruction simulation support > >> > >> Pratyush Anand (2): > >> arm64: Blacklist non-kprobe-able symbol > >> arm64: Treat all entry code as non-kprobe-able > >> > >> Sandeepa Prabhu (4): > >> arm64: Kprobes with single stepping support > >> arm64: kprobes instruction simulation support > >> arm64: Add kernel return probes support (kretprobes) > >> kprobes: Add arm64 case in kprobe example module > >> > >> William Cohen (1): > >> arm64: Add trampoline code for kretprobes > > > > I applied these patches on top of the arm64 for-next/core branch an > > tried to run the resulting kernel in a guest (on a Juno platform using > > both kvmtool and qemu) with KPROBES_SANITY_TEST enabled. Unfortunately, > > the kernel fails to boot with lots of "Unexpected kernel single-step > > exception at EL1". > > > > Did you manage to run Kprobes in a guest before? > > I ran the systemtap testsuite several times on a physical machine > running a kernel with the kprobe v15 patches without problem. > Shouldn't the guest machine behave in the same manner as a host > machine for single stepping and exception handling? If the guest > machine is failing, wouldn't that suggest there is a problem with the > KVM handling of single stepping for guests? It didn't fail for me on the host either. What's strange is that on some occasions even the guest managed to get to a prompt. I'll do more tests today on different CPU configurations, just to rule out potential hardware issues. If not hardware related, it's possible that the interaction with KVM doesn't work as expected, maybe the saving/restoring of the guest debug state loses information. -- Catalin