linux-kernel - Re: [PATCHv2] Documentation: add new description of path-name lookup.

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20151027212145.GF14022@fieldses.org>
Date:	Tue, 27 Oct 2015 17:21:45 -0400
From:	"J. Bruce Fields" <bfields@...ldses.org>
To:	Neil Brown <neil@...wn.name>
Cc:	Jonathan Corbet <corbet@....net>,
	Randy Dunlap <rdunlap@...radead.org>,
	Al Viro <viro@...IV.linux.org.uk>,
	lkml <linux-kernel@...r.kernel.org>, linux-doc@...r.kernel.org
Subject: Re: [PATCHv2] Documentation: add new description of path-name lookup.

On Mon, Oct 26, 2015 at 03:35:54PM +0900, Neil Brown wrote:
> From c38784b876a181eda9a5687e618749157dc96a0e Mon Sep 17 00:00:00 2001
> From: NeilBrown <neil@...wn.name>
> Date: Sat, 25 Jul 2015 10:24:41 +1000
> Subject: [PATCH] Documentation: add new description of path-name lookup.
> 
> This document is based on three recent lwn.net articles.
> Some of the introductory material and linkage between articles
> has been removed, and some time-based descriptions have been
> revised.

Thanks for doing this!  Nit:

> +End of the road
> +---------------
> +
> +Despite its complexity, all this pathname lookup code appears to be
> +in good shape - various parts are certainly easier to understand now
> +than even a couple of releases ago.  But that doesn't mean it is
> +"finished".   As already mentioned, RCU-walk currently only follows
> +symlinks that are stored in the inode so, while it handles many ext4
> +symlinks, it doesn't help with NFS, XFS, or Btrfs.  That support
> +is not likely to be long delayed.

This looks likely to go stale quickly.  Maybe just drop it?

I'd personally also probably drop the introduction.  (I think of
Documetation/ as reference material with less of a need to tell a
"story".)  But, I could be wrong.

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