<div dir="ltr">I sent a patch to the mailing list like in the good old days.<div><br></div><div>Do you prefer github PR?</div><div><br></div><div>Let me know.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 5, 2018 at 7:36 PM, Eugene Syromiatnikov <span dir="ltr"><<a href="mailto:esyr@redhat.com" target="_blank">esyr@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, Jun 05, 2018 at 07:13:45PM +0300, Elazar Leibovich wrote:<br>
> Hi,<br>
> <br>
> Currently the format in which BPF programs are printed, is an strace<br>
> specific format.<br>
> <br>
> It's a fine format, but, except of being verbose, it's incompatible with<br>
> kernel's tooling.<br>
> <br>
> Currently abbreviated format, and none-verbose format are identical.<br>
> <br>
> I therefor suggest the following:<br>
> <br>
> None-verbose format would remain as it is today. That is, printout of the<br>
> BPF program address only without traversing it, like the man page says.<br>
> <br>
> Verbose and abbreviated format, would be the bpf_asm format of <len>,<op><br>
> <jt> <jf>  <k>,.... which is indeed shorter.<br>
> <br>
> Verbose and none-abbreviated format would be the current format.<br>
> <br>
> The main advantage of my proposal is that in bpf_asm compatible mode, you<br>
> can dump it to  bpf_debug, run it, dump it etc. I'm not aware of tooling<br>
> that supports the current format.<br>
> <br>
> I have a patch to submit, if you think this is a good solution.<br>
> <br>
> What do you think?<br>
<br>
</div></div>I'm actually planning to continue improving BPF program output format,<br>
as there is a lot of work to do, and making it closer to verifier/bpftool<br>
output is on my todo list, so, yeah, the patch like that will be pretty<br>
much appreciated.<br>
<br>
But, there are two things I have in my mind:<br>
 * We try to stick to C-like syntax as much as possible, so, for a BPF<br>
   assembly dump some syntax should be invented for printing those dumps<br>
   (like, enclose it in some combination of braces/quotes/...).<br>
 * I'm also thinking about (and leaning to) I/O dump-like format, when<br>
   program disassembly is printed after the syscall.<br>
<span class="HOEnZb"><font color="#888888">-- <br>
Strace-devel mailing list<br>
<a href="mailto:Strace-devel@lists.strace.io">Strace-devel@lists.strace.io</a><br>
<a href="https://lists.strace.io/mailman/listinfo/strace-devel" rel="noreferrer" target="_blank">https://lists.strace.io/<wbr>mailman/listinfo/strace-devel</a><br>
</font></span></blockquote></div><br></div>