2018-07-26 05:04:47

by Tobin C. Harding

[permalink] [raw]
Subject: [PATCH v2 0/4] docs: bpf: Fix RST conversion

Hi Dave,

I'm sending v2 of this to you instead of to Jon. Rationale: BPF is to
do with networking anyways and there is a broken link that requires
conversion of Documentation/networking/filter.txt to fix and that will
go to you. FTR there are no merge conflicts between this set and the
other set I just sent (either can be applied on top of the other)

[PATCH v2 net-next 0/3] docs: net: Convert netdev-FAQ to RST


Recently BPF docs were converted to RST format. A couple of things were
missed.

- Use 'index.rst' instead of 'README.rst'. Although README.rst will
work just fine it is more typical to keep the subdirectory indices
in a file called 'index.rst'.

- Integrate files Documentation/bpf/*.rst into build system using
toctree in Documentation/bpf/index.rst

- Include bpf/index in top level toctree so bpf is indexed in the main
kernel docs.

- Make anal change to heading format (inline with rest of Documentation/).


thanks,
Tobin.

v2:
- Fix typo in commit log s/us/use (thanks Sergei)


Tobin C. Harding (4):
docs: bpf: Rename README.rst to index.rst
docs: bpf: Add toctree to index
docs: Add bpf/index to top level index
docs: bpf: Capitalise document heading

Documentation/bpf/{README.rst => index.rst} | 10 +++++-----
Documentation/index.rst | 1 +
2 files changed, 6 insertions(+), 5 deletions(-)
rename Documentation/bpf/{README.rst => index.rst} (82%)

--
2.17.1



2018-07-26 05:04:51

by Tobin C. Harding

[permalink] [raw]
Subject: [PATCH v2 2/4] docs: bpf: Add toctree to index

Recently bpf/ docs were converted to us RST format. bp/index.rst was
created out of README but toctree was not added to include files within
Documentation/bpf/

Add toctree to Documentation/bpf/index.rst

Signed-off-by: Tobin C. Harding <[email protected]>
---
Documentation/bpf/index.rst | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/Documentation/bpf/index.rst b/Documentation/bpf/index.rst
index b9a80c9e9392..ab2ff13a809b 100644
--- a/Documentation/bpf/index.rst
+++ b/Documentation/bpf/index.rst
@@ -22,14 +22,14 @@ Frequently asked questions (FAQ)

Two sets of Questions and Answers (Q&A) are maintained.

-* QA for common questions about BPF see: bpf_design_QA_
+.. toctree::
+ :maxdepth: 1

-* QA for developers interacting with BPF subsystem: bpf_devel_QA_
+ bpf_design_QA
+ bpf_devel_QA


.. Links:
-.. _bpf_design_QA: bpf_design_QA.rst
-.. _bpf_devel_QA: bpf_devel_QA.rst
.. _Documentation/networking/filter.txt: ../networking/filter.txt
.. _man-pages: https://www.kernel.org/doc/man-pages/
.. _bpf(2): http://man7.org/linux/man-pages/man2/bpf.2.html
--
2.17.1


2018-07-26 05:04:51

by Tobin C. Harding

[permalink] [raw]
Subject: [PATCH v2 1/4] docs: bpf: Rename README.rst to index.rst

Recently bpf/ docs were converted to use RST format. 'README.rst' was
created but in order to fit in with the Sphinx build system this file
should be named 'index.rst'. Rename file, fixes to integrate into
Sphinx build system in following patches.

docs: Rename Documentation/bpf/README.rst to Documentation/bpf/index.rst

Signed-off-by: Tobin C. Harding <[email protected]>
---
Documentation/bpf/{README.rst => index.rst} | 0
1 file changed, 0 insertions(+), 0 deletions(-)
rename Documentation/bpf/{README.rst => index.rst} (100%)

diff --git a/Documentation/bpf/README.rst b/Documentation/bpf/index.rst
similarity index 100%
rename from Documentation/bpf/README.rst
rename to Documentation/bpf/index.rst
--
2.17.1


2018-07-26 05:04:55

by Tobin C. Harding

[permalink] [raw]
Subject: [PATCH v2 3/4] docs: Add bpf/index to top level index

Recently bpf docs were converted to RST format. The new files were not
added to the top level toctree. This causes build system to emit a
warning of type

WARNING: document isn't included in any toctree

Add bpf/index.rst to Documentation/index.rst

Signed-off-by: Tobin C. Harding <[email protected]>
---
Documentation/index.rst | 1 +
1 file changed, 1 insertion(+)

diff --git a/Documentation/index.rst b/Documentation/index.rst
index 5d310d960f82..24339a9c2ef8 100644
--- a/Documentation/index.rst
+++ b/Documentation/index.rst
@@ -92,6 +92,7 @@ needed).
crypto/index
filesystems/index
vm/index
+ bpf/index

Architecture-specific documentation
-----------------------------------
--
2.17.1


2018-07-26 05:04:59

by Tobin C. Harding

[permalink] [raw]
Subject: [PATCH v2 4/4] docs: bpf: Capitalise document heading

The majority of files in the kernel documentation index use
capitalisation for all words, especially the shorter ones. BPF docs
better fit in with the rest of the documentation if the heading is all
capitalised.

Capitalise document heading.

Signed-off-by: Tobin C. Harding <[email protected]>
---
Documentation/bpf/index.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Documentation/bpf/index.rst b/Documentation/bpf/index.rst
index ab2ff13a809b..00a8450a602f 100644
--- a/Documentation/bpf/index.rst
+++ b/Documentation/bpf/index.rst
@@ -1,5 +1,5 @@
=================
-BPF documentation
+BPF Documentation
=================

This directory contains documentation for the BPF (Berkeley Packet
--
2.17.1


2018-07-27 05:27:13

by Daniel Borkmann

[permalink] [raw]
Subject: Re: [PATCH v2 0/4] docs: bpf: Fix RST conversion

On 07/26/2018 07:03 AM, Tobin C. Harding wrote:
> Hi Dave,
>
> I'm sending v2 of this to you instead of to Jon. Rationale: BPF is to
> do with networking anyways and there is a broken link that requires
> conversion of Documentation/networking/filter.txt to fix and that will
> go to you. FTR there are no merge conflicts between this set and the
> other set I just sent (either can be applied on top of the other)
>
> [PATCH v2 net-next 0/3] docs: net: Convert netdev-FAQ to RST
>
>
> Recently BPF docs were converted to RST format. A couple of things were
> missed.
>
> - Use 'index.rst' instead of 'README.rst'. Although README.rst will
> work just fine it is more typical to keep the subdirectory indices
> in a file called 'index.rst'.
>
> - Integrate files Documentation/bpf/*.rst into build system using
> toctree in Documentation/bpf/index.rst
>
> - Include bpf/index in top level toctree so bpf is indexed in the main
> kernel docs.
>
> - Make anal change to heading format (inline with rest of Documentation/).
>
>
> thanks,
> Tobin.

Applied to bpf-next, thanks Tobin!