2022-12-01 11:40:51

by Tiezhu Yang

[permalink] [raw]
Subject: [PATCH 0/2] Update links of LoongArch ISA Vol1 and ELF psABI

Tiezhu Yang (2):
docs/LoongArch: Update links of LoongArch ISA Vol1 and ELF psABI
docs/zh_CN/LoongArch: Update links of LoongArch ISA Vol1 and ELF psABI

Documentation/loongarch/introduction.rst | 8 ++++----
Documentation/translations/zh_CN/loongarch/introduction.rst | 8 ++++----
2 files changed, 8 insertions(+), 8 deletions(-)

--
2.1.0


2022-12-01 12:23:54

by Tiezhu Yang

[permalink] [raw]
Subject: [PATCH 1/2] docs/LoongArch: Update links of LoongArch ISA Vol1 and ELF psABI

The current links of LoongArch ISA Vol1 and ELF psABI are invalid,
the latest versions are 1.02 and 2.00 respectively, let us update
the links.

Signed-off-by: Tiezhu Yang <[email protected]>
---
Documentation/loongarch/introduction.rst | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/Documentation/loongarch/introduction.rst b/Documentation/loongarch/introduction.rst
index 6c9160c..49135d4 100644
--- a/Documentation/loongarch/introduction.rst
+++ b/Documentation/loongarch/introduction.rst
@@ -375,15 +375,15 @@ Developer web site of Loongson and LoongArch (Software and Documentation):

Documentation of LoongArch ISA:

- https://github.com/loongson/LoongArch-Documentation/releases/latest/download/LoongArch-Vol1-v1.00-CN.pdf (in Chinese)
+ https://github.com/loongson/LoongArch-Documentation/releases/latest/download/LoongArch-Vol1-v1.02-CN.pdf (in Chinese)

- https://github.com/loongson/LoongArch-Documentation/releases/latest/download/LoongArch-Vol1-v1.00-EN.pdf (in English)
+ https://github.com/loongson/LoongArch-Documentation/releases/latest/download/LoongArch-Vol1-v1.02-EN.pdf (in English)

Documentation of LoongArch ELF psABI:

- https://github.com/loongson/LoongArch-Documentation/releases/latest/download/LoongArch-ELF-ABI-v1.00-CN.pdf (in Chinese)
+ https://github.com/loongson/LoongArch-Documentation/releases/latest/download/LoongArch-ELF-ABI-v2.00-CN.pdf (in Chinese)

- https://github.com/loongson/LoongArch-Documentation/releases/latest/download/LoongArch-ELF-ABI-v1.00-EN.pdf (in English)
+ https://github.com/loongson/LoongArch-Documentation/releases/latest/download/LoongArch-ELF-ABI-v2.00-EN.pdf (in English)

Linux kernel repository of Loongson and LoongArch:

--
2.1.0

2022-12-03 11:47:37

by Jonathan Corbet

[permalink] [raw]
Subject: Re: [PATCH 0/2] Update links of LoongArch ISA Vol1 and ELF psABI

Tiezhu Yang <[email protected]> writes:

> Tiezhu Yang (2):
> docs/LoongArch: Update links of LoongArch ISA Vol1 and ELF psABI
> docs/zh_CN/LoongArch: Update links of LoongArch ISA Vol1 and ELF psABI
>
> Documentation/loongarch/introduction.rst | 8 ++++----
> Documentation/translations/zh_CN/loongarch/introduction.rst | 8 ++++----
> 2 files changed, 8 insertions(+), 8 deletions(-)

I've applied these - working links are better than broken ones. I
assume, though, that the new URLs, too, will break the next time the
manual is updated? Is there any chance Loongson could provide a stable
URL for the current version of this manual instead?

Thanks,

jon

2022-12-04 06:16:38

by Xi Ruoyao

[permalink] [raw]
Subject: Re: [PATCH 0/2] Update links of LoongArch ISA Vol1 and ELF psABI

On Sat, 2022-12-03 at 03:58 -0700, Jonathan Corbet wrote:
> Tiezhu Yang <[email protected]> writes:
>
> > Tiezhu Yang (2):
> >    docs/LoongArch: Update links of LoongArch ISA Vol1 and ELF psABI
> >    docs/zh_CN/LoongArch: Update links of LoongArch ISA Vol1 and ELF psABI
> >
> >   Documentation/loongarch/introduction.rst                    | 8 ++++----
> >   Documentation/translations/zh_CN/loongarch/introduction.rst | 8 ++++----
> >   2 files changed, 8 insertions(+), 8 deletions(-)
>
> I've applied these - working links are better than broken ones.  I
> assume, though, that the new URLs, too, will break the next time the
> manual is updated?  Is there any chance Loongson could provide a stable
> URL for the current version of this manual instead?

Use a URL with tag name instead of "latest", like

https://github.com/loongson/LoongArch-Documentation/releases/download/2022.08.12/LoongArch-Vol1-v1.02-CN.pdf

It will still refer to an outdated doc if the doc is updated, but at
least it won't be broken.

--
Xi Ruoyao <[email protected]>
School of Aerospace Science and Technology, Xidian University

2022-12-04 09:13:12

by Huacai Chen

[permalink] [raw]
Subject: Re: [PATCH 0/2] Update links of LoongArch ISA Vol1 and ELF psABI

For the whole series,
Acked-by: Huacai Chen <[email protected]>

On Thu, Dec 1, 2022 at 6:59 PM Tiezhu Yang <[email protected]> wrote:
>
> Tiezhu Yang (2):
> docs/LoongArch: Update links of LoongArch ISA Vol1 and ELF psABI
> docs/zh_CN/LoongArch: Update links of LoongArch ISA Vol1 and ELF psABI
>
> Documentation/loongarch/introduction.rst | 8 ++++----
> Documentation/translations/zh_CN/loongarch/introduction.rst | 8 ++++----
> 2 files changed, 8 insertions(+), 8 deletions(-)
>
> --
> 2.1.0
>

2022-12-05 02:17:44

by Tiezhu Yang

[permalink] [raw]
Subject: Re: [PATCH 0/2] Update links of LoongArch ISA Vol1 and ELF psABI



On 12/03/2022 06:58 PM, Jonathan Corbet wrote:
> Tiezhu Yang <[email protected]> writes:
>
>> Tiezhu Yang (2):
>> docs/LoongArch: Update links of LoongArch ISA Vol1 and ELF psABI
>> docs/zh_CN/LoongArch: Update links of LoongArch ISA Vol1 and ELF psABI
>>
>> Documentation/loongarch/introduction.rst | 8 ++++----
>> Documentation/translations/zh_CN/loongarch/introduction.rst | 8 ++++----
>> 2 files changed, 8 insertions(+), 8 deletions(-)
>
> I've applied these - working links are better than broken ones. I
> assume, though, that the new URLs, too, will break the next time the
> manual is updated? Is there any chance Loongson could provide a stable
> URL for the current version of this manual instead?
>
> Thanks,
>
> jon
>

Here are my thoughts:

(1) Just use the release link without file name, simple and timeless.

https://github.com/loongson/LoongArch-Documentation/releases

(2) Use the online html links, but the Chinese html version of ISA
Vol1 is unavailable now.

https://loongson.github.io/LoongArch-Documentation/LoongArch-Vol1-CN.html
(unavailable now)
https://loongson.github.io/LoongArch-Documentation/LoongArch-Vol1-EN.html

https://loongson.github.io/LoongArch-Documentation/LoongArch-ELF-ABI-CN.html
https://loongson.github.io/LoongArch-Documentation/LoongArch-ELF-ABI-EN.html

(3) As Ruoyao said, use a link with tag name instead of "latest",
may be outdated but always valid.

https://github.com/loongson/LoongArch-Documentation/releases/download/2022.08.12/LoongArch-Vol1-v1.02-CN.pdf

Thanks,
Tiezhu