linux-doc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] docs/bpf: Fix invalid link of "Documentation/bpf/btf.rst"
@ 2023-02-22  3:10 Tiezhu Yang
  2023-02-22  8:54 ` Bagas Sanjaya
  0 siblings, 1 reply; 3+ messages in thread
From: Tiezhu Yang @ 2023-02-22  3:10 UTC (permalink / raw)
  To: Alexei Starovoitov, Daniel Borkmann, Andrii Nakryiko,
	Jonathan Corbet
  Cc: bpf, linux-doc, linux-kernel

"Documentation/bpf/btf.rst" is linked to the following invalid web page:

https://www.kernel.org/doc/html/latest/bpf/btf.rst

The correct link should be:

https://www.kernel.org/doc/html/latest/bpf/btf.html

Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
---
 Documentation/bpf/bpf_devel_QA.rst | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/Documentation/bpf/bpf_devel_QA.rst b/Documentation/bpf/bpf_devel_QA.rst
index 03d4993..f54c328 100644
--- a/Documentation/bpf/bpf_devel_QA.rst
+++ b/Documentation/bpf/bpf_devel_QA.rst
@@ -690,6 +690,7 @@ when:
    https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/tools/testing/selftests/bpf/
 .. _Documentation/dev-tools/kselftest.rst:
    https://www.kernel.org/doc/html/latest/dev-tools/kselftest.html
-.. _Documentation/bpf/btf.rst: btf.rst
+.. _Documentation/bpf/btf.rst:
+   https://www.kernel.org/doc/html/latest/bpf/btf.html
 
 Happy BPF hacking!
-- 
2.1.0


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH] docs/bpf: Fix invalid link of "Documentation/bpf/btf.rst"
  2023-02-22  3:10 [PATCH] docs/bpf: Fix invalid link of "Documentation/bpf/btf.rst" Tiezhu Yang
@ 2023-02-22  8:54 ` Bagas Sanjaya
  2023-02-22 10:06   ` Tiezhu Yang
  0 siblings, 1 reply; 3+ messages in thread
From: Bagas Sanjaya @ 2023-02-22  8:54 UTC (permalink / raw)
  To: Tiezhu Yang, Alexei Starovoitov, Daniel Borkmann, Andrii Nakryiko,
	Jonathan Corbet
  Cc: bpf, linux-doc, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 1492 bytes --]

On Wed, Feb 22, 2023 at 11:10:01AM +0800, Tiezhu Yang wrote:
> "Documentation/bpf/btf.rst" is linked to the following invalid web page:
> 
> https://www.kernel.org/doc/html/latest/bpf/btf.rst
> 
> The correct link should be:
> 
> https://www.kernel.org/doc/html/latest/bpf/btf.html
> 
> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>

Hi,

I have already submitted a different fix to the same problem at [1] (v2
at [2]). Please take a look at there.

[1]: https://lore.kernel.org/linux-doc/20230219030956.22662-1-bagasdotme@gmail.com/
[2]: https://lore.kernel.org/linux-doc/20230222083530.26136-1-bagasdotme@gmail.com/

> diff --git a/Documentation/bpf/bpf_devel_QA.rst b/Documentation/bpf/bpf_devel_QA.rst
> index 03d4993..f54c328 100644
> --- a/Documentation/bpf/bpf_devel_QA.rst
> +++ b/Documentation/bpf/bpf_devel_QA.rst
> @@ -690,6 +690,7 @@ when:
>     https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/tools/testing/selftests/bpf/
>  .. _Documentation/dev-tools/kselftest.rst:
>     https://www.kernel.org/doc/html/latest/dev-tools/kselftest.html
> -.. _Documentation/bpf/btf.rst: btf.rst
> +.. _Documentation/bpf/btf.rst:
> +   https://www.kernel.org/doc/html/latest/bpf/btf.html
>  
>  Happy BPF hacking!

For consistency with my fix above, can you please convert these in-tree links
from using external link to simply write the doc path instead? 

Thanks.

-- 
An old man doll... just what I always wanted! - Clara

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] docs/bpf: Fix invalid link of "Documentation/bpf/btf.rst"
  2023-02-22  8:54 ` Bagas Sanjaya
@ 2023-02-22 10:06   ` Tiezhu Yang
  0 siblings, 0 replies; 3+ messages in thread
From: Tiezhu Yang @ 2023-02-22 10:06 UTC (permalink / raw)
  To: Bagas Sanjaya, Alexei Starovoitov, Daniel Borkmann,
	Andrii Nakryiko, Jonathan Corbet
  Cc: bpf, linux-doc, linux-kernel



On 02/22/2023 04:54 PM, Bagas Sanjaya wrote:
> On Wed, Feb 22, 2023 at 11:10:01AM +0800, Tiezhu Yang wrote:
>> "Documentation/bpf/btf.rst" is linked to the following invalid web page:
>>
>> https://www.kernel.org/doc/html/latest/bpf/btf.rst
>>
>> The correct link should be:
>>
>> https://www.kernel.org/doc/html/latest/bpf/btf.html
>>
>> Signed-off-by: Tiezhu Yang <yangtiezhu@loongson.cn>
>
> Hi,
>
> I have already submitted a different fix to the same problem at [1] (v2
> at [2]). Please take a look at there.

Oh, I am sorry, I did not notice these patches, I found this issue when
I read the bpf doc.

>
> [1]: https://lore.kernel.org/linux-doc/20230219030956.22662-1-bagasdotme@gmail.com/
> [2]: https://lore.kernel.org/linux-doc/20230222083530.26136-1-bagasdotme@gmail.com/
>
>> diff --git a/Documentation/bpf/bpf_devel_QA.rst b/Documentation/bpf/bpf_devel_QA.rst
>> index 03d4993..f54c328 100644
>> --- a/Documentation/bpf/bpf_devel_QA.rst
>> +++ b/Documentation/bpf/bpf_devel_QA.rst
>> @@ -690,6 +690,7 @@ when:
>>     https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/tools/testing/selftests/bpf/
>>  .. _Documentation/dev-tools/kselftest.rst:
>>     https://www.kernel.org/doc/html/latest/dev-tools/kselftest.html
>> -.. _Documentation/bpf/btf.rst: btf.rst
>> +.. _Documentation/bpf/btf.rst:
>> +   https://www.kernel.org/doc/html/latest/bpf/btf.html
>>
>>  Happy BPF hacking!
>
> For consistency with my fix above, can you please convert these in-tree links
> from using external link to simply write the doc path instead?
>
> Thanks.
>

Please ignore my patch, you can go on to modify your changes.
Please let me know if there is anything I need to do.

Thanks,
Tiezhu


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-02-22 10:08 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-02-22  3:10 [PATCH] docs/bpf: Fix invalid link of "Documentation/bpf/btf.rst" Tiezhu Yang
2023-02-22  8:54 ` Bagas Sanjaya
2023-02-22 10:06   ` Tiezhu Yang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).