Re: [PATCH v2 08/11] docs: refcount-vs-atomic.rst: prefix url with https

From: Andrea Parri
Date: Wed May 09 2018 - 09:43:34 EST


Hi Mauro,

On Wed, May 09, 2018 at 10:18:51AM -0300, Mauro Carvalho Chehab wrote:
> There's a github URL there, but it is not prefixed by https.
> Add a prefix, to avoid false-positives with:
> ./scripts/documentation-file-ref-check
>
> As a side effect, Sphinx should also generate a cross-ref.
>
> Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@xxxxxxxxxx>

There seems to be a "conflicting" patch ("applied" according to Jon):

http://lkml.kernel.org/r/1525468309-5310-1-git-send-email-andrea.parri@xxxxxxxxxxxxxxxxxxxx

Let me stress here that the github repo. is out-of-date (and we have
no plans to keep that in sync with mainline).

Andrea


> ---
> Documentation/core-api/refcount-vs-atomic.rst | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/core-api/refcount-vs-atomic.rst b/Documentation/core-api/refcount-vs-atomic.rst
> index 83351c258cdb..185d659e350a 100644
> --- a/Documentation/core-api/refcount-vs-atomic.rst
> +++ b/Documentation/core-api/refcount-vs-atomic.rst
> @@ -17,7 +17,7 @@ in order to help maintainers validate their code against the change in
> these memory ordering guarantees.
>
> The terms used through this document try to follow the formal LKMM defined in
> -github.com/aparri/memory-model/blob/master/Documentation/explanation.txt
> +https://github.com/aparri/memory-model/blob/master/Documentation/explanation.txt
>
> memory-barriers.txt and atomic_t.txt provide more background to the
> memory ordering in general and for atomic operations specifically.
> --
> 2.17.0
>