Re: [PATCH] documentation: fix broken lkml archive links in RCU requirements
From: Steven Rostedt
Date: Fri Sep 09 2016 - 10:34:16 EST
On Fri, 9 Sep 2016 16:17:14 +0200
Richard Weinberger <richard.weinberger@xxxxxxxxx> wrote:
> > Signed-off-by: Michael Opdenacker <michael.opdenacker@xxxxxxxxxxxxxxxxxx>
> > ---
> > Documentation/RCU/Design/Requirements/Requirements.html | 8 ++++----
> > 1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/Documentation/RCU/Design/Requirements/Requirements.html b/Documentation/RCU/Design/Requirements/Requirements.html
> > index ece410f40436..2adb3d43ce44 100644
> > --- a/Documentation/RCU/Design/Requirements/Requirements.html
> > +++ b/Documentation/RCU/Design/Requirements/Requirements.html
> > @@ -1527,7 +1527,7 @@ However, as I learned from Matt Mackall's
> > <a href="http://elinux.org/Linux_Tiny-FAQ">bloatwatch</a>
> > efforts, memory footprint is critically important on single-CPU systems with
> > non-preemptible (<tt>CONFIG_PREEMPT=n</tt>) kernels, and thus
> > -<a href="https://lkml.kernel.org/g/20090113221724.GA15307@xxxxxxxxxxxxxxxxxx">tiny RCU</a>
> > +<a href="http://lkml.org/lkml/2009/1/14/449">tiny RCU</a>
> > was born.
> > Josh Triplett has since taken over the small-memory banner with his
> > <a href="https://tiny.wiki.kernel.org/">Linux kernel tinification</a>
> > @@ -1975,7 +1975,7 @@ guard against mishaps and misuse:
> > and cleaned up with <tt>destroy_rcu_head()</tt>.
> > Mathieu Desnoyers made me aware of this requirement, and also
> > supplied the needed
> > - <a href="https://lkml.kernel.org/g/20100319013024.GA28456@Krystal">patch</a>.
> > + <a href="https://lkml.org/lkml/2010/3/18/417">patch</a>.
>
> Please don't add lkml.org. It does not use message ids for indexing.
> With knowing the message id you can query any other archive.
> e.g. http://marc.info/?i=20100319013024.GA28456@Krystal
> By adding lkml.org you kill that information. Archives come and go,
> the message id is the only common query id we have.
>
> IMHO kernel.org admins should fix/improve their redirection service to
> point to a working service.
>
Correct, we avoid any links to lkml.org at all costs. Simple do a
s,/g/,/r/, and all your links should work. For example, using the above
mentioned link:
https://lkml.kernel.org/r/20100319013024.GA28456@Krystal
Works as expected.
-- Steve