Re: [PATCH 3/5] mm: powerpc ptlock comments

[Paul Mackerras]

Hugh Dickins writes:

> Update comments (only) on page_table_lock and mmap_sem in arch/powerpc.

[snip]

>  	/*
> -	 * No need to use ldarx/stdcx here because all who
> -	 * might be updating the pte will hold the
> -	 * page_table_lock
> +	 * No need to use ldarx/stdcx here
>  	 */

If you're going to remove the because clause you might as well remove
the whole comment.  What you have left is either redundant or
mystifying (according to the depth of knowledge of the reader).  And
in fact I don't think I could now say why we don't need an atomic
update sequence there, so I would appreciate something that updates
the "because" part.

Paul.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/