| Message ID | 20240908192307.20733-3-dennis.lamerice@gmail.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | docs:filesystems: fix spelling and grammar mistakes on netfs library page | expand |
On Sun, 08 Sep 2024 15:23:09 -0400, Dennis Lam wrote:
>
Applied to the vfs.netfs branch of the vfs/vfs.git tree.
Patches in the vfs.netfs branch should appear in linux-next soon.
Please report any outstanding bugs that were missed during review in a
new review to the original patch series allowing us to drop it.
It's encouraged to provide Acked-bys and Reviewed-bys even though the
patch has now been applied. If possible patch trailers will be updated.
Note that commit hashes shown below are subject to change due to rebase,
trailer updates or similar. If in doubt, please check the listed branch.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/vfs/vfs.git
branch: vfs.netfs
[1/1] docs:filesystems: fix spelling and grammar mistakes on netfs library page
https://git.kernel.org/vfs/vfs/c/8bec52672c67
Dennis Lam <dennis.lamerice@gmail.com> wrote: > - * Allow the netfs to partially fulfil a read, which will then be resubmitted. > + * Allow the netfs to partially fulfill a read, which will then be resubmitted. "fulfil" is also correct: https://en.wiktionary.org/wiki/fulfil > - * Handle clearing of bufferage that aren't on the server. > + * Handle clearing of bufferages that aren't on the server. "bufferage" wouldn't be countable and can't be pluralised, so the fix should be "aren't" -> "isn't" instead. David
Thanks for the feedback. I'll be sure to revert the "fulfil" change and correct the pluralization in a new patch. Dennis On Mon, Sep 9, 2024 at 5:11 AM David Howells <dhowells@redhat.com> wrote: > > Dennis Lam <dennis.lamerice@gmail.com> wrote: > > > - * Allow the netfs to partially fulfil a read, which will then be resubmitted. > > + * Allow the netfs to partially fulfill a read, which will then be resubmitted. > > "fulfil" is also correct: > > https://en.wiktionary.org/wiki/fulfil > > > - * Handle clearing of bufferage that aren't on the server. > > + * Handle clearing of bufferages that aren't on the server. > > "bufferage" wouldn't be countable and can't be pluralised, so the fix should > be "aren't" -> "isn't" instead. > > David >
For those reading the mailing list, sorry for the top post. I'll be more mindful in the future. Dennis
diff --git a/Documentation/filesystems/netfs_library.rst b/Documentation/filesystems/netfs_library.rst index 4cc657d743f7..cc8e7a249de5 100644 --- a/Documentation/filesystems/netfs_library.rst +++ b/Documentation/filesystems/netfs_library.rst @@ -111,12 +111,12 @@ The following services are provided: * Allow the netfs to expand a readahead request in both directions to meet its needs. - * Allow the netfs to partially fulfil a read, which will then be resubmitted. + * Allow the netfs to partially fulfill a read, which will then be resubmitted. * Handle local caching, allowing cached data and server-read data to be interleaved for a single request. - * Handle clearing of bufferage that aren't on the server. + * Handle clearing of bufferages that aren't on the server. * Handle retrying of reads that failed, switching reads from the cache to the server as necessary.
Signed-off-by: Dennis Lam <dennis.lamerice@gmail.com> --- Documentation/filesystems/netfs_library.rst | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)