On Fri, Aug 29, 2014 at 3:24 PM, Daniel Borkmann <dborkman@xxxxxxxxxx> 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-api" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
