Messages in this thread | | | Date | Tue, 27 Oct 2015 17:21:45 -0400 | From | "J. Bruce Fields" <> | 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@brown.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.
| |