On Wed, Aug 01, 2018 at 07:52:12AM -0600, Jonathan Corbet wrote: > On Wed, 1 Aug 2018 15:08:58 +1000 > "Tobin C. Harding" <me@xxxxxxxx> 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...:) Thanks for the feedback Jon, I like when I get feedback as apposed to just patches being applied - makes me feel like the work actually matters, thanks. > 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. Will use this method for re-spin of this set and for future conversions. > 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. And we have Documentation/bpf as well? I'll have a go at splitting this file up and/or moving it around. The content is pretty heavy so I may need some comments/ideas from the original authors. I'll spin up a V2 and we can see how it looks. thanks, Tobin. -- To unsubscribe from this list: send the line "unsubscribe linux-doc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html