lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 29 Aug 2014 16:01:30 -0700 From: Alexei Starovoitov <ast@...mgrid.com> To: Daniel Borkmann <dborkman@...hat.com> Cc: Alexei Starovoitov <alexei.starovoitov@...il.com>, "David S. Miller" <davem@...emloft.net>, Ingo Molnar <mingo@...nel.org>, Linus Torvalds <torvalds@...ux-foundation.org>, Andy Lutomirski <luto@...capital.net>, Steven Rostedt <rostedt@...dmis.org>, Chema Gonzalez <chema@...gle.com>, Eric Dumazet <edumazet@...gle.com>, Peter Zijlstra <a.p.zijlstra@...llo.nl>, Brendan Gregg <brendan.d.gregg@...il.com>, Namhyung Kim <namhyung@...nel.org>, "H. Peter Anvin" <hpa@...or.com>, Andrew Morton <akpm@...ux-foundation.org>, Kees Cook <keescook@...omium.org>, Linux API <linux-api@...r.kernel.org>, "netdev@...r.kernel.org" <netdev@...r.kernel.org>, "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org> Subject: Re: [PATCH v8 net-next 2/2] net: filter: split filter.h and expose eBPF to user space On Fri, Aug 29, 2014 at 3:24 PM, Daniel Borkmann <dborkman@...hat.com> wrote: >> >> Also I really dislike 'ebpf' name in all lower case. If we make such >> header >> file name, we would need to rename all macros and function names >> to EBPF_... which I find very ugly looking. I think all good abbreviations >> are >> three letters :) > > > I don't think we would have to name defines that way, really, that would be > terrible. We can keep them simply *as is*. Not sure though why bpf.h + > ebpf.h > would be that bad. ;) I haven't tried it out yet, but if we would indeed run > into a name collision, above proposal would resolve that. imo it's a consistency issue. If main uapi header is ebpf.h then corresponding kernel internal header should be ebpf.h as well and kernel/ebpf/ directory and so on. That's why I insist on uapi/linux/bpf.h and no other name. Note I didn't move any of the BPF_ALU64_REG, BPF_ALU32_IMM macros from linux/filter.h. Without them my verifier testsuite won't compile, so more lines would be added to bpf.h in the future. At that time we can take 45 lines out of uapi/linux/filter.h and move them into bpf_common.h. My request is let's not fight about it right now. We didn't even cross the bridge yet and arguing about beauty of user apps that come in 30 patches from now... These two patches are about _intent_ of making eBPF usable from userspace, so I can move along with llvm. Also worth noting that llmv will not be including this uapi/linux/bpf.h It has its own infra to generate instructions. Look at: tools/bpf/llvm/lib/Target/BPF/BPFInstrInfo.td it's a special 'table definition' language for describing bits and fields of instructions. So these two patches are mainly establishing _intent_ and bpf.h file name. That's why I'm so paranoid about naming. btw, I've spent last two days writing syscall manpage :( What is the best way to present it for review? If I just attach it raw, it's unreadable... I can include a link to html page, but man2html produces ugly pages comparing to what 'man' command shows. Any nice man converters that generate stuff seen on man7.org ? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists