On Mon, Aug 26, 2019 at 05:05:58PM -0700, Andy Lutomirski wrote: > >> > >> The BPF verifier and interpreter, taken in isolation, may be extremely > >> safe, but attaching BPF programs to various hooks can easily take down > >> the system, deliberately or by accident. A handler, especially if it > >> can access user memory or otherwise fault, will explode if attached to > >> an inappropriate kprobe, hw_breakpoint, or function entry trace event. > > > > absolutely not true. > > This is not a constructive way to have a conversation. When you get > an email that contains a statement you disagree with, perhaps you > could try to give some argument as to why you disagree rather than > just saying "absolutely not true". Especially when you are talking to > one of the maintainers of the affected system who has a > not-yet-finished branch that addresses some of the bugs that you claim > absolutely don't exist. If it's really truly necessary, I can go and > write an example that will crash an x86 kernel, but I feel like it > would be a waste of everyone's time. Please do write an example and prove your earlier sensational statement that "can _easily_ take down the system" by attaching bpf to kprobe. Most of the functions where kprobes are not allowed are already marked by 'nokprobe'. All of them marked? Probably not. There could be places where kprobe will blow the system, but 1. it's not easy to do. unlike your claim 2. that issue has nothing to do with bpf safety guarantees. > How confident are you that the BPF program that calls bpf_probe_read() > on an MMIO address has well-defined semantics? How confident are you > that the system will still work if such a program runs? bpf_probe_read is a wrapper of probe_read. Nothing more. I'm confident that probe_read maintainers are doing good job. All of the bpf tracing is relying on existing kernel mechanisms like kprobe, uprobe, perf, probe_read, etc. bpf verifier cannot make them safer. If reading mmio via bpf_probe_read will trigger undesired hw behavior there is nothing bpf verifier can do about it.
