lkml.org 
[lkml]   [2018]   [Aug]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH bpf-next 03/13] docs: net: Add filter to index toctree
On Wed,  1 Aug 2018 15:08:58 +1000
"Tobin C. Harding" <me@tobin.cc> wrote:

> filter.rst was just created (from filter.txt). We should include it in
> the networking docs toctree.
>
> Add filter to Documentation/networking/index.rst toctree.

So...the minorest of nits just to prove that I actually looked at
this...:) Adding it to the toctree at this point is going to make the
docs build messy (OK, even messier than it already is) because the RST
conversion hasn't really been done yet. I'd do this at the end of the
series, just in case somebody, somewhere, is crazy enough to try to
bisect a docs build.

Something that's a bit more relevant, perhaps: this document is a bit of
a mishmash. It starts by talking about how to use cBPF for socket
filters, then includes a long discussion about the features of eBPF
which, as I understand it, is still not accessible to user space for
socket filters. That latter part certainly merits being in its own file
where people will actually find it? And if it were me, I would move that
file into the core API guide, since it will be relevant beyond networking.

Thanks,

jon

\
 
 \ /
  Last update: 2018-08-01 15:54    [W:0.186 / U:1.840 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site