From: Leo Yan <leo.yan@linaro.org> To: Masami Hiramatsu <mhiramat@kernel.org> Cc: Russell King <linux@armlinux.org.uk>, Oleg Nesterov <oleg@redhat.com>, Catalin Marinas <catalin.marinas@arm.com>, Will Deacon <will@kernel.org>, Alexei Starovoitov <ast@kernel.org>, Daniel Borkmann <daniel@iogearbox.net>, Martin KaFai Lau <kafai@fb.com>, Song Liu <songliubraving@fb.com>, Yonghong Song <yhs@fb.com>, Arnd Bergmann <arnd@arndb.de>, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, bpf@vger.kernel.org, Justin He <Justin.He@arm.com> Subject: Re: [PATCH 0/2] arm/arm64: Add support for function error injection Date: Wed, 17 Jul 2019 16:14:56 +0800 [thread overview] Message-ID: <20190717081456.GB20476@leoy-ThinkPad-X240s> (raw) In-Reply-To: <20190717165222.62e02b99ebc16e23c3b81de2@kernel.org> On Wed, Jul 17, 2019 at 04:52:22PM +0900, Masami Hiramatsu wrote: > On Tue, 16 Jul 2019 19:12:59 +0800 > Leo Yan <leo.yan@linaro.org> wrote: > > > This small patch set is to add support for function error injection; > > this can be used to eanble more advanced debugging feature, e.g. > > CONFIG_BPF_KPROBE_OVERRIDE. > > > > I only tested the first patch on arm64 platform Juno-r2 with below > > steps; the second patch is for arm arch, but I absent the platform > > for the testing so only pass compilation. > > > > - Enable kernel configuration: > > CONFIG_BPF_KPROBE_OVERRIDE > > CONFIG_BTRFS_FS > > CONFIG_BPF_EVENTS=y > > CONFIG_KPROBES=y > > CONFIG_KPROBE_EVENTS=y > > CONFIG_BPF_KPROBE_OVERRIDE=y > > - Build samples/bpf on Juno-r2 board with Debian rootFS: > > # cd $kernel > > # make headers_install > > # make samples/bpf/ LLC=llc-7 CLANG=clang-7 > > - Run the sample tracex7: > > # ./tracex7 /dev/sdb1 > > [ 1975.211781] BTRFS error (device (efault)): open_ctree failed > > mount: /mnt/linux-kernel/linux-cs-dev/samples/bpf/tmpmnt: mount(2) system call failed: Cannot allocate memory. > > This series looks good to me from the view point of override usage :) > > Reviewed-by: Masami Hiramatsu <mhiramat@kernel.org> > > For this series. > > Thank you, Thank you for reviewing, Masami. > > > > > > Leo Yan (2): > > arm64: Add support for function error injection > > arm: Add support for function error injection > > > > arch/arm/Kconfig | 1 + > > arch/arm/include/asm/error-injection.h | 13 +++++++++++++ > > arch/arm/include/asm/ptrace.h | 5 +++++ > > arch/arm/lib/Makefile | 2 ++ > > arch/arm/lib/error-inject.c | 19 +++++++++++++++++++ > > arch/arm64/Kconfig | 1 + > > arch/arm64/include/asm/error-injection.h | 13 +++++++++++++ > > arch/arm64/include/asm/ptrace.h | 5 +++++ > > arch/arm64/lib/Makefile | 2 ++ > > arch/arm64/lib/error-inject.c | 19 +++++++++++++++++++ > > 10 files changed, 80 insertions(+) > > create mode 100644 arch/arm/include/asm/error-injection.h > > create mode 100644 arch/arm/lib/error-inject.c > > create mode 100644 arch/arm64/include/asm/error-injection.h > > create mode 100644 arch/arm64/lib/error-inject.c > > > > -- > > 2.17.1 > > > > > -- > Masami Hiramatsu <mhiramat@kernel.org>
WARNING: multiple messages have this Message-ID (diff)
From: Leo Yan <leo.yan@linaro.org> To: Masami Hiramatsu <mhiramat@kernel.org> Cc: Song Liu <songliubraving@fb.com>, linux-kernel@vger.kernel.org, Arnd Bergmann <arnd@arndb.de>, Daniel Borkmann <daniel@iogearbox.net>, Catalin Marinas <catalin.marinas@arm.com>, Alexei Starovoitov <ast@kernel.org>, Russell King <linux@armlinux.org.uk>, netdev@vger.kernel.org, Oleg Nesterov <oleg@redhat.com>, Yonghong Song <yhs@fb.com>, bpf@vger.kernel.org, Will Deacon <will@kernel.org>, Martin KaFai Lau <kafai@fb.com>, linux-arm-kernel@lists.infradead.org, Justin He <Justin.He@arm.com> Subject: Re: [PATCH 0/2] arm/arm64: Add support for function error injection Date: Wed, 17 Jul 2019 16:14:56 +0800 [thread overview] Message-ID: <20190717081456.GB20476@leoy-ThinkPad-X240s> (raw) In-Reply-To: <20190717165222.62e02b99ebc16e23c3b81de2@kernel.org> On Wed, Jul 17, 2019 at 04:52:22PM +0900, Masami Hiramatsu wrote: > On Tue, 16 Jul 2019 19:12:59 +0800 > Leo Yan <leo.yan@linaro.org> wrote: > > > This small patch set is to add support for function error injection; > > this can be used to eanble more advanced debugging feature, e.g. > > CONFIG_BPF_KPROBE_OVERRIDE. > > > > I only tested the first patch on arm64 platform Juno-r2 with below > > steps; the second patch is for arm arch, but I absent the platform > > for the testing so only pass compilation. > > > > - Enable kernel configuration: > > CONFIG_BPF_KPROBE_OVERRIDE > > CONFIG_BTRFS_FS > > CONFIG_BPF_EVENTS=y > > CONFIG_KPROBES=y > > CONFIG_KPROBE_EVENTS=y > > CONFIG_BPF_KPROBE_OVERRIDE=y > > - Build samples/bpf on Juno-r2 board with Debian rootFS: > > # cd $kernel > > # make headers_install > > # make samples/bpf/ LLC=llc-7 CLANG=clang-7 > > - Run the sample tracex7: > > # ./tracex7 /dev/sdb1 > > [ 1975.211781] BTRFS error (device (efault)): open_ctree failed > > mount: /mnt/linux-kernel/linux-cs-dev/samples/bpf/tmpmnt: mount(2) system call failed: Cannot allocate memory. > > This series looks good to me from the view point of override usage :) > > Reviewed-by: Masami Hiramatsu <mhiramat@kernel.org> > > For this series. > > Thank you, Thank you for reviewing, Masami. > > > > > > Leo Yan (2): > > arm64: Add support for function error injection > > arm: Add support for function error injection > > > > arch/arm/Kconfig | 1 + > > arch/arm/include/asm/error-injection.h | 13 +++++++++++++ > > arch/arm/include/asm/ptrace.h | 5 +++++ > > arch/arm/lib/Makefile | 2 ++ > > arch/arm/lib/error-inject.c | 19 +++++++++++++++++++ > > arch/arm64/Kconfig | 1 + > > arch/arm64/include/asm/error-injection.h | 13 +++++++++++++ > > arch/arm64/include/asm/ptrace.h | 5 +++++ > > arch/arm64/lib/Makefile | 2 ++ > > arch/arm64/lib/error-inject.c | 19 +++++++++++++++++++ > > 10 files changed, 80 insertions(+) > > create mode 100644 arch/arm/include/asm/error-injection.h > > create mode 100644 arch/arm/lib/error-inject.c > > create mode 100644 arch/arm64/include/asm/error-injection.h > > create mode 100644 arch/arm64/lib/error-inject.c > > > > -- > > 2.17.1 > > > > > -- > Masami Hiramatsu <mhiramat@kernel.org> _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2019-07-17 8:15 UTC|newest] Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-07-16 11:12 [PATCH 0/2] arm/arm64: Add support for function error injection Leo Yan 2019-07-16 11:12 ` Leo Yan 2019-07-16 11:13 ` [PATCH 1/2] arm64: " Leo Yan 2019-07-16 11:13 ` Leo Yan 2019-07-25 1:42 ` Leo Yan 2019-07-25 1:42 ` Leo Yan 2019-07-31 16:08 ` Will Deacon 2019-07-31 16:08 ` Will Deacon 2019-07-31 16:58 ` Leo Yan 2019-07-31 16:58 ` Leo Yan 2019-07-16 11:13 ` [PATCH 2/2] arm: " Leo Yan 2019-07-16 11:13 ` Leo Yan 2019-07-25 1:48 ` Leo Yan 2019-07-25 1:48 ` Leo Yan 2019-07-17 7:52 ` [PATCH 0/2] arm/arm64: " Masami Hiramatsu 2019-07-17 7:52 ` Masami Hiramatsu 2019-07-17 8:14 ` Leo Yan [this message] 2019-07-17 8:14 ` Leo Yan
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=20190717081456.GB20476@leoy-ThinkPad-X240s \ --to=leo.yan@linaro.org \ --cc=Justin.He@arm.com \ --cc=arnd@arndb.de \ --cc=ast@kernel.org \ --cc=bpf@vger.kernel.org \ --cc=catalin.marinas@arm.com \ --cc=daniel@iogearbox.net \ --cc=kafai@fb.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux@armlinux.org.uk \ --cc=mhiramat@kernel.org \ --cc=netdev@vger.kernel.org \ --cc=oleg@redhat.com \ --cc=songliubraving@fb.com \ --cc=will@kernel.org \ --cc=yhs@fb.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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.