Re: [PATCH v3 4/6] folio: Add a function to get the host inode for a folio

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Matthew Wilcox <willy@xxxxxxxxxxxxx>
Subject: Re: [PATCH v3 4/6] folio: Add a function to get the host inode for a folio
From: David Howells <dhowells@xxxxxxxxxx>
Date: Wed, 03 Nov 2021 14:34:33 +0000
Cc: dhowells@xxxxxxxxxx, Jeff Layton <jlayton@xxxxxxxxxx>, Ilya Dryomov <idryomov@xxxxxxxxx>, Dominique Martinet <asmadeus@xxxxxxxxxxxxx>, linux-fsdevel@xxxxxxxxxxxxxxx, linux-cachefs@xxxxxxxxxx, v9fs-developer@xxxxxxxxxxxxxxxxxxxxx, linux-afs@xxxxxxxxxxxxxxxxxxx, linux-nfs@xxxxxxxxxxxxxxx, linux-cifs@xxxxxxxxxxxxxxx, ceph-devel@xxxxxxxxxxxxxxx, devel@xxxxxxxxxxxxxxxxxx, linux-mm@xxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <YYKLkBwQdtn4ja+i@casper.infradead.org>
Organization: Red Hat UK Ltd. Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod Street, Windsor, Berkshire, SI4 1TE, United Kingdom. Registered in England and Wales under Company Registration No. 3798903
References: <YYKLkBwQdtn4ja+i@casper.infradead.org> <163584174921.4023316.8927114426959755223.stgit@warthog.procyon.org.uk> <163584184628.4023316.9386282630968981869.stgit@warthog.procyon.org.uk>

Matthew Wilcox <willy@xxxxxxxxxxxxx> wrote:

> > + * For folios which are in the page cache, return the inode that is hosting
> > + * this folio belongs to.
> 
> This looks like an editing mistake.  Either you meant
> 'return the inode that hosts this folio' or
> 'return the inode this folio belongs to'
> (and i prefer the second).

Yeah - I'll go with that.

David

References:
- Re: [PATCH v3 4/6] folio: Add a function to get the host inode for a folio
  - From: Matthew Wilcox
- [PATCH v3 0/6] netfs, 9p, afs, ceph: Support folios, at least partially
  - From: David Howells
- [PATCH v3 4/6] folio: Add a function to get the host inode for a folio
  - From: David Howells

Prev by Date: [linux-next:master 5531/13070] sound/soc/codecs/max98927.c:902:19: error: implicit declaration of function 'devm_gpiod_get_optional'; did you mean 'devm_regulator_get_optional'?
Next by Date: [linux-next:master 2951/11937] sound/soc/codecs/wm_adsp.c:1441 wm_adsp_control_add() error: uninitialized symbol 'ret'.
Previous by thread: Re: [PATCH v3 4/6] folio: Add a function to get the host inode for a folio
Next by thread: [PATCH v3 5/6] netfs, 9p, afs, ceph: Use folios
Index(es):
- Date
- Thread

[Index of Archives] [Linux ARM Kernel] [Linux ARM] [Linux Omap] [Fedora ARM] [IETF Annouce] [Bugtraq] [Linux OMAP] [Linux MIPS] [eCos] [Asterisk Internet PBX] [Linux API]